fix(tests): fix test that was leaking a context (#933)

microsoft · Feb 11, 2020 · 6105d8a · 6105d8a
1 parent aae5fca
commit 6105d8a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/test/browser.spec.js b/test/browser.spec.js
@@ -42,6 +42,7 @@ module.exports.describe = function({testRunner, expect, playwright, CHROMIUM, WE
       const page = await browser.newPage();
       let error;
       await page.context().newPage().catch(e => error = e);
+      await page.close();
       expect(error.message).toContain('Please use browser.newContext()');
     });
   });

diff --git a/test/playwright.spec.js b/test/playwright.spec.js
@@ -118,6 +118,7 @@ module.exports.describe = ({testRunner, product, playwrightPath}) => {
 
       state.tearDown = async () => {
         await Promise.all(contexts.map(c => c.close()));
+        expect((await state.browser.contexts()).length).toBe(0, `"${test.fullName}" leaked a context`);
         if (rl) {
           rl.removeListener('line', onLine);
           rl.close();