Apache
/
commons-collections
mirror of https://github.com/apache/commons-collections.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[COMPRESS-872] Add MultiKeyMapTest.testCompress872()

This commit is contained in:
Gary Gregory 2024-12-08 07:28:09 -05:00
parent bdb26ce0ad
commit bcd7973998
1 changed files with 39 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

50
src/test/java/org/apache/commons/collections4/map/MultiKeyMapTest.java
View File

@ -25,6 +25,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
import static org.junit.jupiter.api.Assertions.fail;
import java.util.Map;
import java.util.concurrent.atomic.AtomicReference;
import org.apache.commons.collections4.MapIterator;
import org.apache.commons.collections4.keyvalue.MultiKey;
@ -136,6 +137,32 @@ public class MultiKeyMapTest<K, V> extends AbstractIterableMapTest<MultiKey<? ex
assertSame(map.get(new MultiKey<>((K) I1, (K) I2)), cloned.get(new MultiKey<>((K) I1, (K) I2)));
}
/**
* Tests COMPRESS-872
* <p>
* Claim:
* </p>
* <ol>
* <li>Create a MultiKeyMap, with key(s) of a type (class/record) which has some fields.
* <li>Use multiKeyMap.put(T... keys, V value), to create an entry in the Map, to map the keys to a value
* <li>Use multiKeyMap.get(T... keys), to verify that the mapping exists and returns the expected value.
* <li>Modify/alter any of the objects used as a key. It is enough to change the value of any member field of any of the objects.
* <li>Use multiKeyMap.get(T... keys) again, however, now there is no mapping for these keys!
* <li>Use multiKeyMap.get(T... keys) with the new modified/altered objects, and it will return the expected value
* </ol>
*/
@Test
public void testCompress872() {
final AtomicReference<String> k1 = new AtomicReference<>("K1v1");
final AtomicReference<String> k2 = new AtomicReference<>("K2v1");
final MultiKeyMap<AtomicReference<String>, String> map = (MultiKeyMap<AtomicReference<String>, String>) makeObject();
assertNull(map.put(k1, k2, "V"));
assertEquals("V", map.get(k1, k2));
k1.set("K1v2");
assertEquals("V", map.get(k1, k2));
assertEquals("V", map.get(k1, k2));
}
@Test
@SuppressWarnings("unchecked")
public void testLRUMultiKeyMap() {
@ -437,6 +464,17 @@ public class MultiKeyMapTest<K, V> extends AbstractIterableMapTest<MultiKey<? ex
}
}
// public void testCreate() throws Exception {
// resetEmpty();
// writeExternalFormToDisk(
// (java.io.Serializable) map,
// "src/test/resources/data/test/MultiKeyMap.emptyCollection.version4.obj");
// resetFull();
// writeExternalFormToDisk(
// (java.io.Serializable) map,
// "src/test/resources/data/test/MultiKeyMap.fullCollection.version4.obj");
// }
@Test
public void testMultiKeyRemoveAll4() {
resetFull();
@ -451,17 +489,6 @@ public class MultiKeyMapTest<K, V> extends AbstractIterableMapTest<MultiKey<? ex
}
}
// public void testCreate() throws Exception {
// resetEmpty();
// writeExternalFormToDisk(
// (java.io.Serializable) map,
// "src/test/resources/data/test/MultiKeyMap.emptyCollection.version4.obj");
// resetFull();
// writeExternalFormToDisk(
// (java.io.Serializable) map,
// "src/test/resources/data/test/MultiKeyMap.fullCollection.version4.obj");
// }
@Test
@SuppressWarnings("unchecked")
public void testNullHandling() {
@ -480,4 +507,5 @@ public class MultiKeyMapTest<K, V> extends AbstractIterableMapTest<MultiKey<? ex
assertThrows(NullPointerException.class, () -> map.put(null, (V) new Object()));
}
}