Merge pull request #9 from cloudgraphdev/fix/unit-tests

tyler-dunkel · web-flow · commit 664bfa1ebb7c · 2022-04-25T11:59:29.000-05:00
fix: Fixed empty rules edge case
diff --git a/src/rules-engine/data-processors/dgraph-data-processor.ts b/src/rules-engine/data-processors/dgraph-data-processor.ts
@@ -198,6 +198,11 @@ export default class DgraphDataProcessor implements DataProcessor {
   }
 
   prepareRulesMetadataMutations = (rules: Rule[] = []): Entity[] => {
+    // Return an empty array if there are no rules
+    if (isEmpty(rules)) {
+      return []
+    }
+
     return [
       {
         name: 'ruleMetadata',

Original file line number	Diff line number	Diff line change
`@@ -198,6 +198,11 @@ export default class DgraphDataProcessor implements DataProcessor {`
`198`	`198`	`}`
`199`	`199`
`200`	`200`	`prepareRulesMetadataMutations = (rules: Rule[] = []): Entity[] => {`
	`201`	`+ // Return an empty array if there are no rules`
	`202`	`+ if (isEmpty(rules)) {`
	`203`	`+ return []`
	`204`	`+ }`
	`205`	`+`
`201`	`206`	`return [`
`202`	`207`	`{`
`203`	`208`	`name: 'ruleMetadata',`