[TS-38628] [Impacted Test Engine] Kotlin Migration

[Avanatiker]

Migration of the Impacted Test Engine. Follow up of #617

Summary

This Pull Request implements several changes across the project. The most notable updates include the migration to Kotlin conventions in various Gradle files, Java toolchain updates, dependency updates, and the introduction of new classes and test cases for an impacted test execution engine.

• Build System Updates:

  • Updated Java versions across Gradle files to align with recent conventions (Java 21 for some modules, Java 17 in others).
  • Adoption of kotlin-convention plugin in multiple Gradle scripts to unify project conventions.
  • Gradle dependency updates for libraries such as Mockito, Logback, and OkHttp.
  • Adjustments in settings.gradle.kts to remove unnecessary toolchains plugin and simplify includes.
  • Minor cleanup in .gitignore.

• Code Changes:

  • Enhanced impacted test execution engine, introducing new functionality for ordering and filtering impacted tests.
  • Added new classes for impacted tests: ImpactedTestEngine, ImpactedTestsSorter, InternalImpactedTestEngine, etc.
  • Refactored or migrated existing old implementation while making it Kotlin-friendly for better consistency.

• Test Cases:

  • Comprehensive new tests for intricate scenarios within impacted test execution workflows.
  • Enhanced support for dynamic or prioritized tests using mock-based unit cases.

[Avanatiker]

I have solved all true positives. Rest is false positive

[Avanatiker]

I have changed some docker code to comply to the best practices. I not sure if i tested it correctly. Is it possible to add docker tests?

[Avanatiker]

What do we do about the bazillion test gaps?

[cqse-teamscale-io]

• agent/src/main/java/com/teamscale/jacoco/agent/JaCoCoPreMain.java:63-77: [Test Gap] Changed method createRuntime has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/commons/IndentingWriter.kt:9-13: [Test Gap] Added method indent has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/commons/IndentingWriter.kt:16-20: [Test Gap] Added method writeLine has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/commons/IndentingWriter.kt:22-22: [Test Gap] Added method toString has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/engine/ImpactedTestEngine.kt:40-40: [Test Gap] Added method getArtifactId has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/engine/ImpactedTestEngine.kt:14-27: [Test Gap] Added method discover has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/engine/ImpactedTestEngine.kt:29-36: [Test Gap] Added method execute has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/engine/ImpactedTestEngine.kt:38-38: [Test Gap] Added method getGroupId has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/engine/TestDataWriter.kt:15-22: [Test Gap] Added method dumpTestExecutions has not been tested. (view in Teamscale)
• impacted-test-engine/src/main/kotlin/com/teamscale/test_impacted/engine/TestDataWriter.kt:25-32: [Test Gap] Added method dumpTestDetails has not been tested. (view in Teamscale)

See other 7 changed and 41 added untested methods.

[DreierF]

I have changed some docker code to comply to the best practices. I not sure if i tested it correctly. Is it possible to add docker tests?

I also did a manual test and seems to work as expected.

What do we do about the bazillion test gaps?
We need to ignore those as we cover them, but don't collect coverage for them at the moment, because it is very difficult to instrument due to the multiple nested JVM being involved in the process.