Read encoding from attachment info instead of headers

Clean up by removing now unused headers from attachment info, and add a
unit test for attachment rendering

1 files changed, 10 insertions, 0 deletions

diff --git a/web-ui/test/spec/mail_view/ui/mail_view.spec.js b/web-ui/test/spec/mail_view/ui/mail_view.spec.js
index deb7fb88..fe763919 100644
--- a/web-ui/test/spec/mail_view/ui/mail_view.spec.js
+++ b/web-ui/test/spec/mail_view/ui/mail_view.spec.js
@@ -265,6 +265,16 @@ describeComponent('mail_view/ui/mail_view', function () {
     expect(openNoMessageSelectedEvent).toHaveBeenTriggeredOn(document);
   });
 
+  it('shows a download link for attachments', function() {
+    var withAttachments = {mail: Pixelated.testData().parsedMail.withAttachments};
+
+    this.component.displayMail({}, withAttachments);
+
+    var attachmentLink = $(this.component.$node.find('.attachmentsArea li').html());
+    var expectedLink = '/attachment/912ec803b2ce49e4a541068d495ab570?encoding=base64&filename=filename.txt';
+    expect(attachmentLink.attr('href')) .toBe(expectedLink);
+  });
+
   function creatingEvent(event, keyCode) {
     var e = $.Event(event);
     e.which = keyCode;