chore: update cli tests

Author: andypols

packages/git-proxy-cli/index.ts

@@ -230,8 +230,9 @@ async function authoriseGitPush(id: string) {
 /**
  * Reject git push by ID
  * @param {string} id The ID of the git push to reject
+ * @param {string} reason The reason for rejecting the push
  */
-async function rejectGitPush(id: string) {
+async function rejectGitPush(id: string, reason: string) {
   if (!fs.existsSync(GIT_PROXY_COOKIE_FILE)) {
     console.error('Error: Reject: Authentication required');
     process.exitCode = 1;
@@ -247,7 +248,11 @@ async function rejectGitPush(id: string) {
 
     await axios.post(
       `${baseUrl}/api/v1/push/${id}/reject`,
-      {},
+      {
+        params: {
+          reason,
+        },
+      },
       {
         headers: { Cookie: cookies },
       },
@@ -261,13 +266,17 @@ async function rejectGitPush(id: string) {
 
     if (error.response) {
       switch (error.response.status) {
+        case 400:
+          errorMessage = `Error: Reject: ${error.response.data.message}`;
+          process.exitCode = 3;
+          break;
         case 401:
           errorMessage = 'Error: Reject: Authentication required';
-          process.exitCode = 3;
+          process.exitCode = 4;
           break;
         case 404:
           errorMessage = `Error: Reject: ID: '${id}': Not Found`;
-          process.exitCode = 4;
+          process.exitCode = 5;
       }
     }
     console.error(errorMessage);
@@ -553,9 +562,14 @@ yargs(hideBin(process.argv)) // eslint-disable-line @typescript-eslint/no-unused
         demandOption: true,
         type: 'string',
       },
+      reason: {
+        describe: 'Reason for rejecting the push',
+        demandOption: true,
+        type: 'string',
+      },
     },
     handler(argv) {
-      rejectGitPush(argv.id);
+      rejectGitPush(argv.id, argv.reason);
     },
   })
   .command({

packages/git-proxy-cli/test/testCli.test.ts

@@ -440,7 +440,7 @@ describe('test git-proxy-cli', function () {
       }
 
       const id = GHOST_PUSH_ID;
-      const cli = `${CLI_PATH} reject --id ${id}`;
+      const cli = `${CLI_PATH} reject --id ${id} --reason "Test rejection"`;
       const expectedExitCode = 2;
       const expectedMessages = null;
       const expectedErrorMessages = ['Error: Reject:'];
@@ -451,7 +451,7 @@ describe('test git-proxy-cli', function () {
       await helper.removeCookiesFile();
 
       const id = GHOST_PUSH_ID;
-      const cli = `${CLI_PATH} reject --id ${id}`;
+      const cli = `${CLI_PATH} reject --id ${id} --reason "Test rejection"`;
       const expectedExitCode = 1;
       const expectedMessages = null;
       const expectedErrorMessages = ['Error: Reject: Authentication required'];
@@ -463,8 +463,8 @@ describe('test git-proxy-cli', function () {
         await helper.createCookiesFileWithExpiredCookie();
         await helper.startServer();
         const id = pushId;
-        const cli = `${CLI_PATH} reject --id ${id}`;
-        const expectedExitCode = 3;
+        const cli = `${CLI_PATH} reject --id ${id} --reason "Test rejection"`;
+        const expectedExitCode = 4;
         const expectedMessages = null;
         const expectedErrorMessages = ['Error: Reject: Authentication required'];
         await helper.runCli(cli, expectedExitCode, expectedMessages, expectedErrorMessages);
@@ -479,8 +479,8 @@ describe('test git-proxy-cli', function () {
         await helper.runCli(`${CLI_PATH} login --username admin --password admin`);
 
         const id = GHOST_PUSH_ID;
-        const cli = `${CLI_PATH} reject --id ${id}`;
-        const expectedExitCode = 4;
+        const cli = `${CLI_PATH} reject --id ${id} --reason "Test rejection"`;
+        const expectedExitCode = 5;
         const expectedMessages = null;
         const expectedErrorMessages = [`Error: Reject: ID: '${id}': Not Found`];
         await helper.runCli(cli, expectedExitCode, expectedMessages, expectedErrorMessages);
@@ -756,7 +756,7 @@ describe('test git-proxy-cli', function () {
         let expectedErrorMessages = null;
         await helper.runCli(cli, expectedExitCode, expectedMessages, expectedErrorMessages);
 
-        cli = `${CLI_PATH} reject --id ${pushId}`;
+        cli = `${CLI_PATH} reject --id ${pushId} --reason "Test rejection"`;
         expectedExitCode = 0;
         expectedMessages = [`Reject: ID: '${pushId}': OK`];
         expectedErrorMessages = null;