Skip to content

Commit 49e7fb7

Browse files
Sort dependencies in user build file (#868)
* Sort dependencies and update descriptors * Revert unnecessary min version for JUnit4 * Convert mavenCoordinates into property
1 parent 4446a9d commit 49e7fb7

File tree

3 files changed

+12
-8
lines changed

3 files changed

+12
-8
lines changed

utbot-intellij/src/main/kotlin/org/utbot/intellij/plugin/models/ExternalLibraryDescriptors.kt

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,13 @@
11
package org.utbot.intellij.plugin.models
22

33
import com.intellij.openapi.roots.ExternalLibraryDescriptor
4+
import org.jetbrains.idea.maven.utils.library.RepositoryLibraryDescription
5+
6+
val ExternalLibraryDescriptor.mavenCoordinates: String
7+
get() = "$libraryGroupId:$libraryArtifactId:${preferredVersion ?: RepositoryLibraryDescription.ReleaseVersionId}"
8+
9+
val ExternalLibraryDescriptor.id: String
10+
get() = "$libraryGroupId:$libraryArtifactId"
411

512
//TODO: think about using JUnitExternalLibraryDescriptor from intellij-community sources (difficult to install)
613
fun jUnit4LibraryDescriptor(versionInProject: String?) =
@@ -10,7 +17,7 @@ fun jUnit5LibraryDescriptor(versionInProject: String?) =
1017
ExternalLibraryDescriptor("org.junit.jupiter", "junit-jupiter", "5.8.1", null, versionInProject ?: "5.8.1")
1118

1219
fun testNgLibraryDescriptor(versionInProject: String?) =
13-
ExternalLibraryDescriptor("org.testng", "testng", "6.8.8", null, versionInProject ?: "6.9.6")
20+
ExternalLibraryDescriptor("org.testng", "testng", "7.6.0", null, versionInProject ?: "7.6.0")
1421

1522
fun jUnit5ParametrizedTestsLibraryDescriptor(versionInProject: String?) =
1623
ExternalLibraryDescriptor("org.junit.jupiter", "junit-jupiter-params", "5.8.1", null, versionInProject ?: "5.8.1")

utbot-intellij/src/main/kotlin/org/utbot/intellij/plugin/ui/GenerateTestsDialogWindow.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,7 @@ import org.utbot.framework.plugin.api.MockStrategyApi
114114
import org.utbot.framework.plugin.api.TreatOverflowAsError
115115
import org.utbot.framework.util.Conflict
116116
import org.utbot.intellij.plugin.models.GenerateTestsModel
117+
import org.utbot.intellij.plugin.models.id
117118
import org.utbot.intellij.plugin.models.jUnit4LibraryDescriptor
118119
import org.utbot.intellij.plugin.models.jUnit5LibraryDescriptor
119120
import org.utbot.intellij.plugin.models.jUnit5ParametrizedTestsLibraryDescriptor
@@ -792,7 +793,7 @@ class GenerateTestsDialogWindow(val model: GenerateTestsModel) : DialogWrapper(m
792793
.addDependency(model.testModule, libraryDescriptor, DependencyScope.TEST)
793794
promise.thenRun {
794795
module.allLibraries()
795-
.lastOrNull { library -> library.libraryName == libraryDescriptor.presentableName }?.let {
796+
.lastOrNull { library -> library.presentableName.contains(libraryDescriptor.id) }?.let {
796797
ModuleRootModificationUtil.updateModel(module) { model -> placeEntryToCorrectPlace(model, it) }
797798
}
798799
}

utbot-intellij/src/main/kotlin/org/utbot/intellij/plugin/util/UtProjectModelModifier.kt

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,9 @@ import com.intellij.util.PathUtil
1717
import com.intellij.util.containers.ContainerUtil
1818
import org.jetbrains.concurrency.Promise
1919
import org.jetbrains.concurrency.resolvedPromise
20-
import org.jetbrains.idea.maven.utils.library.RepositoryLibraryDescription
2120
import org.jetbrains.idea.maven.utils.library.RepositoryLibraryProperties
2221
import org.jetbrains.jps.model.library.JpsMavenRepositoryLibraryDescriptor
22+
import org.utbot.intellij.plugin.models.mavenCoordinates
2323

2424
class UtProjectModelModifier(val project: Project) : IdeaProjectModelModifier(project) {
2525
override fun addExternalLibraryDependency(
@@ -38,7 +38,7 @@ class UtProjectModelModifier(val project: Project) : IdeaProjectModelModifier(pr
3838
} else {
3939
val roots = JarRepositoryManager.loadDependenciesModal(
4040
project,
41-
RepositoryLibraryProperties(JpsMavenRepositoryLibraryDescriptor(descriptor.mavenCoordinates())),
41+
RepositoryLibraryProperties(JpsMavenRepositoryLibraryDescriptor(descriptor.mavenCoordinates)),
4242
/* loadSources = */ false,
4343
/* loadJavadoc = */ false,
4444
/* copyTo = */ null,
@@ -78,8 +78,4 @@ class UtProjectModelModifier(val project: Project) : IdeaProjectModelModifier(pr
7878
}
7979
return resolvedPromise()
8080
}
81-
82-
private fun ExternalLibraryDescriptor.mavenCoordinates(): String {
83-
return "$libraryGroupId:$libraryArtifactId:${preferredVersion ?: RepositoryLibraryDescription.ReleaseVersionId}"
84-
}
8581
}

0 commit comments

Comments
 (0)