fixup! fixup! fixup! fixup! Strict mode now always return validationE…

…rror
loopbackio · Sep 7, 2016 · dd9dee6 · dd9dee6
1 parent 8f2b55d
commit dd9dee6
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/test/manipulation.test.js b/test/manipulation.test.js
@@ -622,13 +622,13 @@ describe('manipulation', function() {
       Person.definition.settings.strict = 'validate';
       Person.findById(person.id, function(err, p) {
         p.updateAttributes({foo: 'bar'},
-          function(err1, p1) {
-            should.exist(err1);
-            err1.name.should.equal('ValidationError');
-            err1.message.should.containEql('`foo` is not defined in the model');
-            Person.findById(person.id, function(e2, p2) {
-              if (e2) return done(e2);
-              should.not.exist(p2.foo);
+          function(err, p) {
+            should.exist(err);
+            err.name.should.equal('ValidationError');
+            err.message.should.containEql('`foo` is not defined in the model');
+            Person.findById(person.id, function(e, p) {
+              if (e) return done(e);
+              should.not.exist(p.foo);
               done();
             });
           });