Fix tests

GitTools · Sep 10, 2024 · 5efec84 · 5efec84
1 parent 697c973
commit 5efec84
Show file tree

Hide file tree

Showing 7 changed files with 59 additions and 8 deletions.
diff --git a/gitreleasemanager/addasset/action.yml b/gitreleasemanager/addasset/action.yml
@@ -32,3 +32,7 @@ inputs:
     description: The directory on which GitReleaseManager should be executed. Defaults to current directory
     required: false
     default: ''
+  logFilePath:
+    description: Path to where log file should be created. Defaults to logging to console.
+    required: false
+    default: ''
diff --git a/gitreleasemanager/close/action.yml b/gitreleasemanager/close/action.yml
@@ -28,3 +28,7 @@ inputs:
     description: The directory on which GitReleaseManager should be executed. Defaults to current directory
     required: false
     default: ''
+  logFilePath:
+    description: Path to where log file should be created. Defaults to logging to console.
+    required: false
+    default: ''
diff --git a/gitreleasemanager/discard/action.yml b/gitreleasemanager/discard/action.yml
@@ -28,3 +28,7 @@ inputs:
     description: The directory on which GitReleaseManager should be executed. Defaults to current directory
     required: false
     default: ''
+  logFilePath:
+    description: Path to where log file should be created. Defaults to logging to console.
+    required: false
+    default: ''
diff --git a/gitreleasemanager/open/action.yml b/gitreleasemanager/open/action.yml
@@ -28,3 +28,7 @@ inputs:
     description: The directory on which GitReleaseManager should be executed. Defaults to current directory
     required: false
     default: ''
+  logFilePath:
+    description: Path to where log file should be created. Defaults to logging to console.
+    required: false
+    default: ''
diff --git a/gitreleasemanager/publish/action.yml b/gitreleasemanager/publish/action.yml
@@ -28,3 +28,8 @@ inputs:
     description: The directory on which GitReleaseManager should be executed. Defaults to current directory
     required: false
     default: ''
+  logFilePath:
+    description: Path to where log file should be created. Defaults to logging to console.
+    required: false
+    default: ''
+
diff --git a/src/__tests__/tools/gitreleasemanager/settings.spec.ts b/src/__tests__/tools/gitreleasemanager/settings.spec.ts
@@ -46,7 +46,8 @@ describe('GitReleaseManager settings', () => {
             owner: 'owner',
             token: 'token',
             targetDirectory: 'path',
-            milestone: 'milestone'
+            milestone: 'milestone',
+            logFilePath: './output.log'
         }
 
         const buildAgent = {
@@ -66,7 +67,8 @@ describe('GitReleaseManager settings', () => {
             owner: 'owner',
             token: 'token',
             targetDirectory: 'path',
-            milestone: 'milestone'
+            milestone: 'milestone',
+            logFilePath: './output.log'
         }
 
         const buildAgent = {
@@ -86,7 +88,8 @@ describe('GitReleaseManager settings', () => {
             owner: 'owner',
             token: 'token',
             targetDirectory: 'path',
-            milestone: 'milestone'
+            milestone: 'milestone',
+            logFilePath: './output.log'
         }
 
         const buildAgent = {
@@ -106,7 +109,8 @@ describe('GitReleaseManager settings', () => {
             owner: 'owner',
             token: 'token',
             targetDirectory: 'path',
-            milestone: 'milestone'
+            milestone: 'milestone',
+            logFilePath: './output.log'
         }
 
         const buildAgent = {
@@ -127,7 +131,8 @@ describe('GitReleaseManager settings', () => {
             token: 'token',
             targetDirectory: 'path',
             milestone: 'milestone',
-            assets: ['asset']
+            assets: ['asset'],
+            logFilePath: './output.log'
         }
 
         const buildAgent = {

diff --git a/src/__tests__/tools/gitreleasemanager/tool.spec.ts b/src/__tests__/tools/gitreleasemanager/tool.spec.ts
@@ -136,7 +136,8 @@ describe('GitReleaseManagerTool', () => {
             owner: 'owner',
             repository: 'repo',
             token: 'token',
-            targetDirectory: 'targetDirectory'
+            targetDirectory: 'targetDirectory',
+            logFilePath: './logFilePath'
         } as CommonSettings
 
         beforeEach(() => {
@@ -162,7 +163,18 @@ describe('GitReleaseManagerTool', () => {
                     ...commonSettings
                 } as CommonSettings)
 
-                expect(args).toEqual(['--owner', 'owner', '--repository', 'repo', '--token', 'token', '--targetDirectory', 'targetDirectory'])
+                expect(args).toEqual([
+                    '--owner',
+                    'owner',
+                    '--repository',
+                    'repo',
+                    '--token',
+                    'token',
+                    '--targetDirectory',
+                    'targetDirectory',
+                    '--logFilePath',
+                    './logFilePath'
+                ])
             })
 
             it('should throw error for non-existing target path', async () => {
@@ -192,7 +204,8 @@ describe('GitReleaseManagerTool', () => {
                     milestone: 'milestone',
                     inputFileName: 'inputFileName',
                     isPreRelease: true,
-                    assets: ['asset1', 'asset2']
+                    assets: ['asset1', 'asset2'],
+                    logFilePath: './createLogFilePath'
                 } as CreateSettings)
 
                 expect(args).toEqual([
@@ -205,6 +218,8 @@ describe('GitReleaseManagerTool', () => {
                     'token',
                     '--targetDirectory',
                     'targetDirectory',
+                    '--logFilePath',
+                    './createLogFilePath',
                     '--milestone',
                     'milestone',
                     '--name',
@@ -237,6 +252,8 @@ describe('GitReleaseManagerTool', () => {
                     'token',
                     '--targetDirectory',
                     'targetDirectory',
+                    '--logFilePath',
+                    './logFilePath',
                     '--milestone',
                     'milestone'
                 ])
@@ -260,6 +277,8 @@ describe('GitReleaseManagerTool', () => {
                     'token',
                     '--targetDirectory',
                     'targetDirectory',
+                    '--logFilePath',
+                    './logFilePath',
                     '--milestone',
                     'milestone'
                 ])
@@ -283,6 +302,8 @@ describe('GitReleaseManagerTool', () => {
                     'token',
                     '--targetDirectory',
                     'targetDirectory',
+                    '--logFilePath',
+                    './logFilePath',
                     '--milestone',
                     'milestone'
                 ])
@@ -306,6 +327,8 @@ describe('GitReleaseManagerTool', () => {
                     'token',
                     '--targetDirectory',
                     'targetDirectory',
+                    '--logFilePath',
+                    './logFilePath',
                     '--tagName',
                     'milestone'
                 ])
@@ -330,6 +353,8 @@ describe('GitReleaseManagerTool', () => {
                     'token',
                     '--targetDirectory',
                     'targetDirectory',
+                    '--logFilePath',
+                    './logFilePath',
                     '--tagName',
                     'milestone',
                     '--assets',