Commit 0bc3124
committed
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 <refack@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>1 parent 033773c commit 0bc3124
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -41,9 +41,9 @@ class EnvironmentTest : public NodeTestFixture { | |||
| 41 | 41 | } | |
| 42 | 42 | ||
| 43 | 43 | ~Env() { | |
| 44 | - FreeIsolateData(isolate_data_); | ||
| 45 | 44 | environment_->CleanupHandles(); | |
| 46 | 45 | FreeEnvironment(environment_); | |
| 46 | + FreeIsolateData(isolate_data_); | ||
| 47 | 47 | } | |
| 48 | 48 | ||
| 49 | 49 | Environment* operator*() const { | |
0 commit comments