Merge pull request #6 from ikenox/add-test-cases

Add test cases

README.md

@@ -71,6 +71,8 @@ const singletonContainer = singletonScope.instanciate({});
 // Define another scope. You can specify scope-specific parameter. As an example,
 // here is `{ request: Request }`.
 const requestScope = scope<{ request: Request }>()
+  // Provide other-scoped dependencies in this scope together
+  // Note that the merged singleton-scoped dependencies are still singleton
   .static(singletonContainer)
   .provide({
     // Define request-scoped dependencies.

src/example.test.ts

@@ -45,6 +45,8 @@ const singletonContainer = singletonScope.instanciate({});
 // Define another scope. You can specify scope-specific parameter. As an example,
 // here is `{ request: Request }`.
 const requestScope = scope<{ request: Request }>()
+  // Provide other-scoped dependencies in this scope together
+  // Note that the merged singleton-scoped dependencies are still singleton
   .static(singletonContainer)
   .provide({
     // Define request-scoped dependencies.

src/index.test.ts

@@ -78,20 +78,32 @@ test('cache', ({ expect }) => {
   expect(container1.num).not.toBe(container2.num);
 });
 
-test("container instantiation processses doen't evaluate a passed another container dependency instances", ({
-  expect,
-}) => {
+describe('lazy evaluation', () => {
   const container = scope()
     .provide({
-      dep: () => {
+      depA: () => {
         throw new Error('this code should never be called');
       },
+      depB: () => 123,
     })
     .instanciate({});
 
-  expect(() => {
-    scope().static(container).instanciate({});
-  }).not.toThrowError();
+  test('dependencies are not evaluated when merged by `static` method', ({
+    expect,
+  }) => {
+    expect(() => {
+      scope().static(container).instanciate({});
+    }).not.toThrowError();
+  });
+
+  test('unspecified variable is not evaluated on object destruction', ({
+    expect,
+  }) => {
+    expect(() => {
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
+      const { depB } = container;
+    }).not.toThrowError();
+  });
 });
 
 describe('type-level tests', () => {

src/index.ts

@@ -35,7 +35,9 @@ export class ContainerScope<
   /**
    * Merges external static objects (mainly other container instances) to provide
    * via the container. It allows the container to provide other-scoped container's
-   * dependency instances together. The merged container keeps original scope.
+   * dependency instances together.
+   * The merged container keeps original scope, meaning that a singleton-scoped
+   * container which is merged into request-scoped keeps singletion.
    */
   static<P extends Record<string, unknown> & WithoutReserved<Instances>>(
     p: P