Don't throw on missing header (#722)

* Don't throw on missing header

* Adding nasty tests to verify the change

Author: WilliamBZA

src/ServicePulse.Host.Tests/tests/js/views/message/controller.spec.js

@@ -69,6 +69,23 @@
             expect(controller.message.retried).toEqual(false);
             expect(serviceControlService.retryFailedMessages).toHaveBeenCalled();
         });
+
+        it('and the headers contain content type, GetContentType returns it', function () {
+            var originalValue = 'xml';
+            var headers = [];
+            headers.push({key: 'NServiceBus.ContentType', value: originalValue});
+            headers.push({key: 'bla', value:1});
+            var headerValue = controller.getContentType(headers);            
+
+            expect(headerValue).toEqual(originalValue);
+        });
+
+        it('and the headers are empty GetContentType returns null', function () {
+            var headers = [];            
+            var headerValue = controller.getContentType(headers);
+
+            expect(headerValue).toEqual(null);
+        });
     });
 
     describe('when archiving a message', function () {

src/ServicePulse.Host/app/js/views/message/controller.js

@@ -86,8 +86,11 @@
         }
 
         function getContentType(headers) {
-            return headers.find(function (element) { return element.key === 'NServiceBus.ContentType'; }).value;
+            var header = headers.find(function (element) { return element.key === 'NServiceBus.ContentType'; });
+            return header ? header.value : null;
         }
+        
+        vm.getContentType = getContentType;
 
         vm.retryMessage = function () {
             toastService.showInfo("Retrying the message " + vm.message.message_id + " ...");