From 940df3e2e12ff862c9e190fdfcd9a0278164a318 Mon Sep 17 00:00:00 2001 From: Gislene Pereira Date: Fri, 19 Feb 2016 14:56:09 -0300 Subject: Close the compose/inline box right after user clicks Send button and show message later when the email is sent or not. The Growl will be yellow if success and red if the email failed to be sent. Issue #609 --- .../spec/mail_list_actions/ui/compose_trigger.spec.js | 15 +++++++++++++++ web-ui/test/spec/user_alerts/ui/user_alerts.spec.js | 8 +++++++- 2 files changed, 22 insertions(+), 1 deletion(-) (limited to 'web-ui/test/spec') diff --git a/web-ui/test/spec/mail_list_actions/ui/compose_trigger.spec.js b/web-ui/test/spec/mail_list_actions/ui/compose_trigger.spec.js index 5e65ddbb..20deeae3 100644 --- a/web-ui/test/spec/mail_list_actions/ui/compose_trigger.spec.js +++ b/web-ui/test/spec/mail_list_actions/ui/compose_trigger.spec.js @@ -13,4 +13,19 @@ describeComponent('mail_list_actions/ui/compose_trigger', function () { expect(spyEvent).toHaveBeenTriggeredOn(document); }); + it('trigger showEmailSuccess message when message is sent', function () { + var spyEvent = spyOnEvent(document, Pixelated.events.ui.userAlerts.displayMessage); + + this.component.trigger(document, Pixelated.events.mail.sent); + + expect(spyEvent).toHaveBeenTriggeredOnAndWith(document, {message: 'Your message was sent!', class: 'success'}); + }); + + it('trigger showEmailError message when message is not sent', function () { + var spyEvent = spyOnEvent(document, Pixelated.events.ui.userAlerts.displayMessage); + + this.component.trigger(document, Pixelated.events.mail.send_failed, {responseJSON: {message: 'failure'}}); + + expect(spyEvent).toHaveBeenTriggeredOnAndWith(document, {message: 'Error, message not sent: failure', class: 'error'}); + }); }); diff --git a/web-ui/test/spec/user_alerts/ui/user_alerts.spec.js b/web-ui/test/spec/user_alerts/ui/user_alerts.spec.js index 5b25b5fb..5d87795a 100644 --- a/web-ui/test/spec/user_alerts/ui/user_alerts.spec.js +++ b/web-ui/test/spec/user_alerts/ui/user_alerts.spec.js @@ -10,7 +10,13 @@ describeComponent('user_alerts/ui/user_alerts', function () { it('should render message when ui:user_alerts:displayMessage is triggered', function () { this.component.trigger(Pixelated.events.ui.userAlerts.displayMessage, { message: 'a message' }); - expect(this.component.$node.html()).toMatch('a message'); + expect(this.component.$node.html()).toEqual('a message\n'); + }); + + it('should render error message', function () { + this.component.trigger(Pixelated.events.ui.userAlerts.displayMessage, { message: 'send failed', class: 'error' }); + + expect(this.component.$node.html()).toEqual('send failed\n'); }); it('should be emptied and hidden when hide is called', function() { -- cgit v1.2.3