Merge "Initialize parameters maps before reading params"
diff --git a/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js b/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js
index 9709e8f..02242cb 100644
--- a/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js
+++ b/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js
@@ -433,7 +433,6 @@
} else {
let source = vm.getRemoteWorkflowSource(vm.changeManagement.workflow);
if( source === "NATIVE"){
- vm.localWorkflowsParameters = vm.remoteWorkflowsParameters;
vm.triggerLocalWorkflow();
}else {
vm.triggerRemoteWorkflow();
@@ -687,7 +686,9 @@
};
vm.loadWorkFlows = function () {
- if (featureFlags.isOn(COMPONENT.FEATURE_FLAGS.FLAG_HANDLE_SO_WORKFLOWS)) {
+ vm.localWorkflowsParameters = new Map();
+ vm.remoteWorkflowsParameters = new Map();
+ if (featureFlags.isOn(COMPONENT.FEATURE_FLAGS.FLAG_HANDLE_SO_WORKFLOWS)) {
return vm.loadRemoteWorkFlows()
.then(function () {
vm.workflows = vm.remoteWorkflows.map(item => item.name);
@@ -723,7 +724,6 @@
};
vm.loadLocalWorkFlowsParameters = function () {
- vm.localWorkflowsParameters = new Map();
vm.localWorkflows.forEach(function(workflow) {
vm.loadLocalWorkFlowParameters(workflow);
});
@@ -740,12 +740,10 @@
};
vm.loadRemoteWorkFlowsParameters = function () {
- vm.remoteWorkflowsParameters = new Map();
vm.remoteWorkflows.forEach(function(workflow) {
if (workflow.source ==='SDC' || workflow.source === 'sdc' ){
vm.loadRemoteWorkFlowParameters(workflow);
} else {
- vm.localWorkflowsParameters = new Map();
vm.loadLocalWorkFlowParameters(workflow.name);
}
});
diff --git a/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js b/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js
index 709e8ca..b11070a 100644
--- a/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js
+++ b/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js
@@ -127,7 +127,7 @@
// when
return $controller.loadWorkFlows().then(() => {
expect($controller.workflows).toContain('inPlaceSoftwareUpdate');
- expect($controller.localWorkflowsParameters).toBeUndefined();
+ expect($controller.localWorkflowsParameters).toEqual(new Map());
});
});
@@ -196,7 +196,7 @@
return $controller.loadWorkFlows()
.then(() => {
expect($controller.workflows).toEqual(["workflow 0"]);
- expect($controller.remoteWorkflowsParameters).toEqual(undefined);
+ expect($controller.remoteWorkflowsParameters).toEqual(new Map());
});
});