From ad57ec9d7764a091885d5d997a93f6b5032c9d0d Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Mon, 17 Jul 2017 06:40:19 +0200 Subject: [PATCH] test: add DISABLED_ prefix to commented out test Commit 95ab966a742d23d7271c7b4c36fb84aa2bbece59 ("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 --- test/cctest/test_environment.cc | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/test/cctest/test_environment.cc b/test/cctest/test_environment.cc index d116b4d995b8b7..db08743261c843 100644 --- 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;