mongojs-hooks - npm Package Compare versions

Comparing version 0.1.5 to 0.1.6

mongojs-hooks.js

@@ -68,3 +68,3 @@ var mongojs = require("mongojs");
 	sanitise: function (object) {
-		if (!object)
+		if (!object || object._bsontype)
 			return object;
@@ -75,3 +75,3 @@
 
-		var newObject = { };
+		var newObject = _.isArray(object) ? [ ] : { };
 
@@ -98,3 +98,3 @@ Object.keys(object).forEach(function (key) {
 	unsanitise: function (object) {
-		if (!object)
+		if (!object || object._bsontype)
 			return object;
@@ -105,3 +105,3 @@
 
-		var newObject = { };
+		var newObject = _.isArray(object) ? [ ] : { };
 
@@ -108,0 +108,0 @@ Object.keys(object).forEach(function (sKey) {

package.json

 {
 	"name": "mongojs-hooks",
 	"description": "Wrap monogjs in pre and post hooks",
-	"version": "0.1.5",
+	"version": "0.1.6",
 	"main": "./mongojs-hooks",
@@ -6,0 +6,0 @@ "author": {

test/util.js

@@ -87,3 +87,40 @@ var mongo = require("../mongojs-hooks");
 			});
+
+			it("should skip object if it contains a _bsontype key", function () {
+				// These are special Mongo objects so don't malform them
+				var object = {
+					_bsontype: "ObjectID",
+					"test.this": "ignore"
+				};
+
+				var sanitised = mongo.util.sanitise(object);
+				sanitised.should.not.have.property("testU+FF0Ethis");
+				sanitised.should.have.property("test.this");
+			});
 		});
+
+		describe("when an Array", function () {
+			it("should maintain all elements in the array", function () {
+				var array = [
+					{ key: "value" },
+					{ second: "object" }
+				];
+
+				var sanitised = mongo.util.sanitise(array);
+				(sanitised instanceof Array).should.be.true;
+				sanitised.length.should.equal(2);
+				sanitised.should.deep.equal(array);
+			});
+
+			it("should sanitise keys in any elements", function () {
+				var array = [
+					{ "test.one": "value" },
+					{ "test.two": "object" }
+				];
+
+				var sanitised = mongo.util.sanitise(array);
+				sanitised[0].should.have.property("testU+FF0Eone");
+				sanitised[1].should.have.property("testU+FF0Etwo");
+			});
+		});
 	});
@@ -174,4 +211,41 @@
 			});
+
+			it("should skip object if it contains a _bsontype key", function () {
+				// These are special Mongo objects so don't malform them
+				var object = {
+					_bsontype: "ObjectID",
+					"testU+FF0Ethis": "ignore"
+				};
+
+				var unsanitised = mongo.util.unsanitise(object);
+				unsanitised.should.not.have.property("test.this");
+				unsanitised.should.have.property("testU+FF0Ethis");
+			});
 		});
+
+		describe("when an Array", function () {
+			it("should maintain all elements in the array", function () {
+				var array = [
+					{ key: "value" },
+					{ second: "object" }
+				];
+
+				var unsanitised = mongo.util.unsanitise(array);
+				(unsanitised instanceof Array).should.be.true;
+				unsanitised.length.should.equal(2);
+				unsanitised.should.deep.equal(array);
+			});
+
+			it("should unsanitise keys in any elements", function () {
+				var array = [
+					{ "testU+FF0Eone": "value" },
+					{ "testU+FF0Etwo": "object" }
+				];
+
+				var unsanitised = mongo.util.unsanitise(array);
+				unsanitised[0].should.have.property("test.one");
+				unsanitised[1].should.have.property("test.two");
+			});
+		});
 	});
 });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

26426

increased by9.33%

Lines of code

522

increased by13.48%

No dependency changes