Add Publish Immutable Action (#584)

* Upgrade-actions/cache for v3

* Upgrade actions/upload-artifact from v3 to v4

* audit fix

* Replaced macos-latest with macos-13

* fix check failures

.eslintignore

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

.eslintrc.js

@@ -1,50 +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-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}]
-  },
-  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,2 @@
-* text=auto eol=lf
+* text=auto
 .licenses/** -diff linguist-generated=true

.github/ISSUE_TEMPLATE/config.yml

@@ -1 +1 @@
-blank_issues_enabled: false
+blank_issues_enabled: false

.github/workflows/basic-validation.yml

@@ -1,16 +0,0 @@
-name: Basic validation
-
-on:
-  push:
-    branches:
-      - main
-    paths-ignore:
-      - '**.md'
-  pull_request:
-    paths-ignore:
-      - '**.md'
-
-jobs:
-  call-basic-validation:
-    name: Basic validation
-    uses: actions/reusable-workflows/.github/workflows/basic-validation.yml@main

.github/workflows/check-dist.yml

@@ -1,3 +1,8 @@
+# `dist/index.js` is a special file in Actions.
+# When you reference an action with `uses:` in a workflow,
+# `index.js` is the code that will run.
+# For our project, we generate this file through a build process from other source files.
+# We need to make sure the checked-in `index.js` actually matches what we expect it to be.
 name: Check dist/
 
 on:
@@ -12,6 +17,36 @@ on:
   workflow_dispatch:
 
 jobs:
-  call-check-dist:
-    name: Check dist/
-    uses: actions/reusable-workflows/.github/workflows/check-dist.yml@main
+  check-dist:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Set Node.js 16.x
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+          cache: npm
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Rebuild the dist/ directory
+        run: npm run build
+
+      - name: Compare the expected and actual dist/ directories
+        run: |
+          if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
+            echo "Detected uncommitted changes after build.  See status below:"
+            git diff
+            exit 1
+          fi
+        id: diff
+
+      # If index.js was different than expected, upload the expected version as an artifact
+      - uses: actions/upload-artifact@v4
+        if: ${{ failure() && steps.diff.conclusion == 'failure' }}
+        with:
+          name: dist
+          path: dist/

.github/workflows/codeql-analysis.yml

@@ -1,14 +0,0 @@
-name: CodeQL analysis
-
-on:
-  push:
-    branches: [main]
-  pull_request:
-    branches: [main]
-  schedule:
-    - cron: '0 3 * * 0'
-
-jobs:
-  call-codeQL-analysis:
-    name: CodeQL analysis
-    uses: actions/reusable-workflows/.github/workflows/codeql-analysis.yml@main

.github/workflows/licensed.yml

@@ -9,6 +9,20 @@ on:
       - main
 
 jobs:
-  call-licensed:
-    name: Licensed
-    uses: actions/reusable-workflows/.github/workflows/licensed.yml@main
+  test:
+    runs-on: ubuntu-latest
+    name: Check licenses
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set Node.js 16.x
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+      - run: npm ci
+      - name: Install licensed
+        run: |
+          cd $RUNNER_TEMP
+          curl -Lfs -o licensed.tar.gz https://github.com/github/licensed/releases/download/3.4.4/licensed-3.4.4-linux-x64.tar.gz
+          sudo tar -xzf licensed.tar.gz
+          sudo mv licensed /usr/local/bin/licensed
+      - run: licensed status

.github/workflows/publish-immutable-actions.yml

@@ -0,0 +1,20 @@
+name: 'Publish Immutable Action Version'
+
+on:
+  release:
+    types: [published]
+
+jobs:
+  publish:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      id-token: write
+      packages: write
+
+    steps:
+      - name: Checking out
+        uses: actions/checkout@v4
+      - name: Publish
+        id: publish
+        uses: actions/publish-immutable-action@v0.0.4

.github/workflows/release-new-action-version.yml

@@ -1,5 +1,4 @@
 name: Release new action version
-
 on:
   release:
     types: [released]
@@ -22,7 +21,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Update the ${{ env.TAG_NAME }} tag
-        uses: actions/publish-action@v0.2.2
+        uses: actions/publish-action@v0.2.1
         with:
           source-tag: ${{ env.TAG_NAME }}
           slack-webhook: ${{ secrets.SLACK_WEBHOOK }}

.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

@@ -1,5 +1,4 @@
 name: Validate 'setup-go'
-
 on:
   push:
     branches:
@@ -42,7 +41,7 @@ jobs:
           go-version: oldstable
       - name: Verify Go
         run: go version
-
+  
   aliases-arch:
     runs-on: ${{ matrix.os }}
     strategy:
@@ -52,8 +51,8 @@ jobs:
         version: [stable, oldstable]
         architecture: [x64, x32]
         exclude:
-          - os: macos-latest
-            architecture: x32
+        - os: macos-latest
+          architecture: x32
     steps:
       - uses: actions/checkout@v3
       - name: Setup Go ${{ matrix.version }} ${{ matrix.architecture }}
@@ -107,7 +106,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest, windows-latest, macos-latest]
+        os: [ubuntu-latest, windows-latest, macos-13]
     steps:
       - uses: actions/checkout@v3
       - name: Setup Go and check latest
@@ -140,7 +139,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [macos-latest, windows-latest, ubuntu-latest]
+        os: [macos-13, windows-latest, ubuntu-latest]
         go: [1.12.16, 1.13.11, 1.14.3]
     steps:
       - name: Checkout
@@ -161,8 +160,11 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [macos-latest, windows-latest, ubuntu-latest]
-        go: [1.9, 1.8.6]
+        os: [macos-13, windows-latest, ubuntu-latest]
+        go: [1.11.12, 1.8.6]
+        exclude:
+          - os: macos-13
+            go: 1.8.6
     steps:
       - name: Checkout
         uses: actions/checkout@v3

.github/workflows/workflow.yml

@@ -0,0 +1,39 @@
+name: build-test
+on:
+  push:
+    branches:
+      - main
+    paths-ignore:
+      - '**.md'
+  pull_request:
+    paths-ignore:
+      - '**.md'
+jobs:
+  run:
+    name: Run
+    runs-on: ${{ matrix.operating-system }}
+    strategy:
+      matrix:
+        operating-system: [ubuntu-latest, windows-latest]
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Setup node 16
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16
+          cache: npm
+
+      - name: npm ci
+        run: npm ci
+
+      - name: Lint
+        run: npm run format-check
+
+      - name: npm test
+        run: npm test
+
+      - name: audit packages
+        run: npm audit --audit-level=high
+        if: matrix.operating-system == 'ubuntu-latest'

.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"
+  }

CODEOWNERS

@@ -1 +1 @@
-* @actions/setup-actions-team
+* @actions/actions-service

README.md

@@ -1,6 +1,6 @@
 # setup-go
 
-[![Basic validation](https://github.com/actions/setup-go/actions/workflows/basic-validation.yml/badge.svg)](https://github.com/actions/setup-go/actions/workflows/basic-validation.yml)
+[![build-test](https://github.com/actions/setup-go/actions/workflows/workflow.yml/badge.svg)](https://github.com/actions/setup-go/actions/workflows/workflow.yml)
 [![Validate 'setup-go'](https://github.com/actions/setup-go/actions/workflows/versions.yml/badge.svg)](https://github.com/actions/setup-go/actions/workflows/versions.yml)
 
 This action sets up a go environment for use in actions by:
@@ -8,16 +8,6 @@ This action sets up a go environment for use in actions by:
 - Optionally downloading and caching a version of Go by version and adding to `PATH`.
 - Registering problem matchers for error output.
 
-# V4
-
-The V4 edition of the action offers:
-
- - Enabled caching by default
-
-The action will try to enable caching unless the `cache` input is explicitly set to false.
-
-Please see "[Caching dependency files and build outputs](https://github.com/actions/setup-go#caching-dependency-files-and-build-outputs)" for more information.
-
 # V3
 
 The V3 edition of the action offers:
@@ -29,14 +19,9 @@ The V3 edition of the action offers:
 - stable and oldstable aliases
 - Bug Fixes (including issues around version matching and semver)
 
-The action will first check the local cache for a version match. If a version is not found locally, it will pull it from
-the `main` branch of the [go-versions](https://github.com/actions/go-versions/blob/main/versions-manifest.json)
-repository. On miss or failure, it will fall back to downloading directly
-from [go dist](https://storage.googleapis.com/golang). To change the default behavior, please use
-the [check-latest input](#check-latest-version).
+The action will first check the local cache for a version match. If a version is not found locally, it will pull it from the `main` branch of the [go-versions](https://github.com/actions/go-versions/blob/main/versions-manifest.json) repository. On miss or failure, it will fall back to downloading directly from [go dist](https://storage.googleapis.com/golang). To change the default behavior, please use the [check-latest input](#check-latest-version).
 
-**Note:** The `setup-go` action uses executable binaries which are built by Golang side. The action does not build
-golang from source code.
+**Note:** The `setup-go` action uses executable binaries which are built by Golang side. The action does not build golang from source code.
 
 Matching by [semver spec](https://github.com/npm/node-semver):
 
@@ -93,17 +78,14 @@ steps:
   - run: go run hello.go
 ```
 
+
 ## Check latest version
 
-The `check-latest` flag defaults to `false`. Use the default or set `check-latest` to `false` if you prefer stability
-and if you want to ensure a specific Go version is always used.
+The `check-latest` flag defaults to `false`. Use the default or set `check-latest` to `false` if you prefer stability and if you want to ensure a specific Go version is always used.
 
-If `check-latest` is set to `true`, the action first checks if the cached version is the latest one. If the locally
-cached version is not the most up-to-date, a Go version will then be downloaded. Set `check-latest` to `true` if you
-want the most up-to-date Go version to always be used.
+If `check-latest` is set to `true`, the action first checks if the cached version is the latest one. If the locally cached version is not the most up-to-date, a Go version will then be downloaded. Set `check-latest` to `true` if you want the most up-to-date Go version to always be used.
 
-> Setting `check-latest` to `true` has performance implications as downloading Go versions is slower than using cached
-> versions.
+> Setting `check-latest` to `true` has performance implications as downloading Go versions is slower than using cached versions.
 
 ```yaml
 steps:
@@ -117,14 +99,11 @@ steps:
 
 ## Using stable/oldstable aliases
 
-If `stable` is provided, action will get the latest stable version from
-the [`go-versions`](https://github.com/actions/go-versions/blob/main/versions-manifest.json) repository manifest.
+If `stable` is provided, action will get the latest stable version from the [`go-versions`](https://github.com/actions/go-versions/blob/main/versions-manifest.json) repository manifest. 
 
-If `oldstable` is provided, when current release is 1.19.x, action will resolve version as 1.18.x, where x is the latest
-patch release.
+If `oldstable` is provided, when current release is 1.19.x, action will resolve version as 1.18.x, where x is the latest patch release.
 
-**Note:** using these aliases will result in same version as using corresponding minor release with `check-latest` input
-set to `true`
+**Note:** using these aliases will result in same version as using corresponding minor release with `check-latest` input set to `true`
 
 ```yaml
 steps:
@@ -146,18 +125,11 @@ steps:
 
 ## Caching dependency files and build outputs:
 
-The action has a built-in functionality for caching and restoring go modules and build outputs. It
-uses [toolkit/cache](https://github.com/actions/toolkit/tree/main/packages/cache) under the hood but requires less configuration settings.
-The `cache` input is optional, and caching is turned on by default.
+The action has a built-in functionality for caching and restoring go modules and build outputs. It uses [actions/cache](https://github.com/actions/cache) under the hood but requires less configuration settings. The `cache` input is optional, and caching is turned off by default.
 
-The action defaults to search for the dependency file - go.sum in the repository root, and uses its hash as a part of
-the cache key. Use `cache-dependency-path` input for cases when multiple dependency files are used, or they are located
-in different subdirectories.
-
-If some problem that prevents success caching happens then the action issues the warning in the log and continues the execution of the pipeline. 
-
-**Caching in monorepos**
+The action defaults to search for the dependency file - go.sum in the repository root, and uses its hash as a part of the cache key. Use `cache-dependency-path` input for cases when multiple dependency files are used, or they are located in different subdirectories.
 
+**Caching without specifying dependency file path**
 ```yaml
 steps:
   - uses: actions/checkout@v3
@@ -165,28 +137,36 @@ steps:
     with:
       go-version: '1.17'
       check-latest: true
+      cache: true
+  - run: go run hello.go
+```
+
+**Caching in monorepos**
+```yaml
+steps:
+  - uses: actions/checkout@v3
+  - uses: actions/setup-go@v3
+    with:
+      go-version: '1.17'
+      check-latest: true
+      cache: true
       cache-dependency-path: subdir/go.sum
   - run: go run hello.go
   ```
-
 ## Getting go version from the go.mod file
 
-The `go-version-file` input accepts a path to a `go.mod` file or a `go.work` file that contains the version of Go to be
-used by a project. As the `go.mod` file contains only major and minor (e.g. 1.18) tags, the action will search for the
-latest available patch version sequentially in the runner's directory with the cached tools, in
-the [versions-manifest.json](https://github.com/actions/go-versions/blob/main/versions-manifest.json) file or at the go
-servers.
+The `go-version-file` input accepts a path to a `go.mod` file or a `go.work` file that contains the version of Go to be used by a project. As the `go.mod` file contains only major and minor (e.g. 1.18) tags, the action will search for the latest available patch version sequentially in the runner's directory with the cached tools, in the [version-manifest.json](https://github.com/actions/go-versions/blob/main/versions-manifest.json) file or at the go servers.
 
 If both the `go-version` and the `go-version-file` inputs are provided then the `go-version` input is used.
 > The action will search for the `go.mod` file relative to the repository root
 
 ```yaml
 steps:
-  - uses: actions/checkout@v3
-  - uses: actions/setup-go@v3
-    with:
-      go-version-file: 'path/to/go.mod'
-  - run: go version
+- uses: actions/checkout@v3
+- uses: actions/setup-go@v3
+  with:
+    go-version-file: 'path/to/go.mod'
+- run: go version
 ```
 
 ## Matrix testing
@@ -215,23 +195,13 @@ The `go-version` input supports the following syntax:
 - Specific versions: `1.15`, `1.16.1`, `1.17.0-rc.2`, `1.16.0-beta.1`
 - SemVer's version range syntax: `^1.13.1`, `>=1.18.0-rc.1`
 
-For more information about semantic versioning, please refer to [semver](https://github.com/npm/node-semver)
-documentation.
+For more information about semantic versioning, please refer to [semver](https://github.com/npm/node-semver) documentation.
 
 ## Using `setup-go` on GHES
 
-`setup-go` comes pre-installed on the appliance with GHES if Actions is enabled. When dynamically downloading Go
-distributions, `setup-go` downloads distributions from [`actions/go-versions`](https://github.com/actions/go-versions)
-on github.com (outside of the appliance). These calls to `actions/go-versions` are made via unauthenticated requests,
-which are limited
-to [60 requests per hour per IP](https://docs.github.com/en/rest/overview/resources-in-the-rest-api#rate-limiting). If
-more requests are made within the time frame, then you will start to see rate-limit errors during downloading that looks
-like: `##[error]API rate limit exceeded for...`. After that error the action will try to download versions directly
-from https://storage.googleapis.com/golang, but it also can have rate limit so it's better to put token.
+`setup-go` comes pre-installed on the appliance with GHES if Actions is enabled. When dynamically downloading Go distributions, `setup-go` downloads distributions from [`actions/go-versions`](https://github.com/actions/go-versions) on github.com (outside of the appliance). These calls to `actions/go-versions` are made via unauthenticated requests, which are limited to [60 requests per hour per IP](https://docs.github.com/en/rest/overview/resources-in-the-rest-api#rate-limiting). If more requests are made within the time frame, then you will start to see rate-limit errors during downloading that looks like: `##[error]API rate limit exceeded for...`. After that error the action will try to download versions directly from https://storage.googleapis.com/golang, but it also can have rate limit so it's better to put token.
 
-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:
+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-go@v3
@@ -240,10 +210,7 @@ with:
   go-version: 1.18
 ```
 
-If the runner is not able to access github.com, any Go 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.
+If the runner is not able to access github.com, any Go 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.
 
 # License
 

__tests__/cache-restore.test.ts

@@ -8,14 +8,14 @@ import {PackageManagerInfo} from '../src/package-managers';
 
 describe('restoreCache', () => {
   //Arrange
-  const hashFilesSpy = jest.spyOn(glob, 'hashFiles');
-  const getCacheDirectoryPathSpy = jest.spyOn(
+  let hashFilesSpy = jest.spyOn(glob, 'hashFiles');
+  let getCacheDirectoryPathSpy = jest.spyOn(
     cacheUtils,
     'getCacheDirectoryPath'
   );
-  const restoreCacheSpy = jest.spyOn(cache, 'restoreCache');
-  const infoSpy = jest.spyOn(core, 'info');
-  const setOutputSpy = jest.spyOn(core, 'setOutput');
+  let restoreCacheSpy = jest.spyOn(cache, 'restoreCache');
+  let infoSpy = jest.spyOn(core, 'info');
+  let setOutputSpy = jest.spyOn(core, 'setOutput');
 
   const versionSpec = '1.13.1';
   const packageManager = 'default';
@@ -40,13 +40,13 @@ describe('restoreCache', () => {
     });
 
     //Act + Assert
-    await expect(async () => {
+    expect(async () => {
       await cacheRestore.restoreCache(
         versionSpec,
         packageManager,
         cacheDependencyPath
       );
-    }).rejects.toThrow(
+    }).rejects.toThrowError(
       'Some specified paths were not resolved, unable to cache dependencies.'
     );
   });
@@ -71,7 +71,7 @@ describe('restoreCache', () => {
       packageManager,
       cacheDependencyPath
     );
-    expect(infoSpy).toHaveBeenCalledWith(`Cache is not found`);
+    expect(infoSpy).toBeCalledWith(`Cache is not found`);
   });
 
   it('should set output if cache hit is occured', async () => {
@@ -94,6 +94,6 @@ describe('restoreCache', () => {
       packageManager,
       cacheDependencyPath
     );
-    expect(setOutputSpy).toHaveBeenCalledWith('cache-hit', true);
+    expect(setOutputSpy).toBeCalledWith('cache-hit', true);
   });
 });

__tests__/cache-utils.test.ts

@@ -6,7 +6,7 @@ import {PackageManagerInfo} from '../src/package-managers';
 
 describe('getCommandOutput', () => {
   //Arrange
-  const getExecOutputSpy = jest.spyOn(exec, 'getExecOutput');
+  let getExecOutputSpy = jest.spyOn(exec, 'getExecOutput');
 
   it('should return trimmed stdout in case of successful exit code', async () => {
     //Arrange
@@ -36,7 +36,7 @@ describe('getCommandOutput', () => {
     });
 
     //Act + Assert
-    await expect(async () => {
+    expect(async () => {
       await cacheUtils.getCommandOutput('command');
     }).rejects.toThrow();
   });
@@ -62,7 +62,7 @@ describe('getPackageManagerInfo', () => {
     const packageManagerName = 'invalidName';
 
     //Act + Assert
-    await expect(async () => {
+    expect(async () => {
       await cacheUtils.getPackageManagerInfo(packageManagerName);
     }).rejects.toThrow();
   });
@@ -70,7 +70,7 @@ describe('getPackageManagerInfo', () => {
 
 describe('getCacheDirectoryPath', () => {
   //Arrange
-  const getExecOutputSpy = jest.spyOn(exec, 'getExecOutput');
+  let getExecOutputSpy = jest.spyOn(exec, 'getExecOutput');
 
   const validPackageManager: PackageManagerInfo = {
     dependencyFilePattern: 'go.sum',
@@ -93,41 +93,6 @@ describe('getCacheDirectoryPath', () => {
       .then(data => expect(data).toEqual(expectedResult));
   });
 
-  it('should return path to the cache folder if one command return empty str', async () => {
-    //Arrange
-    getExecOutputSpy.mockImplementationOnce((commandLine: string) => {
-      return new Promise<exec.ExecOutput>(resolve => {
-        resolve({exitCode: 0, stdout: 'path/to/cache/folder', stderr: ''});
-      });
-    });
-
-    getExecOutputSpy.mockImplementationOnce((commandLine: string) => {
-      return new Promise<exec.ExecOutput>(resolve => {
-        resolve({exitCode: 0, stdout: '', stderr: ''});
-      });
-    });
-
-    const expectedResult = ['path/to/cache/folder'];
-
-    //Act + Assert
-    return cacheUtils
-      .getCacheDirectoryPath(validPackageManager)
-      .then(data => expect(data).toEqual(expectedResult));
-  });
-
-  it('should throw if the both commands return empty str', async () => {
-    getExecOutputSpy.mockImplementation((commandLine: string) => {
-      return new Promise<exec.ExecOutput>(resolve => {
-        resolve({exitCode: 10, stdout: '', stderr: ''});
-      });
-    });
-
-    //Act + Assert
-    await expect(async () => {
-      await cacheUtils.getCacheDirectoryPath(validPackageManager);
-    }).rejects.toThrow();
-  });
-
   it('should throw if the specified package name is invalid', async () => {
     getExecOutputSpy.mockImplementation((commandLine: string) => {
       return new Promise<exec.ExecOutput>(resolve => {
@@ -136,7 +101,7 @@ describe('getCacheDirectoryPath', () => {
     });
 
     //Act + Assert
-    await expect(async () => {
+    expect(async () => {
       await cacheUtils.getCacheDirectoryPath(validPackageManager);
     }).rejects.toThrow();
   });
@@ -144,8 +109,8 @@ describe('getCacheDirectoryPath', () => {
 
 describe('isCacheFeatureAvailable', () => {
   //Arrange
-  const isFeatureAvailableSpy = jest.spyOn(cache, 'isFeatureAvailable');
-  const warningSpy = jest.spyOn(core, 'warning');
+  let isFeatureAvailableSpy = jest.spyOn(cache, 'isFeatureAvailable');
+  let warningSpy = jest.spyOn(core, 'warning');
 
   it('should return true when cache feature is available', () => {
     //Arrange
@@ -153,14 +118,16 @@ describe('isCacheFeatureAvailable', () => {
       return true;
     });
 
+    let functionResult;
+
     //Act
-    const functionResult = cacheUtils.isCacheFeatureAvailable();
+    functionResult = cacheUtils.isCacheFeatureAvailable();
 
     //Assert
     expect(functionResult).toBeTruthy();
   });
 
-  it('should warn when cache feature is unavailable and GHES is not used', () => {
+  it('should warn when cache feature is unavailable and GHES is not used ', () => {
     //Arrange
     isFeatureAvailableSpy.mockImplementation(() => {
       return false;
@@ -168,7 +135,7 @@ describe('isCacheFeatureAvailable', () => {
 
     process.env['GITHUB_SERVER_URL'] = 'https://github.com';
 
-    const warningMessage =
+    let warningMessage =
       'The runner was not able to contact the cache service. Caching will be skipped';
 
     //Act
@@ -186,14 +153,16 @@ describe('isCacheFeatureAvailable', () => {
 
     process.env['GITHUB_SERVER_URL'] = 'https://github.com';
 
+    let functionResult;
+
     //Act
-    const functionResult = cacheUtils.isCacheFeatureAvailable();
+    functionResult = cacheUtils.isCacheFeatureAvailable();
 
     //Assert
     expect(functionResult).toBeFalsy();
   });
 
-  it('should warn when cache feature is unavailable and GHES is used', () => {
+  it('should throw when cache feature is unavailable and GHES is used', () => {
     //Arrange
     isFeatureAvailableSpy.mockImplementation(() => {
       return false;
@@ -201,11 +170,10 @@ describe('isCacheFeatureAvailable', () => {
 
     process.env['GITHUB_SERVER_URL'] = 'https://nongithub.com';
 
-    const warningMessage =
+    let errorMessage =
       'Cache action is only supported on GHES version >= 3.5. If you are on version >=3.5 Please check with GHES admin if Actions cache service is enabled or not.';
 
     //Act + Assert
-    expect(cacheUtils.isCacheFeatureAvailable()).toBeFalsy();
-    expect(warningSpy).toHaveBeenCalledWith(warningMessage);
+    expect(() => cacheUtils.isCacheFeatureAvailable()).toThrow(errorMessage);
   });
 });

__tests__/setup-go.test.ts

@@ -8,13 +8,13 @@ import path from 'path';
 import * as main from '../src/main';
 import * as im from '../src/installer';
 
-import goJsonData from './data/golang-dl.json';
-import matchers from '../matchers.json';
-import goTestManifest from './data/versions-manifest.json';
-const matcherPattern = matchers.problemMatcher[0].pattern[0];
-const matcherRegExp = new RegExp(matcherPattern.regexp);
-const win32Join = path.win32.join;
-const posixJoin = path.posix.join;
+let goJsonData = require('./data/golang-dl.json');
+let matchers = require('../matchers.json');
+let goTestManifest = require('./data/versions-manifest.json');
+let matcherPattern = matchers.problemMatcher[0].pattern[0];
+let matcherRegExp = new RegExp(matcherPattern.regexp);
+let win32Join = path.win32.join;
+let posixJoin = path.posix.join;
 
 describe('setup-go', () => {
   let inputs = {} as any;
@@ -133,7 +133,7 @@ describe('setup-go', () => {
     os.platform = 'darwin';
     os.arch = 'x64';
 
-    const match = await im.getInfoFromManifest('1.9.7', true, 'mocktoken');
+    let match = await im.getInfoFromManifest('1.9.7', true, 'mocktoken');
     expect(match).toBeDefined();
     expect(match!.resolvedVersion).toBe('1.9.7');
     expect(match!.type).toBe('manifest');
@@ -146,7 +146,7 @@ describe('setup-go', () => {
     os.platform = 'linux';
     os.arch = 'x64';
 
-    const match = await im.getInfoFromManifest('1.9.7', true, 'mocktoken');
+    let match = await im.getInfoFromManifest('1.9.7', true, 'mocktoken');
     expect(match).toBeDefined();
     expect(match!.resolvedVersion).toBe('1.9.7');
     expect(match!.type).toBe('manifest');
@@ -159,7 +159,7 @@ describe('setup-go', () => {
     os.platform = 'win32';
     os.arch = 'x64';
 
-    const match = await im.getInfoFromManifest('1.9.7', true, 'mocktoken');
+    let match = await im.getInfoFromManifest('1.9.7', true, 'mocktoken');
     expect(match).toBeDefined();
     expect(match!.resolvedVersion).toBe('1.9.7');
     expect(match!.type).toBe('manifest');
@@ -173,11 +173,11 @@ describe('setup-go', () => {
     os.arch = 'x64';
 
     // spec: 1.13.0 => 1.13
-    const match: im.IGoVersion | undefined = await im.findMatch('1.13.0');
+    let match: im.IGoVersion | undefined = await im.findMatch('1.13.0');
     expect(match).toBeDefined();
-    const version: string = match ? match.version : '';
+    let version: string = match ? match.version : '';
     expect(version).toBe('go1.13');
-    const fileName = match ? match.files[0].filename : '';
+    let fileName = match ? match.files[0].filename : '';
     expect(fileName).toBe('go1.13.darwin-amd64.tar.gz');
   });
 
@@ -186,11 +186,11 @@ describe('setup-go', () => {
     os.arch = 'x64';
 
     // spec: 1.13 => 1.13.7 (latest)
-    const match: im.IGoVersion | undefined = await im.findMatch('1.13');
+    let match: im.IGoVersion | undefined = await im.findMatch('1.13');
     expect(match).toBeDefined();
-    const version: string = match ? match.version : '';
+    let version: string = match ? match.version : '';
     expect(version).toBe('go1.13.7');
-    const fileName = match ? match.files[0].filename : '';
+    let fileName = match ? match.files[0].filename : '';
     expect(fileName).toBe('go1.13.7.linux-amd64.tar.gz');
   });
 
@@ -199,11 +199,11 @@ describe('setup-go', () => {
     os.arch = 'x64';
 
     // spec: ^1.13.6 => 1.13.7
-    const match: im.IGoVersion | undefined = await im.findMatch('^1.13.6');
+    let match: im.IGoVersion | undefined = await im.findMatch('^1.13.6');
     expect(match).toBeDefined();
-    const version: string = match ? match.version : '';
+    let version: string = match ? match.version : '';
     expect(version).toBe('go1.13.7');
-    const fileName = match ? match.files[0].filename : '';
+    let fileName = match ? match.files[0].filename : '';
     expect(fileName).toBe('go1.13.7.linux-amd64.tar.gz');
   });
 
@@ -212,11 +212,11 @@ describe('setup-go', () => {
     os.arch = 'x32';
 
     // spec: 1 => 1.13.7 (latest)
-    const match: im.IGoVersion | undefined = await im.findMatch('1');
+    let match: im.IGoVersion | undefined = await im.findMatch('1');
     expect(match).toBeDefined();
-    const version: string = match ? match.version : '';
+    let version: string = match ? match.version : '';
     expect(version).toBe('go1.13.7');
-    const fileName = match ? match.files[0].filename : '';
+    let fileName = match ? match.files[0].filename : '';
     expect(fileName).toBe('go1.13.7.windows-386.zip');
   });
 
@@ -225,11 +225,11 @@ describe('setup-go', () => {
     os.arch = 'x64';
 
     // spec: 1.14, stable=false => go1.14rc1
-    const match: im.IGoVersion | undefined = await im.findMatch('1.14.0-rc.1');
+    let match: im.IGoVersion | undefined = await im.findMatch('1.14.0-rc.1');
     expect(match).toBeDefined();
-    const version: string = match ? match.version : '';
+    let version: string = match ? match.version : '';
     expect(version).toBe('go1.14rc1');
-    const fileName = match ? match.files[0].filename : '';
+    let fileName = match ? match.files[0].filename : '';
     expect(fileName).toBe('go1.14rc1.linux-amd64.tar.gz');
   });
 
@@ -237,7 +237,7 @@ describe('setup-go', () => {
     inputs['go-version'] = '1.13.0';
     inputs.stable = 'true';
 
-    const toolPath = path.normalize('/cache/go/1.13.0/x64');
+    let toolPath = path.normalize('/cache/go/1.13.0/x64');
     findSpy.mockImplementation(() => toolPath);
     await main.run();
 
@@ -249,7 +249,7 @@ describe('setup-go', () => {
 
     inSpy.mockImplementation(name => inputs[name]);
 
-    const toolPath = path.normalize('/cache/go/1.13.0/x64');
+    let toolPath = path.normalize('/cache/go/1.13.0/x64');
     findSpy.mockImplementation(() => toolPath);
     await main.run();
 
@@ -260,10 +260,10 @@ describe('setup-go', () => {
     inputs['go-version'] = '1.13.0';
     inSpy.mockImplementation(name => inputs[name]);
 
-    const toolPath = path.normalize('/cache/go/1.13.0/x64');
+    let toolPath = path.normalize('/cache/go/1.13.0/x64');
     findSpy.mockImplementation(() => toolPath);
 
-    const vars: {[key: string]: string} = {};
+    let vars: {[key: string]: string} = {};
     exportVarSpy.mockImplementation((name: string, val: string) => {
       vars[name] = val;
     });
@@ -276,10 +276,10 @@ describe('setup-go', () => {
     inputs['go-version'] = '1.8';
     inSpy.mockImplementation(name => inputs[name]);
 
-    const toolPath = path.normalize('/cache/go/1.8.0/x64');
+    let toolPath = path.normalize('/cache/go/1.8.0/x64');
     findSpy.mockImplementation(() => toolPath);
 
-    const vars: {[key: string]: string} = {};
+    let vars: {[key: string]: string} = {};
     exportVarSpy.mockImplementation((name: string, val: string) => {
       vars[name] = val;
     });
@@ -293,7 +293,7 @@ describe('setup-go', () => {
   it('finds a version of go already in the cache', async () => {
     inputs['go-version'] = '1.13.0';
 
-    const toolPath = path.normalize('/cache/go/1.13.0/x64');
+    let toolPath = path.normalize('/cache/go/1.13.0/x64');
     findSpy.mockImplementation(() => toolPath);
     await main.run();
 
@@ -302,16 +302,16 @@ describe('setup-go', () => {
 
   it('finds a version in the cache and adds it to the path', async () => {
     inputs['go-version'] = '1.13.0';
-    const toolPath = path.normalize('/cache/go/1.13.0/x64');
+    let toolPath = path.normalize('/cache/go/1.13.0/x64');
     findSpy.mockImplementation(() => toolPath);
     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 error and reports error', async () => {
-    const errMsg = 'unhandled error message';
+    let errMsg = 'unhandled error message';
     inputs['go-version'] = '1.13.0';
 
     findSpy.mockImplementation(() => {
@@ -329,12 +329,12 @@ describe('setup-go', () => {
 
     findSpy.mockImplementation(() => '');
     dlSpy.mockImplementation(() => '/some/temp/path');
-    const toolPath = path.normalize('/cache/go/1.13.0/x64');
+    let toolPath = path.normalize('/cache/go/1.13.0/x64');
     extractTarSpy.mockImplementation(() => '/some/other/temp/path');
     cacheSpy.mockImplementation(() => toolPath);
     await main.run();
 
-    const expPath = path.join(toolPath, 'bin');
+    let expPath = path.join(toolPath, 'bin');
 
     expect(dlSpy).toHaveBeenCalled();
     expect(extractTarSpy).toHaveBeenCalled();
@@ -352,12 +352,12 @@ describe('setup-go', () => {
     dlSpy.mockImplementation(() => 'C:\\temp\\some\\path');
     extractZipSpy.mockImplementation(() => 'C:\\temp\\some\\other\\path');
 
-    const toolPath = path.normalize('C:\\cache\\go\\1.13.0\\x64');
+    let toolPath = path.normalize('C:\\cache\\go\\1.13.0\\x64');
     cacheSpy.mockImplementation(() => toolPath);
 
     await main.run();
 
-    const expPath = path.win32.join(toolPath, 'bin');
+    let expPath = path.win32.join(toolPath, 'bin');
     expect(dlSpy).toHaveBeenCalledWith(
       'https://storage.googleapis.com/golang/go1.13.1.windows-amd64.zip',
       'C:\\temp\\go1.13.1.windows-amd64.zip',
@@ -384,25 +384,25 @@ describe('setup-go', () => {
     os.platform = 'linux';
     os.arch = 'x64';
 
-    const versionSpec = '1.12.16';
+    let versionSpec = '1.12.16';
 
     inputs['go-version'] = versionSpec;
     inputs['token'] = 'faketoken';
 
-    const expectedUrl =
+    let expectedUrl =
       'https://github.com/actions/go-versions/releases/download/1.12.16-20200616.20/go-1.12.16-linux-x64.tar.gz';
 
     // ... but not in the local cache
     findSpy.mockImplementation(() => '');
 
     dlSpy.mockImplementation(async () => '/some/temp/path');
-    const toolPath = path.normalize('/cache/go/1.12.16/x64');
+    let toolPath = path.normalize('/cache/go/1.12.16/x64');
     extractTarSpy.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(extractTarSpy).toHaveBeenCalled();
@@ -421,25 +421,25 @@ describe('setup-go', () => {
     os.platform = 'linux';
     os.arch = 'x64';
 
-    const versionSpec = '1.12';
+    let versionSpec = '1.12';
 
     inputs['go-version'] = versionSpec;
     inputs['token'] = 'faketoken';
 
-    const expectedUrl =
+    let expectedUrl =
       'https://github.com/actions/go-versions/releases/download/1.12.17-20200616.21/go-1.12.17-linux-x64.tar.gz';
 
     // ... but not in the local cache
     findSpy.mockImplementation(() => '');
 
     dlSpy.mockImplementation(async () => '/some/temp/path');
-    const toolPath = path.normalize('/cache/go/1.12.17/x64');
+    let toolPath = path.normalize('/cache/go/1.12.17/x64');
     extractTarSpy.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(extractTarSpy).toHaveBeenCalled();
@@ -458,7 +458,7 @@ describe('setup-go', () => {
     os.platform = 'linux';
     os.arch = 'x64';
 
-    const versionSpec = '1.12.14';
+    let versionSpec = '1.12.14';
 
     inputs['go-version'] = versionSpec;
     inputs['token'] = 'faketoken';
@@ -467,13 +467,13 @@ describe('setup-go', () => {
     findSpy.mockImplementation(() => '');
 
     dlSpy.mockImplementation(async () => '/some/temp/path');
-    const toolPath = path.normalize('/cache/go/1.12.14/x64');
+    let toolPath = path.normalize('/cache/go/1.12.14/x64');
     extractTarSpy.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(logSpy).toHaveBeenCalledWith('Setup go version spec 1.12.14');
     expect(findSpy).toHaveBeenCalled();
     expect(logSpy).toHaveBeenCalledWith('Attempting to download 1.12.14...');
@@ -489,7 +489,7 @@ describe('setup-go', () => {
   });
 
   it('reports a failed download', async () => {
-    const errMsg = 'unhandled download message';
+    let errMsg = 'unhandled download message';
     os.platform = 'linux';
     os.arch = 'x64';
 
@@ -510,7 +510,7 @@ describe('setup-go', () => {
     whichSpy.mockImplementation(async () => {
       return '';
     });
-    const added = await main.addBinToPath();
+    let added = await main.addBinToPath();
     expect(added).toBeFalsy();
   });
 
@@ -528,8 +528,8 @@ describe('setup-go', () => {
       return false;
     });
 
-    const added = await main.addBinToPath();
-    expect(added).toBeTruthy();
+    let added = await main.addBinToPath();
+    expect(added).toBeTruthy;
   });
 
   interface Annotation {
@@ -543,9 +543,9 @@ describe('setup-go', () => {
   // problem matcher regex pattern tests
 
   function testMatch(line: string): Annotation {
-    const annotation = <Annotation>{};
+    let annotation = <Annotation>{};
 
-    const match = matcherRegExp.exec(line);
+    let match = matcherRegExp.exec(line);
     if (match) {
       annotation.line = parseInt(match[matcherPattern.line], 10);
       annotation.column = parseInt(match[matcherPattern.column], 10);
@@ -557,8 +557,8 @@ describe('setup-go', () => {
   }
 
   it('matches on relative unix path', async () => {
-    const line = './main.go:13:2: undefined: fmt.Printl';
-    const annotation = testMatch(line);
+    let line = './main.go:13:2: undefined: fmt.Printl';
+    let annotation = testMatch(line);
     expect(annotation).toBeDefined();
     expect(annotation.line).toBe(13);
     expect(annotation.column).toBe(2);
@@ -567,8 +567,8 @@ describe('setup-go', () => {
   });
 
   it('matches on unix path up the tree', async () => {
-    const line = '../main.go:13:2: undefined: fmt.Printl';
-    const annotation = testMatch(line);
+    let line = '../main.go:13:2: undefined: fmt.Printl';
+    let annotation = testMatch(line);
     expect(annotation).toBeDefined();
     expect(annotation.line).toBe(13);
     expect(annotation.column).toBe(2);
@@ -577,8 +577,8 @@ describe('setup-go', () => {
   });
 
   it('matches on unix path down the tree', async () => {
-    const line = 'foo/main.go:13:2: undefined: fmt.Printl';
-    const annotation = testMatch(line);
+    let line = 'foo/main.go:13:2: undefined: fmt.Printl';
+    let annotation = testMatch(line);
     expect(annotation).toBeDefined();
     expect(annotation.line).toBe(13);
     expect(annotation.column).toBe(2);
@@ -587,8 +587,8 @@ describe('setup-go', () => {
   });
 
   it('matches on rooted unix path', async () => {
-    const line = '/assert.go:4:1: missing return at end of function';
-    const annotation = testMatch(line);
+    let line = '/assert.go:4:1: missing return at end of function';
+    let annotation = testMatch(line);
     expect(annotation).toBeDefined();
     expect(annotation.line).toBe(4);
     expect(annotation.column).toBe(1);
@@ -597,8 +597,8 @@ describe('setup-go', () => {
   });
 
   it('matches on unix path with spaces', async () => {
-    const line = '   ./assert.go:5:2: missing return at end of function   ';
-    const annotation = testMatch(line);
+    let line = '   ./assert.go:5:2: missing return at end of function   ';
+    let annotation = testMatch(line);
     expect(annotation).toBeDefined();
     expect(annotation.line).toBe(5);
     expect(annotation.column).toBe(2);
@@ -607,8 +607,8 @@ describe('setup-go', () => {
   });
 
   it('matches on unix path with tabs', async () => {
-    const line = '\t./assert.go:5:2: missing return at end of function   ';
-    const annotation = testMatch(line);
+    let line = '\t./assert.go:5:2: missing return at end of function   ';
+    let annotation = testMatch(line);
     expect(annotation).toBeDefined();
     expect(annotation.line).toBe(5);
     expect(annotation.column).toBe(2);
@@ -617,8 +617,8 @@ describe('setup-go', () => {
   });
 
   it('matches on relative windows path', async () => {
-    const line = '.\\main.go:13:2: undefined: fmt.Printl';
-    const annotation = testMatch(line);
+    let line = '.\\main.go:13:2: undefined: fmt.Printl';
+    let annotation = testMatch(line);
     expect(annotation).toBeDefined();
     expect(annotation.line).toBe(13);
     expect(annotation.column).toBe(2);
@@ -627,8 +627,8 @@ describe('setup-go', () => {
   });
 
   it('matches on windows path up the tree', async () => {
-    const line = '..\\main.go:13:2: undefined: fmt.Printl';
-    const annotation = testMatch(line);
+    let line = '..\\main.go:13:2: undefined: fmt.Printl';
+    let annotation = testMatch(line);
     expect(annotation).toBeDefined();
     expect(annotation.line).toBe(13);
     expect(annotation.column).toBe(2);
@@ -730,7 +730,7 @@ describe('setup-go', () => {
       os.platform = 'linux';
       os.arch = 'x64';
 
-      const versionSpec = '1.13';
+      let versionSpec = '1.13';
 
       inputs['go-version'] = versionSpec;
       inputs['check-latest'] = true;
@@ -741,13 +741,13 @@ describe('setup-go', () => {
       findSpy.mockImplementation(() => '');
 
       dlSpy.mockImplementation(async () => '/some/temp/path');
-      const toolPath = path.normalize('/cache/go/1.13.7/x64');
+      let toolPath = path.normalize('/cache/go/1.13.7/x64');
       extractTarSpy.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(extractTarSpy).toHaveBeenCalled();
@@ -767,7 +767,7 @@ describe('setup-go', () => {
       os.platform = 'linux';
       os.arch = 'x64';
 
-      const versionSpec = '1.13';
+      let versionSpec = '1.13';
 
       process.env['GITHUB_PATH'] = '';
 
@@ -784,13 +784,13 @@ describe('setup-go', () => {
       getAllVersionsSpy.mockImplementationOnce(() => undefined);
 
       dlSpy.mockImplementation(async () => '/some/temp/path');
-      const toolPath = path.normalize('/cache/go/1.13.7/x64');
+      let toolPath = path.normalize('/cache/go/1.13.7/x64');
       extractTarSpy.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(logSpy).toHaveBeenCalledWith(
         `Failed to resolve version ${versionSpec} from manifest`
@@ -910,7 +910,7 @@ use .
         inputs['go-version'] = version;
         inputs['architecture'] = arch;
 
-        const expectedUrl =
+        let expectedUrl =
           platform === 'win32'
             ? `https://github.com/actions/go-versions/releases/download/${version}/go-${version}-${platform}-${arch}.${fileExtension}`
             : `https://storage.googleapis.com/golang/go${version}.${osSpec}-${arch}.${fileExtension}`;
@@ -919,7 +919,7 @@ use .
         findSpy.mockImplementation(() => '');
 
         dlSpy.mockImplementation(async () => '/some/temp/path');
-        const toolPath = path.normalize(`/cache/go/${version}/${arch}`);
+        let toolPath = path.normalize(`/cache/go/${version}/${arch}`);
         cacheSpy.mockImplementation(async () => toolPath);
 
         await main.run();
@@ -944,7 +944,7 @@ use .
         findSpy.mockImplementation(() => '');
 
         dlSpy.mockImplementation(async () => '/some/temp/path');
-        const toolPath = path.normalize(`/cache/go/${alias}/${arch}`);
+        let toolPath = path.normalize(`/cache/go/${alias}/${arch}`);
         cacheSpy.mockImplementation(async () => toolPath);
 
         await main.run();

action.yml

@@ -14,7 +14,7 @@ inputs:
     default: ${{ github.server_url == 'https://github.com' && github.token || '' }}
   cache:
     description: Used to specify whether caching is needed. Set to true, if you'd like to enable caching.
-    default: true
+    default: false
   cache-dependency-path:
     description: 'Used to specify the path to a dependency file - go.sum'
   architecture:

docs/contributors.md

@@ -52,7 +52,7 @@ Pull requests are the easiest way to contribute changes to git repos at GitHub.
 
 - Please check that no one else has already created a pull request with these changes
 - Use a "feature branch" for your changes. That separates the changes in the pull request from your other changes and makes it easy to edit/amend commits in the pull request
--  **Run `pre-checkin` script to format, lint, build and test changes**
+-  **Run `pre-checkin` script to format, build and test changes**
 - Make sure your changes are well formatted and that all tests are passing
 - If your pull request is connected to an open issue, please, leave a link to this issue in the `Related issue:` section
 - If you later need to add new commits to the pull request, you can simply commit the changes to the local branch and then push them. The pull request gets automatically updated
@@ -61,7 +61,6 @@ Pull requests are the easiest way to contribute changes to git repos at GitHub.
 
 - To implement new features or fix bugs, you need to make changes to the `.ts` files, which are located in the `src` folder
 - To comply with the code style, **you need to run the `format` script**
-- To lint the code, **you need to run the `lint:fix` script**
 - To transpile source code to `javascript` we use [NCC](https://github.com/vercel/ncc). **It is very important to run the `build` script after making changes**, otherwise your changes will not get into the final `javascript` build
 - You can also start formatting, building code, and testing with a single `pre-checkin` command
 

package.json

@@ -1,17 +1,15 @@
 {
   "name": "setup-go",
-  "version": "4.0.0",
+  "version": "3.3.0",
   "private": true,
   "description": "setup go action",
   "main": "lib/setup-go.js",
   "scripts": {
     "build": "tsc && ncc build -o dist/setup src/setup-go.ts && ncc build -o dist/cache-save src/cache-save.ts",
-    "format": "prettier --no-error-on-unmatched-pattern --config ./.prettierrc.js --write **/*.{ts,yml,yaml}",
-    "format-check": "prettier --no-error-on-unmatched-pattern --config ./.prettierrc.js --check **/*.{ts,yml,yaml}",
-    "lint": "eslint --config ./.eslintrc.js **/*.ts",
-    "lint:fix": "eslint --config ./.eslintrc.js **/*.ts --fix",
+    "format": "prettier --write **/*.ts",
+    "format-check": "prettier --check **/*.ts",
     "test": "jest --coverage",
-    "pre-checkin": "npm run format && npm run lint:fix && npm run build && npm test"
+    "pre-checkin": "npm run format && npm run build && npm test"
   },
   "repository": {
     "type": "git",
@@ -25,7 +23,7 @@
   "author": "GitHub",
   "license": "MIT",
   "dependencies": {
-    "@actions/cache": "^3.0.0",
+    "@actions/cache": "^4.0.3",
     "@actions/core": "^1.10.0",
     "@actions/exec": "^1.1.0",
     "@actions/glob": "^0.2.0",
@@ -38,16 +36,11 @@
     "@types/jest": "^27.0.2",
     "@types/node": "^16.11.25",
     "@types/semver": "^6.0.0",
-    "@typescript-eslint/eslint-plugin": "^5.54.0",
-    "@typescript-eslint/parser": "^5.54.0",
     "@vercel/ncc": "^0.33.4",
-    "eslint": "^8.35.0",
-    "eslint-config-prettier": "^8.6.0",
-    "eslint-plugin-jest": "^27.2.1",
     "jest": "^27.2.5",
     "jest-circus": "^27.2.5",
     "nock": "^10.0.6",
-    "prettier": "^2.8.4",
+    "prettier": "^1.17.1",
     "ts-jest": "^27.0.5",
     "typescript": "^4.3.3"
   }

src/cache-restore.ts

@@ -48,7 +48,7 @@ export const restoreCache = async (
 };
 
 const findDependencyFile = (packageManager: PackageManagerInfo) => {
-  const dependencyFile = packageManager.dependencyFilePattern;
+  let dependencyFile = packageManager.dependencyFilePattern;
   const workspace = process.env.GITHUB_WORKSPACE!;
   const rootContent = fs.readdirSync(workspace);
 

src/cache-save.ts

@@ -23,7 +23,7 @@ export async function run() {
     if (typeof error === 'string') {
       message = error;
     }
-    core.warning(message);
+    core.setFailed(message);
   }
 }
 
@@ -59,13 +59,6 @@ const cachePackages = async () => {
     );
   }
 
-  if (!primaryKey) {
-    core.info(
-      'Primary key was not generated. Please check the log messages above for more errors or information'
-    );
-    return;
-  }
-
   if (primaryKey === state) {
     core.info(
       `Cache hit occurred on the primary key ${primaryKey}, not saving cache.`

src/cache-utils.ts

@@ -34,29 +34,19 @@ export const getPackageManagerInfo = async (packageManager: string) => {
 export const getCacheDirectoryPath = async (
   packageManagerInfo: PackageManagerInfo
 ) => {
-  const pathOutputs = await Promise.allSettled(
-    packageManagerInfo.cacheFolderCommandList.map(async command =>
+  let pathList = await Promise.all(
+    packageManagerInfo.cacheFolderCommandList.map(command =>
       getCommandOutput(command)
     )
   );
 
-  const results = pathOutputs.map(item => {
-    if (item.status === 'fulfilled') {
-      return item.value;
-    } else {
-      core.info(`[warning]getting cache directory path failed: ${item.reason}`);
-    }
+  const emptyPaths = pathList.filter(item => !item);
 
-    return '';
-  });
-
-  const cachePaths = results.filter(item => item);
-
-  if (!cachePaths.length) {
+  if (emptyPaths.length) {
     throw new Error(`Could not get cache folder paths.`);
   }
 
-  return cachePaths;
+  return pathList;
 };
 
 export function isGhes(): boolean {
@@ -67,19 +57,19 @@ export function isGhes(): boolean {
 }
 
 export function isCacheFeatureAvailable(): boolean {
-  if (cache.isFeatureAvailable()) {
-    return true;
-  }
+  if (!cache.isFeatureAvailable()) {
+    if (isGhes()) {
+      throw new Error(
+        'Cache action is only supported on GHES version >= 3.5. If you are on version >=3.5 Please check with GHES admin if Actions cache service is enabled or not.'
+      );
+    } else {
+      core.warning(
+        'The runner was not able to contact the cache service. Caching will be skipped'
+      );
+    }
 
-  if (isGhes()) {
-    core.warning(
-      'Cache action is only supported on GHES version >= 3.5. If you are on version >=3.5 Please check with GHES admin if Actions cache service is enabled or not.'
-    );
     return false;
   }
 
-  core.warning(
-    'The runner was not able to contact the cache service. Caching will be skipped'
-  );
-  return false;
+  return true;
 }

src/installer.ts

@@ -37,7 +37,7 @@ export async function getGo(
   arch = os.arch()
 ) {
   let manifest: tc.IToolRelease[] | undefined;
-  const osPlat: string = os.platform();
+  let osPlat: string = os.platform();
 
   if (
     versionSpec === StableReleaseAlias.Stable ||
@@ -83,7 +83,8 @@ export async function getGo(
   }
 
   // check cache
-  const toolPath = tc.find('go', versionSpec, arch);
+  let toolPath: string;
+  toolPath = tc.find('go', versionSpec, arch);
   // If not found in cache, download
   if (toolPath) {
     core.info(`Found in cache @ ${toolPath}`);
@@ -245,12 +246,13 @@ async function getInfoFromDist(
   versionSpec: string,
   arch: string
 ): Promise<IGoVersionInfo | null> {
-  const version: IGoVersion | undefined = await findMatch(versionSpec, arch);
+  let version: IGoVersion | undefined;
+  version = await findMatch(versionSpec, arch);
   if (!version) {
     return null;
   }
 
-  const downloadUrl = `https://storage.googleapis.com/golang/${version.files[0].filename}`;
+  let downloadUrl: string = `https://storage.googleapis.com/golang/${version.files[0].filename}`;
 
   return <IGoVersionInfo>{
     type: 'dist',
@@ -264,14 +266,14 @@ export async function findMatch(
   versionSpec: string,
   arch = os.arch()
 ): Promise<IGoVersion | undefined> {
-  const archFilter = sys.getArch(arch);
-  const platFilter = sys.getPlatform();
+  let archFilter = sys.getArch(arch);
+  let platFilter = sys.getPlatform();
 
   let result: IGoVersion | undefined;
   let match: IGoVersion | undefined;
 
-  const dlUrl = 'https://golang.org/dl/?mode=json&include=all';
-  const candidates: IGoVersion[] | null = await module.exports.getVersionsDist(
+  const dlUrl: string = 'https://golang.org/dl/?mode=json&include=all';
+  let candidates: IGoVersion[] | null = await module.exports.getVersionsDist(
     dlUrl
   );
   if (!candidates) {
@@ -280,8 +282,8 @@ export async function findMatch(
 
   let goFile: IGoVersionFile | undefined;
   for (let i = 0; i < candidates.length; i++) {
-    const candidate: IGoVersion = candidates[i];
-    const version = makeSemver(candidate.version);
+    let candidate: IGoVersion = candidates[i];
+    let version = makeSemver(candidate.version);
 
     core.debug(`check ${version} satisfies ${versionSpec}`);
     if (semver.satisfies(version, versionSpec)) {
@@ -313,7 +315,7 @@ export async function getVersionsDist(
   dlUrl: string
 ): Promise<IGoVersion[] | null> {
   // this returns versions descending so latest is first
-  const http: httpm.HttpClient = new httpm.HttpClient('setup-go', [], {
+  let http: httpm.HttpClient = new httpm.HttpClient('setup-go', [], {
     allowRedirects: true,
     maxRedirects: 3
   });
@@ -329,9 +331,9 @@ export async function getVersionsDist(
 export function makeSemver(version: string): string {
   version = version.replace('go', '');
   version = version.replace('beta', '-beta.').replace('rc', '-rc.');
-  const parts = version.split('-');
+  let parts = version.split('-');
 
-  const semVersion = semver.coerce(parts[0])?.version;
+  let semVersion = semver.coerce(parts[0])?.version;
   if (!semVersion) {
     throw new Error(
       `The version: ${version} can't be changed to SemVer notation`
@@ -367,10 +369,10 @@ export function parseGoVersionFile(versionFilePath: string): string {
 }
 
 async function resolveStableVersionDist(versionSpec: string, arch: string) {
-  const archFilter = sys.getArch(arch);
-  const platFilter = sys.getPlatform();
-  const dlUrl = 'https://golang.org/dl/?mode=json&include=all';
-  const candidates: IGoVersion[] | null = await module.exports.getVersionsDist(
+  let archFilter = sys.getArch(arch);
+  let platFilter = sys.getPlatform();
+  const dlUrl: string = 'https://golang.org/dl/?mode=json&include=all';
+  let candidates: IGoVersion[] | null = await module.exports.getVersionsDist(
     dlUrl
   );
   if (!candidates) {

src/main.ts

@@ -27,8 +27,8 @@ export async function run() {
     }
 
     if (versionSpec) {
-      const token = core.getInput('token');
-      const auth = !token ? undefined : `token ${token}`;
+      let token = core.getInput('token');
+      let auth = !token ? undefined : `token ${token}`;
 
       const checkLatest = core.getBooleanInput('check-latest');
 
@@ -51,31 +51,15 @@ export async function run() {
         core.exportVariable('GOROOT', installDir);
       }
 
+      let added = await addBinToPath();
+      core.debug(`add bin ${added}`);
       core.info(`Successfully set up Go version ${versionSpec}`);
-    } else {
-      core.info(
-        '[warning]go-version input was not specified. The action will try to use pre-installed version.'
-      );
     }
 
-    const added = await addBinToPath();
-    core.debug(`add bin ${added}`);
-
-    const goPath = await io.which('go');
-    const goVersion = (cp.execSync(`${goPath} version`) || '').toString();
-
     if (cache && isCacheFeatureAvailable()) {
       const packageManager = 'default';
       const cacheDependencyPath = core.getInput('cache-dependency-path');
-      try {
-        await restoreCache(
-          parseGoVersion(goVersion),
-          packageManager,
-          cacheDependencyPath
-        );
-      } catch (error) {
-        core.warning(`Restore cache failed: ${error.message}`);
-      }
+      await restoreCache(versionSpec, packageManager, cacheDependencyPath);
     }
 
     // add problem matchers
@@ -83,12 +67,14 @@ export async function run() {
     core.info(`##[add-matcher]${matchersPath}`);
 
     // output the version actually being used
+    let goPath = await io.which('go');
+    let goVersion = (cp.execSync(`${goPath} version`) || '').toString();
     core.info(goVersion);
 
     core.setOutput('go-version', parseGoVersion(goVersion));
 
     core.startGroup('go env');
-    const goEnv = (cp.execSync(`${goPath} env`) || '').toString();
+    let goEnv = (cp.execSync(`${goPath} env`) || '').toString();
     core.info(goEnv);
     core.endGroup();
   } catch (error) {
@@ -98,16 +84,16 @@ export async function run() {
 
 export async function addBinToPath(): Promise<boolean> {
   let added = false;
-  const g = await io.which('go');
+  let g = await io.which('go');
   core.debug(`which go :${g}:`);
   if (!g) {
     core.debug('go not in the path');
     return added;
   }
 
-  const buf = cp.execSync('go env GOPATH');
+  let buf = cp.execSync('go env GOPATH');
   if (buf.length > 1) {
-    const gp = buf.toString().trim();
+    let gp = buf.toString().trim();
     core.debug(`go env GOPATH :${gp}:`);
     if (!fs.existsSync(gp)) {
       // some of the hosted images have go install but not profile dir
@@ -115,7 +101,7 @@ export async function addBinToPath(): Promise<boolean> {
       await io.mkdirP(gp);
     }
 
-    const bp = path.join(gp, 'bin');
+    let bp = path.join(gp, 'bin');
     if (!fs.existsSync(bp)) {
       core.debug(`creating ${bp}`);
       await io.mkdirP(bp);

src/system.ts

@@ -1,4 +1,4 @@
-import os from 'os';
+const os = require('os');
 
 export function getPlatform(): string {
   // darwin and linux match already

tsconfig.json

@@ -46,7 +46,6 @@
     // "types": [],                           /* Type declaration files to be included in compilation. */
     // "allowSyntheticDefaultImports": true,  /* Allow default imports from modules with no default export. This does not affect code emit, just typechecking. */
     "esModuleInterop": true,                  /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */
-    "resolveJsonModule": true,                /* Allows importing modules with a '.json' extension, which is a common practice in node projects. */
     "sourceMap": true,
     // "preserveSymlinks": true,              /* Do not resolve the real path of symlinks. */
     // "allowUmdGlobalAccess": true,          /* Allow accessing UMD globals from modules. */