Skip to content

Commit a85bfc7

Browse files
authored
Fix lint tests (#1575)
1 parent 3cf0927 commit a85bfc7

File tree

7 files changed

+14
-15
lines changed

7 files changed

+14
-15
lines changed

sentry-android-okhttp/src/test/java/io/sentry/android/okhttp/SentryOkHttpInterceptorTest.kt

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -185,17 +185,16 @@ class SentryOkHttpInterceptorTest {
185185

186186
@Test
187187
fun `customizer receives request and response`() {
188-
var request: Request? = null
189188
val sut = fixture.getSut(beforeSpan = object : SentryOkHttpInterceptor.BeforeSpanCallback {
190-
override fun execute(span: ISpan, req: Request, res: Response?): ISpan {
191-
assertEquals(request!!.url, req.url)
192-
assertEquals(request!!.method, req.method)
193-
assertNotNull(res) {
189+
override fun execute(span: ISpan, request: Request, response: Response?): ISpan {
190+
assertEquals(request.url, request.url)
191+
assertEquals(request.method, request.method)
192+
assertNotNull(response) {
194193
assertEquals(201, it.code)
195194
}
196195
return span
197196
} })
198-
request = getRequest()
197+
val request = getRequest()
199198
sut.newCall(request).execute()
200199
}
201200

sentry-servlet/src/test/kotlin/io/sentry/servlet/SentryRequestHttpServletRequestProcessorTest.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,8 @@ class SentryRequestHttpServletRequestProcessorTest {
9191

9292
eventProcessor.process(event, null)
9393

94-
assertNotNull(event.request) { request ->
95-
assertNotNull(request.headers) {
94+
assertNotNull(event.request) { req ->
95+
assertNotNull(req.headers) {
9696
assertFalse(it.containsKey("X-FORWARDED-FOR"))
9797
assertFalse(it.containsKey("Authorization"))
9898
assertFalse(it.containsKey("authorization"))

sentry-spring-boot-starter/src/test/kotlin/io/sentry/spring/boot/it/SentrySpringIntegrationTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ class SentrySpringIntegrationTest {
6464
lateinit var hub: IHub
6565

6666
@LocalServerPort
67-
lateinit var port: Integer
67+
var port: Int? = null
6868

6969
@Before
7070
fun reset() {

sentry-spring/src/test/kotlin/io/sentry/spring/mvc/SentrySpringIntegrationTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ class SentrySpringIntegrationTest {
7777
lateinit var hub: IHub
7878

7979
@LocalServerPort
80-
lateinit var port: Integer
80+
var port: Int? = null
8181

8282
@Before
8383
fun `reset mocks`() {

sentry-spring/src/test/kotlin/io/sentry/spring/webflux/SentryWebfluxIntegrationTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ class SentryWebfluxIntegrationTest {
5151
lateinit var transport: ITransport
5252

5353
@LocalServerPort
54-
lateinit var port: Integer
54+
var port: Int? = null
5555

5656
private val testClient = WebTestClient.bindToServer().build()
5757

sentry/src/main/java/io/sentry/OptionsContainer.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,11 @@
77
@ApiStatus.Internal
88
public final class OptionsContainer<T> {
99

10-
public static <T> OptionsContainer<T> create(final Class<T> clazz) {
10+
public @NotNull static <T> OptionsContainer<T> create(final @NotNull Class<T> clazz) {
1111
return new OptionsContainer<>(clazz);
1212
}
1313

14-
private final Class<T> clazz;
14+
private final @NotNull Class<T> clazz;
1515

1616
private OptionsContainer(final @NotNull Class<T> clazz) {
1717
super();

sentry/src/test/java/io/sentry/config/PropertiesProviderTest.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,15 @@ class PropertiesProviderTest {
2828
}
2929

3030
@Test
31-
fun `when property is value is "true" returns true`() {
31+
fun `when property is value is true returns true`() {
3232
whenever(propertiesProvider.getProperty(any())).thenReturn("true")
3333
val result = propertiesProvider.getBooleanProperty("prop")
3434
assertNotNull(result)
3535
assertTrue(result)
3636
}
3737

3838
@Test
39-
fun `when property is value is "false" returns false`() {
39+
fun `when property is value is false returns false`() {
4040
whenever(propertiesProvider.getProperty(any())).thenReturn("false")
4141
val result = propertiesProvider.getBooleanProperty("prop")
4242
assertNotNull(result)

0 commit comments

Comments
 (0)