Skip to content

Commit

Permalink
add 'disabledReason' to conditional disabled
Browse files Browse the repository at this point in the history
  • Loading branch information
aSemy committed Jul 26, 2022
1 parent 8f6cdc8 commit 38b0da2
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions mockk/jvm/src/test/kotlin/io/mockk/jvm/HashMapMockTest.kt
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ class HashMapMockTest {
}

@Test
@DisabledForJreRange(min = JRE.JAVA_17) // https://github.com/mockk/mockk/issues/864
@DisabledForJreRange(min = JRE.JAVA_17, disabledReason = "https://github.com/mockk/mockk/issues/864")
fun canSpyAHashMap() {
val map = spyk<HashMap<String, String>>()
assertDoesNotThrow { map["key"] = "value" }
Expand All @@ -37,7 +37,7 @@ class HashMapMockTest {
}

@Test
@DisabledForJreRange(min = JRE.JAVA_16) // https://github.com/mockk/mockk/issues/864
@DisabledForJreRange(min = JRE.JAVA_16, disabledReason = "https://github.com/mockk/mockk/issues/864")
fun concurrentHashMap_shouldBeSpied_Successfully() {
val map = spyk(ConcurrentHashMap<String, String>())
assertDoesNotThrow { map.put("key", "value") }
Expand All @@ -46,7 +46,7 @@ class HashMapMockTest {
}

@Test
@Disabled(value = "mocking of abstractMap don't work") // https://github.com/mockk/mockk/issues/864
@Disabled(value = "mocking AbstractMap doesn't work https://github.com/mockk/mockk/issues/864")
fun abstractMap_shouldBeMocked_SuccessFully() {
val map = mockk<AbstractMap<String, String>>()
assertDoesNotThrow { map["key"] }
Expand Down

0 comments on commit 38b0da2

Please sign in to comment.