Use addTestJdk&removeTestJdk methods instead addJdk&removeJdk

This commit is contained in:
Alexander Dudinsky
2021-01-20 09:46:03 +03:00
parent 20843b6894
commit 71e74497b5
2 changed files with 11 additions and 4 deletions
@@ -250,7 +250,7 @@ class GradleFacetImportTest : GradleImportingTestCase() {
@Test
fun testJsImportTransitive() {
configureByFiles()
importProject()
importProject(false)
with(facetSettings) {
Assert.assertEquals("1.3", languageLevel!!.versionString)
@@ -867,10 +867,14 @@ class GradleFacetImportTest : GradleImportingTestCase() {
}
override fun importProject() {
importProject(true)
}
fun importProject(skipIndexing: Boolean = true) {
val isCreateEmptyContentRootDirectories = currentExternalProjectSettings.isCreateEmptyContentRootDirectories
try {
currentExternalProjectSettings.isCreateEmptyContentRootDirectories = true
super.importProject(true)
super.importProject(skipIndexing)
} finally {
currentExternalProjectSettings.isCreateEmptyContentRootDirectories = isCreateEmptyContentRootDirectories
}
@@ -31,6 +31,7 @@ import com.intellij.openapi.fileTypes.FileTypeManager
import com.intellij.openapi.projectRoots.JavaSdk
import com.intellij.openapi.projectRoots.ProjectJdkTable
import com.intellij.openapi.projectRoots.Sdk
import com.intellij.openapi.projectRoots.impl.ProjectJdkTableImpl
import com.intellij.openapi.projectRoots.impl.SdkConfigurationUtil
import com.intellij.openapi.roots.ProjectRootManager
import com.intellij.openapi.ui.Messages
@@ -140,7 +141,8 @@ abstract class GradleImportingTestCase : ExternalSystemImportingTestCase() {
val jdk = SdkConfigurationUtil.setupSdk(arrayOfNulls(0), jdkHomeDir, JavaSdk.getInstance(), true, null, GRADLE_JDK_NAME)
TestCase.assertNotNull("Cannot create JDK for $myJdkHome", jdk)
if (!jdkTable.allJdks.contains(jdk)) {
jdkTable.addJdk(jdk!!, testRootDisposable)
// jdkTable.addJdk(jdk!!, testRootDisposable)
(jdkTable as ProjectJdkTableImpl).addTestJdk(jdk!!, testRootDisposable)
ProjectRootManager.getInstance(myProject).projectSdk = jdk
}
FileTypeManager.getInstance().associateExtension(GroovyFileType.GROOVY_FILE_TYPE, "gradle")
@@ -176,7 +178,8 @@ abstract class GradleImportingTestCase : ExternalSystemImportingTestCase() {
ThrowableRunnable {
runWrite {
Arrays.stream(ProjectJdkTable.getInstance().allJdks).forEach { jdk: Sdk ->
ProjectJdkTable.getInstance().removeJdk(jdk)
(ProjectJdkTable.getInstance() as ProjectJdkTableImpl).removeTestJdk(jdk)
// ProjectJdkTable.getInstance().removeJdk(jdk)
}
for (sdk in removedSdks) {
SdkConfigurationUtil.addSdk(sdk)