From 80f4caf4536d81004d18fb6fcf22bc7105d8c274 Mon Sep 17 00:00:00 2001 From: ExE Boss <3889017+ExE-Boss@users.noreply.github.com> Date: Tue, 19 Jun 2018 23:36:50 +0200 Subject: [PATCH] chore: Use `delete window.browser` in `setupTestDOMWindow` (#128) This patch remove a workaround introduced in #2, because it has been fixed upstream and the workaround is not needed anymore (when using a nodejs version >= 8). --- test/setup.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/test/setup.js b/test/setup.js index fdcf53e..7b79adb 100644 --- a/test/setup.js +++ b/test/setup.js @@ -45,8 +45,7 @@ function setupTestDOMWindow(chromeObject, browserObject = undefined) { if (browserObject) { window.browser = browserObject; } else { - // TODO: change into `delete window.browser` once tmpvar/jsdom#1622 has been fixed. - window.browser = undefined; + delete window.browser; } const scriptEl = window.document.createElement("script");