feat: skip git clone when the sd.yaml has config-parser-error (#183)

Author: y-oksaku

index.js

@@ -269,7 +269,9 @@ function parsePipelineYaml({
                                 }
                             ],
                             secrets: [],
-                            environment: {}
+                            environment: {
+                                SD_SKIP_REPOSITORY_CLONE: 'true'
+                            }
                         }
                     ]
                 },

test/data/bad-build-cluster.json

@@ -11,7 +11,9 @@
                     }
                 ],
                 "secrets": [],
-                "environment": {}
+                "environment": {
+                    "SD_SKIP_REPOSITORY_CLONE": "true"
+                }
             }
         ]
     },

test/data/bad-notification-email-settings.json

@@ -11,7 +11,9 @@
                   }
               ],
               "secrets": [],
-              "environment": {}
+              "environment": {
+                "SD_SKIP_REPOSITORY_CLONE": "true"
+              }
           }
       ]
   },

test/data/bad-notification-shared-settings.json

@@ -11,7 +11,9 @@
                   }
               ],
               "secrets": [],
-              "environment": {}
+              "environment": {
+                "SD_SKIP_REPOSITORY_CLONE": "true"
+              }
           }
       ]
   },

test/data/bad-notification-slack-email-settings.json

@@ -11,7 +11,9 @@
                   }
               ],
               "secrets": [],
-              "environment": {}
+              "environment": {
+                "SD_SKIP_REPOSITORY_CLONE": "true"
+              }
           }
       ]
   },

test/data/bad-notification-slack-settings.json

@@ -11,7 +11,9 @@
                   }
               ],
               "secrets": [],
-              "environment": {}
+              "environment": {
+                "SD_SKIP_REPOSITORY_CLONE": "true"
+              }
           }
       ]
   },

test/data/pipeline-cache-false-nonexist-job.json

@@ -11,7 +11,9 @@
                     }
                 ],
                 "secrets": [],
-                "environment": {}
+                "environment": {
+                    "SD_SKIP_REPOSITORY_CLONE": "true"
+                }
             }
         ]
     },

test/data/pipeline-cache-nonexist-job.json

@@ -11,7 +11,9 @@
                     }
                 ],
                 "secrets": [],
-                "environment": {}
+                "environment": {
+                    "SD_SKIP_REPOSITORY_CLONE": "true"
+                }
             }
         ]
     },

test/index.test.js

@@ -56,7 +56,7 @@ describe('config parser', () => {
                     });
                     assert.strictEqual(data.jobs.main[0].image, 'node:18');
                     assert.deepEqual(data.jobs.main[0].secrets, []);
-                    assert.deepEqual(data.jobs.main[0].environment, {});
+                    assert.deepEqual(data.jobs.main[0].environment, { SD_SKIP_REPOSITORY_CLONE: 'true' });
                     assert.strictEqual(data.jobs.main[0].commands[0].name, 'config-parse-error');
                     assert.match(data.jobs.main[0].commands[0].command, YAMLMISSING);
                     assert.match(data.errors[0], YAMLMISSING);
@@ -67,7 +67,7 @@ describe('config parser', () => {
                 parser({ yaml: 'foo: :' }).then(data => {
                     assert.strictEqual(data.jobs.main[0].image, 'node:18');
                     assert.deepEqual(data.jobs.main[0].secrets, []);
-                    assert.deepEqual(data.jobs.main[0].environment, {});
+                    assert.deepEqual(data.jobs.main[0].environment, { SD_SKIP_REPOSITORY_CLONE: 'true' });
                     assert.strictEqual(data.jobs.main[0].commands[0].name, 'config-parse-error');
                     assert.match(data.jobs.main[0].commands[0].command, /YAMLException:/);
                     assert.match(data.errors[0], /YAMLException:/);
@@ -77,7 +77,7 @@ describe('config parser', () => {
                 parser({ yaml: loadData('basic-job-with-duplicate-steps.yaml') }).then(data => {
                     assert.strictEqual(data.jobs.main[0].image, 'node:18');
                     assert.deepEqual(data.jobs.main[0].secrets, []);
-                    assert.deepEqual(data.jobs.main[0].environment, {});
+                    assert.deepEqual(data.jobs.main[0].environment, { SD_SKIP_REPOSITORY_CLONE: 'true' });
                     assert.strictEqual(data.jobs.main[0].commands[0].name, 'config-parse-error');
                     assert.match(data.jobs.main[0].commands[0].command, /Error:.*main has duplicate step: publish/);
                     assert.match(data.errors[0], /Error:.*main has duplicate step: publish/);
@@ -95,7 +95,7 @@ describe('config parser', () => {
                 parser({ yaml: 'foo: bar\n---\nfoo: baz' }).then(data => {
                     assert.strictEqual(data.jobs.main[0].image, 'node:18');
                     assert.deepEqual(data.jobs.main[0].secrets, []);
-                    assert.deepEqual(data.jobs.main[0].environment, {});
+                    assert.deepEqual(data.jobs.main[0].environment, { SD_SKIP_REPOSITORY_CLONE: 'true' });
                     assert.strictEqual(data.jobs.main[0].commands[0].name, 'config-parse-error');
                     assert.match(data.jobs.main[0].commands[0].command, /YAMLException:.*ambigious/);
                     assert.match(data.errors[0], /YAMLException:.*ambigious/);
@@ -105,7 +105,7 @@ describe('config parser', () => {
                 parser({ yaml: 'jobs: {}\n---\nversion: 4' }).then(data => {
                     assert.strictEqual(data.jobs.main[0].image, 'node:18');
                     assert.deepEqual(data.jobs.main[0].secrets, []);
-                    assert.deepEqual(data.jobs.main[0].environment, {});
+                    assert.deepEqual(data.jobs.main[0].environment, { SD_SKIP_REPOSITORY_CLONE: 'true' });
                     assert.strictEqual(data.jobs.main[0].commands[0].name, 'config-parse-error');
                     assert.match(data.errors[0], 'ValidationError: "jobs" is required');
                 }));