Skip to content

Commit 9b5b703

Browse files
committed
fix: node12 inspection of provider
See: nodejs/node#26241
1 parent d50a30b commit 9b5b703

File tree

2 files changed

+7
-18
lines changed

2 files changed

+7
-18
lines changed

lib/registry/injector.js

+6-17
Original file line numberDiff line numberDiff line change
@@ -41,15 +41,7 @@ const { parseDependencyQuery } = require('./query');
4141
/** @typedef {import('../typedefs').DependencyDescriptor} DependencyDescriptor */
4242
/** @typedef {import('../typedefs').DependencyQuery} DependencyQuery */
4343

44-
const INSPECT = util.inspect.custom || Symbol.for('nodejs.util.inspect.custom');
45-
46-
/**
47-
* @param {Injector} injector
48-
*/
49-
function inspectProvider(injector) {
50-
// @ts-ignore
51-
return `Provider { ${injector[INSPECT]()} }`;
52-
}
44+
const INSPECT = util.inspect.custom;
5345

5446
const PROVIDER_HANDLER = {
5547
/**
@@ -59,17 +51,14 @@ const PROVIDER_HANDLER = {
5951
get(injector, key) {
6052
switch (key) {
6153
case 'constructor':
62-
return injector.constructor;
63-
6454
case 'get':
65-
return injector.get;
66-
6755
case 'keys':
68-
return injector.keys;
56+
case 'scope':
57+
return injector[key];
6958

70-
case INSPECT: {
71-
return inspectProvider.bind(null, injector);
72-
}
59+
case INSPECT:
60+
// @ts-ignore
61+
return injector[INSPECT];
7362

7463
default:
7564
return injector.get(key);

test/registry.test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ describe('Registry', () => {
125125
const provider = injector.getProvider();
126126
assert.equal(
127127
inspect(provider),
128-
'Provider { Injector<action> { action, byAction, request, response, byReq, Symbol(byReq), constValue } }'
128+
'Injector<action> { action, byAction, request, response, byReq, Symbol(byReq), constValue }'
129129
);
130130
});
131131
});

0 commit comments

Comments
 (0)