From 20ea035bf53911e7b67a680f9bdf09a53f0b8110 Mon Sep 17 00:00:00 2001
From: Zirro <code@zirro.se>
Date: Thu, 14 Jun 2018 13:52:47 +0200
Subject: [PATCH] Enable Blob-slice.html test in Node.js v10

The issue with `instanceof ArrayBuffer` was fixed in Node.js v10.4.0.

See: https://github.com/nodejs/node/issues/20978
---
 test/web-platform-tests/run-wpts.js | 5 +----
 test/web-platform-tests/to-run.yaml | 1 -
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/test/web-platform-tests/run-wpts.js b/test/web-platform-tests/run-wpts.js
index 8733e1c43a..26fcb70331 100644
--- a/test/web-platform-tests/run-wpts.js
+++ b/test/web-platform-tests/run-wpts.js
@@ -15,7 +15,6 @@ const validReasons = new Set([
   "needs-await",
   "needs-node8",
   "needs-node10",
-  "fails-node10",
   "timeout-node6" // For tests that timeout in Node.js v6, but pass in later versions
 ]);
 
@@ -28,7 +27,6 @@ try {
 
 const hasNode8 = Number(process.versions.node.split(".")[0]) >= 8;
 const hasNode10 = Number(process.versions.node.split(".")[0]) >= 10;
-const isNode10 = Number(process.versions.node.split(".")[0]) === 10;
 
 const manifestFilename = path.resolve(__dirname, "wpt-manifest.json");
 const manifest = readManifest(manifestFilename);
@@ -67,8 +65,7 @@ describe("web-platform-tests", () => {
           const expectFail = (reason === "fail") ||
                              (reason === "needs-await" && !supportsAwait) ||
                              (reason === "needs-node8" && !hasNode8) ||
-                             (reason === "needs-node10" && !hasNode10) ||
-                             (reason === "fails-node10" && isNode10);
+                             (reason === "needs-node10" && !hasNode10);
 
           if (matchingPattern && shouldSkip) {
             specify.skip(`[${reason}] ${testFile}`);
diff --git a/test/web-platform-tests/to-run.yaml b/test/web-platform-tests/to-run.yaml
index bba6f4dd0c..104d82b49d 100644
--- a/test/web-platform-tests/to-run.yaml
+++ b/test/web-platform-tests/to-run.yaml
@@ -9,7 +9,6 @@ blob/Blob-constructor.html: [fail, "- Blob is not a function
                                     - HTMLSelectElement does not have indexed properties
                                     - MessageChannel not implemented
                                     - element attributes does not have indexed properties"]
-blob/Blob-slice.html: [fails-node10, instanceof ArrayBuffer is failing, https://github.com/nodejs/node/issues/20978]
 file/File-constructor-endings.html: [needs-await]
 file/send-file-form*: [fail, DataTransfer not implemented]
 filelist-section/filelist.html: [fail, function is not instanceof Function]