updated on method overload to include generic method call

Author: jagernet-ops

types/index.d.ts

@@ -53,7 +53,7 @@ export class Engine {
 
   on(eventName: "success", handler: EventHandler): this;
   on(eventName: "failure", handler: EventHandler): this;
-  on(eventName: string, handler: EventHandler): this;
+  on<T>(eventName: string, handler: EventHandler<T>): this;
 
   run(facts?: Record<string, any>, runOptions?: RunOptions): Promise<EngineResult>;
   stop(): this;
@@ -119,8 +119,8 @@ export interface Event {
 
 export type PathResolver = (value: object, path: string) => any;
 
-export type EventHandler = (
-  event: Event,
+export type EventHandler<T = Event> = (
+  event: T,
   almanac: Almanac,
   ruleResult: RuleResult
 ) => void;

types/index.test-d.ts

@@ -10,7 +10,8 @@ import rulesEngine, {
   PathResolver,
   Rule,
   RuleProperties,
-  RuleSerializable
+  RuleSerializable,
+  Event
 } from "../";
 
 // setup basic fixture data
@@ -93,6 +94,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) => {
+  expectType<Event>(event)
+})
+engine.on('failure', (event) => {
+  expectType<Event>(event)
+})
+engine.on<{ foo: Array<string> }>('foo', (event) => {
+  expectType<{ foo: Array<string> }>(event)
+})
 
 // Run the Engine
 expectType<Promise<EngineResult>>(engine.run({ displayMessage: true }));