@lumjs/tests 1.8.1 → 1.9.0

package/CHANGELOG.md

@@ -11,6 +11,15 @@ and a reference to a property of a module will be in `@tests.propName` format.
 
 ## [Unreleased]
 
+## [1.9.0] - 2024-07-27
+### Changed
+- Renamed `Harness` constructor name to `LumTestsHarness` to be more unique.
+- The `Stats` class no longer requires the `../harness` module, but instead
+  checks if the object instance's constructor name is `LumTestsHarness`.
+  This will allow the `test` and `test/functional` modules to be bundled
+  by Webpack without bundling `harness` which currently has some issues.
+- Minor bug fixes from `1.8.1` and `1.8.2` unlisted releases.
+
 ## [1.8.0] - 2023-01-06
 ### Changed
 - Bumped `@lumjs/core` to `1.8.0`.
@@ -140,7 +149,8 @@ and a reference to a property of a module will be in `@tests.propName` format.
 - Ported from Lum.js v4 library set.
 - Added a few more features from the PHP version.
 
-[Unreleased]: https://github.com/supernovus/lum.tests.js/compare/v1.8.0...HEAD
+[Unreleased]: https://github.com/supernovus/lum.tests.js/compare/v1.9.0...HEAD
+[1.9.0]: https://github.com/supernovus/lum.tests.js/compare/v1.8.0...v1.9.0
 [1.8.0]: https://github.com/supernovus/lum.tests.js/compare/v1.7.1...v1.8.0
 [1.7.1]: https://github.com/supernovus/lum.tests.js/compare/v1.7.0...v1.7.1
 [1.7.0]: https://github.com/supernovus/lum.tests.js/compare/v1.6.0...v1.7.0

package/TODO.md

@@ -1,5 +1,6 @@
 # TODO
 
+- Update to properly support `async` tests; the `harness` will need some work.
 - Write tests for:
   - `call()`
   - `diesWith()`

package/lib/harness/index.js

@@ -33,7 +33,7 @@ const Plugin = require('./plugin');
  * 
  * @exports module:@lumjs/tests/harness
  */
-class Harness
+class LumTestsHarness
 {
   /**
    * Build a new Harness
@@ -248,12 +248,12 @@ class Harness
 
 } // Harness class
 
-def(Harness, 'Plugin', Plugin);
-def(Harness, 'QueuedTest', QueuedTest);
-def(Harness, 'Errors', require('./errors'));
+def(LumTestsHarness, 'Plugin', Plugin);
+def(LumTestsHarness, 'QueuedTest', QueuedTest);
+def(LumTestsHarness, 'Errors', require('./errors'));
 
 // Export it.
-module.exports = Harness;
+module.exports = LumTestsHarness;
 
 // Some classes required at end for recursive sanity reasons.
 

package/lib/test/stats.js

@@ -100,8 +100,9 @@ class Stats
     }
 
     if (!this.isTop)
-    { // Try to find a Harness instance.
-      if (isObj(require.main) && require.main.exports instanceof Harness)
+    { // Try to find a possible Harness instance using some obscure logic...
+      if (isObj(require.main) && isObj(require.main.exports) 
+        && require.main.exports.constructor.name === 'LumTestsHarness')
       { // We found the Harness instance.
         this.harness = require.main.exports;
       }
@@ -380,7 +381,3 @@ class Stats
 
 // Export the class
 module.exports = Stats;
-
-// Finally at the bottom after `module.exports` has been set, we will load
-// the Harness class to avoid circular references failing.
-const Harness = require('../harness');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lumjs/tests",
-  "version": "1.8.1",
+  "version": "1.9.0",
   "main": "lib/index.js",
   "bin":
   {
@@ -27,7 +27,7 @@
     "@lumjs/core": "^1.8.0"
   },
   "scripts": {
-    "test": "lumtest.js",
+    "test": "./bin/lumtest.js",
     "build-docs": "jsdoc -c ./jsdoc.json",
     "build": "npm run build:grammar",
     "build:grammar": "./bin/build-grammar.sh tap"