make artifacts optional

saucelabs · Mar 19, 2024 · 040d02c · 040d02c
1 parent e8d4fcb
commit 040d02c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/playwright-runner.ts b/src/playwright-runner.ts
@@ -184,8 +184,9 @@ function zipArtifacts(runCfg: RunnerConfig | CucumberRunnerConfig) {
   if (!runCfg.artifacts || !runCfg.artifacts.retain) {
     return;
   }
-  Object.keys(runCfg.artifacts.retain).forEach((source) => {
-    const dest = path.join(runCfg.assetsDir, runCfg.artifacts.retain[source]);
+  const archivesMap = runCfg.artifacts.retain;
+  Object.keys(archivesMap).forEach((source) => {
+    const dest = path.join(runCfg.assetsDir, archivesMap[source]);
     try {
       zip(path.dirname(runCfg.path), source, dest);
     } catch (err) {

diff --git a/src/types.ts b/src/types.ts
@@ -76,7 +76,7 @@ export interface CucumberRunnerConfig {
   path: string;
   preExecTimeout: number;
   projectPath: string;
-  artifacts: Artifacts;
+  artifacts?: Artifacts;
 }
 
 export interface CucumberSuite {
@@ -103,7 +103,7 @@ export interface CucumberSuite {
 }
 
 export interface Artifacts {
-  retain: {
+  retain?: {
     [key: string]: string;
   };
 }