Merge pull request #115 from mattyg/build/fix-tests-replace-jest

Fix test runner setup and fix failing tests

CHANGELOG.md

@@ -5,6 +5,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## Changed
+- Replace jest with vitest
+
+## Fixed
+- Test suite now runs properly
+- Mocking browser globals in tests
+- Failing tests
+
 ## [0.6.18-prerelease] - 2023-11-1 
 ### Changed
 - bumped comb version to 0.3.2

package.json

@@ -8,7 +8,7 @@
   "scripts": {
     "build": "npm run bundle",
     "bundle": "npx webpack --mode production --output-filename holo_hosting_web_sdk.js",
-    "test": "yarn jest",
+    "test": "npx vitest",
     "prepublish": "npm run build"
   },
   "author": "Holo Ltd.",
@@ -34,18 +34,15 @@
     "@babel/preset-env": "^7.9.5",
     "@babel/preset-typescript": "^7.21.0",
     "@holo-host/chaperone": "^8.0.1",
-    "@jest/globals": "^29.4.3",
-    "@types/jest": "^29.4.0",
     "@types/node": "^12.20.55",
     "@whi/stdlog": "^0.3.0",
     "babel-loader": "^8.1.0",
     "babel-polyfill": "^6.26.0",
-    "jest": "^27.1.3",
-    "jest-canvas-mock": "^2.4.0",
+    "jsdom": "^22.1.0",
     "puppeteer": "^2.1.1",
-    "ts-jest": "^27.1.3",
     "ts-loader": "^9.4.1",
     "typescript": "^4.9.3",
+    "vitest": "^0.34.6",
     "webpack": "^5.74.0",
     "webpack-cli": "^4.10.0"
   },

src/index.ts

@@ -160,7 +160,7 @@ class WebSdkApi implements AppAgentClient {
     const { agentState, uiState } = chaperone_state
 
     webSdkApi.agentState = agentState
-    webSdkApi.myPubKey = decodeHashFromBase64(agentState.id)
+    webSdkApi.myPubKey = agentState ? decodeHashFromBase64(agentState.id) : undefined
     webSdkApi.uiState = uiState
     webSdkApi.chaperoneState = chaperone_state
     webSdkApi.happId = happ_id

tests/api.test.ts

@@ -1,15 +1,18 @@
-import {describe, expect, test} from '@jest/globals';
+import {beforeEach, afterEach, expect, describe, it } from 'vitest'
 
 const path = require('path')
 const log = require('@whi/stdlog')(path.basename(__filename), {
   level: process.env.LOG_LEVEL || 'fatal'
 })
 
-require("./mock_browser")
-const mock_comb = require("./mock_comb")
+import "./mock_browser"
+import mock_comb from "./mock_comb"
+import WebSdkApi from "../src/index"
 
-let WebSdkApi
-WebSdkApi = require("../src/index").default
+import { JSDOM } from "jsdom"
+const dom = new JSDOM()
+global.document = dom.window.document
+global.window = dom.window
 
 describe('test API endpoints', () => {
   let holo
@@ -132,8 +135,6 @@ describe("test comb error", () => {
         throw new Error(expectedError)
       }
     }
-
-    WebSdkApi = require("../src/index").default
   })
 
   afterEach(() => {

tests/mock_browser.ts

@@ -1,5 +1,5 @@
 
-(<any>global.window) = {
+export default (<any>global.window) = {
   "location": {
     "hostname": "https://localhost",
   }

tests/mock_comb.ts

@@ -30,7 +30,7 @@ let event_listeners = {}
   }
 }
 
-module.exports = {
+export default {
   nextResponse(value) {
     next_response = value;
   },