fix: node12 inspection of provider

See: nodejs/node#26241
groupon · Dec 9, 2019 · 0cb2b14 · 0cb2b14
1 parent d50a30b
commit 0cb2b14
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 20 deletions.
diff --git a/lib/registry/injector.js b/lib/registry/injector.js
@@ -43,34 +43,18 @@ const { parseDependencyQuery } = require('./query');
 
 const INSPECT = util.inspect.custom || Symbol.for('nodejs.util.inspect.custom');
 
-/**
- * @param {Injector} injector
- */
-function inspectProvider(injector) {
-  // @ts-ignore
-  return `Provider { ${injector[INSPECT]()} }`;
-}
-
 const PROVIDER_HANDLER = {
   /**
    * @param {Injector} injector
-   * @param {string | symbol} key
+   * @param {string} key
    */
   get(injector, key) {
     switch (key) {
       case 'constructor':
-        return injector.constructor;
-
       case 'get':
-        return injector.get;
-
       case 'keys':
-        return injector.keys;
-
-      case INSPECT: {
-        return inspectProvider.bind(null, injector);
-      }
-
+      case 'scope':
+        return injector[key];
       default:
         return injector.get(key);
     }

diff --git a/test/registry.test.js b/test/registry.test.js
@@ -125,7 +125,7 @@ describe('Registry', () => {
       const provider = injector.getProvider();
       assert.equal(
         inspect(provider),
-        'Provider { Injector<action> { action, byAction, request, response, byReq, Symbol(byReq), constValue } }'
+        'Injector<action> { action, byAction, request, response, byReq, Symbol(byReq), constValue }'
       );
     });
   });