increase type test coverage

Increase type test coverage fro the on method

Author: chris-pardy

types/index.test-d.ts

@@ -4,14 +4,15 @@ import rulesEngine, {
   Almanac,
   EngineResult,
   Engine,
+  Event,
   Fact,
   Operator,
   OperatorEvaluator,
   PathResolver,
   Rule,
   RuleProperties,
-  RuleSerializable,
-  Event
+  RuleResult,
+  RuleSerializable
 } from "../";
 
 // setup basic fixture data
@@ -94,14 +95,15 @@ expectType<Engine>(engine.addFact(fact));
 expectType<Engine>(engine.addFact(dynamicFact));
 expectType<boolean>(engine.removeFact(fact));
 expectType<Fact<string>>(engine.getFact<string>("test"));
-engine.on('success', (event) => {
+engine.on('success', (event, almanac, ruleResult) => {
   expectType<Event>(event)
+  expectType<Almanac>(alamanc)
+  expectType<RuleResult>(ruleResult)
 })
-engine.on('failure', (event) => {
-  expectType<Event>(event)
-})
-engine.on<{ foo: Array<string> }>('foo', (event) => {
+engine.on<{ foo: Array<string> }>('foo', (event, alamanc, ruleResult) => {
   expectType<{ foo: Array<string> }>(event)
+  expectType<Alamanc>(alamanc)
+  expectType<RuleResult>(ruleResult)
 })
 
 // Run the Engine