feat: update tests and NoopContextManager

Leave checking special cases for the actual ContextManager and remove
the short-circuit from the API implementation.

Author: rauno56

src/api/context.ts

@@ -83,10 +83,7 @@ export class ContextAPI {
    * @param target function or event emitter to bind
    */
   public bind<T>(context: Context, target: T): T {
-    if (context === undefined) {
-      return target;
-    }
-    return this._getContextManager().bind(target, context);
+    return this._getContextManager().bind(context, target);
   }
 
   private _getContextManager(): ContextManager {

src/context/NoopContextManager.ts

@@ -31,7 +31,7 @@ export class NoopContextManager implements types.ContextManager {
     return fn.call(thisArg, ...args);
   }
 
-  bind<T>(target: T, _context?: types.Context): T {
+  bind<T>(_context: types.Context, target: T): T {
     return target;
   }
 

src/context/types.ts

@@ -65,7 +65,7 @@ export interface ContextManager {
    * @param target Any object to which a context need to be set
    * @param [context] Optionally specify the context which you want to assign
    */
-  bind<T>(target: T, context?: Context): T;
+  bind<T>(context: Context, target: T): T;
 
   /**
    * Enable context management

test/context/NoopContextManager.test.ts

@@ -119,13 +119,13 @@ describe('NoopContextManager', () => {
   describe('.bind()', () => {
     it('should return the same target (when enabled)', () => {
       const test = { a: 1 };
-      assert.deepStrictEqual(contextManager.bind(test), test);
+      assert.deepStrictEqual(contextManager.bind(contextManager.active(), test), test);
     });
 
     it('should return the same target (when disabled)', () => {
       contextManager.disable();
       const test = { a: 1 };
-      assert.deepStrictEqual(contextManager.bind(test), test);
+      assert.deepStrictEqual(contextManager.bind(contextManager.active(), test), test);
       contextManager.enable();
     });
   });