Merge pull request #267 from CacheControl/remove-rule-ts

Author: CacheControl

Diff for: package.json

@@ -1,6 +1,6 @@
 {
   "name": "json-rules-engine",
-  "version": "6.0.1",
+  "version": "6.1.0",
   "description": "Rules Engine expressed in simple json",
   "main": "dist/index.js",
   "types": "types/index.d.ts",

Diff for: types/index.d.ts

@@ -20,7 +20,8 @@ export class Engine {
   constructor(rules?: Array<RuleProperties>, options?: EngineOptions);
 
   addRule(rule: RuleProperties): this;
-  removeRule(rule: Rule): boolean;
+  removeRule(ruleOrName: Rule | string): boolean;
+  updateRule(rule: Rule): void;
 
   addOperator(operator: Operator): Map<string, Operator>;
   addOperator<A, B>(

Diff for: types/index.test-d.ts

@@ -58,6 +58,7 @@ const rule: Rule = new Rule(ruleProps);
 const ruleFromString: Rule = new Rule(JSON.stringify(ruleProps));
 expectType<Engine>(engine.addRule(rule));
 expectType<boolean>(engine.removeRule(ruleFromString));
+expectType<void>(engine.updateRule(ruleFromString));
 
 expectType<Rule>(rule.setConditions({ any: [] }));
 expectType<Rule>(rule.setEvent({ type: "test" }));