fix(gv-policy-studio): allow to reset condition

Fixes #388

src/policy-studio/gv-policy-studio.js

@@ -840,6 +840,9 @@ export class GvPolicyStudio extends KeyboardElement(LitElement) {
   _onSubmitFlow({ detail: { values } }) {
     const selectedFlow = this.getSelectedFlow();
     const selectedFlowUpdated = Object.assign({}, selectedFlow, values);
+    if (!values.condition) {
+      selectedFlowUpdated.condition = null;
+    }
     if (!deepEqual(selectedFlow, selectedFlowUpdated)) {
       Object.assign(selectedFlow, selectedFlowUpdated, { _dirty: true });
       this.isDirty = true;

stories/policy-studio/gv-policy-studio.test.js

@@ -370,5 +370,35 @@ describe('P O L I C Y  S T U D I O', () => {
         ]);
       }
     });
+
+    test('should reset condition', async () => {
+      const _id = 'foobar';
+      const _stepId = 'foobar-step';
+      const step = { _id: _stepId, name: 'step', description: 'step description', configuration: {}, policy: 'api-key' };
+      const flow = {
+        _id,
+        name: 'New flow',
+        description: 'test',
+        condition: '#method == "POST"',
+        pre: [step],
+        post: [],
+      };
+      const values = {
+        _id,
+        name: 'Updated flow',
+        description: 'test',
+        pre: [step],
+        post: [],
+      };
+      component.definition = { flows: [flow] };
+      await component._onSelectFlows({ detail: { flows: [flow._id] } });
+
+      // execute update
+      await component._onSubmitFlow({ detail: { values } });
+
+      expect(component.definition.flows.length).toEqual(1);
+      expect(component.definition.flows[0]._id).toEqual(_id);
+      expect(component.definition.flows[0].condition).toBeNull();
+    });
   });
 });