meocord 2.1.0 → 2.1.1

package/dist/cjs/testing/index.cjs

@@ -152,23 +152,16 @@ function isValueProvider(p) {
  * fn(2)                    // → 99
  * fn.mock.calls             // → [[2], [2]]
  */ function createMockFn(impl) {
+    // One persistent implementation and one queue of single-use ones, which is how
+    // jest and vitest model this. mockReturnValue, mockResolvedValue,
+    // mockRejectedValue and mockImplementation all write the same slot, so the last
+    // call wins; the four `*Once` variants all push onto the same queue, so they are
+    // consumed in the order they were declared regardless of which kind they are.
+    // Keeping a slot per kind instead gave a fixed precedence, where a stored
+    // resolved value beat a later mockRejectedValue and the override was silently
+    // dropped.
     let currentImpl = impl;
-    let returnOnce = [];
-    let returnValue = {
-        set: false,
-        value: undefined
-    };
-    let resolvedOnce = [];
-    let resolvedValue = {
-        set: false,
-        value: undefined
-    };
-    let rejectedOnce = [];
-    let rejectedValue = {
-        set: false,
-        value: undefined
-    };
-    const implOnce = [];
+    let onceQueue = [];
     let name = 'vi.fn';
     const calls = [];
     const results = [];
@@ -179,20 +172,8 @@ function isValueProvider(p) {
         let type = 'return';
         let value;
         try {
-            if (implOnce.length > 0) {
-                value = implOnce.shift().apply(this, args);
-            } else if (returnOnce.length > 0) {
-                value = returnOnce.shift();
-            } else if (returnValue.set) {
-                value = returnValue.value;
-            } else if (resolvedOnce.length > 0) {
-                value = Promise.resolve(resolvedOnce.shift());
-            } else if (resolvedValue.set) {
-                value = Promise.resolve(resolvedValue.value);
-            } else if (rejectedOnce.length > 0) {
-                value = Promise.reject(rejectedOnce.shift());
-            } else if (rejectedValue.set) {
-                value = Promise.reject(rejectedValue.value);
+            if (onceQueue.length > 0) {
+                value = onceQueue.shift().apply(this, args);
             } else if (currentImpl !== undefined) {
                 value = currentImpl.apply(this, args);
             } else {
@@ -234,36 +215,27 @@ function isValueProvider(p) {
         enumerable: false
     });
     mockFn.mockReturnValue = (v)=>{
-        returnValue = {
-            set: true,
-            value: v
-        };
+        currentImpl = ()=>v;
         return mockFn;
     };
     mockFn.mockReturnValueOnce = (v)=>{
-        returnOnce.push(v);
+        onceQueue.push(()=>v);
         return mockFn;
     };
     mockFn.mockResolvedValue = (v)=>{
-        resolvedValue = {
-            set: true,
-            value: v
-        };
+        currentImpl = ()=>Promise.resolve(v);
         return mockFn;
     };
     mockFn.mockResolvedValueOnce = (v)=>{
-        resolvedOnce.push(v);
+        onceQueue.push(()=>Promise.resolve(v));
         return mockFn;
     };
     mockFn.mockRejectedValue = (v)=>{
-        rejectedValue = {
-            set: true,
-            value: v
-        };
+        currentImpl = ()=>Promise.reject(v);
         return mockFn;
     };
     mockFn.mockRejectedValueOnce = (v)=>{
-        rejectedOnce.push(v);
+        onceQueue.push(()=>Promise.reject(v));
         return mockFn;
     };
     mockFn.mockImplementation = (fn)=>{
@@ -271,7 +243,7 @@ function isValueProvider(p) {
         return mockFn;
     };
     mockFn.mockImplementationOnce = (fn)=>{
-        implOnce.push(fn);
+        onceQueue.push(fn);
         return mockFn;
     };
     mockFn.mockClear = ()=>{
@@ -284,22 +256,7 @@ function isValueProvider(p) {
         calls.length = 0;
         results.length = 0;
         instances.length = 0;
-        returnOnce = [];
-        returnValue = {
-            set: false,
-            value: undefined
-        };
-        resolvedOnce = [];
-        resolvedValue = {
-            set: false,
-            value: undefined
-        };
-        rejectedOnce = [];
-        rejectedValue = {
-            set: false,
-            value: undefined
-        };
-        implOnce.length = 0;
+        onceQueue = [];
         currentImpl = impl;
         return mockFn;
     };

package/dist/esm/testing/mock-fn.js

@@ -30,23 +30,16 @@
  * fn(2)                    // → 99
  * fn.mock.calls             // → [[2], [2]]
  */ function createMockFn(impl) {
+    // One persistent implementation and one queue of single-use ones, which is how
+    // jest and vitest model this. mockReturnValue, mockResolvedValue,
+    // mockRejectedValue and mockImplementation all write the same slot, so the last
+    // call wins; the four `*Once` variants all push onto the same queue, so they are
+    // consumed in the order they were declared regardless of which kind they are.
+    // Keeping a slot per kind instead gave a fixed precedence, where a stored
+    // resolved value beat a later mockRejectedValue and the override was silently
+    // dropped.
     let currentImpl = impl;
-    let returnOnce = [];
-    let returnValue = {
-        set: false,
-        value: undefined
-    };
-    let resolvedOnce = [];
-    let resolvedValue = {
-        set: false,
-        value: undefined
-    };
-    let rejectedOnce = [];
-    let rejectedValue = {
-        set: false,
-        value: undefined
-    };
-    const implOnce = [];
+    let onceQueue = [];
     let name = 'vi.fn';
     const calls = [];
     const results = [];
@@ -57,20 +50,8 @@
         let type = 'return';
         let value;
         try {
-            if (implOnce.length > 0) {
-                value = implOnce.shift().apply(this, args);
-            } else if (returnOnce.length > 0) {
-                value = returnOnce.shift();
-            } else if (returnValue.set) {
-                value = returnValue.value;
-            } else if (resolvedOnce.length > 0) {
-                value = Promise.resolve(resolvedOnce.shift());
-            } else if (resolvedValue.set) {
-                value = Promise.resolve(resolvedValue.value);
-            } else if (rejectedOnce.length > 0) {
-                value = Promise.reject(rejectedOnce.shift());
-            } else if (rejectedValue.set) {
-                value = Promise.reject(rejectedValue.value);
+            if (onceQueue.length > 0) {
+                value = onceQueue.shift().apply(this, args);
             } else if (currentImpl !== undefined) {
                 value = currentImpl.apply(this, args);
             } else {
@@ -112,36 +93,27 @@
         enumerable: false
     });
     mockFn.mockReturnValue = (v)=>{
-        returnValue = {
-            set: true,
-            value: v
-        };
+        currentImpl = ()=>v;
         return mockFn;
     };
     mockFn.mockReturnValueOnce = (v)=>{
-        returnOnce.push(v);
+        onceQueue.push(()=>v);
         return mockFn;
     };
     mockFn.mockResolvedValue = (v)=>{
-        resolvedValue = {
-            set: true,
-            value: v
-        };
+        currentImpl = ()=>Promise.resolve(v);
         return mockFn;
     };
     mockFn.mockResolvedValueOnce = (v)=>{
-        resolvedOnce.push(v);
+        onceQueue.push(()=>Promise.resolve(v));
         return mockFn;
     };
     mockFn.mockRejectedValue = (v)=>{
-        rejectedValue = {
-            set: true,
-            value: v
-        };
+        currentImpl = ()=>Promise.reject(v);
         return mockFn;
     };
     mockFn.mockRejectedValueOnce = (v)=>{
-        rejectedOnce.push(v);
+        onceQueue.push(()=>Promise.reject(v));
         return mockFn;
     };
     mockFn.mockImplementation = (fn)=>{
@@ -149,7 +121,7 @@
         return mockFn;
     };
     mockFn.mockImplementationOnce = (fn)=>{
-        implOnce.push(fn);
+        onceQueue.push(fn);
         return mockFn;
     };
     mockFn.mockClear = ()=>{
@@ -162,22 +134,7 @@
         calls.length = 0;
         results.length = 0;
         instances.length = 0;
-        returnOnce = [];
-        returnValue = {
-            set: false,
-            value: undefined
-        };
-        resolvedOnce = [];
-        resolvedValue = {
-            set: false,
-            value: undefined
-        };
-        rejectedOnce = [];
-        rejectedValue = {
-            set: false,
-            value: undefined
-        };
-        implOnce.length = 0;
+        onceQueue = [];
         currentImpl = impl;
         return mockFn;
     };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "meocord",
   "description": "Decorator-based Discord bot framework built on discord.js. Brings NestJS-style controllers, dependency injection, guards, and testing utilities to bot development — with a full CLI and TypeScript-first design.",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "type": "module",
   "engines": {
     "node": ">=22"