Merge pull request #319 from evanp/assert-not-includes

Correct assignment for assert.notInclude() synonym assert.notIncludes()
vowsjs · Nov 4, 2014 · ad95bef · ad95bef
2 parents f18b325 + 875f0d3
commit ad95bef
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/lib/assert/macros.js b/lib/assert/macros.js
@@ -104,7 +104,7 @@ assert.notInclude = function (actual, expected, message) {
         assert.fail(actual, expected, message || "expected {actual} not to include {expected}", "include", assert.notInclude);
     }
 };
-assert.notIncludes = assert.notIncludes;
+assert.notIncludes = assert.notInclude;
 
 assert.deepInclude = function (actual, expected, message) {
     if (!isArray(actual)) {

diff --git a/test/assert-test.js b/test/assert-test.js
@@ -119,6 +119,10 @@ vows.describe('vows/assert').addBatch({
             assertError(assert.isNotEmpty, {});
             assertError(assert.isNotEmpty, []);
             assertError(assert.isNotEmpty, "");
+        },
+        "`notIncludes`": function () {
+            assert.notIncludes([1, 2, 3], 4);
+            assert.notIncludes({"red": 1, "blue": 2}, "green");
         }
     }
 }).export(module);
@@ -134,4 +138,3 @@ function assertError(assertion, args, fail) {
                                "expected an AssertionError for {actual}",
                                "assertError", assertError);
 }
-