Test EventGateway rules

ProcessMaker · May 28, 2021 · 461444b · 461444b
1 parent 4012f85
commit 461444b
Show file tree

Hide file tree

Showing 4 changed files with 367 additions and 1 deletion.
diff --git a/rule-tester.js b/rule-tester.js
@@ -0,0 +1,90 @@
+/* global it, describe, beforeEach */
+
+const assert = require('assert');
+
+const Linter = require('bpmnlint/lib/linter');
+
+function createResolver(rule) {
+  return {
+    resolveRule: () => Promise.resolve(rule)
+  };
+}
+
+function expectEqual(a, b) {
+  assert.deepStrictEqual(a, b);
+}
+
+
+function verify(ruleName, rule, testCases) {
+  const linterConfig = {
+    rules: { [ruleName]: 2 }
+  };
+
+  describe(`rules/${ruleName}`, function() {
+
+    let linter;
+
+    beforeEach(function() {
+      linter = new Linter({
+        resolver: createResolver(rule)
+      });
+    });
+
+
+    describe('should lint valid', function() {
+
+      testCases.valid.forEach(({ moddleElement }, idx) => (
+
+        it(`test case #${idx + 1}`, function() {
+          return (
+            Promise.resolve(moddleElement)
+              .then(moddleRoot => {
+                return linter.lint(moddleRoot.root, linterConfig);
+              })
+              .then(lintResults => {
+                expectEqual(lintResults, {});
+              })
+          );
+        })
+
+      ));
+
+    });
+
+
+    describe('should lint invalid', function() {
+
+      testCases.invalid.forEach(({ moddleElement, report }, idx) => (
+
+        it(`test case #${idx}`, function() {
+
+          const expectedResult = report instanceof Array ? report : [{
+            ...report,
+            category: 'error'
+          }];
+
+          return (
+            Promise.resolve(moddleElement)
+              .then(moddleRoot => {
+                return linter.lint(moddleRoot.root, linterConfig);
+              })
+              .then(lintResults => {
+                expectEqual(lintResults, {
+                  [ruleName]: expectedResult
+                });
+              })
+          );
+        })
+
+      ));
+
+    });
+
+  });
+
+}
+
+
+module.exports = {
+  verify
+};
diff --git a/test-diagrams/event-based-gateway.invalid.bpmn b/test-diagrams/event-based-gateway.invalid.bpmn
@@ -0,0 +1,133 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<bpmn:definitions xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:bpmn="http://www.omg.org/spec/BPMN/20100524/MODEL" xmlns:bpmndi="http://www.omg.org/spec/BPMN/20100524/DI" xmlns:dc="http://www.omg.org/spec/DD/20100524/DC" xmlns:di="http://www.omg.org/spec/DD/20100524/DI" xmlns:pm="http://processmaker.com/BPMN/2.0/Schema.xsd" xmlns:tns="http://sourceforge.net/bpmn/definitions/_1530553328908" xmlns:xsd="http://www.w3.org/2001/XMLSchema" targetNamespace="http://bpmn.io/schema/bpmn" exporter="ProcessMaker Modeler" exporterVersion="1.0" xsi:schemaLocation="http://www.omg.org/spec/BPMN/20100524/MODEL http://bpmn.sourceforge.net/schemas/BPMN20.xsd">
+  <bpmn:process id="ProcessId" name="ProcessName" isExecutable="true">
+    <bpmn:startEvent id="node_1" name="Start Event" pm:allowInterstitial="false" pm:config="{&#34;web_entry&#34;:null}">
+      <bpmn:outgoing>node_5</bpmn:outgoing>
+    </bpmn:startEvent>
+    <bpmn:eventBasedGateway id="node_3" name="Event Based Gateway">
+      <bpmn:incoming>node_5</bpmn:incoming>
+      <bpmn:outgoing>node_9</bpmn:outgoing>
+      <bpmn:outgoing>node_11</bpmn:outgoing>
+      <bpmn:outgoing>node_26</bpmn:outgoing>
+    </bpmn:eventBasedGateway>
+    <bpmn:sequenceFlow id="node_5" name="" sourceRef="node_1" targetRef="node_3" />
+    <bpmn:intermediateCatchEvent id="node_6" name="Intermediate Timer Event">
+      <bpmn:incoming>node_9</bpmn:incoming>
+      <bpmn:incoming>node_10</bpmn:incoming>
+      <bpmn:outgoing>node_18</bpmn:outgoing>
+      <bpmn:timerEventDefinition>
+        <bpmn:timeDuration>PT1H</bpmn:timeDuration>
+      </bpmn:timerEventDefinition>
+    </bpmn:intermediateCatchEvent>
+    <bpmn:sequenceFlow id="node_9" name="" sourceRef="node_3" targetRef="node_6" />
+    <bpmn:endEvent id="node_19" name="Message End Event">
+      <bpmn:incoming>node_18</bpmn:incoming>
+      <bpmn:messageEventDefinition messageRef="node_19_message" />
+    </bpmn:endEvent>
+    <bpmn:sequenceFlow id="node_18" name="" sourceRef="node_6" targetRef="node_19" />
+    <bpmn:endEvent id="node_24" name="Signal End Event">
+      <bpmn:incoming>node_23</bpmn:incoming>
+      <bpmn:signalEventDefinition signalRef="sync_assignment" />
+    </bpmn:endEvent>
+    <bpmn:endEvent id="node_28" name="End Event" pm:screenRef="">
+      <bpmn:incoming>node_30</bpmn:incoming>
+    </bpmn:endEvent>
+    <bpmn:intermediateThrowEvent id="node_2" name="Intermediate Message Throw Event">
+      <bpmn:incoming>node_11</bpmn:incoming>
+      <bpmn:outgoing>node_23</bpmn:outgoing>
+      <bpmn:messageEventDefinition messageRef="node_2_message" />
+    </bpmn:intermediateThrowEvent>
+    <bpmn:sequenceFlow id="node_11" name="" sourceRef="node_3" targetRef="node_2" />
+    <bpmn:sequenceFlow id="node_23" name="" sourceRef="node_2" targetRef="node_24" />
+    <bpmn:intermediateThrowEvent id="node_4" name="Intermediate Message Throw Event">
+      <bpmn:incoming>node_26</bpmn:incoming>
+      <bpmn:outgoing>node_30</bpmn:outgoing>
+      <bpmn:messageEventDefinition messageRef="node_4_message" />
+    </bpmn:intermediateThrowEvent>
+    <bpmn:sequenceFlow id="node_26" name="" sourceRef="node_3" targetRef="node_4" />
+    <bpmn:sequenceFlow id="node_30" name="" sourceRef="node_4" targetRef="node_28" />
+    <bpmn:task id="node_7" name="Form Task" pm:screenRef="" pm:allowInterstitial="false" pm:assignment="requester" pm:assignmentLock="false" pm:allowReassignment="false" pm:config="{&#34;web_entry&#34;:null,&#34;email_notifications&#34;:{&#34;notifications&#34;:[]}}">
+      <bpmn:incoming>node_14</bpmn:incoming>
+      <bpmn:outgoing>node_10</bpmn:outgoing>
+    </bpmn:task>
+    <bpmn:sequenceFlow id="node_10" sourceRef="node_7" targetRef="node_6" />
+    <bpmn:startEvent id="node_12" name="Start Event" pm:allowInterstitial="false" pm:config="{&#34;web_entry&#34;:null}">
+      <bpmn:outgoing>node_14</bpmn:outgoing>
+    </bpmn:startEvent>
+    <bpmn:sequenceFlow id="node_14" sourceRef="node_12" targetRef="node_7" />
+  </bpmn:process>
+  <bpmn:message id="node_19_message" name="node_19_message" />
+  <bpmn:signal id="sync_assignment" name="sync_assignment" />
+  <bpmn:message id="node_2_message" name="node_2_message" />
+  <bpmn:message id="node_4_message" name="node_4_message" />
+  <bpmndi:BPMNDiagram id="BPMNDiagramId">
+    <bpmndi:BPMNPlane id="BPMNPlaneId" bpmnElement="ProcessId">
+      <bpmndi:BPMNShape id="node_1_di" bpmnElement="node_1">
+        <dc:Bounds x="270" y="280" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNShape id="node_3_di" bpmnElement="node_3">
+        <dc:Bounds x="400" y="280" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_5_di" bpmnElement="node_5">
+        <di:waypoint x="288" y="298" />
+        <di:waypoint x="418" y="298" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_6_di" bpmnElement="node_6">
+        <dc:Bounds x="590" y="280" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_9_di" bpmnElement="node_9">
+        <di:waypoint x="418" y="298" />
+        <di:waypoint x="608" y="298" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_19_di" bpmnElement="node_19">
+        <dc:Bounds x="840" y="280" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_18_di" bpmnElement="node_18">
+        <di:waypoint x="608" y="298" />
+        <di:waypoint x="858" y="298" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_24_di" bpmnElement="node_24">
+        <dc:Bounds x="400" y="560" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNShape id="node_28_di" bpmnElement="node_28">
+        <dc:Bounds x="610" y="140" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNShape id="node_2_di" bpmnElement="node_2">
+        <dc:Bounds x="400" y="440" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_11_di" bpmnElement="node_11">
+        <di:waypoint x="418" y="298" />
+        <di:waypoint x="418" y="458" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNEdge id="node_23_di" bpmnElement="node_23">
+        <di:waypoint x="418" y="458" />
+        <di:waypoint x="418" y="578" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_4_di" bpmnElement="node_4">
+        <dc:Bounds x="400" y="140" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_26_di" bpmnElement="node_26">
+        <di:waypoint x="418" y="298" />
+        <di:waypoint x="418" y="158" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNEdge id="node_30_di" bpmnElement="node_30">
+        <di:waypoint x="418" y="158" />
+        <di:waypoint x="628" y="158" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_7_di" bpmnElement="node_7">
+        <dc:Bounds x="550" y="530" width="116" height="76" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_10_di" bpmnElement="node_10">
+        <di:waypoint x="608" y="568" />
+        <di:waypoint x="608" y="298" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_12_di" bpmnElement="node_12">
+        <dc:Bounds x="590" y="690" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_14_di" bpmnElement="node_14">
+        <di:waypoint x="608" y="708" />
+        <di:waypoint x="608" y="568" />
+      </bpmndi:BPMNEdge>
+    </bpmndi:BPMNPlane>
+  </bpmndi:BPMNDiagram>
+</bpmn:definitions>
diff --git a/test-diagrams/event-based-gateway.valid.bpmn b/test-diagrams/event-based-gateway.valid.bpmn
@@ -0,0 +1,107 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<bpmn:definitions xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:bpmn="http://www.omg.org/spec/BPMN/20100524/MODEL" xmlns:bpmndi="http://www.omg.org/spec/BPMN/20100524/DI" xmlns:dc="http://www.omg.org/spec/DD/20100524/DC" xmlns:di="http://www.omg.org/spec/DD/20100524/DI" xmlns:pm="http://processmaker.com/BPMN/2.0/Schema.xsd" xmlns:tns="http://sourceforge.net/bpmn/definitions/_1530553328908" xmlns:xsd="http://www.w3.org/2001/XMLSchema" targetNamespace="http://bpmn.io/schema/bpmn" exporter="ProcessMaker Modeler" exporterVersion="1.0" xsi:schemaLocation="http://www.omg.org/spec/BPMN/20100524/MODEL http://bpmn.sourceforge.net/schemas/BPMN20.xsd">
+  <bpmn:process id="ProcessId" name="ProcessName" isExecutable="true">
+    <bpmn:startEvent id="node_1" name="Start Event" pm:allowInterstitial="false" pm:config="{&#34;web_entry&#34;:null}">
+      <bpmn:outgoing>node_5</bpmn:outgoing>
+    </bpmn:startEvent>
+    <bpmn:eventBasedGateway id="node_3" name="Event Based Gateway">
+      <bpmn:incoming>node_5</bpmn:incoming>
+      <bpmn:outgoing>node_9</bpmn:outgoing>
+      <bpmn:outgoing>node_11</bpmn:outgoing>
+      <bpmn:outgoing>node_26</bpmn:outgoing>
+    </bpmn:eventBasedGateway>
+    <bpmn:sequenceFlow id="node_5" name="" sourceRef="node_1" targetRef="node_3" />
+    <bpmn:intermediateCatchEvent id="node_6" name="Intermediate Timer Event">
+      <bpmn:incoming>node_9</bpmn:incoming>
+      <bpmn:outgoing>node_18</bpmn:outgoing>
+      <bpmn:timerEventDefinition>
+        <bpmn:timeDuration>PT1H</bpmn:timeDuration>
+      </bpmn:timerEventDefinition>
+    </bpmn:intermediateCatchEvent>
+    <bpmn:sequenceFlow id="node_9" name="" sourceRef="node_3" targetRef="node_6" />
+    <bpmn:intermediateCatchEvent id="node_15" name="Intermediate Message Catch Event">
+      <bpmn:incoming>node_11</bpmn:incoming>
+      <bpmn:outgoing>node_23</bpmn:outgoing>
+      <bpmn:messageEventDefinition messageRef="node_19_message" />
+    </bpmn:intermediateCatchEvent>
+    <bpmn:sequenceFlow id="node_11" name="" sourceRef="node_3" targetRef="node_15" />
+    <bpmn:endEvent id="node_19" name="Message End Event">
+      <bpmn:incoming>node_18</bpmn:incoming>
+      <bpmn:messageEventDefinition messageRef="node_19_message" />
+    </bpmn:endEvent>
+    <bpmn:sequenceFlow id="node_18" name="" sourceRef="node_6" targetRef="node_19" />
+    <bpmn:endEvent id="node_24" name="Signal End Event">
+      <bpmn:incoming>node_23</bpmn:incoming>
+      <bpmn:signalEventDefinition signalRef="sync_assignment" />
+    </bpmn:endEvent>
+    <bpmn:sequenceFlow id="node_23" name="" sourceRef="node_15" targetRef="node_24" />
+    <bpmn:intermediateCatchEvent id="node_27" name="Intermediate Signal Catch Event">
+      <bpmn:incoming>node_26</bpmn:incoming>
+      <bpmn:outgoing>node_30</bpmn:outgoing>
+      <bpmn:signalEventDefinition signalRef="sync_assignment" />
+    </bpmn:intermediateCatchEvent>
+    <bpmn:sequenceFlow id="node_26" name="" sourceRef="node_3" targetRef="node_27" />
+    <bpmn:endEvent id="node_28" name="End Event" pm:screenRef="">
+      <bpmn:incoming>node_30</bpmn:incoming>
+    </bpmn:endEvent>
+    <bpmn:sequenceFlow id="node_30" name="" sourceRef="node_27" targetRef="node_28" />
+  </bpmn:process>
+  <bpmn:message id="node_19_message" name="node_19_message" />
+  <bpmn:signal id="sync_assignment" name="sync_assignment" />
+  <bpmndi:BPMNDiagram id="BPMNDiagramId">
+    <bpmndi:BPMNPlane id="BPMNPlaneId" bpmnElement="ProcessId">
+      <bpmndi:BPMNShape id="node_1_di" bpmnElement="node_1">
+        <dc:Bounds x="270" y="280" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNShape id="node_3_di" bpmnElement="node_3">
+        <dc:Bounds x="400" y="280" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_5_di" bpmnElement="node_5">
+        <di:waypoint x="288" y="298" />
+        <di:waypoint x="418" y="298" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_6_di" bpmnElement="node_6">
+        <dc:Bounds x="590" y="280" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_9_di" bpmnElement="node_9">
+        <di:waypoint x="418" y="298" />
+        <di:waypoint x="608" y="298" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_15_di" bpmnElement="node_15">
+        <dc:Bounds x="400" y="440" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_11_di" bpmnElement="node_11">
+        <di:waypoint x="418" y="298" />
+        <di:waypoint x="418" y="458" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_19_di" bpmnElement="node_19">
+        <dc:Bounds x="840" y="280" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_18_di" bpmnElement="node_18">
+        <di:waypoint x="608" y="298" />
+        <di:waypoint x="858" y="298" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_24_di" bpmnElement="node_24">
+        <dc:Bounds x="400" y="560" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_23_di" bpmnElement="node_23">
+        <di:waypoint x="418" y="458" />
+        <di:waypoint x="418" y="578" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_27_di" bpmnElement="node_27">
+        <dc:Bounds x="400" y="140" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_26_di" bpmnElement="node_26">
+        <di:waypoint x="418" y="298" />
+        <di:waypoint x="418" y="158" />
+      </bpmndi:BPMNEdge>
+      <bpmndi:BPMNShape id="node_28_di" bpmnElement="node_28">
+        <dc:Bounds x="610" y="140" width="36" height="36" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNEdge id="node_30_di" bpmnElement="node_30">
+        <di:waypoint x="418" y="158" />
+        <di:waypoint x="628" y="158" />
+      </bpmndi:BPMNEdge>
+    </bpmndi:BPMNPlane>
+  </bpmndi:BPMNDiagram>
+</bpmn:definitions>
diff --git a/test.js b/test.js
@@ -1,11 +1,12 @@
 const readModdle = require('bpmnlint/lib/testers/helper').readModdle;
-const RuleTester = require('bpmnlint/lib/testers/rule-tester');
+const RuleTester = require('./rule-tester');
 
 const gatewayDirectionRule = require('./rules/gateway-direction');
 const callActivityChildProcessRule = require('./rules/call-activity-child-process');
 const callActivitySequenceFlowRule = require('./rules/call-activity-sequence-flow');
 const idRequiredRule = require('./rules/id-required');
 const signalRefRequiredRule = require('./rules/signal-ref-required');
+const eventBasedGatewayRule = require('./rules/event-based-gateway');
 
 RuleTester.verify('gateway-direction', gatewayDirectionRule, {
   valid: [
@@ -165,3 +166,38 @@ RuleTester.verify('signal-ref-required', signalRefRequiredRule, {
     },
   ]
 });
+
+RuleTester.verify('event-based-gateway', eventBasedGatewayRule, {
+  valid: [
+    {
+      moddleElement: readModdle('./test-diagrams/event-based-gateway.valid.bpmn')
+    },
+  ],
+  invalid: [
+    {
+      moddleElement: readModdle('./test-diagrams/event-based-gateway.invalid.bpmn'),
+      report: [
+        {
+          "category": "error",
+          "id": "node_6",
+          "message": "Event Gateway target elements must not have additional incoming Sequence Flows"
+        },
+        {
+          "category": "error",
+          "id": "node_2",
+          "message": "Event Gateways target elements must be Catch Events"
+        },
+        {
+          "category": "error",
+          "id": "node_4",
+          "message": "Event Gateways target elements must be Catch Events"
+        },
+        {
+          "category": "error",
+          "id": "node_3",
+          "message": "Event Gateways target elements must be valid Catch Events"
+        }
+      ]
+    },
+  ]
+});