[dev containers] AT-1851 Draft: using Eel/IJent test environment in S…

…tarter tests GitOrigin-RevId: a633dfdcc6593cbf21962c2adc4ddd36326dbc95
JetBrains · Jan 22, 2025 · 6a1570e · 6a1570e
1 parent de87cdc
commit 6a1570e
Showing 4 changed files with 11 additions and 7 deletions.
diff --git a/...tools.ide.starter.junit5/src/com/intellij/ide/starter/junit5/CurrentTestMethodProvider.kt b/...tools.ide.starter.junit5/src/com/intellij/ide/starter/junit5/CurrentTestMethodProvider.kt
@@ -10,7 +10,7 @@ import org.kodein.di.direct
 import org.kodein.di.instance
 
 /**
- * The listener do provides [CurrentTestMethod] to DI
+ * The listener that provides [CurrentTestMethod] to DI
  */
 open class CurrentTestMethodProvider : TestExecutionListener {
 

diff --git a/intellij.tools.ide.starter/src/com/intellij/ide/starter/path/GlobalPaths.kt b/intellij.tools.ide.starter/src/com/intellij/ide/starter/path/GlobalPaths.kt
@@ -33,10 +33,9 @@ abstract class GlobalPaths(val checkoutDir: Path) {
 
   val testsDirectory = (testHomePath / "tests").createDirectories()
 
-  private val cacheDirectory: Path = if (CIServer.instance.isBuildRunningOnCI &&
-                                         !System.getProperty("agent.persistent.cache").isNullOrEmpty()
-  ) {
-    (Paths.get(System.getProperty("agent.persistent.cache"), "perf-tests-cache")).createDirectories()
+  open val cacheDirectory: Path = if (CIServer.instance.isBuildRunningOnCI &&
+                                      !System.getProperty("agent.persistent.cache").isNullOrEmpty()) {
+    (Paths.get(System.getProperty("agent.persistent.cache"), "test-data-cache")).createDirectories()
   }
   else {
     (testHomePath / "cache").createDirectories()

diff --git a/intellij.tools.ide.starter/src/com/intellij/ide/starter/project/ProjectInfoSpec.kt b/intellij.tools.ide.starter/src/com/intellij/ide/starter/project/ProjectInfoSpec.kt
@@ -5,13 +5,17 @@ import java.nio.file.Path
 import kotlin.time.Duration
 
 interface ProjectInfoSpec {
+  /**
+   * "true" - the same project data can be shared between tests
+   * "false" - before each test the project will be removed and unpacked from scratch/cleaned (depends on the implementation of [ProjectInfoSpec]).
+   * */
   val isReusable: Boolean
   val downloadTimeout: Duration
 
   fun downloadAndUnpackProject(): Path?
 
   /**
-   * Use this to tune/configure project before IDE start
+   * Use this to tune/configure the project before IDE start
    */
   val configureProjectBeforeUse: (IDETestContext) -> Unit
 

diff --git a/intellij.tools.ide.starter/src/com/intellij/ide/starter/runner/CurrentTestMethod.kt b/intellij.tools.ide.starter/src/com/intellij/ide/starter/runner/CurrentTestMethod.kt
@@ -7,7 +7,8 @@ data class TestMethod(val name: String, val clazz: String, val clazzSimpleName:
 }
 
 /**
- * Container that contains current test method reference
+ * Container that contains the current test method reference.
+ * Method is provided by [com.intellij.ide.starter.junit5.CurrentTestMethodProvider]
  */
 object CurrentTestMethod {
   @Volatile