Clean up scratch DOM after tests (#1701)

compat/test/browser/portals.test.js

@@ -1,6 +1,6 @@
 import { createElement as h, render, createPortal, useState, Component } from '../../src';
-import { setupScratch } from '../../../test/_util/helpers';
-import { setupRerender, teardown } from 'preact/test-utils';
+import { setupScratch, teardown } from '../../../test/_util/helpers';
+import { setupRerender } from 'preact/test-utils';
 /* eslint-disable react/jsx-boolean-value, react/display-name, prefer-arrow-callback */
 
 /** @jsx h */
@@ -29,6 +29,8 @@ describe('Portal', () => {
 		render(<Foo>foobar</Foo>, scratch);
 
 		expect(root.innerHTML).to.equal('foobar');
+
+		root.parentNode.removeChild(root);
 	});
 
 	it('should insert the portal', () => {

src/internal.d.ts

@@ -4,7 +4,7 @@ export interface Options extends preact.Options {
 	/** Attach a hook that is invoked before render, mainly to check the arguments. */
 	_root?(vnode: preact.ComponentChild, parent: Element | Document | ShadowRoot | DocumentFragment): void;
 	/** Attach a hook that is invoked before a vnode is diffed. */
-	diff?(vnode: VNode): void;
+	_diff?(vnode: VNode): void;
 	/** Attach a hook that is invoked after a tree was mounted or was updated. */
 	_commit?(vnode: VNode): void;
 	/** Attach a hook that is invoked before a vnode has rendered. */

test/browser/render.test.js

@@ -1017,6 +1017,8 @@ describe('render()', () => {
 			render(<div id="a">new</div>, newScratch, newScratch.querySelector('#a'));
 			expect(newScratch.innerHTML).to.equal('<div id="a">new</div>');
 			expect(unmount).to.be.calledOnce;
+
+			newScratch.parentNode.removeChild(newScratch);
 		});
 
 		it('should render multiple render roots in one parentDom', () => {