Skip to content

Commit b980551

Browse files
committed
Fix compilation warnings
1 parent eac9955 commit b980551

File tree

2 files changed

+30
-16
lines changed

2 files changed

+30
-16
lines changed

spring-ws-core/src/test/java/org/springframework/ws/client/core/WebServiceTemplateTests.java

+19-11
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ public void testSendAndReceiveMessageResponse() throws Exception {
109109
WebServiceMessageCallback requestCallback = mock(WebServiceMessageCallback.class);
110110
requestCallback.doWithMessage(isA(WebServiceMessage.class));
111111

112-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
112+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
113113
Object extracted = new Object();
114114
when(extractorMock.extractData(isA(WebServiceMessage.class))).thenReturn(extracted);
115115

@@ -127,7 +127,7 @@ public void testSendAndReceiveMessageResponse() throws Exception {
127127
@Test
128128
public void testSendAndReceiveMessageNoResponse() throws Exception {
129129

130-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
130+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
131131

132132
this.connectionMock.send(isA(WebServiceMessage.class));
133133
when(this.connectionMock.hasError()).thenReturn(false);
@@ -142,7 +142,7 @@ public void testSendAndReceiveMessageNoResponse() throws Exception {
142142
@Test
143143
public void testSendAndReceiveMessageFault() throws Exception {
144144

145-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
145+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
146146

147147
FaultMessageResolver faultMessageResolverMock = mock(FaultMessageResolver.class);
148148
this.template.setFaultMessageResolver(faultMessageResolverMock);
@@ -165,7 +165,7 @@ public void testSendAndReceiveMessageFault() throws Exception {
165165
@Test
166166
public void testSendAndReceiveConnectionError() throws Exception {
167167

168-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
168+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
169169

170170
this.template.setFaultMessageResolver(null);
171171

@@ -184,7 +184,7 @@ public void testSendAndReceiveConnectionError() throws Exception {
184184
@Test
185185
public void testSendAndReceiveSourceResponse() throws Exception {
186186

187-
SourceExtractor extractorMock = mock(SourceExtractor.class);
187+
SourceExtractor<Object> extractorMock = mockSourceExtractor();
188188
Object extracted = new Object();
189189
when(extractorMock.extractData(isA(Source.class))).thenReturn(extracted);
190190

@@ -202,7 +202,7 @@ public void testSendAndReceiveSourceResponse() throws Exception {
202202
@Test
203203
public void testSendAndReceiveSourceNoResponse() throws Exception {
204204

205-
SourceExtractor extractorMock = mock(SourceExtractor.class);
205+
SourceExtractor<Object> extractorMock = mockSourceExtractor();
206206

207207
this.connectionMock.send(isA(WebServiceMessage.class));
208208
when(this.connectionMock.hasError()).thenReturn(false);
@@ -321,7 +321,7 @@ public boolean supports(URI uri) {
321321
WebServiceMessageCallback requestCallback = mock(WebServiceMessageCallback.class);
322322
requestCallback.doWithMessage(isA(WebServiceMessage.class));
323323

324-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
324+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
325325
Object extracted = new Object();
326326
when(extractorMock.extractData(isA(WebServiceMessage.class))).thenReturn(extracted);
327327

@@ -352,7 +352,7 @@ public void testInterceptors() throws Exception {
352352
WebServiceMessageCallback requestCallback = mock(WebServiceMessageCallback.class);
353353
requestCallback.doWithMessage(isA(WebServiceMessage.class));
354354

355-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
355+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
356356
Object extracted = new Object();
357357
when(extractorMock.extractData(isA(WebServiceMessage.class))).thenReturn(extracted);
358358

@@ -381,7 +381,7 @@ public void testInterceptorsInterceptedNoResponse() throws Exception {
381381
WebServiceMessageCallback requestCallback = mock(WebServiceMessageCallback.class);
382382
requestCallback.doWithMessage(messageContext.getRequest());
383383

384-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
384+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
385385

386386
Object result = this.template.doSendAndReceive(messageContext, this.connectionMock, requestCallback,
387387
extractorMock);
@@ -406,7 +406,7 @@ public void testInterceptorsInterceptedCreateResponse() throws Exception {
406406
WebServiceMessageCallback requestCallback = mock(WebServiceMessageCallback.class);
407407
requestCallback.doWithMessage(messageContext.getRequest());
408408

409-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
409+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
410410
Object extracted = new Object();
411411
when(extractorMock.extractData(messageContext.getResponse())).thenReturn(extracted);
412412

@@ -441,7 +441,7 @@ public boolean supports(URI uri) {
441441
}
442442
});
443443

444-
WebServiceMessageExtractor extractorMock = mock(WebServiceMessageExtractor.class);
444+
WebServiceMessageExtractor<Object> extractorMock = mockWebServiceMessageExtractor();
445445

446446
reset(this.connectionMock);
447447

@@ -456,4 +456,12 @@ public boolean supports(URI uri) {
456456
assertThat(result).isNull();
457457
}
458458

459+
private static WebServiceMessageExtractor<Object> mockWebServiceMessageExtractor() {
460+
return mock(WebServiceMessageExtractor.class);
461+
}
462+
463+
private static SourceExtractor<Object> mockSourceExtractor() {
464+
return mock(SourceExtractor.class);
465+
}
466+
459467
}

spring-ws-core/src/test/java/org/springframework/ws/wsdl/wsdl11/provider/InliningXsdSchemaTypesProviderTests.java

+11-5
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@
1616

1717
package org.springframework.ws.wsdl.wsdl11.provider;
1818

19+
import java.util.List;
20+
1921
import javax.wsdl.Definition;
2022
import javax.wsdl.Types;
2123
import javax.wsdl.extensions.schema.Schema;
@@ -65,9 +67,9 @@ public void testSingle() throws Exception {
6567
Types types = this.definition.getTypes();
6668

6769
assertThat(types).isNotNull();
68-
assertThat(types.getExtensibilityElements()).hasSize(1);
70+
assertThat(getExtensibilityElements(types)).hasSize(1);
6971

70-
Schema wsdlSchema = (Schema) types.getExtensibilityElements().get(0);
72+
Schema wsdlSchema = (Schema) getExtensibilityElements(types).get(0);
7173

7274
assertThat(wsdlSchema.getElement()).isNotNull();
7375
}
@@ -93,15 +95,19 @@ public void testComplex() throws Exception {
9395
Types types = this.definition.getTypes();
9496

9597
assertThat(types).isNotNull();
96-
assertThat(types.getExtensibilityElements()).hasSize(2);
98+
assertThat(getExtensibilityElements(types)).hasSize(2);
9799

98-
Schema wsdlSchema = (Schema) types.getExtensibilityElements().get(0);
100+
Schema wsdlSchema = (Schema) getExtensibilityElements(types).get(0);
99101

100102
assertThat(wsdlSchema.getElement()).isNotNull();
101103

102-
wsdlSchema = (Schema) types.getExtensibilityElements().get(1);
104+
wsdlSchema = (Schema) getExtensibilityElements(types).get(1);
103105

104106
assertThat(wsdlSchema.getElement()).isNotNull();
105107
}
106108

109+
private static List<?> getExtensibilityElements(Types types) {
110+
return types.getExtensibilityElements();
111+
}
112+
107113
}

0 commit comments

Comments
 (0)