fix(context): allow inject to be explicitly invoked for class ctor …

…args Signed-off-by: Miroslav Bajtoš <mbajtoss@gmail.com>
loopbackio · Sep 5, 2019 · 6a0d4f2 · 6a0d4f2
1 parent 06d0967
commit 6a0d4f2
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/packages/context/src/__tests__/unit/inject.unit.ts b/packages/context/src/__tests__/unit/inject.unit.ts
@@ -29,6 +29,16 @@ describe('function argument injection', () => {
     expect(meta.map(m => m.bindingSelector)).to.deepEqual(['foo']);
   });
 
+  it('allows decorator to be explicitly invoked for class ctor args', () => {
+    class TestClass {
+      constructor(foo: string) {}
+    }
+    inject('foo')(TestClass, undefined, 0);
+
+    const meta = describeInjectedArguments(TestClass);
+    expect(meta.map(m => m.bindingSelector)).to.deepEqual(['foo']);
+  });
+
   it('can retrieve information about injected method arguments', () => {
     class TestClass {
       test(@inject('foo') foo: string) {}

diff --git a/packages/context/src/inject.ts b/packages/context/src/inject.ts
@@ -124,7 +124,7 @@ export function inject(
   }
   return function markParameterOrPropertyAsInjected(
     target: Object,
-    member: string,
+    member: string | undefined,
     methodDescriptorOrParameterIndex?:
       | TypedPropertyDescriptor<BoundValue>
       | number,