Add unit tests

actions · Apr 19, 2023 · 15c2532 · 15c2532
1 parent c1712a6
commit 15c2532
Show file tree

Hide file tree

Showing 4 changed files with 71 additions and 5 deletions.
diff --git a/__tests__/authutil.test.ts b/__tests__/authutil.test.ts
@@ -4,6 +4,8 @@ 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';
+import {getCacheDirectoryPath} from '../src/cache-utils';
 
 let rcFile: string;
 
@@ -209,4 +211,66 @@ describe('authutil tests', () => {
       `@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`
     );
   });
+
+  it('getPackageManagerWorkingDir should return null for not yarn', async () => {
+    process.env['INPUT_CACHE'] = 'some';
+    delete process.env['INPUT_CACHE-DEPENDENCY-PATH'];
+    const dir = cacheUtils.getPackageManagerWorkingDir();
+    expect(dir).toBeNull();
+  });
+
+  it('getPackageManagerWorkingDir should return null for not yarn with cache-dependency-path', async () => {
+    process.env['INPUT_CACHE'] = 'some';
+    process.env['INPUT_CACHE-DEPENDENCY-PATH'] = '/foo/bar';
+    const dir = cacheUtils.getPackageManagerWorkingDir();
+    expect(dir).toBeNull();
+  });
+
+  it('getPackageManagerWorkingDir should return null for yarn but without cache-dependency-path', async () => {
+    process.env['INPUT_CACHE'] = 'yarn';
+    delete process.env['INPUT_CACHE-DEPENDENCY-PATH'];
+    const dir = cacheUtils.getPackageManagerWorkingDir();
+    expect(dir).toBeNull();
+  });
+
+  it('getPackageManagerWorkingDir should return path for yarn with cache-dependency-path', async () => {
+    process.env['INPUT_CACHE'] = 'yarn';
+    const cachePath = '/foo/bar';
+    process.env['INPUT_CACHE-DEPENDENCY-PATH'] = cachePath;
+    const dir = cacheUtils.getPackageManagerWorkingDir();
+    expect(dir).toEqual(path.dirname(cachePath));
+  });
+
+  it('getCommandOutput(getPackageManagerVersion) should be called from  with getPackageManagerWorkingDir result', async () => {
+    process.env['INPUT_CACHE'] = 'yarn';
+    const cachePath = '/foo/bar';
+    process.env['INPUT_CACHE-DEPENDENCY-PATH'] = cachePath;
+    const getCommandOutputSpy = jest
+      .spyOn(cacheUtils, 'getCommandOutput')
+      .mockReturnValue(Promise.resolve('baz'));
+
+    const version = await cacheUtils.getPackageManagerVersion('foo', 'bar');
+    expect(getCommandOutputSpy).toHaveBeenCalledWith(
+      `foo bar`,
+      path.dirname(cachePath)
+    );
+  });
+
+  it('getCommandOutput(getCacheDirectoryPath) should be called from  with getPackageManagerWorkingDir result', async () => {
+    process.env['INPUT_CACHE'] = 'yarn';
+    const cachePath = '/foo/bar';
+    process.env['INPUT_CACHE-DEPENDENCY-PATH'] = cachePath;
+    const getCommandOutputSpy = jest
+      .spyOn(cacheUtils, 'getCommandOutput')
+      .mockReturnValue(Promise.resolve('baz'));
+
+    const version = await cacheUtils.getCacheDirectoryPath(
+      {lockFilePatterns: [], getCacheFolderCommand: 'quz'},
+      ''
+    );
+    expect(getCommandOutputSpy).toHaveBeenCalledWith(
+      `quz`,
+      path.dirname(cachePath)
+    );
+  });
 });
diff --git a/dist/cache-save/index.js b/dist/cache-save/index.js
@@ -59628,7 +59628,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.isCacheFeatureAvailable = exports.isGhes = exports.getCacheDirectoryPath = exports.getPackageManagerInfo = exports.getPackageManagerWorkingDir = exports.getCommandOutput = exports.supportedPackageManagers = void 0;
+exports.isCacheFeatureAvailable = exports.isGhes = exports.getCacheDirectoryPath = exports.getPackageManagerInfo = exports.getPackageManagerVersion = exports.getPackageManagerWorkingDir = exports.getCommandOutput = exports.supportedPackageManagers = void 0;
 const core = __importStar(__nccwpck_require__(2186));
 const exec = __importStar(__nccwpck_require__(1514));
 const cache = __importStar(__nccwpck_require__(7799));
@@ -59678,6 +59678,7 @@ const getPackageManagerVersion = (packageManager, command) => __awaiter(void 0,
     }
     return stdOut;
 });
+exports.getPackageManagerVersion = getPackageManagerVersion;
 const getPackageManagerInfo = (packageManager) => __awaiter(void 0, void 0, void 0, function* () {
     if (packageManager === 'npm') {
         return exports.supportedPackageManagers.npm;
@@ -59686,7 +59687,7 @@ const getPackageManagerInfo = (packageManager) => __awaiter(void 0, void 0, void
         return exports.supportedPackageManagers.pnpm;
     }
     else if (packageManager === 'yarn') {
-        const yarnVersion = yield getPackageManagerVersion('yarn', '--version');
+        const yarnVersion = yield exports.getPackageManagerVersion('yarn', '--version');
         core.debug(`Consumed yarn version is ${yarnVersion}`);
         if (yarnVersion.startsWith('1.')) {
             return exports.supportedPackageManagers.yarn1;

diff --git a/dist/setup/index.js b/dist/setup/index.js
@@ -71596,7 +71596,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.isCacheFeatureAvailable = exports.isGhes = exports.getCacheDirectoryPath = exports.getPackageManagerInfo = exports.getPackageManagerWorkingDir = exports.getCommandOutput = exports.supportedPackageManagers = void 0;
+exports.isCacheFeatureAvailable = exports.isGhes = exports.getCacheDirectoryPath = exports.getPackageManagerInfo = exports.getPackageManagerVersion = exports.getPackageManagerWorkingDir = exports.getCommandOutput = exports.supportedPackageManagers = void 0;
 const core = __importStar(__nccwpck_require__(2186));
 const exec = __importStar(__nccwpck_require__(1514));
 const cache = __importStar(__nccwpck_require__(7799));
@@ -71646,6 +71646,7 @@ const getPackageManagerVersion = (packageManager, command) => __awaiter(void 0,
     }
     return stdOut;
 });
+exports.getPackageManagerVersion = getPackageManagerVersion;
 const getPackageManagerInfo = (packageManager) => __awaiter(void 0, void 0, void 0, function* () {
     if (packageManager === 'npm') {
         return exports.supportedPackageManagers.npm;
@@ -71654,7 +71655,7 @@ const getPackageManagerInfo = (packageManager) => __awaiter(void 0, void 0, void
         return exports.supportedPackageManagers.pnpm;
     }
     else if (packageManager === 'yarn') {
-        const yarnVersion = yield getPackageManagerVersion('yarn', '--version');
+        const yarnVersion = yield exports.getPackageManagerVersion('yarn', '--version');
         core.debug(`Consumed yarn version is ${yarnVersion}`);
         if (yarnVersion.startsWith('1.')) {
             return exports.supportedPackageManagers.yarn1;

diff --git a/src/cache-utils.ts b/src/cache-utils.ts
@@ -61,7 +61,7 @@ export const getPackageManagerWorkingDir = (): string | null => {
   return cacheDependencyPath ? path.dirname(cacheDependencyPath) : null;
 };
 
-const getPackageManagerVersion = async (
+export const getPackageManagerVersion = async (
   packageManager: string,
   command: string
 ) => {