summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIttay Stern <ittay.stern@att.com>2019-05-30 08:39:05 +0000
committerGerrit Code Review <gerrit@onap.org>2019-05-30 08:39:05 +0000
commit04b8bbe7938b4f2119b969455e7de03d4987cce1 (patch)
treec7ea6a4d4facf3a9f599a564f58b88836002c017
parentbcc49b982fda33597e22c4463c06610de7dd95ba (diff)
parentac142979f8499af24390290f7800d3fe266583d1 (diff)
Merge "Initialize parameters maps before reading params" into dublin4.3.0
-rw-r--r--vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js8
-rw-r--r--vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js4
2 files changed, 5 insertions, 7 deletions
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 9709e8f32..02242cb37 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 709e8ca00..b11070a9f 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 @@ describe('Testing workFlows from SO', () => {
// when
return $controller.loadWorkFlows().then(() => {
expect($controller.workflows).toContain('inPlaceSoftwareUpdate');
- expect($controller.localWorkflowsParameters).toBeUndefined();
+ expect($controller.localWorkflowsParameters).toEqual(new Map());
});
});
@@ -196,7 +196,7 @@ describe('Testing workFlows from SO', () => {
return $controller.loadWorkFlows()
.then(() => {
expect($controller.workflows).toEqual(["workflow 0"]);
- expect($controller.remoteWorkflowsParameters).toEqual(undefined);
+ expect($controller.remoteWorkflowsParameters).toEqual(new Map());
});
});