review comments

packages/cache/RELEASES.md

@@ -51,5 +51,5 @@
 ### 1.0.10
 - Update `lockfileVersion` to `v2` in `package-lock.json [#1022](https://github.com/actions/toolkit/pull/1022) 
 
-### 1.0.11
+### 2.0.0
 - Added support to check if Actions cache service feature is available or not [#1028](https://github.com/actions/toolkit/pull/1028)

packages/cache/__tests__/cache.test.ts

@@ -0,0 +1,15 @@
+import * as cache from '../src/cache'
+
+test('isFeatureAvailable returns true if server url is set', () => {
+    try {
+      process.env['ACTIONS_CACHE_URL'] = 'http://cache.com'
+      expect(cache.isFeatureAvailable()).toBe(true)
+    } finally {
+      delete process.env['ACTIONS_CACHE_URL']
+    }
+  })
+  
+  test('isFeatureAvailable returns false if server url is not set', () => {
+    expect(cache.isFeatureAvailable()).toBe(false)
+  })
+  

packages/cache/__tests__/cacheUtils.test.ts

@@ -32,16 +32,3 @@ test('assertDefined throws if undefined', () => {
 test('assertDefined returns value', () => {
   expect(cacheUtils.assertDefined('test', 5)).toBe(5)
 })
-
-test('isFeatureAvailable returns true if server url is set', () => {
-  try {
-    process.env['ACTIONS_CACHE_URL'] = 'http://cache.com'
-    expect(cacheUtils.isFeatureAvailable()).toBe(true)
-  } finally {
-    delete process.env['ACTIONS_CACHE_URL']
-  }
-})
-
-test('isFeatureAvailable returns false if server url is not set', () => {
-  expect(cacheUtils.isFeatureAvailable()).toBe(false)
-})

packages/cache/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@actions/cache",
-  "version": "1.0.11",
+  "version": "2.0.0",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "@actions/cache",
-      "version": "1.0.11",
+      "version": "2.0.0",
       "license": "MIT",
       "dependencies": {
         "@actions/core": "^1.2.6",

packages/cache/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@actions/cache",
-  "version": "1.0.11",
+  "version": "2.0.0",
   "preview": true,
   "description": "Actions cache lib",
   "keywords": [

packages/cache/src/cache.ts

@@ -50,7 +50,7 @@ function checkKey(key: string): void {
  */
 
 export function isFeatureAvailable(): boolean {
-  return utils.isFeatureAvailable()
+  return !!process.env['ACTIONS_CACHE_URL']
 }
 
 /**

packages/cache/src/internal/cacheUtils.ts

@@ -123,7 +123,3 @@ export function assertDefined<T>(name: string, value?: T): T {
 
   return value
 }
-
-export function isFeatureAvailable(): boolean {
-  return !!process.env['ACTIONS_CACHE_URL']
-}