Fixed headTail pattern so that one element arrays match

bryanjos · bryanjos · commit 8145680a71d2 · 2017-07-22T09:02:45.000-05:00
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "tailored",
-  "version": "2.6.1",
+  "version": "2.6.2",
   "description": "Pattern matching library",
   "main": "lib/tailored.js",
   "jsnext:main": "src/index.js",
diff --git a/src/tailored/resolvers.js b/src/tailored/resolvers.js
@@ -72,7 +72,7 @@ function resolveVariable(pattern) {
 
 function resolveHeadTail() {
   return function(value, args) {
-    if (!Checks.is_array(value) || value.length < 2) {
+    if (!Checks.is_array(value) || value.length === 0) {
       return false;
     }
 
diff --git a/test/defmatch.spec.js b/test/defmatch.spec.js
@@ -112,6 +112,14 @@ describe('defmatch', () => {
     expect(fn([3, 1, 2, 4]).length).to.equal(3);
   });
 
+  it('must produce a head and a tail with one element array', () => {
+    let fn = Tailored.defmatch(
+      Tailored.clause([Tailored.headTail()], (head, tail) => tail)
+    );
+
+    expect(fn([3]).length).to.equal(0);
+  });
+
   it('must match on tuple', () => {
     let fn = Tailored.defmatch(
       Tailored.clause([Tailored.type(Tuple, { values: [1, 2, 3] })], () => 3)

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "tailored",`
`3`		`- "version": "2.6.1",`
	`3`	`+ "version": "2.6.2",`
`4`	`4`	`"description": "Pattern matching library",`
`5`	`5`	`"main": "lib/tailored.js",`
`6`	`6`	`"jsnext:main": "src/index.js",`
Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ function resolveVariable(pattern) {`
`72`	`72`
`73`	`73`	`function resolveHeadTail() {`
`74`	`74`	`return function(value, args) {`
`75`		`- if (!Checks.is_array(value) \|\| value.length < 2) {`
	`75`	`+ if (!Checks.is_array(value) \|\| value.length === 0) {`
`76`	`76`	`return false;`
`77`	`77`	`}`
`78`	`78`