#153 fix mail checking and unchecking on mail list

3 files changed, 18 insertions, 21 deletions

diff --git a/web-ui/test/spec/mail_list/ui/mail_list.spec.js b/web-ui/test/spec/mail_list/ui/mail_list.spec.js
index 72f61a39..22a10a31 100644
--- a/web-ui/test/spec/mail_list/ui/mail_list.spec.js
+++ b/web-ui/test/spec/mail_list/ui/mail_list.spec.js
@@ -108,7 +108,7 @@ describeComponent('mail_list/ui/mail_list', function () {
 
       $(document).trigger(Pixelated.events.ui.mail.checked, {mail: mailList[0]});
 
-      expect(setCheckAllCheckboxEvent).toHaveBeenTriggeredOnAndWith(document, {hasMailsChecked: true});
+      expect(setCheckAllCheckboxEvent).toHaveBeenTriggeredOnAndWith(document, true);
     });
 
     it('unchecks the check all checkbox if no mail is left checked', function () {
@@ -118,7 +118,7 @@ describeComponent('mail_list/ui/mail_list', function () {
 
       $(document).trigger(Pixelated.events.ui.mail.unchecked, {mail: {ident: '1'}});
 
-      expect(setCheckAllCheckboxEvent).toHaveBeenTriggeredOnAndWith(document, {hasMailsChecked: false});
+      expect(setCheckAllCheckboxEvent).toHaveBeenTriggeredOnAndWith(document, false);
     });
   });
 
diff --git a/web-ui/test/spec/services/mail_service.spec.js b/web-ui/test/spec/services/mail_service.spec.js
index 778d79da..685ad5bc 100644
--- a/web-ui/test/spec/services/mail_service.spec.js
+++ b/web-ui/test/spec/services/mail_service.spec.js
@@ -3,9 +3,13 @@
 describeComponent('services/mail_service', function () {
 
   var email1, i18n;
+  var features;
 
   beforeEach( function () {
+    features = require('features');
+    spyOn(features, 'isAutoRefreshEnabled').and.returnValue(false);
     this.setupComponent();
+
     email1 = Pixelated.testData().parsedMail.simpleTextPlain;
     i18n = require('views/i18n');
   } );
@@ -21,7 +25,7 @@ describeComponent('services/mail_service', function () {
   });
 
   describe('when marks many emails as read', function () {
-    var readRequest, checkedMails, uncheckedEmailsEvent, setCheckAllEvent, deferred;
+    var readRequest, checkedMails, uncheckAllEvent, deferred;
 
     beforeEach(function () {
       checkedMails = {
@@ -32,8 +36,7 @@ describeComponent('services/mail_service', function () {
       deferred = $.Deferred();
       readRequest = spyOn($, 'ajax').and.returnValue(deferred);
 
-      uncheckedEmailsEvent = spyOnEvent(document, Pixelated.events.ui.mail.unchecked);
-      setCheckAllEvent = spyOnEvent(document, Pixelated.events.ui.mails.hasMailsChecked);
+      uncheckAllEvent = spyOnEvent(document, Pixelated.events.ui.mails.uncheckAll);
       spyOn(this.component, 'refreshMails');
 
       this.component.trigger(Pixelated.events.mail.read, {checkedMails: checkedMails});
@@ -49,14 +52,9 @@ describeComponent('services/mail_service', function () {
       expect(this.component.refreshMails).toHaveBeenCalled();
     });
 
-    it('unchecks read emails', function () {
-      deferred.resolve(checkedMails);
-      expect(uncheckedEmailsEvent).toHaveBeenTriggeredOnAndWith(document, {mails: checkedMails});
-    });
-
     it('clears the check all checkbox', function () {
       deferred.resolve(checkedMails);
-      expect(setCheckAllEvent).toHaveBeenTriggeredOnAndWith(document, false);
+      expect(uncheckAllEvent).toHaveBeenTriggeredOn(document);
     });
   });
 
@@ -133,12 +131,11 @@ describeComponent('services/mail_service', function () {
   });
 
   describe('when successfuly deletes an email', function () {
-    var displayMessageEvent, uncheckedEmailsEvent, setCheckAllEvent, mailsDeletedEvent;
+    var displayMessageEvent, uncheckAllEvent, mailsDeletedEvent;
 
     beforeEach(function () {
       displayMessageEvent = spyOnEvent(document, Pixelated.events.ui.userAlerts.displayMessage);
-      uncheckedEmailsEvent = spyOnEvent(document, Pixelated.events.ui.mail.unchecked);
-      setCheckAllEvent = spyOnEvent(document, Pixelated.events.ui.mails.hasMailsChecked);
+      uncheckAllEvent = spyOnEvent(document, Pixelated.events.ui.mails.uncheckAll);
       mailsDeletedEvent = spyOnEvent(document, Pixelated.events.mail.deleted);
       spyOn(this.component, 'refreshMails');
 
@@ -156,16 +153,12 @@ describeComponent('services/mail_service', function () {
       expect(displayMessageEvent).toHaveBeenTriggeredOnAndWith(document, {message: 'A success message'});
     });
 
-    it('unchecks deleted emails', function () {
-      expect(uncheckedEmailsEvent).toHaveBeenTriggeredOnAndWith(document, { mails: {1: 'email 1', 2: 'email 2'} });
-    });
-
     it('tells about deleted emails', function () {
       expect(mailsDeletedEvent).toHaveBeenTriggeredOnAndWith(document, { mails: {1: 'email 1', 2: 'email 2'} });
     });
 
-    it('clears the check all checkbox', function () {
-      expect(setCheckAllEvent).toHaveBeenTriggeredOnAndWith(document, false);
+    it('unchecks all checked mails', function () {
+      expect(uncheckAllEvent).toHaveBeenTriggeredOn(document);
     });
   });
 
diff --git a/web-ui/test/spec/tags/data/tags.spec.js b/web-ui/test/spec/tags/data/tags.spec.js
index adb279ac..469ab0ce 100644
--- a/web-ui/test/spec/tags/data/tags.spec.js
+++ b/web-ui/test/spec/tags/data/tags.spec.js
@@ -3,7 +3,11 @@
 describeComponent('tags/data/tags', function () {
   'use strict';
 
-  beforeEach(function () {
+  var features;
+
+  beforeEach( function () {
+    features = require('features');
+    spyOn(features, 'isAutoRefreshEnabled').and.returnValue(false);
     this.setupComponent();
   });