test: add DISABLED_ prefix to commented out test

Commit 95ab966 ("test: disable MultipleEnvironmentsPerIsolate") commented out MultipleEnvironmentsPerIsolate but it migth be better to disable the test so that this gets reported and not forgotten: YOU HAVE 1 DISABLED TEST PR-URL: #14317 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
nodejs · Jul 29, 2017 · 75bf8a9 · 75bf8a9
1 parent 46d3ff2
commit 75bf8a9
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/test/cctest/test_environment.cc b/test/cctest/test_environment.cc
@@ -85,8 +85,7 @@ TEST_F(EnvironmentTest, AtExitWithArgument) {
   EXPECT_EQ(arg, cb_1_arg);
 }
 
-/*
-TEST_F(EnvironmentTest, MultipleEnvironmentsPerIsolate) {
+TEST_F(EnvironmentTest, DISABLED_MultipleEnvironmentsPerIsolate) {
   const v8::HandleScope handle_scope(isolate_);
   const Argv argv;
   Env env1 {handle_scope, isolate_, argv};
@@ -101,7 +100,6 @@ TEST_F(EnvironmentTest, MultipleEnvironmentsPerIsolate) {
   RunAtExit(*env2);
   EXPECT_TRUE(called_cb_2);
 }
-*/
 
 static void at_exit_callback1(void* arg) {
   called_cb_1 = true;