Fix bug with extends in multi levels. Apparent extends is using deep-merge, not deep-extends like the rest of the YAML (#112)

Author: firecow

src/job-expanders.ts

@@ -28,6 +28,10 @@ export function jobExtends(gitlabData: any) {
                 delete jobData.extends;
             }
 
+            if (parentData.extends) {
+                jobData.extends = (jobData.extends || []).concat(parentData.extends);
+            }
+
             gitlabData[jobName] = deepExtend({}, parentData, jobData);
         }
 

tests/cases.test.ts

@@ -280,11 +280,15 @@ test('dotenv <test-job>', async () => {
     expect(mockProcessStdout).toHaveBeenCalledWith("Test something\n");
 });
 
-test('extends', async () => {
+test('extends <test-job>', async () => {
     await defaultCmd.handler({
-        cwd: 'tests/test-cases/extends'
+        cwd: 'tests/test-cases/extends',
+        job: 'test-job'
     });
+
+    expect(mockProcessStdout).toHaveBeenCalledWith("Test something (before_script)\n");
     expect(mockProcessStdout).toHaveBeenCalledWith("Test something\n");
+    expect(mockProcessStdout).toHaveBeenCalledWith("Test something (after_script)\n");
     expect(mockProcessExit).toBeCalledTimes(0);
 });
 

tests/test-cases/extends/.gitlab-ci.yml

@@ -1,13 +1,21 @@
 ---
-.parent1:
+.after:
+  after_script:
+    - echo "Test something (after_script)"
+
+.before:
   stage: test
+  before_script:
+    - echo "Test something (before_script)"
   script:
     - echo "Should not be printed"
 
-.parent2:
-  extends: .parent1
+.parent:
+  extends: .before
   script:
     - echo "Test something"
 
 test-job:
-  extends: .parent2
+  extends:
+    - .after
+    - .parent