From 2f2baacbcc8c0161e6e529452d49112fd4379bfc Mon Sep 17 00:00:00 2001
From: Clark Du <clark.duxin@gmail.com>
Date: Mon, 11 Mar 2019 13:22:09 +0000
Subject: [PATCH] test: try to fix unhandled request

---
 packages/server/src/jsdom.js |  1 +
 test/unit/basic.dev.test.js  | 11 +++++------
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/packages/server/src/jsdom.js b/packages/server/src/jsdom.js
index d71c94c121..4b5d9e74cb 100644
--- a/packages/server/src/jsdom.js
+++ b/packages/server/src/jsdom.js
@@ -56,6 +56,7 @@ export default async function renderAndGetWindow(
   if (!nuxtExists) {
     const error = new Error('Could not load the nuxt app')
     error.body = window.document.body.innerHTML
+    window.close()
     throw error
   }
 
diff --git a/test/unit/basic.dev.test.js b/test/unit/basic.dev.test.js
index 1ff42361a2..d9d12add92 100644
--- a/test/unit/basic.dev.test.js
+++ b/test/unit/basic.dev.test.js
@@ -107,10 +107,9 @@ describe('basic dev', () => {
     const window = await nuxt.server.renderAndGetWindow(url('/stateless'))
     const html = window.document.body.innerHTML
     expect(html).toContain('<h1>My component!</h1>')
-  })
-
-  test('Check render:routeDone hook called', () => {
+    // Check render:routeDone hook called
     expect(nuxt.__hook_render_routeDone__).toBe('/stateless')
+    window.close()
   })
 
   // test('/_nuxt/test.hot-update.json should returns empty html', async t => {
@@ -122,7 +121,7 @@ describe('basic dev', () => {
   //   }
   // })
 
-  test.skip('/__open-in-editor (open-in-editor)', async () => {
+  test('/__open-in-editor (open-in-editor)', async () => {
     const { body } = await rp(
       url('/__open-in-editor?file=pages/index.vue'),
       { resolveWithFullResponse: true }
@@ -130,7 +129,7 @@ describe('basic dev', () => {
     expect(body).toBe('')
   })
 
-  test.skip('/__open-in-editor should return error (open-in-editor)', async () => {
+  test('/__open-in-editor should return error (open-in-editor)', async () => {
     await expect(rp(url('/__open-in-editor?file='))).rejects.toMatchObject({
       statusCode: 500,
       error: 'launch-editor-middleware: required query param "file" is missing.'
@@ -143,7 +142,7 @@ describe('basic dev', () => {
     })
   })
 
-  test.skip('/error should return json format error (Youch)', async () => {
+  test('/error should return json format error (Youch)', async () => {
     const opts = {
       headers: {
         accept: 'application/json'