feat(context): allow namespace tag for createBindingFromClass

Author: raymondfeng

packages/context/src/__tests__/unit/binding-inspector.unit.ts

@@ -185,6 +185,27 @@ describe('createBindingFromClass()', () => {
     expect(binding.key).to.eql('classes.MyClass');
   });
 
+  it('honors namespace with @bind', () => {
+    @bind({tags: {namespace: 'services'}})
+    class MyService {}
+
+    const ctx = new Context();
+    const binding = givenBindingFromClass(MyService, ctx);
+
+    expect(binding.key).to.eql('services.MyService');
+  });
+
+  it('honors namespace with options', () => {
+    class MyService {}
+
+    const ctx = new Context();
+    const binding = givenBindingFromClass(MyService, ctx, {
+      namespace: 'services',
+    });
+
+    expect(binding.key).to.eql('services.MyService');
+  });
+
   function givenBindingFromClass(
     cls: Constructor<unknown>,
     ctx: Context = new Context(),

packages/context/src/binding-inspector.ts

@@ -255,6 +255,7 @@ function getNamespace(type: string, typeNamespaces = DEFAULT_TYPE_NAMESPACES) {
  * `<namespace>.<name>`.
  *   - namespace
  *     - `options.namespace`
+ *     - `namespace` tag value
  *     - Map `options.type` or `type` tag value to a namespace, for example,
  *       'controller` to 'controller'.
  *   - name
@@ -276,7 +277,8 @@ function buildBindingKey(
   let key: string = options.key || bindingTemplate.tagMap[ContextTags.KEY];
   if (key) return key;
 
-  let namespace = options.namespace;
+  let namespace =
+    options.namespace || bindingTemplate.tagMap[ContextTags.NAMESPACE];
   if (!namespace) {
     const namespaces = Object.assign(
       {},

packages/context/src/keys.ts

@@ -11,6 +11,10 @@ export namespace ContextTags {
    * Type of the artifact
    */
   export const TYPE = 'type';
+  /**
+   * Namespace of the artifact
+   */
+  export const NAMESPACE = 'namespace';
   /**
    * Name of the artifact
    */