From 6985ab3d3596497e8aa3de45e6a33b0b46d9b902 Mon Sep 17 00:00:00 2001 From: Nogic <24802730+nogic1008@users.noreply.github.com> Date: Thu, 27 Apr 2023 02:09:11 +0000 Subject: [PATCH] test: fix cache-utils unit test --- __tests__/cache-utils.test.ts | 38 +++++++++++++---------------------- 1 file changed, 14 insertions(+), 24 deletions(-) diff --git a/__tests__/cache-utils.test.ts b/__tests__/cache-utils.test.ts index e13309f..ba73835 100644 --- a/__tests__/cache-utils.test.ts +++ b/__tests__/cache-utils.test.ts @@ -99,34 +99,24 @@ Options: }); }); - describe('isCacheFeatureAvailable()', () => { - // Save & Restore env - let serverUrlEnv: string | undefined; - beforeAll(() => (serverUrlEnv = process.env['GITHUB_SERVER_URL'])); - afterEach(() => (process.env['GITHUB_SERVER_URL'] = serverUrlEnv)); + describe.each(['', 'https://github.com/', 'https://example.com/'])( + 'isCacheFeatureAvailable()', + url => { + // Save & Restore env + let serverUrlEnv: string | undefined; + beforeAll(() => (serverUrlEnv = process.env['GITHUB_SERVER_URL'])); + beforeEach(() => process.env['GITHUB_SERVER_URL'] = url) + afterEach(() => (process.env['GITHUB_SERVER_URL'] = serverUrlEnv)); - it.each(['', 'https://github.com/', 'https://example.com/'])( - 'returns true when cache.isFeatureAvailable() === true', - url => { - process.env['GITHUB_SERVER_URL'] = url; + it('returns true when cache.isFeatureAvailable() === true', () => { jest.mocked(cache.isFeatureAvailable).mockReturnValue(true); expect(isCacheFeatureAvailable()).toBe(true); - } - ); + }); - it.each(['', 'https://github.com/'])( - 'returns false when cache.isFeatureAvailable() === false', - url => { - process.env['GITHUB_SERVER_URL'] = url; + it('returns false when cache.isFeatureAvailable() === false', () => { jest.mocked(cache.isFeatureAvailable).mockReturnValue(false); expect(isCacheFeatureAvailable()).toBe(false); - } - ); - - it('throws Error when GHES version < 3.5.', () => { - process.env['GITHUB_SERVER_URL'] = 'https://example.com/'; - jest.mocked(cache.isFeatureAvailable).mockReturnValue(false); - expect(() => isCacheFeatureAvailable()).toThrow(); - }); - }); + }); + } + ); });