Undoing a recent change to set(), creating a test for an existing bug

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "haro",
-  "version": "1.7.5",
+  "version": "1.7.8",
   "homepage": "http://haro.rocks",
   "authors": [
     "Jason Mulligan <jason.mulligan@avoidwork.com>"

lib/haro.es6.js

@@ -5,7 +5,7 @@
  * @copyright 2015
  * @license BSD-3-Clause
  * @link http://haro.rocks
- * @version 1.7.7
+ * @version 1.7.8
  */
 "use strict";
 
@@ -770,10 +770,8 @@ class Haro {
 			}
 
 			if (method === "post") {
-				if (!this.data.has(lkey)) {
-					this.registry[this.total] = lkey;
-					++this.total;
-				}
+				this.registry[this.total] = lkey;
+				++this.total;
 
 				if (this.versioning) {
 					this.versions.set(lkey, new Set());
@@ -1109,7 +1107,7 @@ function factory (data = null, config = {}, indexes = []) {
 }
 
 factory.transform = cast;
-factory.version = "1.7.7";
+factory.version = "1.7.8";
 
 // Node, AMD & window supported
 if (typeof exports !== "undefined") {

lib/haro.js

@@ -5,7 +5,7 @@
  * @copyright 2015
  * @license BSD-3-Clause
  * @link http://haro.rocks
- * @version 1.7.7
+ * @version 1.7.8
  */
 "use strict";
 
@@ -864,10 +864,8 @@ function _classCallCheck(instance, Constructor) { if (!(instance instanceof Cons
 					}
 
 					if (method === "post") {
-						if (!_this10.data.has(lkey)) {
-							_this10.registry[_this10.total] = lkey;
-							++_this10.total;
-						}
+						_this10.registry[_this10.total] = lkey;
+						++_this10.total;
 
 						if (_this10.versioning) {
 							_this10.versions.set(lkey, new Set());
@@ -1241,7 +1239,7 @@ function _classCallCheck(instance, Constructor) { if (!(instance instanceof Cons
 	}
 
 	factory.transform = cast;
-	factory.version = "1.7.7";
+	factory.version = "1.7.8";
 
 	// Node, AMD & window supported
 	if (typeof exports !== "undefined") {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "haro",
-  "version": "1.7.7",
+  "version": "1.7.8",
   "description": "Harō is a modern immutable DataStore using Maps, Sets, Promises, & Tuples",
   "main": "lib/haro.js",
   "scripts": {

src/haro.js

@@ -505,10 +505,8 @@ class Haro {
 			}
 
 			if (method === "post") {
-				if (!this.data.has(lkey)) {
-					this.registry[this.total] = lkey;
-					++this.total;
-				}
+				this.registry[this.total] = lkey;
+				++this.total;
 
 				if (this.versioning) {
 					this.versions.set(lkey, new Set());

test/haro_test.js

@@ -104,6 +104,50 @@ exports["create (batch)"] = {
 	}
 };
 
+exports["update (batch)"] = {
+	setUp: function (done) {
+		this.store = haro();
+		done();
+	},
+	test: function (test) {
+		var self = this;
+
+		test.expect(14);
+		test.equal(this.store.total, 0, "Should be '0'");
+		test.equal(this.store.data.size, 0, "Should be '0'");
+		this.store.batch(data, "set").then(function () {
+			test.equal(self.store.total, 6, "Should be '6'");
+			test.equal(self.store.data.size, 6, "Should be '6'");
+			test.equal(self.store.registry.length, 6, "Should be '6'");
+			return self.store.batch(data, "set");
+		}, function (e) {
+			throw e;
+		}).then(function () {
+			test.equal(self.store.total, 6, "Should be '6'");
+			test.equal(self.store.data.size, 6, "Should be '6'");
+			test.equal(self.store.registry.length, 6, "Should be '6'");
+			test.equal(self.store.limit(2)[1][0], self.store.get(self.store.registry[1])[0], "Should be a match");
+			test.equal(self.store.limit(2, 2)[1][0], self.store.get(self.store.registry[3])[0], "Should be a match");
+			test.equal(self.store.limit(10, 5).length, 1, "Should be '1'");
+			test.equal(self.store.filter(function (i) {
+				return /decker/i.test(i.name);
+			}).length, 1, "Should be '1'");
+			test.equal(self.store.map(function (i) {
+				i.name = 'John Doe';
+				return i;
+			}).length, 6, "Should be '6'");
+			test.equal(self.store.map(function (i) {
+				i.name = 'John Doe';
+				return i;
+			})[0].name, 'John Doe', "Should be a match");
+			test.done();
+		}, function (e) {
+			console.log(e.stack);
+			test.done();
+		});
+	}
+};
+
 exports["read (valid)"] = {
 	setUp: function (done) {
 		this.store = haro();