Merge pull request #29 from Brightspace/cpang/US127307-added-null-end…

…point-check

US127307 Added null endpoint check for logUserEvent

src/client.js

@@ -4,6 +4,10 @@ export class Client {
 	}
 
 	logUserEvent(event) {
+		if (!this.options.endpoint) {
+			return;
+		}
+
 		var requestObject = {
 			method: 'POST'
 		};

test/index.js

@@ -68,4 +68,36 @@ describe('test suite', () => {
 			});
 		});
 	});
+
+	describe('logUserEvent - with null endpoint', () => {
+		var fetch, client, event;
+
+		before(() => {
+			client = new Client({
+				endpoint: null
+			});
+
+			event = new TelemetryEvent();
+			const eventBody = new EventBody();
+			event.setBody(eventBody);
+
+			fetch = sinon.stub(window.d2lfetch, 'fetch');
+			var promise = Promise.resolve({
+				ok: true,
+				json: function() {
+					return Promise.resolve();
+				}
+			});
+			fetch.returns(promise);
+		});
+
+		after(() => {
+			fetch && fetch.restore();
+		});
+
+		it ('POSTs an event', () => {
+			client.logUserEvent(event);
+			expect(fetch.calledWith(sinon.match.has('method', 'POST'))).to.be.false;
+		});
+	});
 });