From 0e89a9c837db402fd05a5294cd9fc3753e3ab632 Mon Sep 17 00:00:00 2001 From: Alex Pitigoi Date: Thu, 19 May 2016 15:47:31 -0400 Subject: [PATCH] fix avoid duplicate record on scope with promise --- test/relations.test.js | 25 +++++++++++-------------- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/test/relations.test.js b/test/relations.test.js index 779f4a43..61694f5b 100644 --- a/test/relations.test.js +++ b/test/relations.test.js @@ -5107,20 +5107,17 @@ describe('relations', function() { it('should not allow duplicate record on scope with promises', function(done) { Category.findOne() - .then(function(cat) { - cat.jobIds = [job2.id, job2.id]; - cat.save(); - }) - .then(function(p) { - should.not.exist(p); - done(); - }) - .catch(function(err) { - should.exist(err); - err.name.should.equal('ValidationError'); - err.details.codes.jobs.should.eql(['uniqueness']); - done(); - }); + .then(function(cat) { + cat.jobIds = [job2.id, job2.id]; + return cat.save(); + }) + .then( + function(p) { done(new Error('save() should have failed')); }, + function(err) { + err.name.should.equal('ValidationError'); + err.details.codes.jobs.should.eql(['uniqueness']); + done(); + }); }); it('should find items on scope with promises', function(done) {