diff options
| author | Benjamin J. Culkin <bjculkin@mix.wvu.edu> | 2020-12-14 19:29:06 -0400 |
|---|---|---|
| committer | Benjamin J. Culkin <bjculkin@mix.wvu.edu> | 2020-12-14 19:29:06 -0400 |
| commit | 7783bf18664f45c76fe4e5f583dac0803087de95 (patch) | |
| tree | f5328e5942a17e5d93d1110583f05d533684cef7 | |
| parent | c7a398170fe3bc8b0a9c0014d8e8183e392eda83 (diff) | |
Fix failing tests
| -rw-r--r-- | src/test/java/bjc/funcdata/TestMapCreation.java | 2 | ||||
| -rw-r--r-- | src/test/java/bjc/funcdata/TestMapOperations.java | 14 |
2 files changed, 5 insertions, 11 deletions
diff --git a/src/test/java/bjc/funcdata/TestMapCreation.java b/src/test/java/bjc/funcdata/TestMapCreation.java index 20dbe35..eeea591 100644 --- a/src/test/java/bjc/funcdata/TestMapCreation.java +++ b/src/test/java/bjc/funcdata/TestMapCreation.java @@ -33,6 +33,6 @@ public class TestMapCreation { MapEx<String, String> map = MapEx.of("a", "A", "b", "B"); assertTrue("Constructed map contains key 'a'", map.containsKey("a")); - assertEquals("Constructed map has key 'a' mapped to value 'A'", "A", map.get("a")); + assertEquals("Constructed map has key 'a' mapped to value 'A'", "A", map.get("a").get()); } } diff --git a/src/test/java/bjc/funcdata/TestMapOperations.java b/src/test/java/bjc/funcdata/TestMapOperations.java index d982b5e..7523d02 100644 --- a/src/test/java/bjc/funcdata/TestMapOperations.java +++ b/src/test/java/bjc/funcdata/TestMapOperations.java @@ -27,7 +27,7 @@ public class TestMapOperations { @Test public void getYieldsExpectedValue() { - assertEquals("Constructed map has key 'a' mapped to value 'A'", "A", map.get("a")); + assertEquals("Constructed map has key 'a' mapped to value 'A'", "A", map.get("a").get()); } @Test @@ -35,14 +35,8 @@ public class TestMapOperations { assertFalse("Constructed map doesn't contain key 'c'", map.containsKey("c")); } - @Test(expected = IllegalArgumentException.class) public void getOfNonexistentKeyThrows() { - map.get("c"); - } - - @Test - public void getOrDefaultOfExistingKeyYieldsExistingValue() { - assertEquals("Constructed map getOrDefault for an existing key yields the existing value", "A", map.get("a")); + assertFalse("Getting a non-existant key yields an absent optional", map.get("c").isPresent()); } @Test @@ -50,7 +44,7 @@ public class TestMapOperations { map.put("c", "C"); assertEquals("Constructed map now has 3 items", 3, map.size()); - assertEquals("Constructed map now has 'c' mapped to 'C'", "C", map.get("c")); + assertEquals("Constructed map now has 'c' mapped to 'C'", "C", map.get("c").get()); } @Test @@ -58,7 +52,7 @@ public class TestMapOperations { String val = map.put("a", "D"); assertEquals("Constructed map still contains 2 items", 2, map.size()); - assertEquals("Constructed map now has 'a' mapped to 'D'", "D", map.get("a")); + assertEquals("Constructed map now has 'a' mapped to 'D'", "D", map.get("a").get()); assertEquals("put method returned old value of 'A'", "A", val); } |
