feat(3475): Preserve DISABLED state of child pipelines during sync

lib/pipeline.js

@@ -1076,6 +1076,9 @@ class PipelineModel extends BaseModel {
                     break;
                 }
 
+                // Preserve DISABLED state; otherwise set to ACTIVE
+                const newState = pipeline.state === 'DISABLED' ? 'DISABLED' : 'ACTIVE';
+
                 createOrUpdatePromises.push(
                     this._updateChildPipeline(
                         pipeline,
@@ -1085,7 +1088,7 @@ class PipelineModel extends BaseModel {
                             scmUri,
                             scmRepo,
                             name: scmRepo.name,
-                            state: 'ACTIVE'
+                            state: newState
                         },
                         scmUrl
                     )

package.json

@@ -58,7 +58,7 @@
     "js-yaml": "^4.1.1",
     "lodash": "^4.17.23",
     "screwdriver-config-parser": "^12.0.0",
-    "screwdriver-data-schema": "^25.12.0",
+    "screwdriver-data-schema": "^26.1.0",
     "screwdriver-logger": "^3.0.0",
     "screwdriver-workflow-parser": "^6.0.0"
   }

test/lib/pipeline.test.js

@@ -1719,6 +1719,36 @@ describe('Pipeline Model', () => {
             });
         });
 
+        it('preserves DISABLED state of child pipeline during sync', () => {
+            const parsedYaml = hoek.clone(EXTERNAL_PARSED_YAML);
+            const childPipelineFooMock = getChildPipelineMock({ ...childPipelineGitHubFooConfig, state: 'DISABLED' });
+
+            parsedYaml.childPipelines = {
+                scmUrls: [SCM_URL_GITHUB_FOO]
+            };
+
+            jobs = [mainJob, publishJob];
+            jobFactoryMock.list.resolves(jobs);
+            getUserPermissionMocks({ username: 'batman', push: true, admin: true });
+            scmMock.getFile.resolves('yamlcontentwithscmurls');
+            parserMock.withArgs({ ...parserConfig, ...{ yaml: 'yamlcontentwithscmurls' } }).resolves(parsedYaml);
+
+            pipelineFactoryMock.list
+                .withArgs({ params: { configPipelineId: testId } })
+                .resolves([childPipelineFooMock]);
+
+            pipeline.childPipelines = {
+                scmUrls: [SCM_URL_GITHUB_FOO]
+            };
+
+            return pipeline.sync().then(p => {
+                assert.equal(p.id, testId);
+                assert.notCalled(childPipelineFooMock.sync);
+                assert.calledOnce(childPipelineFooMock.update);
+                assert.equal(childPipelineFooMock.state, 'DISABLED');
+            });
+        });
+
         it('does not update child pipelines if does not belong to this parent', () => {
             jobs = [mainJob, publishJob];
             jobFactoryMock.list.resolves(jobs);