fix: New factory binding doesn't reset a cached value

packages/ditox/src/container.test.ts

@@ -272,6 +272,46 @@ describe('Container', () => {
       expect(container.get(CONTAINER)).toBe(container);
       expect(container.get(PARENT_CONTAINER)).toBe(parent);
     });
+
+    it('should rebind a value binding by the same token', () => {
+      const container = createContainer();
+
+      container.bindValue(NUMBER, 1);
+      container.bindFactory(NUMBER, () => 2);
+      expect(container.get(NUMBER)).toBe(2);
+    });
+
+    it('should rebind a factory binding by the same token', () => {
+      const container = createContainer();
+
+      container.bindFactory(NUMBER, () => 1);
+      container.bindFactory(NUMBER, () => 2);
+      expect(container.get(NUMBER)).toBe(2);
+    });
+
+    it('should reset a cached value during rebinding a factory', () => {
+      const container = createContainer();
+
+      container.bindFactory(NUMBER, () => 1);
+      expect(container.get(NUMBER)).toBe(1);
+
+      container.bindFactory(NUMBER, () => 2);
+      expect(container.get(NUMBER)).toBe(2);
+    });
+
+    it('should not reset a cached value if the factory is the same', () => {
+      const container = createContainer();
+
+      let value = 1;
+      const factory = () => value;
+
+      container.bindFactory(NUMBER, factory);
+      expect(container.get(NUMBER)).toBe(1);
+
+      value = 2;
+      container.bindFactory(NUMBER, factory);
+      expect(container.get(NUMBER)).toBe(1);
+    });
   });
 
   describe('remove()', () => {

packages/ditox/src/container.ts

@@ -164,7 +164,17 @@ export function createContainer(parentContainer?: Container): Container {
         return;
       }
 
-      factories.set(token.symbol, {factory, options});
+      const prev = factories.get(token.symbol);
+      const next = {factory, options};
+
+      const isSame =
+        prev && prev.factory === next.factory && prev.options === options;
+
+      if (!isSame) {
+        values.delete(token.symbol);
+      }
+
+      factories.set(token.symbol, next);
     },
 
     remove<T>(token: Token<T>): void {