diff --git a/packages/cache/__tests__/cache.test.ts b/packages/cache/__tests__/cache.test.ts index b7c3ca45..b3c7f430 100644 --- a/packages/cache/__tests__/cache.test.ts +++ b/packages/cache/__tests__/cache.test.ts @@ -22,9 +22,9 @@ describe('isFeatureAvailable', () => { expect(cache.isFeatureAvailable()).toBe(true) }) - test('returns true for cache service v1 when ACTIONS_RESULTS_URL is set', () => { + test('returns false for cache service v1 when only ACTIONS_RESULTS_URL is set', () => { process.env['ACTIONS_RESULTS_URL'] = 'http://results.com' - expect(cache.isFeatureAvailable()).toBe(true) + expect(cache.isFeatureAvailable()).toBe(false) }) test('returns true for cache service v1 when both URLs are set', () => { @@ -61,9 +61,9 @@ describe('isFeatureAvailable', () => { expect(cache.isFeatureAvailable()).toBe(true) }) - test('returns true for GHES with ACTIONS_RESULTS_URL', () => { + test('returns false for GHES with only ACTIONS_RESULTS_URL', () => { process.env['GITHUB_SERVER_URL'] = 'https://my-enterprise.github.com' process.env['ACTIONS_RESULTS_URL'] = 'http://results.com' - expect(cache.isFeatureAvailable()).toBe(true) + expect(cache.isFeatureAvailable()).toBe(false) }) }) diff --git a/packages/cache/src/cache.ts b/packages/cache/src/cache.ts index 47d92466..ba4cf1c7 100644 --- a/packages/cache/src/cache.ts +++ b/packages/cache/src/cache.ts @@ -67,10 +67,8 @@ export function isFeatureAvailable(): boolean { return !!process.env['ACTIONS_RESULTS_URL'] case 'v1': default: - // For v1, we need either ACTIONS_CACHE_URL or ACTIONS_RESULTS_URL - return !!( - process.env['ACTIONS_CACHE_URL'] || process.env['ACTIONS_RESULTS_URL'] - ) + // For v1, we only need ACTIONS_CACHE_URL + return !!process.env['ACTIONS_CACHE_URL'] } }