Enhance smoke tests

diktat-ruleset/src/test/kotlin/com/saveourtool/diktat/ruleset/smoke/DiktatSmokeTest.kt

@@ -23,11 +23,17 @@
         expected: String,
         test: String,
     ) {
-        Assertions.assertTrue(
-            getTestComparatorUnit(config)
-                .compareFilesFromResources(expected, test)
-                .isSuccessful
+        val result = getTestComparatorUnit(config)
+            .compareFilesFromResources(expected, test)
+        Assertions.assertAll(
+            {
+                Assertions.assertTrue(result.isSuccessful)
+            },
+            {
+                Assertions.assertEquals(result.expectedContentWithoutWarns, result.actualContent)
+            }
         )
+
     }
 
     @BeforeEach
@@ -40,7 +46,7 @@
     }
 
     private fun getTestComparatorUnit(config: Path) = TestComparatorUnit(
-        resourceFilePath = RESOURCE_FILE_PATH,
+        resourceFilePath = tempDir.toString(),
         function = { testFile ->
             format(
                 ruleSetSupplier = {

diktat-ruleset/src/test/kotlin/com/saveourtool/diktat/ruleset/smoke/DiktatSmokeTestBase.kt

@@ -45,14 +45,24 @@
 import org.junit.jupiter.api.Tag
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.Timeout
+import org.junit.jupiter.api.io.TempDir
 import java.io.File
 import java.nio.file.Path
 import java.nio.file.Paths
 
 import java.time.LocalDate
 import java.util.concurrent.TimeUnit.SECONDS
+import kotlin.io.path.ExperimentalPathApi
+import kotlin.io.path.PathWalkOption
+import kotlin.io.path.copyTo
+import kotlin.io.path.createDirectories
 import kotlin.io.path.createTempFile
 import kotlin.io.path.inputStream
+import kotlin.io.path.isDirectory
+import kotlin.io.path.moveTo
+import kotlin.io.path.name
+import kotlin.io.path.toPath
+import kotlin.io.path.walk
 import kotlin.io.path.writeText
 
 import kotlinx.serialization.builtins.ListSerializer
@@ -70,7 +80,10 @@
      * @param rulesToOverride
      */
     @Suppress("UnsafeCallOnNullableType")
-    private fun prepareOverriddenRulesConfig(rulesToDisable: List<Warnings> = emptyList(), rulesToOverride: RuleToConfig = emptyMap()): Path {
+    private fun prepareOverriddenRulesConfig(
+        rulesToDisable: List<Warnings> = emptyList(),
+        rulesToOverride: RuleToConfig = emptyMap(),
+    ): Path {
         val rulesConfig = RulesConfigReader().read(Paths.get(DEFAULT_CONFIG_PATH).inputStream())!!
             .toMutableList()
             .also { rulesConfig ->
@@ -294,7 +307,6 @@
     @Test
     @Tag("DiktatRuleSetProvider")
     @Timeout(TEST_TIMEOUT_SECONDS, unit = SECONDS)
-    @Disabled("https://github.com/saveourtool/diktat/issues/1737")
     fun `fix can cause long line`() {
         val configFilePath = prepareOverriddenRulesConfig(
             rulesToDisable = emptyList(),
@@ -390,36 +402,41 @@
 
     companion object {
         private const val DEFAULT_CONFIG_PATH = "../diktat-analysis.yml"
-        const val RESOURCE_FILE_PATH = "test/smoke/src/main/kotlin"
+        private const val ROOT_RESOURCE_FILE_PATH = "test/smoke"
+        private const val RESOURCE_FILE_PATH = "$ROOT_RESOURCE_FILE_PATH/src/main/kotlin"
         private const val TEST_TIMEOUT_SECONDS = 30L
-        private val tmpFiles: MutableList<File> = mutableListOf()
+
+        @JvmStatic
+        @TempDir
+        internal var tempDir: Path = Paths.get("/invalid")
 
         @BeforeAll
         @JvmStatic
+        @OptIn(ExperimentalPathApi::class)
         @Suppress("AVOID_NULL_CHECKS")
         internal fun createTmpFiles() {
-            listOf(
-                "$RESOURCE_FILE_PATH/../../../build.gradle.kts_" to "build.gradle.kts",
-                "$RESOURCE_FILE_PATH/Example1Test.kt" to "Example1-2Test.kt",
-            )
-                .map { (resource, targetFileName) ->
-                    DiktatSmokeTestBase::class.java
-                        .classLoader
-                        .getResource(resource)!!
-                        .toURI()
-                        .let {
-                            val tmpTestFile = File(it).parentFile.resolve(targetFileName)
-                            File(it).copyTo(tmpTestFile, true)
-                        }
-                        .let { tmpFiles.add(it) }
+            val resourceFilePath = DiktatSmokeTestBase::class.java
+                .classLoader
+                .getResource(ROOT_RESOURCE_FILE_PATH)
+                .let { resource ->
+                    requireNotNull(resource) {
+                        "$ROOT_RESOURCE_FILE_PATH not found"
+                    }
+                }
+                .toURI()
+                .toPath()
+            resourceFilePath.walk(PathWalkOption.INCLUDE_DIRECTORIES).forEach { file ->
+                if (file.isDirectory()) {
+                    tempDir.resolve(resourceFilePath.relativize(file)).createDirectories()
+                } else {
+                    val dest = tempDir.resolve(resourceFilePath.relativize(file))
+                    file.copyTo(dest)
+                    when (file.name) {
+                        "build.gradle.kts_" -> dest.moveTo(dest.parent.resolve("build.gradle.kts"))
+                        "Example1Test.kt" -> dest.copyTo(dest.parent.resolve("Example1-2Test.kt"))
+                    }
                 }
-        }
 
-        @AfterAll
-        @JvmStatic
-        internal fun deleteTmpFiles() {
-            tmpFiles.forEach {
-                it.toPath().deleteIfExistsSilently()
             }
         }
     }