add tests

Author: shruti2522

oscars/src/collectors/mark_sweep_arena2/tests.rs

@@ -437,7 +437,7 @@ fn alive_wm() {
 
 #[test]
 fn two_distinct_keys_wm() {
-    // ensure two distinct Gc keys produce unique addresses and don't collide
+    // Ensure two distinct Gc keys produce unique addresses and don't collide
     // in the HashTable or corrupt each other's entries
     let collector = &mut MarkSweepGarbageCollector::default()
         .with_arena_size(256)
@@ -450,7 +450,7 @@ fn two_distinct_keys_wm() {
     map.insert(&key1, 10u64, collector);
     map.insert(&key2, 20u64, collector);
 
-    // verify both entries are independent and accessible
+    // Verify both entries are independent and accessible
     assert_eq!(
         map.get(&key1),
         Some(&10u64),
@@ -470,7 +470,7 @@ fn two_distinct_keys_wm() {
         "key2 should be alive",
     );
 
-    // dropping one key must prune only its entry, leaving the other intact
+    // Dropping one key must prune only its entry, leaving the other intact
     drop(key1);
     collector.collect();
 
@@ -484,7 +484,7 @@ fn two_distinct_keys_wm() {
         "key2 reported dead while still rooted",
     );
 
-    // both keys dead and collected means no leaks
+    // Both keys dead and collected means no leaks
     drop(key2);
     collector.collect();
 
@@ -497,7 +497,7 @@ fn two_distinct_keys_wm() {
 
 #[test]
 fn two_maps_same_key_wm() {
-    // same Gc key registered in two independent WeakMaps; each must hold its
+    // Same Gc key registered in two independent WeakMaps, each must hold its
     // own value and neither map's operations must bleed into the other
     let collector = &mut MarkSweepGarbageCollector::default()
         .with_arena_size(256)
@@ -525,8 +525,8 @@ fn two_maps_same_key_wm() {
 
 #[test]
 fn drop_map_with_live_key_wm() {
-    // dropping a WeakMap while its key is still alive must not corrupt the
-    // collector; it skips dead maps during prune_dead_entries
+    // Dropping a WeakMap while its key is still alive must not corrupt the
+    // collector, it skips dead maps during prune_dead_entries
     let collector = &mut MarkSweepGarbageCollector::default()
         .with_arena_size(256)
         .with_heap_threshold(512);
@@ -536,13 +536,13 @@ fn drop_map_with_live_key_wm() {
     {
         let mut map = WeakMap::<u64, u64>::new(collector);
         map.insert(&key, 100u64, collector);
-        // map dropped here; WeakMap::drop sets is_alive = false
+        // Map dropped here, WeakMap::drop sets is_alive = false
     }
 
-    // collector must handle the dead map entry without panic or corruption
+    // Collector must handle the dead map entry without panic or corruption
     collector.collect();
 
-    // key still live; drop it then collect to verify no leak
+    // Key still live, drop it then collect to verify no leak
     drop(key);
     collector.collect();