@sprucelabs/test-utils 5.5.18 → 5.5.20

package/build/decorators.d.ts

@@ -1,5 +1,5 @@
 /** Test decorator */
-declare function test(description?: string, ...args: any[]): (target: any, propertyKey: string, descriptor: PropertyDescriptor) => void;
+declare function test(description?: string, ...args: any[]): (Target: any, propertyKey: string, descriptor: PropertyDescriptor) => void;
 declare namespace test {
     var only: (description?: string, ...args: any[]) => (target: any, propertyKey: string, descriptor: PropertyDescriptor) => void;
     var todo: (description?: string, ..._args: any[]) => (target: any, propertyKey: string) => void;

package/build/decorators.js

@@ -40,6 +40,7 @@ if (typeof it === 'undefined') {
     //@ts-ignore
     global.it = () => { };
 }
+let areLifecycleHooksInPlace = false;
 //recursive function to get static method by name looping up through constructor chain
 function resolveMethod(Target, name) {
     if (Target[name]) {
@@ -52,10 +53,10 @@ function resolveMethod(Target, name) {
 }
 /** Hooks up before, after, etc. */
 function hookupTestClassToJestLifecycle(Target) {
-    if (Target.__areLifecycleHooksInPlace) {
+    if (areLifecycleHooksInPlace) {
         return;
     }
-    Target.__areLifecycleHooksInPlace = true;
+    areLifecycleHooksInPlace = true;
     const hooks = ['beforeAll', 'beforeEach', 'afterAll', 'afterEach'];
     hooks.forEach((hook) => {
         // @ts-ignore
@@ -125,15 +126,18 @@ async function runBeforeEach(Target) {
 }
 /** Test decorator */
 function test(description, ...args) {
-    return function (target, propertyKey, descriptor) {
-        hookupTestClassToJestLifecycle(target);
+    return function (Target, propertyKey, descriptor) {
+        hookupTestClassToJestLifecycle(Target);
         // Make sure each test gets the spruce
         it(description ?? propertyKey, async () => {
-            const testClass = SpruceTestResolver_1.default.resolveTestClass(target);
-            const bound = descriptor.value.bind(testClass);
+            const Resolved = SpruceTestResolver_1.default.resolveTestClass(Target);
+            if (!Resolved[propertyKey]) {
+                throw new Error(`The test '${propertyKey}()' should NOT be static when tests run with suite()`);
+            }
+            const bound = descriptor.value.bind(Resolved);
             //@ts-ignore
             global.activeTest = {
-                file: target.name,
+                file: Target.name,
                 test: propertyKey,
             };
             return bound(...args);

package/build/esm/decorators.d.ts

@@ -1,5 +1,5 @@
 /** Test decorator */
-declare function test(description?: string, ...args: any[]): (target: any, propertyKey: string, descriptor: PropertyDescriptor) => void;
+declare function test(description?: string, ...args: any[]): (Target: any, propertyKey: string, descriptor: PropertyDescriptor) => void;
 declare namespace test {
     var only: (description?: string, ...args: any[]) => (target: any, propertyKey: string, descriptor: PropertyDescriptor) => void;
     var todo: (description?: string, ..._args: any[]) => (target: any, propertyKey: string) => void;

package/build/esm/decorators.js

@@ -3,6 +3,7 @@ if (typeof it === 'undefined') {
     //@ts-ignore
     global.it = () => { };
 }
+let areLifecycleHooksInPlace = false;
 //recursive function to get static method by name looping up through constructor chain
 function resolveMethod(Target, name) {
     if (Target[name]) {
@@ -15,10 +16,10 @@ function resolveMethod(Target, name) {
 }
 /** Hooks up before, after, etc. */
 function hookupTestClassToJestLifecycle(Target) {
-    if (Target.__areLifecycleHooksInPlace) {
+    if (areLifecycleHooksInPlace) {
         return;
     }
-    Target.__areLifecycleHooksInPlace = true;
+    areLifecycleHooksInPlace = true;
     const hooks = ['beforeAll', 'beforeEach', 'afterAll', 'afterEach'];
     hooks.forEach((hook) => {
         // @ts-ignore
@@ -91,15 +92,18 @@ async function runBeforeEach(Target) {
 }
 /** Test decorator */
 export default function test(description, ...args) {
-    return function (target, propertyKey, descriptor) {
-        hookupTestClassToJestLifecycle(target);
+    return function (Target, propertyKey, descriptor) {
+        hookupTestClassToJestLifecycle(Target);
         // Make sure each test gets the spruce
         it(description !== null && description !== void 0 ? description : propertyKey, async () => {
-            const testClass = SpruceTestResolver.resolveTestClass(target);
-            const bound = descriptor.value.bind(testClass);
+            const Resolved = SpruceTestResolver.resolveTestClass(Target);
+            if (!Resolved[propertyKey]) {
+                throw new Error(`The test '${propertyKey}()' should NOT be static when tests run with suite()`);
+            }
+            const bound = descriptor.value.bind(Resolved);
             //@ts-ignore
             global.activeTest = {
-                file: target.name,
+                file: Target.name,
                 test: propertyKey,
             };
             return bound(...args);

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "5.5.18",
+  "version": "5.5.20",
   "files": [
     "build"
   ],
@@ -60,7 +60,7 @@
     "watch.tsc": "tsc -w"
   },
   "dependencies": {
-    "@sprucelabs/error": "^6.0.584",
+    "@sprucelabs/error": "^6.0.585",
     "deep-equal": "^2.2.3",
     "just-clone": "^6.2.0",
     "lodash": "^4.17.21",
@@ -72,13 +72,13 @@
     "@sprucelabs/jest-json-reporter": "^8.0.585",
     "@sprucelabs/jest-sheets-reporter": "^2.0.20",
     "@sprucelabs/semantic-release": "^5.0.2",
-    "@sprucelabs/test": "^9.0.75",
+    "@sprucelabs/test": "^9.0.76",
     "@types/deep-equal": "^1.0.4",
     "@types/jest": "^29.5.14",
     "@types/lodash": "^4.17.16",
-    "@types/node": "^22.13.9",
+    "@types/node": "^22.13.10",
     "chokidar-cli": "^3.0.0",
-    "eslint": "^9.21.0",
+    "eslint": "^9.22.0",
     "eslint-config-spruce": "^11.2.26",
     "jest": "^29.7.0",
     "jest-circus": "^29.7.0",