added single test for _pull

dskoda1 · Sep 22, 2016 · faaed8c · faaed8c
1 parent 71828be
commit faaed8c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/Deck.js b/src/Deck.js
@@ -137,7 +137,8 @@ class Deck {
         return this._pull(Array.prototype.removeRandom, n);
     }
 
-
+    // Internal method: called with an Array.prototype method,
+    // and a number of cards to remove.
     _pull(arrMethod, n) {
         if (n === undefined) {
             n = 1;

diff --git a/test/Deck-spec.js b/test/Deck-spec.js
@@ -311,6 +311,12 @@ describe('Deck', () => {
             });
         });
     })
+    describe('Deck._pull()', () => {
+        it('Will fail if called without a function for the first argument', () => {
+            let deck = new Deck();
+            expect(() => {deck._pull()}).to.throw();
+        })
+    })
 //////////////////////////////////////////////////////////////////////////
     describe('Deck.shuffle()', () => {
         let original_shuffle;