Skip to content

Commit 0c8aa99

Browse files
buenaflorclaude
andcommitted
test(extend-app-start): Align NoOpAppStartExtenderTest with repo NoOp test idiom
Use the expression-body "does not throw" form for void no-op cases and drop the getInstance singleton-identity test that no other NoOp test asserts. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent c5edb6e commit 0c8aa99

1 file changed

Lines changed: 3 additions & 14 deletions

File tree

sentry/src/test/java/io/sentry/NoOpAppStartExtenderTest.kt

Lines changed: 3 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -6,23 +6,12 @@ import kotlin.test.assertSame
66
class NoOpAppStartExtenderTest {
77
private val extender = NoOpAppStartExtender.getInstance()
88

9-
@Test
10-
fun `getInstance returns the same singleton`() {
11-
assertSame(NoOpAppStartExtender.getInstance(), NoOpAppStartExtender.getInstance())
12-
}
9+
@Test fun `extendAppStart does not throw`() = extender.extendAppStart()
10+
11+
@Test fun `finishAppStart does not throw`() = extender.finishAppStart()
1312

1413
@Test
1514
fun `getExtendedAppStartSpan returns NoOpSpan`() {
1615
assertSame(NoOpSpan.getInstance(), extender.extendedAppStartSpan)
1716
}
18-
19-
@Test
20-
fun `extendAppStart is a no-op`() {
21-
extender.extendAppStart()
22-
}
23-
24-
@Test
25-
fun `finishAppStart is a no-op`() {
26-
extender.finishAppStart()
27-
}
2817
}

0 commit comments

Comments
 (0)