Run npm run build as suggested

Signed-off-by: Matthieu MOREL <mmorel-35@users.noreply.github.com>

.eslintignore

@@ -1,6 +0,0 @@
-# Ignore list
-/*
-
-# Do not ignore these folders:
-!__tests__/
-!src/

.eslintrc.js

@@ -1,51 +0,0 @@
-// This is a reusable configuration file copied from https://github.com/actions/reusable-workflows/tree/main/reusable-configurations. Please don't make changes to this file as it's the subject of an automatic update.
-module.exports = {
-  extends: [
-    'eslint:recommended',
-    'plugin:@typescript-eslint/recommended',
-    'plugin:eslint-plugin-jest/recommended',
-    'eslint-config-prettier'
-  ],
-  parser: '@typescript-eslint/parser',
-  plugins: ['@typescript-eslint', 'eslint-plugin-node', 'eslint-plugin-jest'],
-  rules: {
-    '@typescript-eslint/no-require-imports': 'error',
-    '@typescript-eslint/no-non-null-assertion': 'off',
-    '@typescript-eslint/no-explicit-any': 'off',
-    '@typescript-eslint/no-empty-function': 'off',
-    '@typescript-eslint/ban-ts-comment': [
-      'error',
-      {
-        'ts-ignore': 'allow-with-description'
-      }
-    ],
-    'no-console': 'error',
-    'yoda': 'error',
-    'prefer-const': [
-      'error',
-      {
-        destructuring: 'all'
-      }
-    ],
-    'no-control-regex': 'off',
-    'no-constant-condition': ['error', {checkLoops: false}],
-    'node/no-extraneous-import': 'error'
-  },
-  overrides: [
-    {
-      files: ['**/*{test,spec}.ts'],
-      rules: {
-        '@typescript-eslint/no-unused-vars': 'off',
-        'jest/no-standalone-expect': 'off',
-        'jest/no-conditional-expect': 'off',
-        'no-console': 'off',
-
-      }
-    }
-  ],
-  env: {
-    node: true,
-    es6: true,
-    'jest/globals': true
-  }
-};

.gitattributes

@@ -1,2 +1 @@
-* text=auto eol=lf
 .licenses/** -diff linguist-generated=true

.github/workflows/basic-validation.yml

@@ -15,5 +15,3 @@ jobs:
   call-basic-validation:
     name: Basic validation
     uses: actions/reusable-workflows/.github/workflows/basic-validation.yml@main
-    with:
-      node-version: '20.x'

.github/workflows/check-dist.yml

@@ -1,4 +1,4 @@
-name: Check dist
+name: Check dist/
 
 on:
   push:
@@ -15,5 +15,3 @@ jobs:
   call-check-dist:
     name: Check dist/
     uses: actions/reusable-workflows/.github/workflows/check-dist.yml@main
-    with:
-      node-version: '20.x'

.github/workflows/codeql-analysis.yml

@@ -2,9 +2,9 @@ name: CodeQL analysis
 
 on:
   push:
-    branches: [main]
+    branches: [ main ]
   pull_request:
-    branches: [main]
+    branches: [ main ]
   schedule:
     - cron: '0 3 * * 0'
 

.github/workflows/e2e-cache.yml

@@ -21,7 +21,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [12, 14, 16]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Clean global cache
         run: npm cache clean --force
       - name: Setup Node
@@ -44,7 +44,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [12, 14, 16]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Install pnpm
         uses: pnpm/action-setup@v2
         with:
@@ -77,7 +77,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [14, 16]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Yarn version
         run: yarn --version
       - name: Generate yarn file
@@ -93,13 +93,13 @@ jobs:
           node-version: ${{ matrix.node-version }}
           cache: 'yarn'
       - name: Install dependencies
-        run: yarn install --ignore-engines
+        run: yarn install
       - name: Verify node and yarn
         run: __tests__/verify-node.sh "${{ matrix.node-version }}"
         shell: bash
 
-  node-yarn3-depencies-caching:
+  node-yarn2-depencies-caching:
-    name: Test yarn 3 (Node ${{ matrix.node-version}}, ${{ matrix.os }})
+    name: Test yarn 2 (Node ${{ matrix.node-version}}, ${{ matrix.os }})
     runs-on: ${{ matrix.os }}
     env:
       YARN_ENABLE_IMMUTABLE_INSTALLS: false
@@ -109,9 +109,9 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [12, 14, 16]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Update yarn
-        run: yarn set version 3.6.4
+        run: yarn set version berry
       - name: Yarn version
         run: yarn --version
       - name: Generate simple .yarnrc.yml
@@ -134,112 +134,3 @@ jobs:
       - name: Verify node and yarn
         run: __tests__/verify-node.sh "${{ matrix.node-version }}"
         shell: bash
-
-  yarn-subprojects:
-    name: Test yarn subprojects
-    strategy:
-      matrix:
-        node-version: [12, 14, 16]
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: prepare sub-projects
-        run: __tests__/prepare-yarn-subprojects.sh yarn1
-
-      # expect
-      #  - no errors
-      #  - log
-      #    ##[debug]Cache Paths:
-      #    ##[debug]["sub2/.yarn/cache","sub3/.yarn/cache","../../../.cache/yarn/v6"]
-      - name: Setup Node
-        uses: ./
-        with:
-          node-version: ${{ matrix.node-version }}
-          cache: 'yarn'
-          cache-dependency-path: |
-            **/*.lock
-            yarn.lock
-
-  yarn-subprojects-berry-local:
-    name: Test yarn subprojects all locally managed
-    strategy:
-      matrix:
-        node-version: [12, 14, 16]
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: prepare sub-projects
-        run: __tests__/prepare-yarn-subprojects.sh keepcache keepcache
-
-      # expect
-      #  - no errors
-      #  - log
-      #    ##[info]All dependencies are managed locally by yarn3, the previous cache can be used
-      #    ##[debug]["node-cache-Linux-yarn-401024703386272f1a950c9f014cbb1bb79a7a5b6e1fb00e8b90d06734af41ee","node-cache-Linux-yarn"]
-      - name: Setup Node
-        uses: ./
-        with:
-          node-version: ${{ matrix.node-version }}
-          cache: 'yarn'
-          cache-dependency-path: |
-            sub2/*.lock
-            sub3/*.lock
-
-  yarn-subprojects-berry-global:
-    name: Test yarn subprojects some locally managed
-    strategy:
-      matrix:
-        node-version: [12, 14, 16]
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: prepare sub-projects
-        run: __tests__/prepare-yarn-subprojects.sh global
-
-      # expect
-      #  - no errors
-      #  - log must
-      #    ##[debug]"/home/runner/work/setup-node-test/setup-node-test/sub2" dependencies are managed by yarn 3 locally
-      #    ##[debug]"/home/runner/work/setup-node-test/setup-node-test/sub3" dependencies are not managed by yarn 3 locally
-      - name: Setup Node
-        uses: ./
-        with:
-          node-version: ${{ matrix.node-version }}
-          cache: 'yarn'
-          cache-dependency-path: |
-            sub2/*.lock
-            sub3/*.lock
-
-  yarn-subprojects-berry-git:
-    name: Test yarn subprojects managed by git
-    strategy:
-      matrix:
-        node-version: [12, 14, 16]
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: prepare sub-projects
-        run: /bin/bash __tests__/prepare-yarn-subprojects.sh keepcache
-
-      # expect
-      #  - no errors
-      #  - log
-      #    [debug]"/home/runner/work/setup-node-test/setup-node-test/sub2" has .yarn/cache - dependencies are kept in the repository
-      #    [debug]"/home/runner/work/setup-node-test/setup-node-test/sub3" has .yarn/cache - dependencies are kept in the repository
-      #    [debug]["node-cache-Linux-yarn-401024703386272f1a950c9f014cbb1bb79a7a5b6e1fb00e8b90d06734af41ee"]
-      - name: Setup Node
-        uses: ./
-        with:
-          node-version: ${{ matrix.node-version }}
-          cache: 'yarn'
-          cache-dependency-path: |
-            sub2/*.lock
-            sub3/*.lock

.github/workflows/proxy.yml

@@ -25,7 +25,7 @@ jobs:
     env:
       https_proxy: http://squid-proxy:3128
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Clear tool cache
         run: rm -rf $RUNNER_TOOL_CACHE/*
       - name: Setup node 14
@@ -41,7 +41,7 @@ jobs:
       https_proxy: http://no-such-proxy:3128
       no_proxy: api.github.com,github.com,nodejs.org,registry.npmjs.org,*.s3.amazonaws.com,s3.amazonaws.com
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Clear tool cache
         run: rm -rf $RUNNER_TOOL_CACHE/*
       - name: Setup node 11

.github/workflows/update-config-files.yml

@@ -1,11 +0,0 @@
-name: Update configuration files
-
-on:
-  schedule:
-    - cron: '0 3 * * 0'
-  workflow_dispatch:
-
-jobs:
-  call-update-configuration-files:
-    name: Update configuration files
-    uses: actions/reusable-workflows/.github/workflows/update-config-files.yml@main

.github/workflows/versions.yml

@@ -20,7 +20,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [10, 12, 14]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node
         uses: ./
         with:
@@ -37,7 +37,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [lts/dubnium, lts/erbium, lts/fermium, lts/*, lts/-1]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node
         uses: ./
         with:
@@ -57,14 +57,9 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
-        node-version:
+        node-version: ['20-v8-canary', '20.0.0-v8-canary','20.0.0-v8-canary20221103f7e2421e91']
-          [
-            '20-v8-canary',
-            '20.0.0-v8-canary',
-            '20.0.0-v8-canary20221101e50e45c9f8'
-          ]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node
         uses: ./
         with:
@@ -82,10 +77,9 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
-        node-version:
+        node-version: [16.0.0-nightly20210420a0261d231c, 17-nightly, 18.0.0-nightly]
-          [16.0.0-nightly20210420a0261d231c, 17-nightly, 18.0.0-nightly]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node
         uses: ./
         with:
@@ -105,7 +99,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [16.0.0-rc.1, 18.0.0-rc.2, 19.0.0-rc.0]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node
         uses: ./
         with:
@@ -125,7 +119,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [10.15, 12.16.0, 14.2.0, 16.3.0]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node
         uses: ./
         with:
@@ -142,7 +136,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [10, 12, 14]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node and check latest
         uses: ./
         with:
@@ -158,10 +152,12 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
-        node-version-file:
+        node-version-file: [.nvmrc, .tool-versions, package.json]
-          [.nvmrc, .tool-versions, .tool-versions-node, package.json]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
+      - name: Remove volta from package.json
+        shell: bash
+        run: cat <<< "$(jq 'del(.volta)' ./__tests__/data/package.json)" > ./__tests__/data/package.json
       - name: Setup node from node version file
         uses: ./
         with:
@@ -174,28 +170,13 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
+        os: [ ubuntu-latest, windows-latest, macos-latest ]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup node from node version file
         uses: ./
         with:
-          node-version-file: '__tests__/data/package-volta.json'
+          node-version-file: '__tests__/data/package.json'
-      - name: Verify node
-        run: __tests__/verify-node.sh 16
-
-  version-file-volta-extends:
-    runs-on: ${{ matrix.os }}
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
-    steps:
-      - uses: actions/checkout@v4
-      - name: Setup node from node version file
-        uses: ./
-        with:
-          node-version-file: '__tests__/data/package-volta-extends.json'
       - name: Verify node
         run: __tests__/verify-node.sh 16
 
@@ -207,7 +188,7 @@ jobs:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [11, 13]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node from dist
         uses: ./
         with:
@@ -223,7 +204,7 @@ jobs:
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       # test old versions which didn't have npm and layout different
       - name: Setup node 0.12.18 from dist
         uses: ./
@@ -236,7 +217,7 @@ jobs:
   arch:
     runs-on: windows-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup node 14 x86 from dist
         uses: ./
         with:
@@ -260,7 +241,7 @@ jobs:
           echo "LATEST_NODE_VERSION=$latestNodeVersion" >> $GITHUB_OUTPUT
         id: version
         shell: bash
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup Node
         uses: ./
         with:

.prettierignore

@@ -1,7 +0,0 @@
-# Ignore list
-/*
-
-# Do not ignore these folders:
-!__tests__/
-!.github/
-!src/

.prettierrc.js

@@ -1,11 +0,0 @@
-// This is a reusable configuration file copied from https://github.com/actions/reusable-workflows/tree/main/reusable-configurations. Please don't make changes to this file as it's the subject of an automatic update.
-module.exports = {
-  printWidth: 80,
-  tabWidth: 2,
-  useTabs: false,
-  semi: true,
-  singleQuote: true,
-  trailingComma: 'none',
-  bracketSpacing: false,
-  arrowParens: 'avoid'
-};

.prettierrc.json

@@ -0,0 +1,11 @@
+{
+    "printWidth": 80,
+    "tabWidth": 2,
+    "useTabs": false,
+    "semi": true,
+    "singleQuote": true,
+    "trailingComma": "none",
+    "bracketSpacing": false,
+    "arrowParens": "avoid",
+    "parser": "typescript"
+  }

README.md

@@ -16,77 +16,14 @@ This action provides the following functionality for GitHub Actions users:
 
 See [action.yml](action.yml)
 
-<!-- start usage -->
-```yaml
-- uses: actions/setup-node@v4
-  with:
-    # Version Spec of the version to use in SemVer notation.
-    # It also emits such aliases as lts, latest, nightly and canary builds
-    # Examples: 12.x, 10.15.1, >=10.15.0, lts/Hydrogen, 16-nightly, latest, node
-    node-version: ''
-
-    # File containing the version Spec of the version to use.  Examples: package.json, .nvmrc, .node-version, .tool-versions.
-    # If node-version and node-version-file are both provided the action will use version from node-version. 
-    node-version-file: ''
-
-    # Set this option if you want the action to check for the latest available version 
-    # that satisfies the version spec.
-    # It will only get affect for lts Nodejs versions (12.x, >=10.15.0, lts/Hydrogen). 
-    # Default: false
-    check-latest: false
-
-    # Target architecture for Node to use. Examples: x86, x64. Will use system architecture by default.
-    # Default: ''. The action use system architecture by default 
-    architecture: ''
-
-    # Used to pull node distributions from https://github.com/actions/node-versions. 
-    # Since there's a default, this is typically not supplied by the user. 
-    # When running this action on github.com, the default value is sufficient. 
-    # When running on GHES, you can pass a personal access token for github.com if you are experiencing rate limiting.
-    #
-    # We recommend using a service account with the least permissions necessary. Also
-    # when generating a new PAT, select the least scopes necessary.
-    #
-    # [Learn more about creating and using encrypted secrets](https://help.github.com/en/actions/automating-your-workflow-with-github-actions/creating-and-using-encrypted-secrets)
-    #
-    # Default: ${{ github.server_url == 'https://github.com' && github.token || '' }}
-    token: ''
-
-    # Used to specify a package manager for caching in the default directory. Supported values: npm, yarn, pnpm.
-    # Package manager should be pre-installed
-    # Default: ''
-    cache: ''
-
-    # Used to specify the path to a dependency file: package-lock.json, yarn.lock, etc. 
-    # It will generate hash from the target file for primary key. It works only If cache is specified.  
-    # Supports wildcards or a list of file names for caching multiple dependencies.
-    # Default: ''
-    cache-dependency-path: ''
-
-    # Optional registry to set up for auth. Will set the registry in a project level .npmrc and .yarnrc file, 
-    # and set up auth to read in from env.NODE_AUTH_TOKEN.
-    # Default: ''
-    registry-url: ''
-
-    # Optional scope for authenticating against scoped registries. 
-    # Will fall back to the repository owner when using the GitHub Packages registry (https://npm.pkg.github.com/).
-    # Default: ''
-    scope: ''
-
-    # Set always-auth option in npmrc file.
-    # Default: ''
-    always-auth: ''
-```
-<!-- end usage -->
-
 **Basic:**
 
 ```yaml
 steps:
-- uses: actions/checkout@v4
+- uses: actions/checkout@v3
-- uses: actions/setup-node@v4
+- uses: actions/setup-node@v3
   with:
-    node-version: 18
+    node-version: 16
 - run: npm ci
 - run: npm test
 ```
@@ -99,16 +36,16 @@ For information regarding locally cached versions of Node.js on GitHub hosted ru
 
 ### Supported version syntax
 
-The `node-version` input supports the Semantic Versioning Specification, for more detailed examples please refer to [the semver package documentation](https://github.com/npm/node-semver).
+The `node-version` input supports the Semantic Versioning Specification, for more detailed examples please refer to the [documentation](https://github.com/npm/node-semver).
 
 Examples:
 
- - Major versions: `18`, `20`
+ - Major versions: `14`, `16`, `18`
  - More specific versions: `10.15`, `16.15.1` , `18.4.0`
  - NVM LTS syntax: `lts/erbium`, `lts/fermium`, `lts/*`, `lts/-n`
  - Latest release: `*` or `latest`/`current`/`node`
 
-**Note:** Like the other values, `*` will get the latest [locally-cached Node.js version](https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2204-Readme.md#nodejs), or the latest version from [actions/node-versions](https://github.com/actions/node-versions/blob/main/versions-manifest.json), depending on the [`check-latest`](docs/advanced-usage.md#check-latest-version) input.
+**Note:** Like the other values, `*` will get the latest [locally-cached Node.js version](https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md#nodejs), or the latest version from [actions/node-versions](https://github.com/actions/node-versions/blob/main/versions-manifest.json), depending on the [`check-latest`](docs/advanced-usage.md#check-latest-version) input.
 
 `current`/`latest`/`node` always resolve to the latest [dist version](https://nodejs.org/dist/index.json).
 That version is then downloaded from actions/node-versions if possible, or directly from Node.js if not.
@@ -134,10 +71,10 @@ See the examples of using cache for `yarn`/`pnpm` and `cache-dependency-path` in
 
 ```yaml
 steps:
-- uses: actions/checkout@v4
+- uses: actions/checkout@v3
-- uses: actions/setup-node@v4
+- uses: actions/setup-node@v3
   with:
-    node-version: 20
+    node-version: 16
     cache: 'npm'
 - run: npm ci
 - run: npm test
@@ -147,10 +84,10 @@ steps:
 
 ```yaml
 steps:
-- uses: actions/checkout@v4
+- uses: actions/checkout@v3
-- uses: actions/setup-node@v4
+- uses: actions/setup-node@v3
   with:
-    node-version: 20
+    node-version: 16
     cache: 'npm'
     cache-dependency-path: subdir/package-lock.json
 - run: npm ci
@@ -168,9 +105,9 @@ jobs:
         node: [ 14, 16, 18 ]
     name: Node ${{ matrix.node }} sample
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Setup node
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v3
         with:
           node-version: ${{ matrix.node }}
       - run: npm ci
@@ -184,10 +121,10 @@ jobs:
 To get a higher rate limit, you can [generate a personal access token on github.com](https://github.com/settings/tokens/new) and pass it as the `token` input for the action:
 
 ```yaml
-uses: actions/setup-node@v4
+uses: actions/setup-node@v3
 with:
   token: ${{ secrets.GH_DOTCOM_TOKEN }}
-  node-version: 20
+  node-version: 16
 ```
 
 If the runner is not able to access github.com, any Nodejs versions requested during a workflow run must come from the runner's tool cache. See "[Setting up the tool cache on self-hosted runners without internet access](https://docs.github.com/en/enterprise-server@3.2/admin/github-actions/managing-access-to-actions-from-githubcom/setting-up-the-tool-cache-on-self-hosted-runners-without-internet-access)" for more information.

__tests__/authutil.test.ts

@@ -1,10 +1,9 @@
 import os from 'os';
-import fs from 'fs';
+import * as fs from 'fs';
 import * as path from 'path';
 import * as core from '@actions/core';
 import * as io from '@actions/io';
 import * as auth from '../src/authutil';
-import * as cacheUtils from '../src/cache-utils';
 
 let rcFile: string;
 
@@ -16,7 +15,11 @@ describe('authutil tests', () => {
   let dbgSpy: jest.SpyInstance;
 
   beforeAll(async () => {
-    const randPath = path.join(Math.random().toString(36).substring(7));
+    const randPath = path.join(
+      Math.random()
+        .toString(36)
+        .substring(7)
+    );
     console.log('::stop-commands::stoptoken'); // Disable executing of runner commands when running tests in actions
     process.env['GITHUB_ENV'] = ''; // Stub out Environment file functionality so we can verify it writes to standard out (toolkit is backwards compatible)
     const tempDir = path.join(_runnerDir, randPath, 'temp');
@@ -64,10 +67,10 @@ describe('authutil tests', () => {
   }, 100000);
 
   function readRcFile(rcFile: string) {
-    const rc = {};
+    let rc = {};
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     for (const line of contents.split(os.EOL)) {
-      const parts = line.split('=');
+      let parts = line.split('=');
       if (parts.length == 2) {
         rc[parts[0].trim()] = parts[1].trim();
       }
@@ -79,8 +82,8 @@ describe('authutil tests', () => {
     await auth.configAuthentication('https://registry.npmjs.org/', 'false');
 
     expect(fs.statSync(rcFile)).toBeDefined();
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
-    const rc = readRcFile(rcFile);
+    let rc = readRcFile(rcFile);
     expect(rc['registry']).toBe('https://registry.npmjs.org/');
     expect(rc['always-auth']).toBe('false');
   });
@@ -89,7 +92,7 @@ describe('authutil tests', () => {
     await auth.configAuthentication('https://registry.npmjs.org', 'false');
 
     expect(fs.statSync(rcFile)).toBeDefined();
-    const rc = readRcFile(rcFile);
+    let rc = readRcFile(rcFile);
     expect(rc['registry']).toBe('https://registry.npmjs.org/');
     expect(rc['always-auth']).toBe('false');
   });
@@ -99,7 +102,7 @@ describe('authutil tests', () => {
     await auth.configAuthentication('https://registry.npmjs.org', 'false');
 
     expect(fs.statSync(rcFile)).toBeDefined();
-    const rc = readRcFile(rcFile);
+    let rc = readRcFile(rcFile);
     expect(rc['@myscope:registry']).toBe('https://registry.npmjs.org/');
     expect(rc['always-auth']).toBe('false');
   });
@@ -108,7 +111,7 @@ describe('authutil tests', () => {
     await auth.configAuthentication('npm.pkg.github.com', 'false');
 
     expect(fs.statSync(rcFile)).toBeDefined();
-    const rc = readRcFile(rcFile);
+    let rc = readRcFile(rcFile);
     expect(rc['@ownername:registry']).toBe('npm.pkg.github.com/');
     expect(rc['always-auth']).toBe('false');
   });
@@ -116,16 +119,16 @@ describe('authutil tests', () => {
   it('Sets up npmrc for always-auth true', async () => {
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
     expect(fs.statSync(rcFile)).toBeDefined();
-    const rc = readRcFile(rcFile);
+    let rc = readRcFile(rcFile);
     expect(rc['registry']).toBe('https://registry.npmjs.org/');
     expect(rc['always-auth']).toBe('true');
   });
 
-  it('is already set the NODE_AUTH_TOKEN export it', async () => {
+  it('It is already set the NODE_AUTH_TOKEN export it ', async () => {
     process.env.NODE_AUTH_TOKEN = 'foobar';
     await auth.configAuthentication('npm.pkg.github.com', 'false');
     expect(fs.statSync(rcFile)).toBeDefined();
-    const rc = readRcFile(rcFile);
+    let rc = readRcFile(rcFile);
     expect(rc['@ownername:registry']).toBe('npm.pkg.github.com/');
     expect(rc['always-auth']).toBe('false');
     expect(process.env.NODE_AUTH_TOKEN).toEqual('foobar');
@@ -134,7 +137,7 @@ describe('authutil tests', () => {
   it('configAuthentication should overwrite non-scoped with non-scoped', async () => {
     fs.writeFileSync(rcFile, 'registry=NNN');
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     expect(contents).toBe(
       `//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}${os.EOL}registry=https://registry.npmjs.org/${os.EOL}always-auth=true`
     );
@@ -143,7 +146,7 @@ describe('authutil tests', () => {
   it('configAuthentication should overwrite only non-scoped', async () => {
     fs.writeFileSync(rcFile, `registry=NNN${os.EOL}@myscope:registry=MMM`);
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     expect(contents).toBe(
       `@myscope:registry=MMM${os.EOL}//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}${os.EOL}registry=https://registry.npmjs.org/${os.EOL}always-auth=true`
     );
@@ -152,7 +155,7 @@ describe('authutil tests', () => {
   it('configAuthentication should add non-scoped to scoped', async () => {
     fs.writeFileSync(rcFile, '@myscope:registry=NNN');
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     expect(contents).toBe(
       `@myscope:registry=NNN${os.EOL}//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}${os.EOL}registry=https://registry.npmjs.org/${os.EOL}always-auth=true`
     );
@@ -162,7 +165,7 @@ describe('authutil tests', () => {
     process.env['INPUT_SCOPE'] = 'myscope';
     fs.writeFileSync(rcFile, `@myscope:registry=NNN`);
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     expect(contents).toBe(
       `//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}${os.EOL}@myscope:registry=https://registry.npmjs.org/${os.EOL}always-auth=true`
     );
@@ -172,7 +175,7 @@ describe('authutil tests', () => {
     process.env['INPUT_SCOPE'] = 'myscope';
     fs.writeFileSync(rcFile, `registry=NNN${os.EOL}@myscope:registry=MMM`);
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     expect(contents).toBe(
       `registry=NNN${os.EOL}//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}${os.EOL}@myscope:registry=https://registry.npmjs.org/${os.EOL}always-auth=true`
     );
@@ -182,7 +185,7 @@ describe('authutil tests', () => {
     process.env['INPUT_SCOPE'] = 'myscope';
     fs.writeFileSync(rcFile, `registry=MMM`);
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     expect(contents).toBe(
       `registry=MMM${os.EOL}//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}${os.EOL}@myscope:registry=https://registry.npmjs.org/${os.EOL}always-auth=true`
     );
@@ -195,7 +198,7 @@ describe('authutil tests', () => {
       `@otherscope:registry=NNN${os.EOL}@myscope:registry=MMM`
     );
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     expect(contents).toBe(
       `@otherscope:registry=NNN${os.EOL}//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}${os.EOL}@myscope:registry=https://registry.npmjs.org/${os.EOL}always-auth=true`
     );
@@ -205,7 +208,7 @@ describe('authutil tests', () => {
     process.env['INPUT_SCOPE'] = 'myscope';
     fs.writeFileSync(rcFile, `@otherscope:registry=MMM`);
     await auth.configAuthentication('https://registry.npmjs.org/', 'true');
-    const contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
+    let contents = fs.readFileSync(rcFile, {encoding: 'utf8'});
     expect(contents).toBe(
       `@otherscope:registry=MMM${os.EOL}//registry.npmjs.org/:_authToken=\${NODE_AUTH_TOKEN}${os.EOL}@myscope:registry=https://registry.npmjs.org/${os.EOL}always-auth=true`
     );

__tests__/cache-restore.test.ts

@@ -32,13 +32,13 @@ describe('cache-restore', () => {
 
   function findCacheFolder(command: string) {
     switch (command) {
-      case 'npm config get cache':
+      case utils.supportedPackageManagers.npm.getCacheFolderCommand:
         return npmCachePath;
-      case 'pnpm store path --silent':
+      case utils.supportedPackageManagers.pnpm.getCacheFolderCommand:
         return pnpmCachePath;
-      case 'yarn cache dir':
+      case utils.supportedPackageManagers.yarn1.getCacheFolderCommand:
         return yarn1CachePath;
-      case 'yarn config get cacheFolder':
+      case utils.supportedPackageManagers.yarn2.getCacheFolderCommand:
         return yarn2CachePath;
       default:
         return 'packge/not/found';
@@ -108,7 +108,7 @@ describe('cache-restore', () => {
     it.each([['npm7'], ['npm6'], ['pnpm6'], ['yarn1'], ['yarn2'], ['random']])(
       'Throw an error because %s is not supported',
       async packageManager => {
-        await expect(restoreCache(packageManager, '')).rejects.toThrow(
+        await expect(restoreCache(packageManager)).rejects.toThrowError(
           `Caching for '${packageManager}' is not supported`
         );
       }
@@ -132,7 +132,7 @@ describe('cache-restore', () => {
           }
         });
 
-        await restoreCache(packageManager, '', '0');
+        await restoreCache(packageManager, undefined, '0');
         expect(hashFilesSpy).toHaveBeenCalled();
         expect(infoSpy).toHaveBeenCalledWith(
           `Cache restored from key: ${platform}-0-setup-node-${packageManager}-${fileHash}`
@@ -163,7 +163,7 @@ describe('cache-restore', () => {
         });
 
         restoreCacheSpy.mockImplementationOnce(() => undefined);
-        await restoreCache(packageManager, '', '0');
+        await restoreCache(packageManager, undefined, '0');
         expect(hashFilesSpy).toHaveBeenCalled();
         expect(infoSpy).toHaveBeenCalledWith(
           `Cache not found for input keys: ${platform}-0-setup-node-${packageManager}-${fileHash}, ${platform}-0-setup-node-${packageManager}-`

__tests__/cache-save.test.ts

@@ -18,7 +18,7 @@ describe('run', () => {
   const commonPath = '/some/random/path';
   process.env['GITHUB_WORKSPACE'] = path.join(__dirname, 'data');
 
-  const inputs = {} as any;
+  let inputs = {} as any;
 
   let getInputSpy: jest.SpyInstance;
   let infoSpy: jest.SpyInstance;
@@ -92,9 +92,6 @@ describe('run', () => {
 
     it('Package manager is not valid, skip caching', async () => {
       inputs['cache'] = 'yarn3';
-      getStateSpy.mockImplementation(key =>
-        key === State.CachePackageManager ? inputs['cache'] : ''
-      );
 
       await run();
 
@@ -110,22 +107,18 @@ describe('run', () => {
   describe('Validate unchanged cache is not saved', () => {
     it('should not save cache for yarn1', async () => {
       inputs['cache'] = 'yarn';
-      getStateSpy.mockImplementation(key =>
+      getStateSpy.mockImplementation(() => yarnFileHash);
-        key === State.CachePackageManager
+      getCommandOutputSpy
-          ? inputs['cache']
+        .mockImplementationOnce(() => '1.2.3')
-          : key === State.CachePrimaryKey || key === State.CacheMatchedKey
+        .mockImplementationOnce(() => `${commonPath}/yarn1`);
-          ? yarnFileHash
-          : key === State.CachePaths
-          ? '["/foo/bar"]'
-          : 'not expected'
-      );
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(2);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`yarn path is ${commonPath}/yarn1`);
+      expect(debugSpy).toHaveBeenCalledWith('Consumed yarn version is 1.2.3');
       expect(infoSpy).toHaveBeenCalledWith(
         `Cache hit occurred on the primary key ${yarnFileHash}, not saving cache.`
       );
@@ -134,22 +127,18 @@ describe('run', () => {
 
     it('should not save cache for yarn2', async () => {
       inputs['cache'] = 'yarn';
-      getStateSpy.mockImplementation(key =>
+      getStateSpy.mockImplementation(() => yarnFileHash);
-        key === State.CachePackageManager
+      getCommandOutputSpy
-          ? inputs['cache']
+        .mockImplementationOnce(() => '2.2.3')
-          : key === State.CachePrimaryKey || key === State.CacheMatchedKey
+        .mockImplementationOnce(() => `${commonPath}/yarn2`);
-          ? yarnFileHash
-          : key === State.CachePaths
-          ? '["/foo/bar"]'
-          : 'not expected'
-      );
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(2);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`yarn path is ${commonPath}/yarn2`);
+      expect(debugSpy).toHaveBeenCalledWith('Consumed yarn version is 2.2.3');
       expect(infoSpy).toHaveBeenCalledWith(
         `Cache hit occurred on the primary key ${yarnFileHash}, not saving cache.`
       );
@@ -158,44 +147,35 @@ describe('run', () => {
 
     it('should not save cache for npm', async () => {
       inputs['cache'] = 'npm';
-      getStateSpy.mockImplementation(key =>
+      getStateSpy.mockImplementation(() => npmFileHash);
-        key === State.CachePackageManager
-          ? inputs['cache']
-          : key === State.CachePrimaryKey || key === State.CacheMatchedKey
-          ? yarnFileHash
-          : key === State.CachePaths
-          ? '["/foo/bar"]'
-          : 'not expected'
-      );
       getCommandOutputSpy.mockImplementationOnce(() => `${commonPath}/npm`);
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(1);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`npm path is ${commonPath}/npm`);
+      expect(infoSpy).toHaveBeenCalledWith(
+        `Cache hit occurred on the primary key ${npmFileHash}, not saving cache.`
+      );
       expect(setFailedSpy).not.toHaveBeenCalled();
     });
 
     it('should not save cache for pnpm', async () => {
       inputs['cache'] = 'pnpm';
-      getStateSpy.mockImplementation(key =>
+      getStateSpy.mockImplementation(() => pnpmFileHash);
-        key === State.CachePackageManager
+      getCommandOutputSpy.mockImplementationOnce(() => `${commonPath}/pnpm`);
-          ? inputs['cache']
-          : key === State.CachePrimaryKey || key === State.CacheMatchedKey
-          ? yarnFileHash
-          : key === State.CachePaths
-          ? '["/foo/bar"]'
-          : 'not expected'
-      );
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(1);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`pnpm path is ${commonPath}/pnpm`);
+      expect(infoSpy).toHaveBeenCalledWith(
+        `Cache hit occurred on the primary key ${pnpmFileHash}, not saving cache.`
+      );
       expect(setFailedSpy).not.toHaveBeenCalled();
     });
   });
@@ -203,24 +183,24 @@ describe('run', () => {
   describe('action saves the cache', () => {
     it('saves cache from yarn 1', async () => {
       inputs['cache'] = 'yarn';
-      getStateSpy.mockImplementation((key: string) =>
+      getStateSpy.mockImplementation((name: string) => {
-        key === State.CachePackageManager
+        if (name === State.CacheMatchedKey) {
-          ? inputs['cache']
+          return yarnFileHash;
-          : key === State.CacheMatchedKey
+        } else {
-          ? yarnFileHash
+          return npmFileHash;
-          : key === State.CachePrimaryKey
+        }
-          ? npmFileHash
+      });
-          : key === State.CachePaths
+      getCommandOutputSpy
-          ? '["/foo/bar"]'
+        .mockImplementationOnce(() => '1.2.3')
-          : 'not expected'
+        .mockImplementationOnce(() => `${commonPath}/yarn1`);
-      );
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(2);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`yarn path is ${commonPath}/yarn1`);
+      expect(debugSpy).toHaveBeenCalledWith('Consumed yarn version is 1.2.3');
       expect(infoSpy).not.toHaveBeenCalledWith(
         `Cache hit occurred on the primary key ${yarnFileHash}, not saving cache.`
       );
@@ -233,24 +213,24 @@ describe('run', () => {
 
     it('saves cache from yarn 2', async () => {
       inputs['cache'] = 'yarn';
-      getStateSpy.mockImplementation((key: string) =>
+      getStateSpy.mockImplementation((name: string) => {
-        key === State.CachePackageManager
+        if (name === State.CacheMatchedKey) {
-          ? inputs['cache']
+          return yarnFileHash;
-          : key === State.CacheMatchedKey
+        } else {
-          ? yarnFileHash
+          return npmFileHash;
-          : key === State.CachePrimaryKey
+        }
-          ? npmFileHash
+      });
-          : key === State.CachePaths
+      getCommandOutputSpy
-          ? '["/foo/bar"]'
+        .mockImplementationOnce(() => '2.2.3')
-          : 'not expected'
+        .mockImplementationOnce(() => `${commonPath}/yarn2`);
-      );
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(2);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`yarn path is ${commonPath}/yarn2`);
+      expect(debugSpy).toHaveBeenCalledWith('Consumed yarn version is 2.2.3');
       expect(infoSpy).not.toHaveBeenCalledWith(
         `Cache hit occurred on the primary key ${yarnFileHash}, not saving cache.`
       );
@@ -263,24 +243,21 @@ describe('run', () => {
 
     it('saves cache from npm', async () => {
       inputs['cache'] = 'npm';
-      getStateSpy.mockImplementation((key: string) =>
+      getStateSpy.mockImplementation((name: string) => {
-        key === State.CachePackageManager
+        if (name === State.CacheMatchedKey) {
-          ? inputs['cache']
+          return npmFileHash;
-          : key === State.CacheMatchedKey
+        } else {
-          ? npmFileHash
+          return yarnFileHash;
-          : key === State.CachePrimaryKey
+        }
-          ? yarnFileHash
+      });
-          : key === State.CachePaths
+      getCommandOutputSpy.mockImplementationOnce(() => `${commonPath}/npm`);
-          ? '["/foo/bar"]'
-          : 'not expected'
-      );
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(1);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`npm path is ${commonPath}/npm`);
       expect(infoSpy).not.toHaveBeenCalledWith(
         `Cache hit occurred on the primary key ${npmFileHash}, not saving cache.`
       );
@@ -293,24 +270,21 @@ describe('run', () => {
 
     it('saves cache from pnpm', async () => {
       inputs['cache'] = 'pnpm';
-      getStateSpy.mockImplementation((key: string) =>
+      getStateSpy.mockImplementation((name: string) => {
-        key === State.CachePackageManager
+        if (name === State.CacheMatchedKey) {
-          ? inputs['cache']
+          return pnpmFileHash;
-          : key === State.CacheMatchedKey
+        } else {
-          ? pnpmFileHash
+          return npmFileHash;
-          : key === State.CachePrimaryKey
+        }
-          ? npmFileHash
+      });
-          : key === State.CachePaths
+      getCommandOutputSpy.mockImplementationOnce(() => `${commonPath}/pnpm`);
-          ? '["/foo/bar"]'
-          : 'not expected'
-      );
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(1);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`pnpm path is ${commonPath}/pnpm`);
       expect(infoSpy).not.toHaveBeenCalledWith(
         `Cache hit occurred on the primary key ${pnpmFileHash}, not saving cache.`
       );
@@ -323,27 +297,24 @@ describe('run', () => {
 
     it('save with -1 cacheId , should not fail workflow', async () => {
       inputs['cache'] = 'npm';
-      getStateSpy.mockImplementation((key: string) =>
+      getStateSpy.mockImplementation((name: string) => {
-        key === State.CachePackageManager
+        if (name === State.CacheMatchedKey) {
-          ? inputs['cache']
+          return npmFileHash;
-          : key === State.CacheMatchedKey
+        } else {
-          ? npmFileHash
+          return yarnFileHash;
-          : key === State.CachePrimaryKey
+        }
-          ? yarnFileHash
+      });
-          : key === State.CachePaths
+      getCommandOutputSpy.mockImplementationOnce(() => `${commonPath}/npm`);
-          ? '["/foo/bar"]'
-          : 'not expected'
-      );
       saveCacheSpy.mockImplementation(() => {
         return -1;
       });
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(1);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`npm path is ${commonPath}/npm`);
       expect(infoSpy).not.toHaveBeenCalledWith(
         `Cache hit occurred on the primary key ${npmFileHash}, not saving cache.`
       );
@@ -356,27 +327,24 @@ describe('run', () => {
 
     it('saves with error from toolkit, should fail workflow', async () => {
       inputs['cache'] = 'npm';
-      getStateSpy.mockImplementation((key: string) =>
+      getStateSpy.mockImplementation((name: string) => {
-        key === State.CachePackageManager
+        if (name === State.CacheMatchedKey) {
-          ? inputs['cache']
+          return npmFileHash;
-          : key === State.CacheMatchedKey
+        } else {
-          ? npmFileHash
+          return yarnFileHash;
-          : key === State.CachePrimaryKey
+        }
-          ? yarnFileHash
+      });
-          : key === State.CachePaths
+      getCommandOutputSpy.mockImplementationOnce(() => `${commonPath}/npm`);
-          ? '["/foo/bar"]'
-          : 'not expected'
-      );
       saveCacheSpy.mockImplementation(() => {
         throw new cache.ValidationError('Validation failed');
       });
 
       await run();
 
-      expect(getInputSpy).not.toHaveBeenCalled();
+      expect(getInputSpy).toHaveBeenCalled();
-      expect(getStateSpy).toHaveBeenCalledTimes(4);
+      expect(getStateSpy).toHaveBeenCalledTimes(2);
-      expect(getCommandOutputSpy).toHaveBeenCalledTimes(0);
+      expect(getCommandOutputSpy).toHaveBeenCalledTimes(1);
-      expect(debugSpy).toHaveBeenCalledTimes(0);
+      expect(debugSpy).toHaveBeenCalledWith(`npm path is ${commonPath}/npm`);
       expect(infoSpy).not.toHaveBeenCalledWith(
         `Cache hit occurred on the primary key ${npmFileHash}, not saving cache.`
       );

__tests__/cache-utils.test.ts

@@ -2,18 +2,7 @@ import * as core from '@actions/core';
 import * as cache from '@actions/cache';
 import path from 'path';
 import * as utils from '../src/cache-utils';
-import {
+import {PackageManagerInfo, isCacheFeatureAvailable} from '../src/cache-utils';
-  PackageManagerInfo,
-  isCacheFeatureAvailable,
-  supportedPackageManagers,
-  getCommandOutput,
-  resetProjectDirectoriesMemoized
-} from '../src/cache-utils';
-import fs from 'fs';
-import * as cacheUtils from '../src/cache-utils';
-import * as glob from '@actions/glob';
-import {Globber} from '@actions/glob';
-import {MockGlobber} from './mock/glob-mock';
 
 describe('cache-utils', () => {
   const versionYarn1 = '1.2.3';
@@ -23,10 +12,8 @@ describe('cache-utils', () => {
   let isFeatureAvailable: jest.SpyInstance;
   let info: jest.SpyInstance;
   let warningSpy: jest.SpyInstance;
-  let fsRealPathSyncSpy: jest.SpyInstance;
 
   beforeEach(() => {
-    console.log('::stop-commands::stoptoken');
     process.env['GITHUB_WORKSPACE'] = path.join(__dirname, 'data');
     debugSpy = jest.spyOn(core, 'debug');
     debugSpy.mockImplementation(msg => {});
@@ -37,29 +24,13 @@ describe('cache-utils', () => {
     isFeatureAvailable = jest.spyOn(cache, 'isFeatureAvailable');
 
     getCommandOutputSpy = jest.spyOn(utils, 'getCommandOutput');
-
-    fsRealPathSyncSpy = jest.spyOn(fs, 'realpathSync');
-    fsRealPathSyncSpy.mockImplementation(dirName => {
-      return dirName;
   });
-  });
-
-  afterEach(() => {
-    jest.resetAllMocks();
-    jest.clearAllMocks();
-    //jest.restoreAllMocks();
-  });
-
-  afterAll(async () => {
-    console.log('::stoptoken::');
-    jest.restoreAllMocks();
-  }, 100000);
 
   describe('getPackageManagerInfo', () => {
     it.each<[string, PackageManagerInfo | null]>([
       ['npm', utils.supportedPackageManagers.npm],
       ['pnpm', utils.supportedPackageManagers.pnpm],
-      ['yarn', utils.supportedPackageManagers.yarn],
+      ['yarn', utils.supportedPackageManagers.yarn1],
       ['yarn1', null],
       ['yarn2', null],
       ['npm7', null]
@@ -101,263 +72,4 @@ describe('cache-utils', () => {
     jest.resetAllMocks();
     jest.clearAllMocks();
   });
-
-  describe('getCacheDirectoriesPaths', () => {
-    let existsSpy: jest.SpyInstance;
-    let lstatSpy: jest.SpyInstance;
-    let globCreateSpy: jest.SpyInstance;
-
-    beforeEach(() => {
-      existsSpy = jest.spyOn(fs, 'existsSync');
-      existsSpy.mockImplementation(() => true);
-
-      lstatSpy = jest.spyOn(fs, 'lstatSync');
-      lstatSpy.mockImplementation(arg => ({
-        isDirectory: () => true
-      }));
-
-      globCreateSpy = jest.spyOn(glob, 'create');
-
-      globCreateSpy.mockImplementation(
-        (pattern: string): Promise<Globber> =>
-          MockGlobber.create(['/foo', '/bar'])
-      );
-
-      resetProjectDirectoriesMemoized();
-    });
-
-    afterEach(() => {
-      existsSpy.mockRestore();
-      lstatSpy.mockRestore();
-      globCreateSpy.mockRestore();
-    });
-
-    it.each([
-      [supportedPackageManagers.npm, ''],
-      [supportedPackageManagers.npm, '/dir/file.lock'],
-      [supportedPackageManagers.npm, '/**/file.lock'],
-      [supportedPackageManagers.pnpm, ''],
-      [supportedPackageManagers.pnpm, '/dir/file.lock'],
-      [supportedPackageManagers.pnpm, '/**/file.lock']
-    ])(
-      'getCacheDirectoriesPaths should return one dir for non yarn',
-      async (packageManagerInfo, cacheDependency) => {
-        getCommandOutputSpy.mockImplementation(() => 'foo');
-
-        const dirs = await cacheUtils.getCacheDirectories(
-          packageManagerInfo,
-          cacheDependency
-        );
-        expect(dirs).toEqual(['foo']);
-        // to do not call for a version
-        // call once for get cache folder
-        expect(getCommandOutputSpy).toHaveBeenCalledTimes(1);
-      }
-    );
-
-    it('getCacheDirectoriesPaths should return one dir for yarn without cacheDependency', async () => {
-      getCommandOutputSpy.mockImplementation(() => 'foo');
-
-      const dirs = await cacheUtils.getCacheDirectories(
-        supportedPackageManagers.yarn,
-        ''
-      );
-      expect(dirs).toEqual(['foo']);
-    });
-
-    it.each([
-      [supportedPackageManagers.npm, ''],
-      [supportedPackageManagers.npm, '/dir/file.lock'],
-      [supportedPackageManagers.npm, '/**/file.lock'],
-      [supportedPackageManagers.pnpm, ''],
-      [supportedPackageManagers.pnpm, '/dir/file.lock'],
-      [supportedPackageManagers.pnpm, '/**/file.lock'],
-      [supportedPackageManagers.yarn, ''],
-      [supportedPackageManagers.yarn, '/dir/file.lock'],
-      [supportedPackageManagers.yarn, '/**/file.lock']
-    ])(
-      'getCacheDirectoriesPaths should throw for getCommandOutput returning empty',
-      async (packageManagerInfo, cacheDependency) => {
-        getCommandOutputSpy.mockImplementation((command: string) =>
-          // return empty string to indicate getCacheFolderPath failed
-          //        --version still works
-          command.includes('version') ? '1.' : ''
-        );
-
-        await expect(
-          cacheUtils.getCacheDirectories(packageManagerInfo, cacheDependency)
-        ).rejects.toThrow(); //'Could not get cache folder path for /dir');
-      }
-    );
-
-    it.each([
-      [supportedPackageManagers.yarn, '/dir/file.lock'],
-      [supportedPackageManagers.yarn, '/**/file.lock']
-    ])(
-      'getCacheDirectoriesPaths should nothrow in case of having not directories',
-      async (packageManagerInfo, cacheDependency) => {
-        lstatSpy.mockImplementation(arg => ({
-          isDirectory: () => false
-        }));
-
-        await cacheUtils.getCacheDirectories(
-          packageManagerInfo,
-          cacheDependency
-        );
-        expect(warningSpy).toHaveBeenCalledTimes(1);
-        expect(warningSpy).toHaveBeenCalledWith(
-          `No existing directories found containing cache-dependency-path="${cacheDependency}"`
-        );
-      }
-    );
-
-    it.each(['1.1.1', '2.2.2'])(
-      'getCacheDirectoriesPaths yarn v%s should return one dir without cacheDependency',
-      async version => {
-        getCommandOutputSpy.mockImplementationOnce(() => version);
-        getCommandOutputSpy.mockImplementationOnce(() => `foo${version}`);
-
-        const dirs = await cacheUtils.getCacheDirectories(
-          supportedPackageManagers.yarn,
-          ''
-        );
-        expect(dirs).toEqual([`foo${version}`]);
-      }
-    );
-
-    it.each(['1.1.1', '2.2.2'])(
-      'getCacheDirectoriesPaths yarn v%s should return 2 dirs with globbed cacheDependency',
-      async version => {
-        let dirNo = 1;
-        getCommandOutputSpy.mockImplementation((command: string) =>
-          command.includes('version') ? version : `file_${version}_${dirNo++}`
-        );
-        globCreateSpy.mockImplementation(
-          (pattern: string): Promise<Globber> =>
-            MockGlobber.create(['/tmp/dir1/file', '/tmp/dir2/file'])
-        );
-
-        const dirs = await cacheUtils.getCacheDirectories(
-          supportedPackageManagers.yarn,
-          '/tmp/**/file'
-        );
-        expect(dirs).toEqual([`file_${version}_1`, `file_${version}_2`]);
-      }
-    );
-
-    it.each(['1.1.1', '2.2.2'])(
-      'getCacheDirectoriesPaths yarn v%s should return 2 dirs  with globbed cacheDependency expanding to duplicates',
-      async version => {
-        let dirNo = 1;
-        getCommandOutputSpy.mockImplementation((command: string) =>
-          command.includes('version') ? version : `file_${version}_${dirNo++}`
-        );
-        globCreateSpy.mockImplementation(
-          (pattern: string): Promise<Globber> =>
-            MockGlobber.create([
-              '/tmp/dir1/file',
-              '/tmp/dir2/file',
-              '/tmp/dir1/file'
-            ])
-        );
-
-        const dirs = await cacheUtils.getCacheDirectories(
-          supportedPackageManagers.yarn,
-          '/tmp/**/file'
-        );
-        expect(dirs).toEqual([`file_${version}_1`, `file_${version}_2`]);
-      }
-    );
-
-    it.each(['1.1.1', '2.2.2'])(
-      'getCacheDirectoriesPaths yarn v%s should return 2 uniq dirs despite duplicate cache directories',
-      async version => {
-        let dirNo = 1;
-        getCommandOutputSpy.mockImplementation((command: string) =>
-          command.includes('version')
-            ? version
-            : `file_${version}_${dirNo++ % 2}`
-        );
-        globCreateSpy.mockImplementation(
-          (pattern: string): Promise<Globber> =>
-            MockGlobber.create([
-              '/tmp/dir1/file',
-              '/tmp/dir2/file',
-              '/tmp/dir3/file'
-            ])
-        );
-
-        const dirs = await cacheUtils.getCacheDirectories(
-          supportedPackageManagers.yarn,
-          '/tmp/**/file'
-        );
-        expect(dirs).toEqual([`file_${version}_1`, `file_${version}_0`]);
-        expect(getCommandOutputSpy).toHaveBeenCalledTimes(6);
-        expect(getCommandOutputSpy).toHaveBeenCalledWith(
-          'yarn --version',
-          '/tmp/dir1'
-        );
-        expect(getCommandOutputSpy).toHaveBeenCalledWith(
-          'yarn --version',
-          '/tmp/dir2'
-        );
-        expect(getCommandOutputSpy).toHaveBeenCalledWith(
-          'yarn --version',
-          '/tmp/dir3'
-        );
-        expect(getCommandOutputSpy).toHaveBeenCalledWith(
-          version.startsWith('1.')
-            ? 'yarn cache dir'
-            : 'yarn config get cacheFolder',
-          '/tmp/dir1'
-        );
-        expect(getCommandOutputSpy).toHaveBeenCalledWith(
-          version.startsWith('1.')
-            ? 'yarn cache dir'
-            : 'yarn config get cacheFolder',
-          '/tmp/dir2'
-        );
-        expect(getCommandOutputSpy).toHaveBeenCalledWith(
-          version.startsWith('1.')
-            ? 'yarn cache dir'
-            : 'yarn config get cacheFolder',
-          '/tmp/dir3'
-        );
-      }
-    );
-
-    it.each(['1.1.1', '2.2.2'])(
-      'getCacheDirectoriesPaths yarn v%s should return 4 dirs with multiple globs',
-      async version => {
-        // simulate wrong indents
-        const cacheDependencyPath = `/tmp/dir1/file
-          /tmp/dir2/file
-/tmp/**/file
-          `;
-        globCreateSpy.mockImplementation(
-          (pattern: string): Promise<Globber> =>
-            MockGlobber.create([
-              '/tmp/dir1/file',
-              '/tmp/dir2/file',
-              '/tmp/dir3/file',
-              '/tmp/dir4/file'
-            ])
-        );
-        let dirNo = 1;
-        getCommandOutputSpy.mockImplementation((command: string) =>
-          command.includes('version') ? version : `file_${version}_${dirNo++}`
-        );
-        const dirs = await cacheUtils.getCacheDirectories(
-          supportedPackageManagers.yarn,
-          cacheDependencyPath
-        );
-        expect(dirs).toEqual([
-          `file_${version}_1`,
-          `file_${version}_2`,
-          `file_${version}_3`,
-          `file_${version}_4`
-        ]);
-      }
-    );
-  });
 });

__tests__/canary-installer.test.ts

@@ -12,11 +12,11 @@ import * as main from '../src/main';
 import * as auth from '../src/authutil';
 import {INodeVersion} from '../src/distributions/base-models';
 
-import nodeTestManifest from './data/versions-manifest.json';
+const nodeTestManifest = require('./data/versions-manifest.json');
-import nodeTestDist from './data/node-dist-index.json';
+const nodeTestDist = require('./data/node-dist-index.json');
-import nodeTestDistNightly from './data/node-nightly-index.json';
+const nodeTestDistNightly = require('./data/node-nightly-index.json');
-import nodeTestDistRc from './data/node-rc-index.json';
+const nodeTestDistRc = require('./data/node-rc-index.json');
-import nodeV8CanaryTestDist from './data/v8-canary-dist-index.json';
+const nodeV8CanaryTestDist = require('./data/v8-canary-dist-index.json');
 
 describe('setup-node', () => {
   let inputs = {} as any;
@@ -95,13 +95,13 @@ describe('setup-node', () => {
     getJsonSpy.mockImplementation(url => {
       let res: any;
       if (url.includes('/rc')) {
-        res = <INodeVersion[]>nodeTestDistRc;
+        res = <INodeVersion>nodeTestDistRc;
       } else if (url.includes('/nightly')) {
-        res = <INodeVersion[]>nodeTestDistNightly;
+        res = <INodeVersion>nodeTestDistNightly;
       } else if (url.includes('/v8-canary')) {
-        res = <INodeVersion[]>nodeV8CanaryTestDist;
+        res = <INodeVersion>nodeV8CanaryTestDist;
       } else {
-        res = <INodeVersion[]>nodeTestDist;
+        res = <INodeVersion>nodeTestDist;
       }
 
       return {result: res};
@@ -154,7 +154,7 @@ describe('setup-node', () => {
     os['arch'] = 'x64';
     inputs.stable = 'true';
 
-    const toolPath = path.normalize(
+    let toolPath = path.normalize(
       '/cache/node/20.0.0-v8-canary20221103f7e2421e91/x64'
     );
     findSpy.mockImplementation(() => toolPath);
@@ -180,7 +180,7 @@ describe('setup-node', () => {
 
     inSpy.mockImplementation(name => inputs[name]);
 
-    const toolPath = path.normalize(
+    let toolPath = path.normalize(
       '/cache/node/20.0.0-v8-canary20221103f7e2421e91/x64'
     );
     findSpy.mockImplementation(() => toolPath);
@@ -192,13 +192,13 @@ describe('setup-node', () => {
     ]);
     await main.run();
 
-    const expPath = path.join(toolPath, 'bin');
+    let expPath = path.join(toolPath, 'bin');
     expect(cnSpy).toHaveBeenCalledWith(`::add-path::${expPath}${osm.EOL}`);
   });
 
   it('handles unhandled find error and reports error', async () => {
     os.platform = 'linux';
-    const errMsg = 'unhandled error message';
+    let errMsg = 'unhandled error message';
     inputs['node-version'] = '20.0.0-v8-canary20221103f7e2421e91';
 
     findSpy.mockImplementation(() => {
@@ -224,7 +224,7 @@ describe('setup-node', () => {
     os.arch = 'x64';
 
     // a version which is not in the manifest but is in node dist
-    const versionSpec = '11.15.0';
+    let versionSpec = '11.15.0';
 
     inputs['node-version'] = versionSpec;
     inputs['always-auth'] = false;
@@ -234,13 +234,13 @@ describe('setup-node', () => {
     findSpy.mockImplementation(() => '');
 
     dlSpy.mockImplementation(async () => '/some/temp/path');
-    const toolPath = path.normalize('/cache/node/11.11.0/x64');
+    let toolPath = path.normalize('/cache/node/11.11.0/x64');
     exSpy.mockImplementation(async () => '/some/other/temp/path');
     cacheSpy.mockImplementation(async () => toolPath);
 
     await main.run();
 
-    const expPath = path.join(toolPath, 'bin');
+    let expPath = path.join(toolPath, 'bin');
 
     expect(dlSpy).toHaveBeenCalled();
     expect(exSpy).toHaveBeenCalled();
@@ -257,7 +257,7 @@ describe('setup-node', () => {
     os.platform = 'linux';
     os.arch = 'x64';
 
-    const versionSpec = '23.0.0-v8-canary20221103f7e2421e91';
+    let versionSpec = '23.0.0-v8-canary20221103f7e2421e91';
     inputs['node-version'] = versionSpec;
 
     findSpy.mockImplementation(() => '');
@@ -275,12 +275,12 @@ describe('setup-node', () => {
   });
 
   it('reports a failed download', async () => {
-    const errMsg = 'unhandled download message';
+    let errMsg = 'unhandled download message';
     os.platform = 'linux';
     os.arch = 'x64';
 
     // a version which is in the manifest
-    const versionSpec = '19.0.0-v8-canary';
+    let versionSpec = '19.0.0-v8-canary';
 
     inputs['node-version'] = versionSpec;
     inputs['always-auth'] = false;
@@ -327,14 +327,14 @@ describe('setup-node', () => {
       inputs['always-auth'] = false;
       inputs['token'] = 'faketoken';
 
-      const expectedUrl = `https://nodejs.org/download/v8-canary/v${version}/node-v${version}-${platform}-${arch}.${fileExtension}`;
+      let expectedUrl = `https://nodejs.org/download/v8-canary/v${version}/node-v${version}-${platform}-${arch}.${fileExtension}`;
 
       // ... but not in the local cache
       findSpy.mockImplementation(() => '');
       findAllVersionsSpy.mockImplementation(() => []);
 
       dlSpy.mockImplementation(async () => '/some/temp/path');
-      const toolPath = path.normalize(`/cache/node/${version}/${arch}`);
+      let toolPath = path.normalize(`/cache/node/${version}/${arch}`);
       exSpy.mockImplementation(async () => '/some/other/temp/path');
       cacheSpy.mockImplementation(async () => toolPath);
 
@@ -502,7 +502,7 @@ describe('setup-node', () => {
 
   describe('setup-node v8 canary tests', () => {
     it('v8 canary setup node flow with cached', async () => {
-      const versionSpec = 'v20-v8-canary';
+      let versionSpec = 'v20-v8-canary';
 
       inputs['node-version'] = versionSpec;
       inputs['always-auth'] = false;

__tests__/data/.tool-versions-node

@@ -1 +0,0 @@
-node 14.0.0

__tests__/data/package-volta-extends.json

@@ -1,5 +0,0 @@
-{
-  "volta": {
-    "extends": "./package-volta.json"
-  }
-}

__tests__/data/package-volta.json

@@ -1,8 +0,0 @@
-{
-  "engines": {
-    "node": "^14.0.0"
-  },
-  "volta": {
-    "node": "16.0.0"
-  }
-}

__tests__/data/package.json

@@ -1,5 +1,8 @@
 {
   "engines": {
     "node": "^14.0.0"
+  },
+  "volta": {
+    "node": "16.0.0"
   }
 }