test: properly order freeing resources in cctest

The `IsolateData` instance is created before the `Environment` instance, so free in reverse order. Fixes: nodejs#14206
addaleax · Aug 10, 2017 · 682a951 · 682a951
1 parent 5a0ee6d
commit 682a951
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 {