Add tests for empty Set(|Hash).pick handling

Raku · May 21, 2017 · 4dfee69 · 4dfee69
1 parent c4c10d5
commit 4dfee69
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 2 deletions.
diff --git a/S02-types/set.t b/S02-types/set.t
@@ -1,7 +1,7 @@
 use v6;
 use Test;
 
-plan 189;
+plan 197;
 
 sub showset($s) { $s.keys.sort.join(' ') }
 
@@ -283,6 +283,22 @@ dies-ok { set(1, 2) «+» set(3, 4) }, 'Set «+» Set is illegal';
     is $s.total, 3, '.pick should not change Set';
 }
 
+# empty set handling of .pick
+{
+    is-deeply set().pick, Nil,         'empty set.pick -> Nil';
+    for
+      1,    '1',
+      *-1,  '*-1',
+      *,    '*',
+      Inf,  'Inf',
+      -1,   '-1',
+      -Inf, '-Inf'
+    -> $p, $t {
+        is-deeply set().pick($p), ().Seq, "empty set.rpick($t) -> ().Seq"
+    }
+    dies-ok { set().pick(NaN) }, 'empty set.pick(NaN) should die';
+}
+
 # L<S32::Containers/Set/grab>
 
 {

diff --git a/S02-types/sethash.t b/S02-types/sethash.t
@@ -1,7 +1,7 @@
 use v6;
 use Test;
 
-plan 229;
+plan 237;
 
 # L<S02/Mutable types/"QuantHash of Bool">
 
@@ -296,6 +296,22 @@ sub showset($s) { $s.keys.sort.join(' ') }
     is $s.elems, 3, '.pick should not change the SetHash';
 }
 
+# empty SetHash handling of .pick
+{
+    is-deeply ().SetHash.pick, Nil,            '().SetHash.pick -> Nil';
+    for
+      1,    '1',
+      *-1,  '*-1',
+      *,    '*',
+      Inf,  'Inf',
+      -1,   '-1',
+      -Inf, '-Inf'
+    -> $p, $t {
+        is-deeply ().SetHash.pick($p), ().Seq, "().SetHash.pick($t) -> ().Seq"
+    }
+    dies-ok { ().SetHash.pick(NaN) }, '().SetHash.pick(NaN) should die';
+}
+
 # L<S32::Containers/SetHash/grab>
 
 {