Merge pull request #22 from srijken/master

Make sure createLog works with null sourceOrMessage and null caller

Author: niemyjski

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "exceptionless",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "JavaScript client for Exceptionless",
   "license": "Apache",
   "main": "dist/exceptionless.js",

dist/exceptionless.js

@@ -1,6 +1,6 @@
 {
   "name": "exceptionless",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "JavaScript client for Exceptionless",
   "license": "Apache",
   "main": "dist/exceptionless.node.js",

dist/exceptionless.js.map

@@ -25,4 +25,13 @@ describe('ExceptionlessClient', () => {
       done();
     });
   }, 5000);
+
+  it('should accept null sourceOrMessage', () => {
+    var client = new ExceptionlessClient('LhhP1C9gijpSKCslHHCvwdSIz298twx271n1l6xw', 'http://localhost:50000');
+    var builder = client.createLog(null, 'Unit Test message', 'Trace');
+
+    expect(builder.target.source).toBeUndefined();
+    expect(builder.target.message).toBe('Unit Test message');
+    expect(builder.target.data['@level']).toBe('Trace');
+  });
 });

dist/exceptionless.min.js

@@ -59,14 +59,14 @@ export class ExceptionlessClient {
   public createLog(sourceOrMessage:string, message?:string, level?:string): EventBuilder {
     var builder = this.createEvent().setType('log');
 
-    if (sourceOrMessage && message && level) {
+    if (message && level) {
       builder = builder.setSource(sourceOrMessage).setMessage(message).setProperty('@level', level);
-    } else if (sourceOrMessage && message) {
+    } else if (message) {
       builder = builder.setSource(sourceOrMessage).setMessage(message);
     } else {
       // TODO: Look into using https://www.stevefenton.co.uk/Content/Blog/Date/201304/Blog/Obtaining-A-Class-Name-At-Runtime-In-TypeScript/
-      var source = (<any>(arguments.callee.caller)).name;
-      builder = builder.setSource(source).setMessage(sourceOrMessage);
+      var caller = (<any>(arguments.callee.caller));
+      builder = builder.setSource(caller && caller.name).setMessage(sourceOrMessage);
     }
 
     return builder;