Merge pull request #263 from TroyAlford/task/move-to-bun-tests

Task: Move to Bun for tests

.eslintrc

@@ -18,7 +18,7 @@
 		"import/extensions": ["error", { "js": "never", "scss": "always", "ts": "never" }],
 		"import/no-unresolved": "off",
 		"import/prefer-default-export": "off",
-		"indent": ["error", "tab"],
+		"indent": ["error", "tab", { "SwitchCase": 1 }],
 		"lines-between-class-members": "off",
 		"no-case-declarations": "off",
 		"no-tabs": "off",

.github/workflows/pull-request.yml

@@ -0,0 +1,16 @@
+name: Pull Request
+
+on:
+  pull_request:
+    branches: [develop]
+
+jobs:
+  build-and-test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: oven-sh/setup-bun@v1
+      - run: bun install
+      - run: bun eslint
+      - run: bun test
+      - run: bun run build

bunfig.toml

@@ -0,0 +1,2 @@
+[test]
+preload = ["./test/happydom.ts","./test/enzyme.ts"]

dist/components/JsxParser.d.ts

@@ -1,5 +1,5 @@
 import React, { ComponentType, ExoticComponent } from 'react';
-export declare type TProps = {
+export type TProps = {
     allowUnknownElements?: boolean;
     autoCloseVoidElements?: boolean;
     bindings?: {

dist/helpers/parseStyle.d.ts

@@ -1,4 +1,4 @@
-declare type Style = string | Partial<CSSStyleDeclaration>;
+type Style = string | Partial<CSSStyleDeclaration>;
 /**
  * Converts a CSS Style string
  * @param {string | Partial<CSSStyleDeclaration>} style A string to convert, or object to return

package.json

@@ -30,6 +30,7 @@
     "@babel/preset-react": "^7.12.13",
     "@babel/preset-typescript": "^7.12.13",
     "@babel/runtime-corejs3": "^7.12.13",
+    "@happy-dom/global-registrator": "^14.12.3",
     "@types/acorn": "^4.0.5",
     "@types/enzyme": "^3.10.8",
     "@types/jest": "^26.0.20",

source/components/JsxParser.test.tsx

@@ -1,13 +1,10 @@
 // @ts-nocheck
-/* eslint-disable function-paren-newline, no-console, no-underscore-dangle */
+/* eslint-disable function-paren-newline, no-console, no-underscore-dangle, no-useless-escape */
 import React from 'react'
 import TestUtils from 'react-dom/test-utils'
 import { mount, shallow } from 'enzyme' // eslint-disable-line import/no-extraneous-dependencies
 import JsxParser from './JsxParser'
 
-jest.unmock('acorn-jsx')
-jest.unmock('./JsxParser')
-
 const Custom = ({ children = [], className, text }) => (
 	<div className={className}>
 		{text}
@@ -18,24 +15,24 @@ const Custom = ({ children = [], className, text }) => (
 describe('JsxParser Component', () => {
 	let parent = null
 	let originalConsoleError = null
-	let originalJsDomEmit = null
+	// let originalJsDomEmit = null
 
 	beforeAll(() => {
 		originalConsoleError = console.error
 		console.error = jest.fn()
 
-		originalJsDomEmit = window._virtualConsole.emit
-		window._virtualConsole.emit = jest.fn()
+		// originalJsDomEmit = window._virtualConsole.emit
+		// window._virtualConsole.emit = jest.fn()
 	})
 
 	afterAll(() => {
 		console.error = originalConsoleError
-		window._virtualConsole.emit = originalJsDomEmit
+		// window._virtualConsole.emit = originalJsDomEmit
 	})
 
 	beforeEach(() => {
 		console.error.mockReset()
-		window._virtualConsole.emit.mockReset()
+		// window._virtualConsole.emit.mockReset()
 		parent = document.createElement('div')
 	})
 
@@ -488,7 +485,9 @@ describe('JsxParser Component', () => {
 					message: expect.stringContaining('<bar> is unrecognized'),
 				}),
 			)
-			expect(wrapper.html()).toMatchSnapshot()
+			expect(wrapper.html()).toMatch(
+				'<div class=\"jsx-parser\"><div>Before </div><div> After</div></div>',
+			)
 		})
 		test('renders errors with renderError prop, if supplied', () => {
 			const onError = jest.fn()
@@ -882,7 +881,8 @@ describe('JsxParser Component', () => {
 
 			expect(window.foo).toHaveBeenCalledTimes(0)
 			expect(wrapper.find('span')).toHaveLength(0)
-			expect(wrapper.html()).toMatchSnapshot()
+			expect(wrapper.html())
+				.toMatch('<div class=\"jsx-parser\"><div>Before </div><div> After</div></div>')
 		})
 		test('can execute binary mathematical operations', () => {
 			const { rendered } = render(<JsxParser jsx="<span>{ 1 + 2 * 4 / 8 - 1 }</span>" />)
@@ -947,15 +947,17 @@ describe('JsxParser Component', () => {
 			expect(rendered.childNodes[0].textContent).toEqual('Nope')
 			expect(component.ParsedChildren[0].props.testProp).toEqual(true)
 		})
-		test('will render options', () => {
+		test.only('will render options', () => {
 			window.foo = jest.fn(() => true)
 			const wrapper = mount(
 				<JsxParser
 					jsx="<select><option>Some value</option></select>"
 				/>,
 			)
 
-			expect(wrapper.html()).toMatchSnapshot()
+			expect(wrapper.html()).toMatch(
+				'<div class=\"jsx-parser\"><select><option>Some value</option></select></div>',
+			)
 		})
 		describe('can evaluate multi-level property accessors', () => {
 			/* eslint-disable dot-notation,no-useless-concat */
@@ -1266,15 +1268,17 @@ describe('JsxParser Component', () => {
 		})
 
 		it('passes attributes', () => {
-			const PropTest = (props: { booleanAttribute: boolean}) => <>{`val:${props.booleanAttribute}`}</>
+			const PropTest = (props: { booleanAttribute: boolean }) => <>{`val:${props.booleanAttribute}`}</>
 			const { html, component } = render(
 				<JsxParser
 					renderInWrapper={false}
 					components={{ PropTest }}
-					bindings={{ items: [
-						{ name: 'Megeara', friend: true },
-						{ name: 'Austerious', friend: false },
-					] }}
+					bindings={{
+						items: [
+							{ name: 'Megeara', friend: true },
+							{ name: 'Austerious', friend: false },
+						],
+					}}
 					jsx="{items.map(item => <p><PropTest booleanAttribute={item.friend} /></p>)}"
 				/>,
 			)
@@ -1290,9 +1294,11 @@ describe('JsxParser Component', () => {
 				<JsxParser
 					renderInWrapper={false}
 					components={{ PropTest }}
-					bindings={{ items: [
-						{ name: 'Megeara', friend: true },
-					] }}
+					bindings={{
+						items: [
+							{ name: 'Megeara', friend: true },
+						],
+					}}
 					jsx="{items.map(item => <PropTest {...item} />)}"
 				/>,
 			)