From 186ce7e837bde20bd2726d662721a93c95ba24a6 Mon Sep 17 00:00:00 2001
From: Julien Gilli <jgilli@netflix.com>
Date: Mon, 8 Oct 2018 17:22:10 -0700
Subject: [PATCH] test: remove obsolete domain test

test-microtask-queue-run-domain.js tests that the behavior of another
test, test-microtask-queue-run.js, is still consistent when the core
domain module is loaded.

This was needed because before the changes in #16222 were merged, the
core domain module would replace the function that would call nextTick
callbacks with a different implementation.

This is no longer the case, and therefore that test is no longer needed.

PR-URL: https://github.com/nodejs/node/pull/23343
Reviewed-By: Vladimir de Turckheim <vlad2t@hotmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 .../test-microtask-queue-run-domain.js        | 66 -------------------
 1 file changed, 66 deletions(-)
 delete mode 100644 test/parallel/test-microtask-queue-run-domain.js

diff --git a/test/parallel/test-microtask-queue-run-domain.js b/test/parallel/test-microtask-queue-run-domain.js
deleted file mode 100644
index 39baf930232411..00000000000000
--- a/test/parallel/test-microtask-queue-run-domain.js
+++ /dev/null
@@ -1,66 +0,0 @@
-// Copyright Joyent, Inc. and other Node contributors.
-//
-// Permission is hereby granted, free of charge, to any person obtaining a
-// copy of this software and associated documentation files (the
-// "Software"), to deal in the Software without restriction, including
-// without limitation the rights to use, copy, modify, merge, publish,
-// distribute, sublicense, and/or sell copies of the Software, and to permit
-// persons to whom the Software is furnished to do so, subject to the
-// following conditions:
-//
-// The above copyright notice and this permission notice shall be included
-// in all copies or substantial portions of the Software.
-//
-// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
-// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN
-// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
-// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
-// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
-// USE OR OTHER DEALINGS IN THE SOFTWARE.
-
-'use strict';
-require('../common');
-const assert = require('assert');
-
-// Requiring the domain module here changes the function that is used by node to
-// call process.nextTick's callbacks to a variant that specifically handles
-// domains. We want to test this specific variant in this test, and so even if
-// the domain module is not used, this require call is needed and must not be
-// removed.
-require('domain');
-
-function enqueueMicrotask(fn) {
-  Promise.resolve().then(fn);
-}
-
-let done = 0;
-
-process.on('exit', function() {
-  assert.strictEqual(done, 2);
-});
-
-// no nextTick, microtask
-setTimeout(function() {
-  enqueueMicrotask(function() {
-    done++;
-  });
-}, 0);
-
-
-// no nextTick, microtask with nextTick
-setTimeout(function() {
-  let called = false;
-
-  enqueueMicrotask(function() {
-    process.nextTick(function() {
-      called = true;
-    });
-  });
-
-  setTimeout(function() {
-    if (called)
-      done++;
-  }, 0);
-
-}, 0);