Skip to content

Commit 22dfa7e

Browse files
committed
Correct type attributions in before sources
1 parent 48fecc0 commit 22dfa7e

File tree

2 files changed

+28
-8
lines changed

2 files changed

+28
-8
lines changed

src/test/java/org/openrewrite/java/testing/hamcrest/RemoveIsMatcherTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,8 +135,8 @@ void noReplacementForOtherMethodInvocations() {
135135
java(
136136
"""
137137
import org.junit.jupiter.api.Test;
138+
import org.hamcrest.Matcher;
138139
import static org.hamcrest.MatcherAssert.assertThat;
139-
import static org.hamcrest.Matcher;
140140
import static org.hamcrest.Matchers.*;
141141
142142
class ATest {

src/test/java/org/openrewrite/java/testing/mockito/CleanupMockitoImportsTest.java

Lines changed: 27 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ void doNotRemoveImportsPossiblyAssociatedWithAnUntypedMockitoMethod() {
8080
"""
8181
import static org.mockito.Mockito.when;
8282
import static org.mockito.BDDMockito.given;
83-
import static org.mockito.Mockito.verifyNoInteractions;
83+
import static org.mockito.Mockito.verifyZeroInteractions;
8484
8585
class MyObjectTest {
8686
MyObject myObject;
@@ -89,9 +89,14 @@ class MyObjectTest {
8989
void test() {
9090
when(myObject.getSomeField()).thenReturn("testValue");
9191
given(myObject.getSomeField()).willReturn("testValue");
92-
verifyNoInteractions(myMock);
92+
verifyZeroInteractions(myMock);
9393
}
9494
}
95+
class MyObject {
96+
String getSomeField() { return null; }
97+
}
98+
class MyMockClass {
99+
}
95100
"""
96101
)
97102
);
@@ -106,7 +111,7 @@ void doNotRemoveImportsAssociatedWithAnUntypedMockitoMethodMixed() {
106111
import org.mockito.Mockito;
107112
import static org.mockito.Mockito.when;
108113
import static org.mockito.BDDMockito.given;
109-
import static org.mockito.Mockito.verifyNoInteractions;
114+
import static org.mockito.Mockito.verifyZeroInteractions;
110115
111116
class MyObjectTest {
112117
MyObject myObject;
@@ -115,10 +120,15 @@ class MyObjectTest {
115120
void test() {
116121
when(myObject.getSomeField()).thenReturn("testValue");
117122
given(myObject.getSomeField()).willReturn("testValue");
118-
verifyNoInteractions(myMock);
123+
verifyZeroInteractions(myMock);
119124
Mockito.reset();
120125
}
121126
}
127+
class MyObject {
128+
String getSomeField() { return null; }
129+
}
130+
class MyMockClass {
131+
}
122132
"""
123133
)
124134
);
@@ -133,7 +143,7 @@ void doNotRemoveImportsAssociatedWithATypedMockitoMethodMixed() {
133143
import org.mockito.Mockito;
134144
import static org.mockito.Mockito.when;
135145
import static org.mockito.BDDMockito.given;
136-
import static org.mockito.Mockito.verifyNoInteractions;
146+
import static org.mockito.Mockito.verifyZeroInteractions;
137147
138148
class MyObjectTest {
139149
MyObject myObject;
@@ -142,10 +152,17 @@ class MyObjectTest {
142152
void test() {
143153
when(myObject.getSomeField()).thenReturn("testValue");
144154
given(myObject.getSomeField()).willReturn("testValue");
145-
verifyNoInteractions(myMock);
155+
verifyZeroInteractions(myMock);
146156
Mockito.mock(OtherClass.class);
147157
}
148158
}
159+
class MyObject {
160+
String getSomeField() { return null; }
161+
}
162+
class MyMockClass {
163+
}
164+
class OtherClass {
165+
}
149166
"""
150167
)
151168
);
@@ -157,7 +174,7 @@ void doNotRemoveStartImportsPossiblyAssociatedWithAnUntypedMockitoMethod() {
157174
rewriteRun(
158175
java(
159176
"""
160-
import static org.mockito.Mockito.*;
177+
import static org.mockito.Mockito.when;
161178
162179
class MyObjectTest {
163180
MyObject myObject;
@@ -166,6 +183,9 @@ void test() {
166183
when(myObject.getSomeField()).thenReturn("testValue");
167184
}
168185
}
186+
class MyObject {
187+
String getSomeField() { return null; }
188+
}
169189
"""
170190
)
171191
);

0 commit comments

Comments
 (0)