separated attachment list UI as a component and the corresponding attachment list payload as a mixin

Issue #548

8 files changed, 373 insertions, 308 deletions

diff --git a/web-ui/app/js/mail_view/data/attachment_list.js b/web-ui/app/js/mail_view/data/attachment_list.js
index 7ea2f9d9..9193e37a 100644
--- a/web-ui/app/js/mail_view/data/attachment_list.js
+++ b/web-ui/app/js/mail_view/data/attachment_list.js
@@ -17,72 +17,25 @@
 
 define(
     [
-        'flight/lib/component',
         'page/events'
     ],
 
-    function (defineComponent, events) {
+    function (events) {
         'use strict';
 
         function attachmentList() {
             this.defaultAttrs({
-                inputFileUpload: '#fileupload',
-                attachmentListItem: '#attachment-list-item',
-                progressBar: '#progress .progress-bar',
-                attachmentBaseUrl: '/attachment',
                 attachments: []
             });
 
             this.addAttachment = function (event, data) {
                 this.attr.attachments.push(data);
-                this.renderAttachmentListView(data);
-            };
-
-            this.renderAttachmentListView = function (data) {
-                var item = this.buildAttachmentListItem(data);
-                this.select('attachmentListItem').html(item);
-            };
-
-            function humanReadable(bytes) {
-                var e = Math.floor(Math.log(bytes) / Math.log(1024));
-                return (bytes / Math.pow(1024, e)).toFixed(2) + ' ' + ' KMGTP'.charAt(e) + 'b';
-            }
-
-            this.buildAttachmentListItem = function (data) {
-                return '<a href="' + this.attr.attachmentBaseUrl + '/' + data.attachment_id + '?filename=' +
-                    data.filename + '&encoding=base64">' + data.filename + ' (' + humanReadable(data.filesize) + ')' +
-                    '</a>';
-            };
-
-            function addJqueryFileUploadConfig(self) {
-                self.select('inputFileUpload').fileupload({
-                    url: self.attr.attachmentBaseUrl,
-                    dataType: 'json',
-                    done: function (e, response) {
-                        var data = response.result;
-                        self.trigger(document, events.mail.uploadedAttachment, data);
-                    },
-                    progressall: function (e, data) {
-                        var progressRate = parseInt(data.loaded / data.total * 100, 10);
-                        self.select('progressBar').css('width', progressRate + '%');
-                    }
-                }).bind('fileuploadstart', function (e) {
-                    self.trigger(document, events.mail.uploadingAttachment);
-                }).bind('fileuploadadd', function (e) {
-                    $('.attachmentsAreaWrap').show();
-                });
-            }
-
-            this.startUpload = function () {
-                addJqueryFileUploadConfig(this);
-                this.select('inputFileUpload').click();
             };
 
             this.after('initialize', function () {
-                this.on(document, events.mail.uploadedAttachment, this.addAttachment);
-                this.on(document, events.mail.startUploadAttachment, this.startUpload);
+                this.on(document, events.mail.appendAttachment, this.addAttachment);
             });
         }
 
-        return defineComponent(attachmentList);
+        return attachmentList;
     });
\ No newline at end of file
diff --git a/web-ui/app/js/mail_view/ui/attachment_list.js b/web-ui/app/js/mail_view/ui/attachment_list.js
new file mode 100644
index 00000000..36899056
--- /dev/null
+++ b/web-ui/app/js/mail_view/ui/attachment_list.js
@@ -0,0 +1,87 @@
+/*
+ * Copyright (c) 2015 ThoughtWorks, Inc.
+ *
+ * Pixelated is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Pixelated is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with Pixelated. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+define(
+    [
+        'flight/lib/component',
+        'page/events'
+    ],
+
+    function (defineComponent, events) {
+        'use strict';
+
+        function attachmentList() {
+            this.defaultAttrs({
+                inputFileUpload: '#fileupload',
+                attachmentListItem: '#attachment-list-item',
+                progressBar: '#progress .progress-bar',
+                attachmentBaseUrl: '/attachment'
+            });
+
+            this.addAttachment = function (event, data) {
+                this.trigger(document, events.mail.appendAttachment, data);
+                this.renderAttachmentListView(data);
+            };
+
+            this.renderAttachmentListView = function (data) {
+                var item = this.buildAttachmentListItem(data);
+                this.select('attachmentListItem').html(item);
+            };
+
+            function humanReadable(bytes) {
+                var e = Math.floor(Math.log(bytes) / Math.log(1024));
+                return (bytes / Math.pow(1024, e)).toFixed(2) + ' ' + ' KMGTP'.charAt(e) + 'b';
+            }
+
+            this.buildAttachmentListItem = function (data) {
+                return '<a href="' + this.attr.attachmentBaseUrl + '/' + data.attachment_id + '?filename=' +
+                    data.filename + '&encoding=base64">' + data.filename + ' (' + humanReadable(data.filesize) + ')' +
+                    '</a>';
+            };
+
+            function addJqueryFileUploadConfig(self) {
+                self.select('inputFileUpload').fileupload({
+                    url: self.attr.attachmentBaseUrl,
+                    dataType: 'json',
+                    done: function (e, response) {
+                        var data = response.result;
+                        self.trigger(document, events.mail.uploadedAttachment, data);
+                    },
+                    progressall: function (e, data) {
+                        var progressRate = parseInt(data.loaded / data.total * 100, 10);
+                        self.select('progressBar').css('width', progressRate + '%');
+                    }
+                }).bind('fileuploadstart', function (e) {
+                    self.trigger(document, events.mail.uploadingAttachment);
+                }).bind('fileuploadadd', function (e) {
+                    $('.attachmentsAreaWrap').show();
+                });
+            }
+
+            this.startUpload = function () {
+                addJqueryFileUploadConfig(this);
+                this.select('inputFileUpload').click();
+            };
+
+            this.after('initialize', function () {
+                this.on(document, events.mail.uploadedAttachment, this.addAttachment);
+                this.on(document, events.mail.startUploadAttachment, this.startUpload);
+            });
+        }
+
+        return defineComponent(attachmentList);
+    });
\ No newline at end of file
diff --git a/web-ui/app/js/mail_view/ui/compose_box.js b/web-ui/app/js/mail_view/ui/compose_box.js
index 8e03c081..993cf880 100644
--- a/web-ui/app/js/mail_view/ui/compose_box.js
+++ b/web-ui/app/js/mail_view/ui/compose_box.js
@@ -20,13 +20,14 @@ define(
     'views/templates',
     'mixins/with_mail_edit_base',
     'page/events',
-    'mail_view/data/mail_builder'
+    'mail_view/data/mail_builder',
+    'mail_view/data/attachment_list'
   ],
 
-  function (defineComponent, templates, withMailEditBase, events, mailBuilder) {
+  function (defineComponent, templates, withMailEditBase, events, mailBuilder, attachmentList) {
     'use strict';
 
-    return defineComponent(composeBox, withMailEditBase);
+    return defineComponent(composeBox, withMailEditBase, attachmentList);
 
     function composeBox() {
 
diff --git a/web-ui/app/js/mixins/with_mail_edit_base.js b/web-ui/app/js/mixins/with_mail_edit_base.js
index f00ad5a3..0aaa6628 100644
--- a/web-ui/app/js/mixins/with_mail_edit_base.js
+++ b/web-ui/app/js/mixins/with_mail_edit_base.js
@@ -16,230 +16,233 @@
  */
 
 define(
-  [
-    'helpers/view_helper',
-    'mail_view/ui/recipients/recipients',
-    'mail_view/ui/draft_save_status',
-    'page/events',
-    'views/i18n',
-    'mail_view/ui/send_button',
-    'mail_view/ui/attachment_icon',
-    'mail_view/data/attachment_list',
-    'flight/lib/utils'
-  ],
-  function(viewHelper, Recipients, DraftSaveStatus, events, i18n, SendButton, AttachmentIcon, AttachmentList, utils) {
-    'use strict';
-
-    function withMailEditBase() {
-
-      this.defaultAttrs({
-        bodyBox: '#text-box',
-        sendButton: '#send-button',
-        attachmentButton: '#attachment-button',
-        attachmentList: '#attachment-list',
-        cancelButton: '#cancel-button',
-        trashButton: '#trash-button',
-        toArea: '#recipients-to-area',
-        toBox: '#recipients-to-box',
-        ccArea: '#recipients-cc-area',
-        bccArea: '#recipients-bcc-area',
-        ccsTrigger: '#ccs-trigger',
-        bccsTrigger: '#bccs-trigger',
-        toTrigger: '#to-trigger',
-        subjectBox: '#subject',
-        tipMsg: '.tip-msg',
-        draftSaveStatus: '#draft-save-status',
-        recipientsFields: '#recipients-fields',
-        currentTag: '',
-        recipientValues: {to: [], cc: [], bcc: []},
-        saveDraftInterval: 3000
-      });
-
-      this.attachRecipients = function (context) {
-        Recipients.attachTo(this.select('toArea'), { name: 'to', addresses: context.recipients.to });
-        Recipients.attachTo(this.select('ccArea'), { name: 'cc', addresses: context.recipients.cc || []});
-        Recipients.attachTo(this.select('bccArea'), { name: 'bcc', addresses: context.recipients.bcc || []});
-      };
-
-      function thereAreRecipientsToDisplay() {
-
-        var allRecipients = _.chain(this.attr.recipientValues).
-          values().
-          flatten().
-          remove(undefined).
-          value();
-
-        return !_.isEmpty(allRecipients);
-      }
-
-      this.warnSendButtonOfRecipients = function () {
-        if (thereAreRecipientsToDisplay.call(this)) {
-          _.forOwn(this.attr.recipientValues, function (recipients, recipientsType) {
-            if (!_.isUndefined(recipients) && !_.isEmpty(recipients)) {
-              var recipientsUpdatedData = {
-                newRecipients: recipients,
-                recipientsName: recipientsType
-              };
-              this.trigger(document, events.ui.recipients.updated, recipientsUpdatedData);
+    [
+        'helpers/view_helper',
+        'mail_view/ui/recipients/recipients',
+        'mail_view/ui/draft_save_status',
+        'page/events',
+        'views/i18n',
+        'mail_view/ui/send_button',
+        'mail_view/ui/attachment_icon',
+        'mail_view/ui/attachment_list',
+        'flight/lib/utils'
+    ],
+    function (viewHelper, Recipients, DraftSaveStatus, events, i18n, SendButton, AttachmentIcon, AttachmentListUI, utils) {
+        'use strict';
+
+        function withMailEditBase() {
+
+            this.defaultAttrs({
+                bodyBox: '#text-box',
+                sendButton: '#send-button',
+                attachmentButton: '#attachment-button',
+                attachmentList: '#attachment-list',
+                cancelButton: '#cancel-button',
+                trashButton: '#trash-button',
+                toArea: '#recipients-to-area',
+                toBox: '#recipients-to-box',
+                ccArea: '#recipients-cc-area',
+                bccArea: '#recipients-bcc-area',
+                ccsTrigger: '#ccs-trigger',
+                bccsTrigger: '#bccs-trigger',
+                toTrigger: '#to-trigger',
+                subjectBox: '#subject',
+                tipMsg: '.tip-msg',
+                draftSaveStatus: '#draft-save-status',
+                recipientsFields: '#recipients-fields',
+                currentTag: '',
+                recipientValues: {to: [], cc: [], bcc: []},
+                saveDraftInterval: 3000
+            });
+
+            this.attachRecipients = function (context) {
+                Recipients.attachTo(this.select('toArea'), {name: 'to', addresses: context.recipients.to});
+                Recipients.attachTo(this.select('ccArea'), {name: 'cc', addresses: context.recipients.cc || []});
+                Recipients.attachTo(this.select('bccArea'), {name: 'bcc', addresses: context.recipients.bcc || []});
+            };
+
+            function thereAreRecipientsToDisplay() {
+
+                var allRecipients = _.chain(this.attr.recipientValues).
+                values().
+                flatten().
+                remove(undefined).
+                value();
+
+                return !_.isEmpty(allRecipients);
             }
-          }.bind(this));
-        }
-      };
 
-      this.render = function(template, context) {
-        this.$node.html(template(context));
+            this.warnSendButtonOfRecipients = function () {
+                if (thereAreRecipientsToDisplay.call(this)) {
+                    _.forOwn(this.attr.recipientValues, function (recipients, recipientsType) {
+                        if (!_.isUndefined(recipients) && !_.isEmpty(recipients)) {
+                            var recipientsUpdatedData = {
+                                newRecipients: recipients,
+                                recipientsName: recipientsType
+                            };
+                            this.trigger(document, events.ui.recipients.updated, recipientsUpdatedData);
+                        }
+                    }.bind(this));
+                }
+            };
+
+            this.render = function (template, context) {
+                this.$node.html(template(context));
+
+                if (!context || _.isEmpty(context)) {
+                    context.recipients = {to: [], cc: [], bcc: []};
+                }
+                this.attr.recipientValues = context.recipients;
+                this.attachRecipients(context);
+
+                this.on(this.select('trashButton'), 'click', this.discardMail);
+                SendButton.attachTo(this.select('sendButton'));
+                AttachmentIcon.attachTo(this.select('attachmentButton'));
+                AttachmentListUI.attachTo(this.select('attachmentList'));
+
+                this.warnSendButtonOfRecipients();
+            };
+
+            this.enableAutoSave = function () {
+                this.select('bodyBox').on('input', this.monitorInput.bind(this));
+                this.select('subjectBox').on('input', this.monitorInput.bind(this));
+                DraftSaveStatus.attachTo(this.select('draftSaveStatus'));
+            };
+
+            this.monitorInput = function () {
+                this.trigger(events.ui.mail.changedSinceLastSave);
+                this.cancelPostponedSaveDraft();
+                var mail = this.buildMail();
+                this.postponeSaveDraft(mail);
+            };
+
+            this.discardMail = function () {
+                this.cancelPostponedSaveDraft();
+                if (this.attr.ident) {
+                    var mail = this.buildMail();
+                    this.trigger(document, events.ui.mail.delete, {mail: mail});
+                } else {
+                    this.trigger(document, events.ui.mail.discard);
+                }
+            };
+
+            this.trim_recipient = function (recipients) {
+                return recipients.map(function (recipient) {
+                    return recipient.trim();
+                });
+            };
+
+            this.sendMail = function () {
+                this.cancelPostponedSaveDraft();
+                var mail = this.buildMail('sent');
+
+                if (allRecipientsAreEmails(mail)) {
+                    mail.header.to = this.trim_recipient(mail.header.to);
+                    mail.header.cc = this.trim_recipient(mail.header.cc);
+                    mail.header.bcc = this.trim_recipient(mail.header.bcc);
+                    this.trigger(events.mail.send, mail);
+                } else {
+                    this.trigger(
+                        events.ui.userAlerts.displayMessage,
+                        {message: i18n('One or more of the recipients are not valid emails')}
+                    );
+                    this.trigger(events.mail.send_failed);
+                }
+            };
+
+            this.buildAndSaveDraft = function () {
+                var mail = this.buildMail();
+                this.saveDraft(mail);
+            };
+
+            this.recipientsUpdated = function (ev, data) {
+                this.attr.recipientValues[data.recipientsName] = data.newRecipients;
+                this.trigger(document, events.ui.mail.recipientsUpdated);
+                if (data.skipSaveDraft) {
+                    return;
+                }
+
+                var mail = this.buildMail();
+                this.postponeSaveDraft(mail);
+            };
+
+            this.saveDraft = function (mail) {
+                this.cancelPostponedSaveDraft();
+                this.trigger(document, events.mail.saveDraft, mail);
+            };
+
+            this.cancelPostponedSaveDraft = function () {
+                clearTimeout(this.attr.timeout);
+            };
+
+            this.postponeSaveDraft = function (mail) {
+                this.cancelPostponedSaveDraft();
+
+                this.attr.timeout = window.setTimeout(_.bind(function () {
+                    this.saveDraft(mail);
+                }, this), this.attr.saveDraftInterval);
+            };
+
+            this.draftSaved = function (event, data) {
+                this.attr.ident = data.ident;
+            };
+
+            this.validateAnyRecipient = function () {
+                return !_.isEmpty(_.flatten(_.values(this.attr.recipientValues)));
+            };
+
+            function allRecipientsAreEmails(mail) {
+                var allRecipients = mail.header.to.concat(mail.header.cc).concat(mail.header.bcc);
+                return _.isEmpty(allRecipients) ? false : _.all(allRecipients, emailFormatChecker);
+            }
 
-        if(!context || _.isEmpty(context)){
-          context.recipients = {to: [], cc: [], bcc: []};
-        }
-        this.attr.recipientValues = context.recipients;
-        this.attachRecipients(context);
-
-        this.on(this.select('trashButton'), 'click', this.discardMail);
-        SendButton.attachTo(this.select('sendButton'));
-        AttachmentIcon.attachTo(this.select('attachmentButton'));
-        AttachmentList.attachTo(this.select('attachmentList'));
-
-        this.warnSendButtonOfRecipients();
-      };
-
-      this.enableAutoSave = function () {
-        this.select('bodyBox').on('input', this.monitorInput.bind(this));
-        this.select('subjectBox').on('input', this.monitorInput.bind(this));
-        DraftSaveStatus.attachTo(this.select('draftSaveStatus'));
-      };
-
-      this.monitorInput = function() {
-        this.trigger(events.ui.mail.changedSinceLastSave);
-        this.cancelPostponedSaveDraft();
-        var mail = this.buildMail();
-        this.postponeSaveDraft(mail);
-      };
-
-      this.discardMail = function() {
-        this.cancelPostponedSaveDraft();
-        if (this.attr.ident) {
-            var mail = this.buildMail();
-            this.trigger(document, events.ui.mail.delete, { mail: mail });
-        } else {
-            this.trigger(document, events.ui.mail.discard);
-        }
-      };
-
-      this.trim_recipient = function(recipients) {
-        return recipients.map(function(recipient) {
-          return recipient.trim();
-        });
-      };
-
-      this.sendMail = function () {
-        this.cancelPostponedSaveDraft();
-        var mail = this.buildMail('sent');
-
-        if (allRecipientsAreEmails(mail)) {
-          mail.header.to = this.trim_recipient(mail.header.to);
-          mail.header.cc = this.trim_recipient(mail.header.cc);
-          mail.header.bcc = this.trim_recipient(mail.header.bcc);
-          this.trigger(events.mail.send, mail);
-        } else {
-          this.trigger(
-            events.ui.userAlerts.displayMessage,
-            {message: i18n('One or more of the recipients are not valid emails')}
-          );
-          this.trigger(events.mail.send_failed);
+            function emailFormatChecker(email) {
+                var emailFormat = /[^\s@]+@[^\s@]+\.[^\s@]+$/;
+                return emailFormat.test(email);
+            }
+
+            this.saveTag = function (ev, data) {
+                this.attr.currentTag = data.tag;
+            };
+
+            this.mailSent = function () {
+                this.trigger(document, events.ui.userAlerts.displayMessage, {message: 'Your message was sent!'});
+            };
+
+            this.enableFloatlabel = function (element) {
+                var showClass = 'showfloatlabel';
+                $(element).bind('keyup', function () {
+                    var label = $(this).prev('label');
+                    if (this.value !== '') {
+                        label.addClass(showClass);
+                        $(this).addClass(showClass);
+                    } else {
+                        label.removeClass(showClass);
+                        $(this).removeClass(showClass);
+                    }
+                });
+            };
+
+            this.before('initialize', function () {
+                if (!this.discardDraft) {
+                    this.discardDraft = function () {
+                    };
+                }
+            });
+
+            this.after('initialize', function () {
+                this.on(document, events.dispatchers.rightPane.clear, this.teardown);
+                this.on(document, events.ui.recipients.updated, this.recipientsUpdated);
+                this.on(document, events.mail.draftSaved, this.draftSaved);
+                this.on(document, events.mail.sent, this.mailSent);
+
+                this.on(document, events.ui.mail.send, this.sendMail);
+
+                this.on(document, events.ui.mail.discard, this.discardDraft);
+                this.on(document, events.ui.tag.selected, this.saveTag);
+                this.on(document, events.ui.tag.select, this.saveTag);
+            });
         }
-      };
-
-      this.buildAndSaveDraft = function () {
-        var mail = this.buildMail();
-        this.saveDraft(mail);
-      };
-
-      this.recipientsUpdated = function (ev, data) {
-        this.attr.recipientValues[data.recipientsName] = data.newRecipients;
-        this.trigger(document, events.ui.mail.recipientsUpdated);
-        if (data.skipSaveDraft) { return; }
-
-        var mail = this.buildMail();
-        this.postponeSaveDraft(mail);
-      };
-
-      this.saveDraft = function (mail) {
-        this.cancelPostponedSaveDraft();
-        this.trigger(document, events.mail.saveDraft, mail);
-      };
-
-      this.cancelPostponedSaveDraft = function() {
-        clearTimeout(this.attr.timeout);
-      };
-
-      this.postponeSaveDraft = function (mail) {
-        this.cancelPostponedSaveDraft();
-
-        this.attr.timeout = window.setTimeout(_.bind(function() {
-          this.saveDraft(mail);
-        }, this), this.attr.saveDraftInterval);
-      };
-
-      this.draftSaved = function(event, data) {
-        this.attr.ident = data.ident;
-      };
-
-      this.validateAnyRecipient = function () {
-        return !_.isEmpty(_.flatten(_.values(this.attr.recipientValues)));
-      };
-
-      function allRecipientsAreEmails(mail) {
-        var allRecipients = mail.header.to.concat(mail.header.cc).concat(mail.header.bcc);
-        return _.isEmpty(allRecipients) ? false : _.all(allRecipients, emailFormatChecker);
-      }
-
-      function emailFormatChecker(email) {
-        var emailFormat = /[^\s@]+@[^\s@]+\.[^\s@]+$/;
-        return emailFormat.test(email);
-      }
-
-      this.saveTag = function(ev, data) {
-        this.attr.currentTag = data.tag;
-      };
-
-      this.mailSent = function() {
-        this.trigger(document, events.ui.userAlerts.displayMessage, { message: 'Your message was sent!' });
-      };
-
-      this.enableFloatlabel = function(element) {
-        var showClass = 'showfloatlabel';
-        $(element).bind('keyup', function() {
-          var label = $(this).prev('label');
-          if (this.value !== '') {
-            label.addClass(showClass);
-            $(this).addClass(showClass);
-          } else {
-            label.removeClass(showClass);
-            $(this).removeClass(showClass);
-          }
-        });
-      };
-
-      this.before('initialize', function () {
-          if (!this.discardDraft){
-            this.discardDraft = function () {};
-          }
-      });
-
-      this.after('initialize', function () {
-        this.on(document, events.dispatchers.rightPane.clear, this.teardown);
-        this.on(document, events.ui.recipients.updated, this.recipientsUpdated);
-        this.on(document, events.mail.draftSaved, this.draftSaved);
-        this.on(document, events.mail.sent, this.mailSent);
-
-        this.on(document, events.ui.mail.send, this.sendMail);
-
-        this.on(document, events.ui.mail.discard, this.discardDraft);
-        this.on(document, events.ui.tag.selected, this.saveTag);
-        this.on(document, events.ui.tag.select, this.saveTag);
-      });
-    }
-
-    return withMailEditBase;
-  });
+
+        return withMailEditBase;
+    });
diff --git a/web-ui/app/js/page/events.js b/web-ui/app/js/page/events.js
index 406c3b23..c85a322d 100644
--- a/web-ui/app/js/page/events.js
+++ b/web-ui/app/js/page/events.js
@@ -145,7 +145,8 @@ define(function () {
       },
       uploadedAttachment: 'mail:uploaded:attachment',
       uploadingAttachment: 'mail:uploading:attachment',
-      startUploadAttachment: 'mail:start:upload:attachment'
+      startUploadAttachment: 'mail:start:upload:attachment',
+      appendAttachment: 'mail:append:attachment'
     },
     mails: {
       available: 'mails:available',
diff --git a/web-ui/test/spec/mail_view/data/attachment_list.spec.js b/web-ui/test/spec/mail_view/data/attachment_list.spec.js
index f1fa0e76..3d93537b 100644
--- a/web-ui/test/spec/mail_view/data/attachment_list.spec.js
+++ b/web-ui/test/spec/mail_view/data/attachment_list.spec.js
@@ -1,45 +1,21 @@
-describeComponent('mail_view/data/attachment_list', function () {
-    'use strict';
+describeMixin('mail_view/data/attachment_list', function () {
+  'use strict';
 
-    describe('initialization', function () {
-        beforeEach(function () {
-            this.setupComponent('<div id="attachment-list">' +
-                '<ul><li id="attachment-list-item"> </li></ul>' +
-                '</div>');
+    describe('initialization', function() {
+        beforeEach(function(){
+            this.setupComponent();
         });
 
         it('should add attachment to the list based on uploadedAttachment event', function () {
             var stubAttachment = {attachment_id: 'faked'};
-            $(document).trigger(Pixelated.events.mail.uploadedAttachment, stubAttachment);
+            $(document).trigger(Pixelated.events.mail.appendAttachment, stubAttachment);
             expect(this.component.attr.attachments).toEqual([stubAttachment]);
 
             var anotherStubAttachment = {attachment_id: 'faked 2'};
-            $(document).trigger(Pixelated.events.mail.uploadedAttachment, anotherStubAttachment);
+            $(document).trigger(Pixelated.events.mail.appendAttachment, anotherStubAttachment);
             expect(this.component.attr.attachments).toEqual([stubAttachment, anotherStubAttachment]);
         });
 
-        it('should render attachment list view based on uploadedAttachment event', function () {
-            var stubAttachment = {attachment_id: 'faked', filename: 'haha.txt', filesize: 4500};
-
-            $(document).trigger(Pixelated.events.mail.uploadedAttachment, stubAttachment);
-
-            var expected_li = '<a href="/attachment/faked?filename=haha.txt&amp;encoding=base64">haha.txt (4.39 Kb)</a>';
-            expect(this.component.select('attachmentListItem').html()).toEqual(expected_li);
-        });
-
-        xit('should start uploading attachments', function () {
-            var stubAttachment = {attachment_id: 'faked', filename: 'haha.txt', filesize: 4500};
-            var mockAjax = spyOn($, 'ajax').and.callFake(function (params) {params.success(stubAttachment);});
-            var uploadedAttachment = spyOnEvent(document, Pixelated.events.mail.uploadedAttachment);
-            var uploading = spyOnEvent(document, Pixelated.events.mail.uploadingAttachment);
-
-            $(document).trigger(Pixelated.events.mail.startUploadAttachment);
-
-            expect(mockAjax).toHaveBeenCalled();
-            expect(uploadedAttachment).toHaveBeenTriggeredOnAndWith(document, stubAttachment);
-            expect(uploading).toHaveBeenTriggeredOn(document);
-        });
-
     });
 
 });
diff --git a/web-ui/test/spec/mail_view/ui/attachment.spec.js b/web-ui/test/spec/mail_view/ui/attachment_icon.spec.js
index d29660b9..d29660b9 100644
--- a/web-ui/test/spec/mail_view/ui/attachment.spec.js
+++ b/web-ui/test/spec/mail_view/ui/attachment_icon.spec.js
diff --git a/web-ui/test/spec/mail_view/ui/attachment_list.spec.js b/web-ui/test/spec/mail_view/ui/attachment_list.spec.js
new file mode 100644
index 00000000..c66b959c
--- /dev/null
+++ b/web-ui/test/spec/mail_view/ui/attachment_list.spec.js
@@ -0,0 +1,44 @@
+describeComponent('mail_view/ui/attachment_list', function () {
+    'use strict';
+
+    describe('initialization', function () {
+        beforeEach(function () {
+            this.setupComponent('<div id="attachment-list">' +
+                '<ul><li id="attachment-list-item"> </li></ul>' +
+                '</div>');
+        });
+
+        it('should trigger add attachment event', function () {
+            var triggerUploadAttachment = spyOnEvent(document, Pixelated.events.mail.appendAttachment);
+            var stubAttachment = {attachment_id: 'faked'};
+
+            $(document).trigger(Pixelated.events.mail.uploadedAttachment, stubAttachment);
+
+            expect(triggerUploadAttachment).toHaveBeenTriggeredOnAndWith(document, stubAttachment);
+        });
+
+        it('should render attachment list view based on uploadedAttachment event', function () {
+            var stubAttachment = {attachment_id: 'faked', filename: 'haha.txt', filesize: 4500};
+
+            $(document).trigger(Pixelated.events.mail.uploadedAttachment, stubAttachment);
+
+            var expected_li = '<a href="/attachment/faked?filename=haha.txt&amp;encoding=base64">haha.txt (4.39 Kb)</a>';
+            expect(this.component.select('attachmentListItem').html()).toEqual(expected_li);
+        });
+
+        xit('should start uploading attachments', function () {
+            var stubAttachment = {attachment_id: 'faked', filename: 'haha.txt', filesize: 4500};
+            var mockAjax = spyOn($, 'ajax').and.callFake(function (params) {params.success(stubAttachment);});
+            var uploadedAttachment = spyOnEvent(document, Pixelated.events.mail.uploadedAttachment);
+            var uploading = spyOnEvent(document, Pixelated.events.mail.uploadingAttachment);
+
+            $(document).trigger(Pixelated.events.mail.startUploadAttachment);
+
+            expect(mockAjax).toHaveBeenCalled();
+            expect(uploadedAttachment).toHaveBeenTriggeredOnAndWith(document, stubAttachment);
+            expect(uploading).toHaveBeenTriggeredOn(document);
+        });
+
+    });
+
+});