Generate Artifacts Outside the Tags Context

tests/jest.config.ts → jest.config.ts

@@ -4,9 +4,9 @@ module.exports = {
   preset: 'ts-jest',
   moduleDirectories: ['node_modules'],
   moduleNameMapper: {
-    '^@/(.*)$': '<rootDir>/../src/$1',
-    '^@model/(.*)$': '<rootDir>/../src/model/$1',
+    '^@/(.*)$': '<rootDir>/src/$1',
+    '^@model/(.*)$': '<rootDir>/src/model/$1',
   },
-  setupFilesAfterEnv: ['<rootDir>/setup-tests.ts'],
+  setupFilesAfterEnv: ['<rootDir>/tests/setup-tests.ts'],
   maxWorkers: 8,
 };

package.json

@@ -30,7 +30,7 @@
     "lint": "eslint ./src --ext .ts",
     "lint:test": "eslint --config ./tests/.eslintrc.json ./tests --ext .ts",
     "prepare": "husky",
-    "test": "jest --config ./tests/jest.config.ts",
+    "test": "jest",
     "qa": "yarn lint && yarn lint:test && yarn test"
   }
 }

src/configuration.ts

@@ -4,7 +4,10 @@ export class Configuration {
   private static readonly COMMAND = 'yarn build';
   private static readonly TARGET_DIR = './build';
 
-  constructor(private readonly read: typeof getInput) {}
+  constructor(
+    private readonly read: typeof getInput,
+    private readonly env: Readonly<NodeJS.ProcessEnv>
+  ) {}
 
   public get command(): string {
     return this.read('command') || Configuration.COMMAND;
@@ -13,4 +16,8 @@ export class Configuration {
   public get targetDir(): string {
     return this.read('target-dir') || Configuration.TARGET_DIR;
   }
+
+  public get isTag(): boolean {
+    return (this.env['GITHUB_REF'] ?? '').startsWith('refs/tags/');
+  }
 }

src/main.ts

@@ -6,17 +6,18 @@ import { createGit } from './create-git';
 import { Configuration } from './configuration';
 
 async function main(): Promise<void> {
-  const configuration = new Configuration(core.getInput.bind(core));
+  const configuration = new Configuration(core.getInput.bind(core), process.env);
 
   const git = await createGit();
   const tags = new Tags(git);
   const artifacts = new Artifacts(git, tags, configuration);
   const temporaryBranch = new TemporaryBranch(git);
+  const { isTag } = configuration;
 
   Promise.resolve()
-    .then(() => temporaryBranch.create())
+    .then(() => (isTag ? temporaryBranch.create() : null))
     .then(() => artifacts.update())
-    .then(() => temporaryBranch.delete())
+    .then(() => (isTag ? temporaryBranch.delete() : null))
 
     .catch((error) => {
       core.setFailed(`Failed to create and push artifacts: ${error}`);

src/model/artifacts.ts

@@ -16,9 +16,9 @@ export class Artifacts {
 
     try {
       await this.compile();
-      await this.tags.collect();
+      this.configuration.isTag && (await this.tags.collect());
       await this.deploy();
-      await this.tags.move();
+      this.configuration.isTag && (await this.tags.move());
     } catch (error: unknown) {
       core.endGroup();
       const message = String(error instanceof Error ? error.message : error);

tests/tsconfig.json

@@ -9,6 +9,6 @@
     },
     "rootDir": "./"
   },
-  "include": ["../src/**/*.ts", "./unit/**/*.ts", "jest.config.ts", "setup-tests.ts"],
+  "include": ["../src/**/*.ts", "./unit/**/*.ts", "../jest.config.ts", "setup-tests.ts"],
   "exclude": ["../node_modules"]
 }

tests/unit/model/artifacts.test.ts

@@ -135,10 +135,49 @@ describe('Artifacts', () => {
 
     expect(collect.mock.invocationCallOrder[0]).toBeLessThan(move.mock.invocationCallOrder[0] ?? 0);
   });
+
+  it('Do not perform any tasks associated to tags when the action is not running for tags', async () => {
+    const git = fromPartial<SimpleGit>({
+      commit: jest.fn(() =>
+        Promise.resolve({ summary: { changes: 0, insertions: 0, deletions: 0 } })
+      ),
+      push: jest.fn(() =>
+        Promise.resolve({
+          remoteMessages: {
+            all: [''],
+          },
+        })
+      ),
+    });
+
+    const collect = jest.fn();
+    const move = jest.fn();
+    const tags = fromPartial<Tags>({ collect, move });
+
+    const _configuration = configuration({
+      GITHUB_REF: 'refs/heads/main',
+    });
+    const artifacts = new Artifacts(git, tags, _configuration);
+
+    jest.mocked(exec).mockImplementation(async () => Promise.resolve(0));
+
+    await artifacts.update();
+
+    expect(collect).not.toHaveBeenCalled();
+    expect(move).not.toHaveBeenCalled();
+  });
 });
 
-function configuration(): Configuration {
-  return new Configuration(stubGetInput());
+function configuration(env?: Readonly<NodeJS.ProcessEnv>): Configuration {
+  let _env = env;
+
+  if (!_env) {
+    _env = {
+      GITHUB_REF: 'refs/tags/v1.0.0',
+    };
+  }
+
+  return new Configuration(stubGetInput(), _env);
 }
 
 function stubGetInput(): typeof getInput {