fix: allow sub class to override event methods

Author: fengmk2

.github/workflows/nodejs.yml

@@ -11,6 +11,6 @@ jobs:
     name: Node.js
     uses: node-modules/github-actions/.github/workflows/node-test.yml@master
     with:
-      version: '18.19.0, 18, 20, 22'
+      version: '18.19.0, 18, 20, 22, 23'
     secrets:
       CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

package.json

@@ -41,6 +41,7 @@
     "lint": "eslint --cache src test --ext .ts",
     "pretest": "npm run lint -- --fix && npm run prepublishOnly",
     "test": "egg-bin test -p --timeout 5000",
+    "test-local": "egg-bin test --timeout 5000",
     "preci": "npm run lint && npm run prepublishOnly && attw --pack",
     "ci": "egg-bin cov -p",
     "prepublishOnly": "tshy && tshy-after"

src/index.ts

@@ -45,7 +45,7 @@ export abstract class Base<T = any> extends ReadyEventEmitter {
     }
     this.options = options ?? {};
     this.#localStorage = this.options.localStorage ?? getAsyncLocalStorage<T>();
-    this.on('error', err => {
+    super.on('error', err => {
       this._defaultErrorHandler(err);
     });
   }

test/index.test.ts

@@ -6,7 +6,23 @@ import { Base, BaseOptions } from '../src/index.js';
 describe('test/index.test.ts', () => {
   class SomeServiceClient extends Base {}
 
+  class SomeServiceClient2 extends Base {
+    protected _lists: any[] = [];
+
+    on(...args: any[]) {
+      this._lists.push(args);
+      super.on(args[0], args[1]);
+      return this;
+    }
+  }
+
   describe('default error handler', () => {
+    it('should allow subclass to override on methods', () => {
+      const c = new SomeServiceClient2();
+      c.on('error', () => {});
+      assert.equal(c.listeners('error').length, 2);
+    });
+
     it('should auto add the default error handler and show error message', () => {
       const c = new SomeServiceClient();
       assert.equal(c.listeners('error').length, 1);