From 824c227d815f3f49f5295d471b34ddcad70f6872 Mon Sep 17 00:00:00 2001 From: Nikolay Krasko Date: Tue, 5 Oct 2021 15:55:59 +0300 Subject: [PATCH] Stop removing libraries from local maven in MavenResolverTest While it's a good practice to remove all files that were downloaded during the test execution, here we delete files in common directory without checking that files were not present before test start. This might be dangerous as we might fail other tests and programs. As we manipulate common maven directory it's better to leave downloaded files untouched --- .../test/kotlin/script/experimental/test/MavenResolverTest.kt | 1 - 1 file changed, 1 deletion(-) diff --git a/libraries/scripting/dependencies-maven/test/kotlin/script/experimental/test/MavenResolverTest.kt b/libraries/scripting/dependencies-maven/test/kotlin/script/experimental/test/MavenResolverTest.kt index fa1ac9ac66f..6cdb34ad94b 100644 --- a/libraries/scripting/dependencies-maven/test/kotlin/script/experimental/test/MavenResolverTest.kt +++ b/libraries/scripting/dependencies-maven/test/kotlin/script/experimental/test/MavenResolverTest.kt @@ -42,7 +42,6 @@ class MavenResolverTest : ResolversTestBase() { if (!checkBody(files)) { Assert.fail("Unexpected resolving results:\n ${files.joinToString("\n ")}") } - files.forEach { it.delete() } } private fun buildOptions(vararg options: Pair): ExternalDependenciesResolver.Options {