From f75bbb6e5f59da87fef30433a57db9b9b0d1582a Mon Sep 17 00:00:00 2001 From: hairlessbear Date: Mon, 22 Jun 2015 20:55:50 -0400 Subject: [PATCH] Modified 'RSVPNeeded is not true' test to not set user.party.quest.completed because it is not needed for the test --- test/spec/controllers/notificationCtrlSpec.js | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/test/spec/controllers/notificationCtrlSpec.js b/test/spec/controllers/notificationCtrlSpec.js index e67c2f5b61..79ce0f7109 100644 --- a/test/spec/controllers/notificationCtrlSpec.js +++ b/test/spec/controllers/notificationCtrlSpec.js @@ -39,12 +39,10 @@ describe('Notification Controller', function() { it('does not open quest invitation modal if RSVPNeeded is not true', function() { user.party.quest.RSVPNeeded = false; - user.party.quest.completed = "hedgebeast"; + delete user.party.quest.completed; scope.$digest(); - // Completion modal should open, but should be the only modal that opens - expect(rootScope.openModal).to.be.calledOnce; - expect(rootScope.openModal).to.be.calledWith('questCompleted', {controller:'InventoryCtrl'}); + expect(rootScope.openModal).to.not.be.called; }); it('does not open quest invitation modal if quest.completed contains a quest key', function() {