refactoring: moved upload logic out of the attachment icon

Issue #548

1 files changed, 5 insertions, 5 deletions

diff --git a/web-ui/test/spec/mail_view/ui/attachment.spec.js b/web-ui/test/spec/mail_view/ui/attachment.spec.js
index bbea2f55..d29660b9 100644
--- a/web-ui/test/spec/mail_view/ui/attachment.spec.js
+++ b/web-ui/test/spec/mail_view/ui/attachment.spec.js
@@ -1,4 +1,4 @@
-describeComponent('mail_view/ui/attachment', function () {
+describeComponent('mail_view/ui/attachment_icon', function () {
   'use strict';
 
   describe('attachment', function () {
@@ -7,14 +7,14 @@ describeComponent('mail_view/ui/attachment', function () {
         this.setupComponent();
     });
 
-    it('render attachment button if feature enabled', function () {
+    it('should render attachment button if feature enabled', function () {
         expect(this.$node.html()).toMatch('<i class="fa fa-paperclip fa-2x"></i>');
     });
 
-    xit('uploads attachment on click', function () {
-        var fileUploads = spyOn($, 'fileupload');
+    it('should trigger starts of attachment upload process', function () {
+        var triggerUploadAttachment = spyOnEvent(document, Pixelated.events.mail.startUploadAttachment);
         this.$node.click();
-        expect(fileUploads).toHaveBeenCalled();
+        expect(triggerUploadAttachment).toHaveBeenTriggeredOn(document);
     });
 
   });