Merge pull request #4369 from preactjs/chore/test-types-and-warnings

chore: test types and warnings

compat/test/browser/testUtils.js

@@ -10,13 +10,13 @@ export function getSymbol(name, fallback) {
 		// eslint-disable-next-line
 		if (
 			Function.prototype.toString
-				.call(eval('Symbol.for'))
+				.call((0, eval)('Symbol.for'))
 				.match(/\[native code\]/)
 		) {
 			// Concatenate these string literals to prevent the test
 			// harness and/or Babel from modifying the symbol value.
 			// eslint-disable-next-line
-			out = eval('Sym' + 'bol.for("' + name + '")');
+			out = (0, eval)('Sym' + 'bol.for("' + name + '")');
 		}
 	} catch (e) {}
 

package.json

@@ -270,6 +270,7 @@
     "@types/chai": "^4.1.2",
     "@types/mocha": "^5.0.0",
     "@types/node": "^14.14.10",
+    "@types/sinon": "^9.0.11",
     "babel-plugin-istanbul": "^6.0.0",
     "babel-plugin-transform-async-to-promises": "^0.8.15",
     "babel-plugin-transform-rename-properties": "0.1.0",

test/browser/getDomSibling.test.js

@@ -5,7 +5,7 @@ import { setupScratch, teardown } from '../_util/helpers';
 /** @jsx createElement */
 
 describe('getDomSibling', () => {
-	/** @type {import('../../src/internal').PreactElement} */
+	/** @type {HTMLDivElement} */
 	let scratch;
 
 	const getRoot = dom => dom._children;

test/extensions.d.ts

@@ -1,5 +1,16 @@
-declare module Chai {
-	export interface Assertion {
-		equalNode(node: Node | null, message?: string): void;
+declare global {
+	namespace Chai {
+		interface Assertion {
+			equalNode(node: Node | null, message?: string): void;
+			called: Assertion;
+			calledOnce: Assertion;
+			calledTwice: Assertion;
+			calledWith(...args: any[]): Assertion;
+			calledWithMatch(...args: Record<string, unknown>[]): Assertion;
+		}
 	}
+	var expect: Chai.ExpectStatic;
+	var sinon: Sinon;
 }
+
+export {};