Skip to content

Commit 96b22ee

Browse files
andreacastelloandreacastello
committed
Made all test classes and methods public
1 parent 9a1f2aa commit 96b22ee

9 files changed

Lines changed: 53 additions & 88 deletions

File tree

‎extra/modules/wurfl-devicedetection/src/test/java/org/prebid/server/hooks/modules/com/scientiamobile/wurfl/devicedetection/config/WURFLDeviceDetectionConfigPropertiesTest.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,10 @@
66

77
import static org.assertj.core.api.Assertions.assertThat;
88

9-
class WURFLDeviceDetectionConfigPropertiesTest {
9+
public class WURFLDeviceDetectionConfigPropertiesTest {
1010

1111
@Test
12-
void shouldInitializeWithEmptyValues() {
12+
public void shouldInitializeWithEmptyValues() {
1313
// given
1414
final WURFLDeviceDetectionConfigProperties properties = new WURFLDeviceDetectionConfigProperties();
1515

@@ -25,7 +25,7 @@ void shouldInitializeWithEmptyValues() {
2525
}
2626

2727
@Test
28-
void shouldSetAndGetProperties() {
28+
public void shouldSetAndGetProperties() {
2929
// given
3030
final WURFLDeviceDetectionConfigProperties properties = new WURFLDeviceDetectionConfigProperties();
3131

‎extra/modules/wurfl-devicedetection/src/test/java/org/prebid/server/hooks/modules/com/scientiamobile/wurfl/devicedetection/config/WURFLDeviceDetectionConfigurationTest.java‎

Lines changed: 0 additions & 35 deletions
This file was deleted.

‎extra/modules/wurfl-devicedetection/src/test/java/org/prebid/server/hooks/modules/com/scientiamobile/wurfl/devicedetection/model/AuctionRequestHeadersContextTest.java‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@
55

66
import static org.assertj.core.api.Assertions.assertThat;
77

8-
class AuctionRequestHeadersContextTest {
8+
public class AuctionRequestHeadersContextTest {
99

1010
@Test
11-
void fromShouldHandleNullHeaders() {
11+
public void fromShouldHandleNullHeaders() {
1212
// when
1313
final AuctionRequestHeadersContext result = AuctionRequestHeadersContext.from(null);
1414

@@ -17,7 +17,7 @@ void fromShouldHandleNullHeaders() {
1717
}
1818

1919
@Test
20-
void fromShouldConvertCaseInsensitiveMultiMapToHeaders() {
20+
public void fromShouldConvertCaseInsensitiveMultiMapToHeaders() {
2121
// given
2222
final CaseInsensitiveMultiMap multiMap = CaseInsensitiveMultiMap.builder()
2323
.add("User-Agent", "Mozilla/5.0 (Windows NT 10.0; Win64; x64) Test")
@@ -35,7 +35,7 @@ void fromShouldConvertCaseInsensitiveMultiMapToHeaders() {
3535
}
3636

3737
@Test
38-
void fromShouldTakeFirstValueForDuplicateHeaders() {
38+
public void fromShouldTakeFirstValueForDuplicateHeaders() {
3939
// given
4040
final CaseInsensitiveMultiMap multiMap = CaseInsensitiveMultiMap.builder()
4141
.add("User-Agent", "Mozilla/5.0 (Windows NT 10.0; Win64; x64) Test")
@@ -52,7 +52,7 @@ void fromShouldTakeFirstValueForDuplicateHeaders() {
5252
}
5353

5454
@Test
55-
void fromShouldHandleEmptyMultiMap() {
55+
public void fromShouldHandleEmptyMultiMap() {
5656
// given
5757
final CaseInsensitiveMultiMap emptyMultiMap = CaseInsensitiveMultiMap.empty();
5858

‎extra/modules/wurfl-devicedetection/src/test/java/org/prebid/server/hooks/modules/com/scientiamobile/wurfl/devicedetection/model/WURFLEngineUtilsTest.java‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,13 @@
1111
import static org.assertj.core.api.Assertions.assertThatThrownBy;
1212

1313
@ExtendWith(MockitoExtension.class)
14-
class WURFLEngineUtilsTest {
14+
public class WURFLEngineUtilsTest {
1515

1616
@Mock(strictness = LENIENT)
1717
private WURFLDeviceDetectionConfigProperties configProperties;
1818

1919
@Test
20-
void extractWURFLFileNameShouldReturnCorrectFileName() {
20+
public void extractWURFLFileNameShouldReturnCorrectFileName() {
2121
// given
2222
final String url = "https://data.examplehost.com/snapshot/wurfl-latest.zip";
2323

@@ -29,7 +29,7 @@ void extractWURFLFileNameShouldReturnCorrectFileName() {
2929
}
3030

3131
@Test
32-
void extractWURFLFileNameShouldHandleSimpleFileName() {
32+
public void extractWURFLFileNameShouldHandleSimpleFileName() {
3333
// given
3434
final String url = "http://example.com/wurfl.zip";
3535

@@ -41,7 +41,7 @@ void extractWURFLFileNameShouldHandleSimpleFileName() {
4141
}
4242

4343
@Test
44-
void extractWURFLFileNameShouldHandleComplexPath() {
44+
public void extractWURFLFileNameShouldHandleComplexPath() {
4545
// given
4646
final String url = "https://examplehost.com/path/to/files/wurfl-snapshot.zip";
4747

@@ -53,7 +53,7 @@ void extractWURFLFileNameShouldHandleComplexPath() {
5353
}
5454

5555
@Test
56-
void extractWURFLFileNameShouldHandleUrlWithQueryParams() {
56+
public void extractWURFLFileNameShouldHandleUrlWithQueryParams() {
5757
// given
5858
final String url = "https://example.com/wurfl.zip?version=latest&format=zip";
5959

@@ -65,15 +65,15 @@ void extractWURFLFileNameShouldHandleUrlWithQueryParams() {
6565
}
6666

6767
@Test
68-
void extractWURFLFileNameShouldThrowExceptionForNullUrl() {
68+
public void extractWURFLFileNameShouldThrowExceptionForNullUrl() {
6969
// when & then
7070
assertThatThrownBy(() -> WURFLEngineUtils.extractWURFLFileName(null))
7171
.isInstanceOf(IllegalArgumentException.class)
7272
.hasMessageContaining("Invalid WURFL snapshot URL: null");
7373
}
7474

7575
@Test
76-
void initializeEngineShouldThrowExceptionForNullDataFilePath() {
76+
public void initializeEngineShouldThrowExceptionForNullDataFilePath() {
7777
// when & then
7878
assertThatThrownBy(() -> WURFLEngineUtils.initializeEngine(configProperties, null))
7979
.isInstanceOf(IllegalArgumentException.class)

‎extra/modules/wurfl-devicedetection/src/test/java/org/prebid/server/hooks/modules/com/scientiamobile/wurfl/devicedetection/resolver/HeadersResolverTest.java‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,12 @@
1212

1313
import static org.assertj.core.api.Assertions.assertThat;
1414

15-
class HeadersResolverTest {
15+
public class HeadersResolverTest {
1616

1717
private static final String TEST_USER_AGENT = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36";
1818

1919
@Test
20-
void resolveWithNullDeviceShouldReturnOriginalHeaders() {
20+
public void resolveWithNullDeviceShouldReturnOriginalHeaders() {
2121
// given
2222
final Map<String, String> headers = new HashMap<>();
2323
headers.put("test", "value");
@@ -30,7 +30,7 @@ void resolveWithNullDeviceShouldReturnOriginalHeaders() {
3030
}
3131

3232
@Test
33-
void resolveWithDeviceUaShouldReturnUserAgentHeader() {
33+
public void resolveWithDeviceUaShouldReturnUserAgentHeader() {
3434
// given
3535
final Device device = Device.builder()
3636
.ua("Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36")
@@ -44,7 +44,7 @@ void resolveWithDeviceUaShouldReturnUserAgentHeader() {
4444
}
4545

4646
@Test
47-
void resolveWithFullSuaShouldReturnAllHeaders() {
47+
public void resolveWithFullSuaShouldReturnAllHeaders() {
4848
// given
4949
final BrandVersion brandVersion = new BrandVersion(
5050
"Chrome",
@@ -82,7 +82,7 @@ void resolveWithFullSuaShouldReturnAllHeaders() {
8282
}
8383

8484
@Test
85-
void resolveWithFullDeviceAndHeadersShouldPrioritizeDevice() {
85+
public void resolveWithFullDeviceAndHeadersShouldPrioritizeDevice() {
8686
// given
8787
final BrandVersion brandVersion = new BrandVersion(
8888
"Chrome",
@@ -130,7 +130,7 @@ void resolveWithFullDeviceAndHeadersShouldPrioritizeDevice() {
130130
}
131131

132132
@Test
133-
void resolveWithMultipleBrandVersionsShouldFormatCorrectly() {
133+
public void resolveWithMultipleBrandVersionsShouldFormatCorrectly() {
134134
// given
135135
final BrandVersion chrome = new BrandVersion("Chrome",
136136
Arrays.asList("100", "0"),
@@ -162,7 +162,7 @@ void resolveWithMultipleBrandVersionsShouldFormatCorrectly() {
162162
}
163163

164164
@Test
165-
void resolveWithNullDeviceAndNullHeadersShouldReturnEmptyMap() {
165+
public void resolveWithNullDeviceAndNullHeadersShouldReturnEmptyMap() {
166166
// when
167167
final Map<String, String> result = HeadersResolver.resolve(null, null);
168168

‎extra/modules/wurfl-devicedetection/src/test/java/org/prebid/server/hooks/modules/com/scientiamobile/wurfl/devicedetection/v1/OrtbDeviceUpdaterTest.java‎

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
import static org.prebid.server.hooks.modules.com.scientiamobile.wurfl.devicedetection.mock.WURFLDeviceMock.WURFLDeviceMockFactory.mockUnknownDevice;
3030

3131
@ExtendWith(MockitoExtension.class)
32-
class OrtbDeviceUpdaterTest {
32+
public class OrtbDeviceUpdaterTest {
3333

3434
private Set<String> staticCaps;
3535
private Set<String> virtualCaps;
@@ -47,7 +47,7 @@ void setUp() {
4747
}
4848

4949
@Test
50-
void updateShouldUpdateDeviceMakeWhenOriginalIsEmpty() {
50+
public void updateShouldUpdateDeviceMakeWhenOriginalIsEmpty() {
5151
// given
5252
final var wurflDevice = mockIPhone();
5353
final OrtbDeviceUpdater target = new OrtbDeviceUpdater(wurflDevice, staticCaps, virtualCaps, true, mapper);
@@ -65,7 +65,7 @@ void updateShouldUpdateDeviceMakeWhenOriginalIsEmpty() {
6565
}
6666

6767
@Test
68-
void updateShouldNotUpdateDeviceMakeWhenOriginalExists() {
68+
public void updateShouldNotUpdateDeviceMakeWhenOriginalExists() {
6969
// given
7070
final Device device = Device.builder().make("Samsung").build();
7171
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -82,7 +82,7 @@ void updateShouldNotUpdateDeviceMakeWhenOriginalExists() {
8282
}
8383

8484
@Test
85-
void updateShouldNotUpdateDeviceMakeWhenOriginalBigIntegerExists() {
85+
public void updateShouldNotUpdateDeviceMakeWhenOriginalBigIntegerExists() {
8686
// given
8787
final Device device = Device.builder().make("Apple").pxratio(new BigDecimal("1.0")).build();
8888
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -100,7 +100,7 @@ void updateShouldNotUpdateDeviceMakeWhenOriginalBigIntegerExists() {
100100
}
101101

102102
@Test
103-
void updateShouldUpdateDeviceModelWhenOriginalIsEmpty() {
103+
public void updateShouldUpdateDeviceModelWhenOriginalIsEmpty() {
104104
// given
105105
final Device device = Device.builder().build();
106106
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -117,7 +117,7 @@ void updateShouldUpdateDeviceModelWhenOriginalIsEmpty() {
117117
}
118118

119119
@Test
120-
void updateShouldUpdateDeviceOsWhenOriginalIsEmpty() {
120+
public void updateShouldUpdateDeviceOsWhenOriginalIsEmpty() {
121121
// given
122122
final Device device = Device.builder().build();
123123
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -134,7 +134,7 @@ void updateShouldUpdateDeviceOsWhenOriginalIsEmpty() {
134134
}
135135

136136
@Test
137-
void updateShouldUpdateResolutionWhenOriginalIsEmpty() {
137+
public void updateShouldUpdateResolutionWhenOriginalIsEmpty() {
138138
// given
139139
final Device device = Device.builder().build();
140140
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -152,7 +152,7 @@ void updateShouldUpdateResolutionWhenOriginalIsEmpty() {
152152
}
153153

154154
@Test
155-
void updateShouldHandleJavascriptSupport() {
155+
public void updateShouldHandleJavascriptSupport() {
156156
// given
157157
final Device device = Device.builder().build();
158158
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -169,7 +169,7 @@ void updateShouldHandleJavascriptSupport() {
169169
}
170170

171171
@Test
172-
void updateShouldHandleOttDeviceType() {
172+
public void updateShouldHandleOttDeviceType() {
173173
// given
174174
final Device device = Device.builder().build();
175175
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -185,7 +185,7 @@ void updateShouldHandleOttDeviceType() {
185185
}
186186

187187
@Test
188-
void updateShouldReturnDeviceOtherMobileWhenMobileIsNotPhoneOrTablet() {
188+
public void updateShouldReturnDeviceOtherMobileWhenMobileIsNotPhoneOrTablet() {
189189
// given
190190
final Device device = Device.builder().build();
191191
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -200,7 +200,7 @@ void updateShouldReturnDeviceOtherMobileWhenMobileIsNotPhoneOrTablet() {
200200
}
201201

202202
@Test
203-
void updateShouldReturnNullWhenMobileTypeIsUnknown() {
203+
public void updateShouldReturnNullWhenMobileTypeIsUnknown() {
204204
// given
205205
final Device device = Device.builder().build();
206206
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -215,7 +215,7 @@ void updateShouldReturnNullWhenMobileTypeIsUnknown() {
215215
}
216216

217217
@Test
218-
void updateShouldHandlePersonalComputerDeviceType() {
218+
public void updateShouldHandlePersonalComputerDeviceType() {
219219
// given
220220
final Device device = Device.builder().build();
221221
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -231,7 +231,7 @@ void updateShouldHandlePersonalComputerDeviceType() {
231231
}
232232

233233
@Test
234-
void updateShouldHandleConnectedTvDeviceType() {
234+
public void updateShouldHandleConnectedTvDeviceType() {
235235
// given
236236
final Device device = Device.builder().build();
237237
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -246,7 +246,7 @@ void updateShouldHandleConnectedTvDeviceType() {
246246
}
247247

248248
@Test
249-
void updateShouldNotUpdateDeviceTypeWhenSet() {
249+
public void updateShouldNotUpdateDeviceTypeWhenSet() {
250250
// given
251251
final Device device = Device.builder()
252252
.devicetype(3)
@@ -263,7 +263,7 @@ void updateShouldNotUpdateDeviceTypeWhenSet() {
263263
}
264264

265265
@Test
266-
void updateShouldHandleTabletDeviceType() {
266+
public void updateShouldHandleTabletDeviceType() {
267267
// given
268268
final Device device = Device.builder().build();
269269
final BidRequest bidRequest = BidRequest.builder().device(device).build();
@@ -279,7 +279,7 @@ void updateShouldHandleTabletDeviceType() {
279279
}
280280

281281
@Test
282-
void updateShouldAddWurflPropertyToExtIfMissingAndPreserveExistingProperties() {
282+
public void updateShouldAddWurflPropertyToExtIfMissingAndPreserveExistingProperties() {
283283
// given
284284
final ExtDevice existingExt = ExtDevice.empty();
285285
existingExt.addProperty("someProperty", TextNode.valueOf("value"));

‎extra/modules/wurfl-devicedetection/src/test/java/org/prebid/server/hooks/modules/com/scientiamobile/wurfl/devicedetection/v1/WURFLDeviceDetectionEntrypointHookTest.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
import static org.mockito.Mockito.mock;
1616
import static org.mockito.Mockito.when;
1717

18-
class WURFLDeviceDetectionEntrypointHookTest {
18+
public class WURFLDeviceDetectionEntrypointHookTest {
1919

2020
private EntrypointPayload payload;
2121
private InvocationContext context;
@@ -27,7 +27,7 @@ void setUp() {
2727
}
2828

2929
@Test
30-
void codeShouldReturnCorrectHookCode() {
30+
public void codeShouldReturnCorrectHookCode() {
3131

3232
// given
3333
final WURFLDeviceDetectionEntrypointHook target = new WURFLDeviceDetectionEntrypointHook();
@@ -40,7 +40,7 @@ void codeShouldReturnCorrectHookCode() {
4040
}
4141

4242
@Test
43-
void callShouldReturnSuccessWithNoAction() {
43+
public void callShouldReturnSuccessWithNoAction() {
4444
// given
4545
final WURFLDeviceDetectionEntrypointHook target = new WURFLDeviceDetectionEntrypointHook();
4646
final CaseInsensitiveMultiMap headers = CaseInsensitiveMultiMap.builder()
@@ -61,7 +61,7 @@ void callShouldReturnSuccessWithNoAction() {
6161
}
6262

6363
@Test
64-
void callShouldHandleNullHeaders() {
64+
public void callShouldHandleNullHeaders() {
6565
// given
6666
final WURFLDeviceDetectionEntrypointHook target = new WURFLDeviceDetectionEntrypointHook();
6767

0 commit comments

Comments
 (0)