test: properly order freeing resources in cctest

The `IsolateData` instance is created before the `Environment` instance, so free in reverse order. Fixes: #14206 PR-URL: #14749 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Daniel Bevenius <[email protected]>
nodejs · Aug 10, 2017 · 0bc3124 · 0bc3124
1 parent 033773c
commit 0bc3124
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/test/cctest/test_environment.cc b/test/cctest/test_environment.cc
@@ -41,9 +41,9 @@ class EnvironmentTest : public NodeTestFixture {
     }
 
     ~Env() {
-      FreeIsolateData(isolate_data_);
       environment_->CleanupHandles();
       FreeEnvironment(environment_);
+      FreeIsolateData(isolate_data_);
     }
 
     Environment* operator*() const {