azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix to address platform expectation for project path

Browse Source 
Project path has to be absolute
This commit is contained in:
Vladimir Dolzhenko
2020-12-03 12:35:31 +01:00
parent 68f8e88d8b
commit 984b3c2f30
2 changed files with 16 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
idea/performanceTests/test/org/jetbrains/kotlin/idea/perf/AbstractPerformanceProjectsTest.kt
+4 -4
View File
@@ -166,9 +166,9 @@ abstract class AbstractPerformanceProjectsTest : UsefulTestCase() {
openAction: ProjectOpenAction,
fast: Boolean = false
): Project {
val projectPath = File(path).canonicalPath
val projectPath = File(path).absolutePath
assertTrue("path $path does not exist, check README.md", File(projectPath).exists())
assertTrue("path $projectPath does not exist, check README.md", File(projectPath).exists())
val warmUpIterations = if (fast) 0 else 5
val iterations = if (fast) 1 else 5
@@ -177,7 +177,7 @@ abstract class AbstractPerformanceProjectsTest : UsefulTestCase() {
var counter = 0
val openProject = OpenProject(
projectPath = path,
projectPath = projectPath,
projectName = name,
jdk = jdk18,
projectOpenAction = openAction
@@ -230,7 +230,7 @@ abstract class AbstractPerformanceProjectsTest : UsefulTestCase() {
myApplication.setDataProvider(TestDataProvider(project))
}
return lastProject ?: error("unable to open project $name at $path")
return lastProject ?: error("unable to open project $name at $projectPath")
}
fun perfTypeAndAutocomplete(
idea/performanceTests/test/org/jetbrains/kotlin/idea/testFramework/ProjectOpenAction.kt
+12 -9
View File
@@ -38,10 +38,11 @@ data class OpenProject(val projectPath: String, val projectName: String, val jdk
enum class ProjectOpenAction {
SIMPLE_JAVA_MODULE {
override fun openProject(projectPath: String, projectName: String, jdk: Sdk): Project {
val project = ProjectManagerEx.getInstanceEx().loadAndOpenProject(projectPath)!!
val path = File(projectPath).absolutePath
val project = ProjectManagerEx.getInstanceEx().loadAndOpenProject(path)!!
val modulePath = "$projectPath/$name${ModuleFileType.DOT_DEFAULT_EXTENSION}"
val projectFile = LocalFileSystem.getInstance().refreshAndFindFileByIoFile(File(projectPath))!!
val modulePath = "$path/$name${ModuleFileType.DOT_DEFAULT_EXTENSION}"
val projectFile = LocalFileSystem.getInstance().refreshAndFindFileByIoFile(File(path))!!
val srcFile = projectFile.findChild("src")!!
val module = runWriteAction {
@@ -61,16 +62,17 @@ enum class ProjectOpenAction {
EXISTING_IDEA_PROJECT {
override fun openProject(projectPath: String, projectName: String, jdk: Sdk): Project {
val path = File(projectPath).absolutePath
val projectManagerEx = ProjectManagerEx.getInstanceEx()
val project = loadProjectWithName(projectPath, projectName)
?: error("project $projectName at $projectPath is not loaded")
val project = loadProjectWithName(path, projectName)
?: error("project $projectName at $path is not loaded")
runWriteAction {
ProjectRootManager.getInstance(project).projectSdk = jdk
}
assertTrue(projectManagerEx.openProject(project), "project $projectName at $projectPath is not opened")
assertTrue(projectManagerEx.openProject(project), "project $projectName at $path is not opened")
return project
}
@@ -78,17 +80,18 @@ enum class ProjectOpenAction {
GRADLE_PROJECT {
override fun openProject(projectPath: String, projectName: String, jdk: Sdk): Project {
val project = ProjectManagerEx.getInstanceEx().loadAndOpenProject(projectPath)!!
val path = File(projectPath).absolutePath
val project = ProjectManagerEx.getInstanceEx().loadAndOpenProject(path)!!
assertTrue(
!project.isDisposed,
"Gradle project $projectName at $projectPath is accidentally disposed immediately after import"
"Gradle project $projectName at $path is accidentally disposed immediately after import"
)
runWriteAction {
ProjectRootManager.getInstance(project).projectSdk = jdk
}
refreshGradleProject(projectPath, project)
refreshGradleProject(path, project)
return project
}