BREAKING CHANGE: Rename TestContainer -> TestIoCContainer

Goodluckhf · Jul 3, 2019 · 58603e4 · 58603e4
1 parent bc049f0
commit 58603e4
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 10 deletions.
diff --git a/README.md b/README.md
@@ -198,12 +198,12 @@ http.listen(port);
 ### Testing
 We provide a comfortable way for testing
 ```javascript
-import {TestContainer} from '@ukitgroup/ioc';
+import { TestIoCContainer } from '@ukitgroup/ioc';
 describe('Unit test', () => {
 	const ctx = {}
 
 	beforeEach(() => {
-		ctx.container = TestContainer.createTestModule([
+		ctx.container = TestIoCContainer.createTestModule([
 		  //... providers definition with mocks
 		])
 		ctx.container.compile();
@@ -222,5 +222,5 @@ More examples you can find in [integration tests](https://github.com/Goodluckhf/
 
 ### TODO:
 * support decorators with typescript
-* TestContainer for integration tests
+* TestIoCContainer for integration tests
 * Get public providers by tag
diff --git a/src/index.ts b/src/index.ts
@@ -13,4 +13,4 @@ export class IoCContainer extends DIContainer {
   }
 }
 
-export { TestContainer } from './test-container';
+export { TestIoCContainer } from './test-ioc-container';
diff --git a/src/integration-tests/test-container.spec.ts b/src/integration-tests/test-container.spec.ts
@@ -1,11 +1,11 @@
 import 'reflect-metadata';
-import { TestContainer } from '../test-container';
+import { TestIoCContainer } from '../test-ioc-container';
 import { ManifestInterface } from '../dto/manifest.interface';
 
 class ServiceA {}
 
-describe.skip('TestContainer', () => {
-  it('should override providers', () => {
+describe('TestContainer', () => {
+  it.skip('should override providers', () => {
     const diManifest: ManifestInterface = {
       moduleName: 'module',
       providers: [
@@ -17,7 +17,7 @@ describe.skip('TestContainer', () => {
       ],
     };
 
-    const testContainerAdapter = new TestContainer([diManifest]);
+    const testContainerAdapter = new TestIoCContainer([diManifest]);
     // @ts-ignore
     testContainerAdapter.override([
       { token: 'ServiceA', useValue: 10, isPublic: true },
@@ -27,4 +27,13 @@ describe.skip('TestContainer', () => {
     const serviceA = testContainerAdapter.get('ServiceA');
     expect(serviceA).toEqual(10);
   });
+
+  it('can get provider without module', () => {
+    const testContainer = TestIoCContainer.createTestModule([
+      { isPublic: true, token: 'testToken', useValue: 10 },
+    ]);
+
+    testContainer.compile();
+    expect(testContainer.get('testToken')).toBe(10);
+  });
 });
diff --git a/src/test-container.ts → src/test-ioc-container.ts b/src/test-container.ts → src/test-ioc-container.ts
@@ -6,7 +6,7 @@ import { ManifestTransformer } from './manifest-transformer';
 
 const testModuleName = 'testModule';
 
-class TestContainer {
+class TestIoCContainer {
   private container: Container;
 
   public constructor(manifests) {
@@ -37,4 +37,4 @@ class TestContainer {
   }
 }
 
-export { TestContainer };
+export { TestIoCContainer };