Updates hooks test to check issue dresende#223 doesn't happen again

dresende · dresende · commit eca145c1c187 · 2013-07-11T22:11:39.000+01:00
diff --git a/test/integration/hook.js b/test/integration/hook.js
@@ -134,7 +134,13 @@ describe("Hook", function() {
 					items.should.have.property("length", 1);
 					items[0].name.should.equal("Jane Doe");
 
-					return done();
+					// ensure it was really saved
+					Person.get(items[0].id, function (err, Item) {
+						should.equal(err, null);
+						Item.name.should.equal("Jane Doe");
+
+						return done();
+					});
 				});
 			});
 		});
@@ -212,6 +218,32 @@ describe("Hook", function() {
 			});
 		});
 
+		describe("when setting properties", function () {
+			before(setup({
+				beforeSave : function () {
+					this.name = "Jane Doe";
+				}
+			}));
+
+			it("should not be discarded", function (done) {
+				Person.create([{ }], function (err, items) {
+					should.equal(err, null);
+
+					items.should.be.a("object");
+					items.should.have.property("length", 1);
+					items[0].name.should.equal("Jane Doe");
+
+					// ensure it was really saved
+					Person.get(items[0].id, function (err, Item) {
+						should.equal(err, null);
+						Item.name.should.equal("Jane Doe");
+
+						return done();
+					});
+				});
+			});
+		});
+
 		describe("if hook method has 1 argument", function () {
 			var beforeSave = false;
 
