Skip to content

Commit 3d595ac

Browse files
Updating AddDependency test to better reflect allowed scopes for Maven (#167)
1 parent a401a0a commit 3d595ac

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/test/java/org/openrewrite/java/dependencies/AddDependencyTest.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,12 +49,12 @@ boolean getMap() {
4949

5050
@DocumentExample
5151
@Test
52-
void addMavenDependencyWithSystemScope() {
52+
void addMavenDependencyWithOnlyIfUsingTestScope() {
5353
rewriteRun(
5454
spec -> spec
55-
.recipe(addDependency("doesnotexist:doesnotexist:1", "com.google.common.math.IntMath", "system")),
55+
.recipe(addDependency("com.google.guava:guava:29.0-jre", "com.google.common.math.IntMath", "test")),
5656
mavenProject("project",
57-
srcMainJava(
57+
srcTestJava(
5858
java(usingGuavaIntMath)
5959
),
6060
//language=xml
@@ -73,10 +73,10 @@ void addMavenDependencyWithSystemScope() {
7373
<version>1</version>
7474
<dependencies>
7575
<dependency>
76-
<groupId>doesnotexist</groupId>
77-
<artifactId>doesnotexist</artifactId>
78-
<version>1</version>
79-
<scope>system</scope>
76+
<groupId>com.google.guava</groupId>
77+
<artifactId>guava</artifactId>
78+
<version>29.0-jre</version>
79+
<scope>test</scope>
8080
</dependency>
8181
</dependencies>
8282
</project>

0 commit comments

Comments
 (0)