Skip to content

Commit db04069

Browse files
committed
patch .build() return type and name
1 parent 63af1a9 commit db04069

File tree

1 file changed

+3
-117
lines changed

1 file changed

+3
-117
lines changed

src/test/java/org/openrewrite/java/testing/junit5/UpgradeOkHttpMockWebServerTest.java

Lines changed: 3 additions & 117 deletions
Original file line numberDiff line numberDiff line change
@@ -50,44 +50,6 @@ public void defaults(RecipeSpec spec) {
5050
//.recipeFromResource("/META-INF/rewrite/junit5.yml", "org.openrewrite.java.testing.junit5.UpgradeOkHttpMockWebServer");
5151
}
5252

53-
// @Test
54-
// void wip() {
55-
// rewriteRun(
56-
// //language=java
57-
// java(
58-
// """
59-
// import okhttp3.Headers;
60-
// import okhttp3.mockwebserver.MockResponse;
61-
//
62-
// class A {
63-
// void someMethod() {
64-
// Headers headers = new Headers.Builder().build();
65-
// MockResponse a = new MockResponse();
66-
// // .status(String): void
67-
// // .getStatus(): String
68-
// // --
69-
// // .setStatus(String): MockResponse[this]
70-
// // ---
71-
// // .headers(Headers): void
72-
// // .setHeaders(Headers): MockResponse
73-
// // .getHeaders(): Headers
74-
// // ---
75-
// // .addHeader(String): MockResponse
76-
// // .addHeader(String,Object): MockResponse
77-
// // .addHeaderLenient(String,Object): MockResponse
78-
// // ---
79-
// // .setHeader(String,Object): MockResponse
80-
// // .removeHeader(String): MockResponse
81-
// // .clearHeaders(): MockResponse
82-
// a.header
83-
// a.trailers(headers);
84-
// }
85-
// }
86-
// """
87-
// )
88-
// );
89-
// }
90-
9153
@DocumentExample
9254
@Test
9355
void shouldUpgradeMavenDependency() {
@@ -280,89 +242,13 @@ void methodA() {
280242
void methodB() {
281243
mockWebServer.enqueue(
282244
new MockResponse.Builder()
283-
.status("hi")
284-
.headers(headersBuilder.build())
285-
.build()
286-
);
245+
.status("hi")
246+
.headers(headersBuilder.build())
247+
.build());
287248
}
288249
}
289250
"""
290251
)
291252
);
292253
}
293-
294-
// @Test
295-
// void wip() {
296-
// rewriteRun(
297-
// //language=java
298-
// java(
299-
// """
300-
// import okhttp3.Headers;
301-
// import okhttp3.mockwebserver.MockResponse;
302-
//
303-
// class A {
304-
// void someMethod() {
305-
// Headers headers = new Headers.Builder().build();
306-
// MockResponse a = new MockResponse();
307-
// // .status(String): void
308-
// // .getStatus(): String
309-
// // --
310-
// // .setStatus(String): MockResponse[this]
311-
// // ---
312-
// // .headers(Headers): void
313-
// // .setHeaders(Headers): MockResponse
314-
// // .getHeaders(): Headers
315-
// // ---
316-
// // .addHeader(String): MockResponse
317-
// // .addHeader(String,Object): MockResponse
318-
// // .addHeaderLenient(String,Object): MockResponse
319-
// // ---
320-
// // .setHeader(String,Object): MockResponse
321-
// // .removeHeader(String): MockResponse
322-
// // .clearHeaders(): MockResponse
323-
// a.header
324-
// a.trailers(headers);
325-
// }
326-
// }
327-
// """
328-
// )
329-
// );
330-
// }
331-
332-
@DocumentExample
333-
@Test
334-
void shouldUpgradeMavenDependency() {
335-
rewriteRun(
336-
spec -> spec.recipeFromResource("/META-INF/rewrite/junit5.yml", "org.openrewrite.java.testing.junit5.UpgradeOkHttpMockWebServer"),
337-
mavenProject("project",
338-
// TODO: handle solely J.NewClass and update declarative recipe to include new one.
339-
//language=xml
340-
pomXml(
341-
"""
342-
<project>
343-
<modelVersion>4.0.0</modelVersion>
344-
<groupId>com.example</groupId>
345-
<artifactId>demo</artifactId>
346-
<version>0.0.1-SNAPSHOT</version>
347-
<dependencies>
348-
<dependency>
349-
<groupId>com.squareup.okhttp3</groupId>
350-
<artifactId>mockwebserver</artifactId>
351-
<version>4.10.0</version>
352-
<scope>test</scope>
353-
</dependency>
354-
</dependencies>
355-
</project>
356-
""",
357-
spec -> spec.after(pom ->
358-
assertThat(pom)
359-
.doesNotContain("<artifactId>mockwebserver</artifactId>")
360-
.contains("<artifactId>mockwebserver3</artifactId>")
361-
.containsPattern("<version>5\\.(.*)</version>")
362-
.actual()
363-
)
364-
)
365-
)
366-
);
367-
}
368254
}

0 commit comments

Comments
 (0)