Revert default server ID removal (#234)

jfrog · Dec 10, 2024 · fd3db35 · fd3db35
1 parent 040913f
commit fd3db35
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 18 deletions.
diff --git a/lib/utils.js b/lib/utils.js
@@ -350,10 +350,10 @@ class Utils {
         return Utils.getRunDefaultServerId();
     }
     /**
-     * Return a server ID that is unique for this workflow run based on the GitHub repository and run ID.
+     * Return the default server ID for JFrog CLI server configuration.
      */
     static getRunDefaultServerId() {
-        return [Utils.SETUP_JFROG_CLI_SERVER_ID_PREFIX, process.env.GITHUB_REPOSITORY, process.env.GITHUB_RUN_ID].join('-');
+        return Utils.SETUP_JFROG_CLI_SERVER_ID;
     }
     static setCliEnv() {
         Utils.exportVariableIfNotSet('JFROG_CLI_ENV_EXCLUDE', '*password*;*secret*;*key*;*token*;*auth*;JF_ARTIFACTORY_*;JF_ENV_*;JF_URL;JF_USER;JF_PASSWORD;JF_ACCESS_TOKEN');
@@ -773,8 +773,8 @@ Utils.LATEST_CLI_VERSION = 'latest';
 Utils.LATEST_RELEASE_VERSION = '[RELEASE]';
 // Placeholder CLI version to use to keep 'latest' in cache.
 Utils.LATEST_SEMVER = '100.100.100';
-// The prefix for the default server id name for JFrog CLI config
-Utils.SETUP_JFROG_CLI_SERVER_ID_PREFIX = 'setup-jfrog-cli-server';
+// The default server id name for separate env config
+Utils.SETUP_JFROG_CLI_SERVER_ID = 'setup-jfrog-cli-server';
 // Environment variable to hold all configured server IDs, separated by ';'
 Utils.JFROG_CLI_SERVER_IDS_ENV_VAR = 'SETUP_JFROG_CLI_SERVER_IDS';
 // Directory name which holds markdown files for the Workflow summary

diff --git a/package-lock.json b/package-lock.json
diff --git a/src/utils.ts b/src/utils.ts
@@ -33,8 +33,8 @@ export class Utils {
     private static readonly LATEST_RELEASE_VERSION: string = '[RELEASE]';
     // Placeholder CLI version to use to keep 'latest' in cache.
     public static readonly LATEST_SEMVER: string = '100.100.100';
-    // The prefix for the default server id name for JFrog CLI config
-    public static readonly SETUP_JFROG_CLI_SERVER_ID_PREFIX: string = 'setup-jfrog-cli-server';
+    // The default server id name for separate env config
+    public static readonly SETUP_JFROG_CLI_SERVER_ID: string = 'setup-jfrog-cli-server';
     // Environment variable to hold all configured server IDs, separated by ';'
     public static readonly JFROG_CLI_SERVER_IDS_ENV_VAR: string = 'SETUP_JFROG_CLI_SERVER_IDS';
     // Directory name which holds markdown files for the Workflow summary
@@ -398,10 +398,10 @@ export class Utils {
     }
 
     /**
-     * Return a server ID that is unique for this workflow run based on the GitHub repository and run ID.
+     * Return the default server ID for JFrog CLI server configuration.
      */
     static getRunDefaultServerId(): string {
-        return [Utils.SETUP_JFROG_CLI_SERVER_ID_PREFIX, process.env.GITHUB_REPOSITORY, process.env.GITHUB_RUN_ID].join('-');
+        return Utils.SETUP_JFROG_CLI_SERVER_ID;
     }
 
     public static setCliEnv() {

diff --git a/test/main.spec.ts b/test/main.spec.ts
@@ -156,15 +156,8 @@ function testConfigCommand(expectedServerId: string) {
 }
 
 describe('JFrog CLI Configuration', () => {
-    beforeAll(() => {
-        process.env.GITHUB_REPOSITORY = 'owner/repo';
-        process.env.GITHUB_RUN_ID = '1';
-    });
-
     afterAll(() => {
-        ['GITHUB_REPOSITORY', 'GITHUB_RUN_ID', Utils.JFROG_CLI_SERVER_IDS_ENV_VAR].forEach((envKey) => {
-            delete process.env[envKey];
-        });
+        delete process.env[Utils.JFROG_CLI_SERVER_IDS_ENV_VAR];
     });
     const myCore: jest.Mocked<typeof core> = core as any;
 
@@ -187,7 +180,7 @@ describe('JFrog CLI Configuration', () => {
     });
 
     test('Get default server ID', async () => {
-        expect(Utils.getRunDefaultServerId()).toStrictEqual('setup-jfrog-cli-server-owner/repo-1');
+        expect(Utils.getRunDefaultServerId()).toStrictEqual('setup-jfrog-cli-server');
     });
 });