⬆️ Upgrade @hover/javascript, restructure tests slightly

.eslintignore

@@ -0,0 +1,5 @@
+node_modules/
+coverage/
+dist/
+extend.js
+extend.d.ts

.eslintrc.js

@@ -1,9 +1,9 @@
-const { overrides, ...config } = require('@hover/javascript/eslint')
+const {overrides, ...config} = require('@hover/javascript/eslint')
 
 module.exports = {
   ...config,
   ignorePatterns: ['dom-testing-library.js', 'rollup.input.js'],
   parserOptions: {
-    project: ['tsconfig.json', 'tsconfig.eslint.json'],
+    project: ['tsconfig.eslint.json'],
   },
 }

lib/__tests__/extend.ts

@@ -6,11 +6,8 @@ describe('lib/extend.ts', () => {
   let page: playwright.Page
   let document: playwright.ElementHandle
 
-  it('should require without error', async () => {
+  beforeAll(async () => {
     await import('../extend')
-  })
-
-  it('should launch playwright', async () => {
     browser = await playwright.chromium.launch({args: ['--no-sandbox', '--disable-setuid-sandbox']})
     page = await browser.newPage()
     await page.goto(`file://${path.join(__dirname, 'fixtures/page.html')}`)
@@ -57,6 +54,7 @@ describe('lib/extend.ts', () => {
       fail() // eslint-disable-line jest/no-jasmine-globals
     } catch (err) {
       err.message = err.message.replace(/(\s*at .*(\n|$))+/gm, '\n    <stack>:X:X')
+      // eslint-disable-next-line jest/no-try-expect
       expect(err.message).toMatchSnapshot()
     }
   })

lib/__tests__/index.ts

@@ -6,7 +6,7 @@ describe('lib/index.ts', () => {
   let browser: playwright.Browser
   let page: playwright.Page
 
-  it('should launch playwright', async () => {
+  beforeAll(async () => {
     browser = await playwright.chromium.launch({args: ['--no-sandbox', '--disable-setuid-sandbox']})
     page = await browser.newPage()
     await page.goto(`file://${path.join(__dirname, 'fixtures/page.html')}`)
@@ -31,16 +31,16 @@ describe('lib/index.ts', () => {
     expect(await queries.getNodeText(element)).toEqual('Hello h1')
   })
 
-  it('should load the defered page', async () => {
-    await page.goto(`file://${path.join(__dirname, 'fixtures/late-page.html')}`)
-  })
+  describe('loading the deferred page', () => {
+    beforeEach(async () => page.goto(`file://${path.join(__dirname, 'fixtures/late-page.html')}`))
 
-  it('should use `wait` properly', async () => {
-    const {getByText} = getQueriesForElement(await getDocument(page))
-    // eslint-disable-next-line @typescript-eslint/no-misused-promises
-    await wait(() => getByText('Loaded!'), {timeout: 7000})
-    expect(await getByText('Loaded!')).toBeTruthy()
-  }, 9000)
+    it('should use `wait` properly', async () => {
+      const {getByText} = getQueriesForElement(await getDocument(page))
+      // eslint-disable-next-line @typescript-eslint/no-misused-promises
+      await wait(() => getByText('Loaded!'), {timeout: 7000})
+      expect(await getByText('Loaded!')).toBeTruthy()
+    }, 9000)
+  })
 
   afterAll(async () => {
     await browser.close()

package.json

@@ -48,7 +48,7 @@
     "wait-for-expect": "^3.0.2"
   },
   "devDependencies": {
-    "@hover/javascript": "^4.4.0",
+    "@hover/javascript": "^4.17.0",
     "@rollup/plugin-commonjs": "^14.0.0",
     "@rollup/plugin-node-resolve": "^8.0.0",
     "@rollup/plugin-replace": "^2.3.1",