test(reactant-di): add unexpected multi-inject testing

Author: unadlib

packages/reactant-di/test/decorators/__snapshots__/multiInject.test.ts.snap

@@ -0,0 +1,3 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`multiInject Unexpected multi-Inject 1`] = `"Ambiguous match found for serviceIdentifier: Foo"`;

packages/reactant-di/test/decorators/multiInject.test.ts

@@ -54,4 +54,25 @@ describe('multiInject', () => {
     expect(bar.length).toBe(2);
     expect(bar.foos[1]).toBe('test');
   });
+
+  test('Unexpected multi-Inject', () => {
+    @injectable()
+    class Foo {
+      public get test() {
+        return 'test';
+      }
+    }
+
+    @injectable()
+    class Bar {
+      constructor(public foos: Foo) {}
+    }
+
+    expect(() => {
+      createContainer({
+        ServiceIdentifiers: new Map(),
+        modules: [Foo, Foo],
+      }).get(Bar);
+    }).toThrowErrorMatchingSnapshot();
+  });
 });

packages/reactant-module/test/__snapshots__/index.test.ts.snap

@@ -0,0 +1,9 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Unexpected multi-inject: module with multiple module injection with same module or others 1`] = `
+"Ambiguous match found for serviceIdentifier: FooIdentifier
+Registered bindings:
+ Foo
+ Foo
+ FooTest"
+`;

packages/reactant-module/test/index.test.ts

@@ -265,3 +265,56 @@ test('module with multiple module injection with same module or others', () => {
   expect(fooBar.foos[0].num).toBe(3);
   expect(computedFn.mock.calls.length).toBe(3);
 });
+
+test('Unexpected multi-inject: module with multiple module injection with same module or others', () => {
+  const computedFn = jest.fn();
+  @injectable({
+    name: 'foo',
+  })
+  class Foo {
+    @state
+    count = 1;
+
+    @action
+    increase() {
+      this.count += 1;
+    }
+
+    @computed(({ count }: Foo) => [count])
+    get num() {
+      computedFn();
+      return this.count + 1;
+    }
+  }
+
+  @injectable({
+    name: 'fooTest',
+  })
+  class FooTest {
+    @state
+    count = 1;
+  }
+
+  @injectable()
+  class FooBar {
+    constructor(@inject('FooIdentifier') public foos: Foo, public foo: Foo) {}
+  }
+
+  const ServiceIdentifiers = new Map();
+  const modules = [
+    FooBar,
+    { provide: 'FooIdentifier', useClass: Foo },
+    { provide: 'FooIdentifier', useClass: Foo },
+    { provide: 'FooIdentifier', useClass: FooTest },
+  ];
+  const container = createContainer({
+    ServiceIdentifiers,
+    modules,
+    options: {
+      defaultScope: 'Singleton',
+    },
+  });
+  expect(() => {
+    container.get(FooBar);
+  }).toThrowErrorMatchingSnapshot();
+});