infra: lume cli update: update tests to run in @web/test-runner (repl…

…aced Karma in lume cli), update GitHub Actions yml to run clean/build/test separately as they are no longer coupled in lume cli, update build output with changes from the updated target:esnext option in lume cli tsconfig. Note, the dev process in the repo, when standalone and not a git submodule in the lume repo, will not work for now and needs to be developed as part of the lume supermodule repo using the link script for bootstrapping and linking of packages. Once we finish the stage-3-decorators and classy-solid migration in Lume (soon) we will get the lume cli working in standalone repos again.

.github/workflows/tests.yml

@@ -21,6 +21,10 @@ jobs:
             - name: npm install, build, and test
               run: |
                   npm i
+                  npm run clean
+                  npm run build
                   npm test
+            - name: ensure no modifications
+              run: git add . && git diff --quiet && git diff --cached --quiet
               env:
                   CI: true

dist/Mixin.test.d.ts

@@ -1,2 +1,5 @@
+declare global {
+    function expect(...args: any[]): any;
+}
 export {};
 //# sourceMappingURL=Mixin.test.d.ts.map

package.json

@@ -8,11 +8,6 @@
 	"type": "module",
 	"main": "dist/index.js",
 	"types": "dist/index.d.ts",
-	"exports COMMENT:": "This removes 'dist' from import statements, as well as replaces the 'main' field. See https://github.com/nodejs/node/issues/14970#issuecomment-571887546",
-	"exports": {
-		".": "./dist/index.js",
-		"./": "./dist/"
-	},
 	"scripts": {
 		"LUME SCRIPTS XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX": "",
 		"clean": "lume clean",
@@ -21,7 +16,7 @@
 		"typecheck": "lume typecheck",
 		"typecheck:watch": "lume typecheckWatch",
 		"test": "lume test",
-		"test:debug": "lume testDebug",
+		"test:watch": "lume test --watch",
 		"prettier": "lume prettier",
 		"prettier:check": "lume prettierCheck",
 		"release:patch": "lume releasePatch",

src/Mixin.test.ts

@@ -4,6 +4,12 @@ import Mixin, {HasInstance, MixinResult} from './Mixin.js'
 import instanceOf from './instanceOf.js'
 import {Constructor} from './utils.js'
 
+// TODO move type def to @lume/cli, map @types/jest's `expect` type into the
+// global env.
+declare global {
+	function expect(...args: any[]): any
+}
+
 describe('Mixin', () => {
 	it('Mixin returns a Function', () => {
 		// const Foo = Mixin(Base => class Foo extends Constructor(Base || Object) {})

src/tests/basics.test.js

@@ -1,6 +1,6 @@
 import {Class, InvalidAccessError, InvalidSuperAccessError} from '../index.js'
-
 import {native} from '../native.js'
+import {spy} from 'sinon'
 
 const test = it
 
@@ -806,8 +806,7 @@ describe('basics', () => {
 	})
 
 	test('valid vs invalid Super access', () => {
-		//const verifyDimensionCall = jest.fn()
-		const verifyDimensionCall = jasmine.createSpy()
+		const verifyDimensionCall = spy()
 
 		// PhysicalObject implicitly extends from Object (no pun intended!):
 		const PhysicalObject = Class({
@@ -883,7 +882,7 @@ describe('basics', () => {
 		const oboe = new Oboe()
 
 		oboe.testFromInstrumentClass()
-		expect(verifyDimensionCall).toHaveBeenCalled()
+		expect(verifyDimensionCall.called).toBe(true)
 
 		expect(oboe.testFromOboeClass()).toBe('wooo')
 	})