From a4fc658fb3b306b7b63e8c23e24c370537d3bfd7 Mon Sep 17 00:00:00 2001 From: Marco Pennekamp Date: Tue, 5 Dec 2023 20:58:11 +0100 Subject: [PATCH] [Test] Use `testRootDisposable` instead of leaking test disposable in `ApiTest` - `testDisposable` is already managed by `KtUsefulTestCase`, so disposal support comes for free. This replaces the previous `TestDisposable`, which was never disposed properly. ^KT-64099 --- js/js.tests/test/org/jetbrains/kotlin/js/testOld/api/ApiTest.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/js/js.tests/test/org/jetbrains/kotlin/js/testOld/api/ApiTest.kt b/js/js.tests/test/org/jetbrains/kotlin/js/testOld/api/ApiTest.kt index 703895f2c7c..08295c73f54 100644 --- a/js/js.tests/test/org/jetbrains/kotlin/js/testOld/api/ApiTest.kt +++ b/js/js.tests/test/org/jetbrains/kotlin/js/testOld/api/ApiTest.kt @@ -129,7 +129,7 @@ class ApiTest : KotlinTestWithEnvironment() { } override fun createEnvironment(): KotlinCoreEnvironment { - return KotlinCoreEnvironment.createForTests(TestDisposable(), CompilerConfiguration(), EnvironmentConfigFiles.JS_CONFIG_FILES) + return KotlinCoreEnvironment.createForTests(testRootDisposable, CompilerConfiguration(), EnvironmentConfigFiles.JS_CONFIG_FILES) } }