v0.0.18: ArrayTome.indexOf and lastIndexOf now do a valueOf on the va…

…lue to be searched for.

component.json

@@ -2,7 +2,7 @@
   "name": "tomes",
   "repo": "Wizcorp/node-tomes",
   "description": "Evented Storage Agnostic Data API",
-  "version": "0.0.17",
+  "version": "0.0.18",
   "keywords": [ "data", "events" ],
   "dependencies": {
     "component/emitter": "*"

index.js

@@ -1407,8 +1407,10 @@ ArrayTome.prototype.unshift = function () {
 	return this._arr.length;
 };
 
-ArrayTome.prototype.indexOf = function (searchElement) {
+ArrayTome.prototype.indexOf = function (s) {
 	// straight from MDN, Tomes have to do valueOf instead of ===
+	var searchElement = s ? s.valueOf() : s;
+
 	var t = this._arr;
 	var len = t.length;
 
@@ -1442,8 +1444,10 @@ ArrayTome.prototype.indexOf = function (searchElement) {
 	return -1;
 };
 
-ArrayTome.prototype.lastIndexOf = function (searchElement) {
+ArrayTome.prototype.lastIndexOf = function (s) {
 	// straight from MDN, Tomes have to do valueOf instead of ===
+	var searchElement = s ? s.valueOf() : s;
+
 	var t = this._arr;
 	var len = t.length;
 

package.json

@@ -1,7 +1,7 @@
 {
     "name": "tomes",
     "description": "Evented Storage Agnostic Data API",
-    "version": "0.0.17",
+    "version": "0.0.18",
     "author": "Wizcorp, Inc. <info@wizcorp.jp>",
     "maintainers": [
         { "name": "Bjorn Stromberg", "email": "bstromberg@wizcorp.jp" }

scripts/lint-all.sh

@@ -17,7 +17,7 @@ then
 	exit 0
 fi
 
-jshint $files --config ./scripts/jshint.cfg
+node_modules/.bin/jshint $files --config ./scripts/jshint.cfg
 exitCode=$?
 
 cd "$path"

scripts/lint-staged.sh

@@ -25,7 +25,7 @@ do
 	# ignore non-existing files (caused by renames or removal)
 	if [[ -e $file ]]
 	then
-		jshint $file --config ./scripts/jshint.cfg
+		node_modules/.bin/jshint $file --config ./scripts/jshint.cfg
 		if [[ $? != 0 ]]
 		then
 			exitCode=1

test/modules/array.js

@@ -560,6 +560,44 @@ exports.testArrayIndexOf = function (test) {
 	test.done();
 };
 
+exports.testArrayIndexOfTome = function (test) {
+	var a = [ '24df74c9-dc07-431e-8eec-cb575bf062e6', 'asdf', null, undefined, 17, '*', false, 0, true, 9, true, false, null, undefined, 'asdf', 17 ];
+	var b = Tome.conjure(a);
+
+	test.expect(a.length * 2);
+
+	var i;
+
+	for (i = 0; i < a.length; i += 1) {
+		test.strictEqual(a.indexOf(b[i].valueOf()), b.indexOf(b[i]));
+	}
+
+	for (i = 0; i < a.length; i += 1) {
+		test.strictEqual(a.indexOf(a[i]), b.indexOf(a[i]));
+	}
+
+	test.done();
+};
+
+exports.testArrayIndexOfTome = function (test) {
+	var a = [ '24df74c9-dc07-431e-8eec-cb575bf062e6', 'asdf', null, undefined, 17, '*', false, 0, true, 9, true, false, null, undefined, 'asdf', 17 ];
+	var b = Tome.conjure(a);
+
+	test.expect(a.length * 2);
+
+	var i;
+
+	for (i = 0; i < a.length; i += 1) {
+		test.strictEqual(a.lastIndexOf(b[i].valueOf()), b.lastIndexOf(b[i]));
+	}
+
+	for (i = 0; i < a.length; i += 1) {
+		test.strictEqual(a.lastIndexOf(a[i]), b.lastIndexOf(a[i]));
+	}
+
+	test.done();
+};
+
 exports.testArrayLastIndexOf = function (test) {
 	test.expect(7);