Fixed a bug where config values may not always be overridden & Config tests

Author: niemyjski

src/Utils.ts

@@ -59,13 +59,13 @@ export class Utils {
     const result: object = {};
 
     for (const key in defaultValues || {}) {
-      if (!!defaultValues[key]) {
+      if (defaultValues[key] !== undefined && defaultValues[key] !== null) {
         result[key] = defaultValues[key];
       }
     }
 
     for (const key in values || {}) {
-      if (!!values[key]) {
+      if (values[key] !== undefined && values[key] !== null) {
         result[key] = values[key];
       }
     }

src/configuration/Configuration-spec.ts

@@ -19,6 +19,31 @@ describe('Configuration', () => {
 
     config = new Configuration({ apiKey: null });
     expect(config.apiKey).to.equal('test');
+    expect(config.includePrivateInformation).to.true;
+    expect(config.includeUserName).to.true;
+    expect(config.includeMachineName).to.true;
+    expect(config.includeIpAddress).to.true;
+    expect(config.includeCookies).to.true;
+    expect(config.includePostData).to.true;
+    expect(config.includeQueryString).to.true;
+
+    config = new Configuration({ includePrivateInformation: false });
+    expect(config.includePrivateInformation).to.false;
+    expect(config.includeUserName).to.false;
+    expect(config.includeMachineName).to.false;
+    expect(config.includeIpAddress).to.false;
+    expect(config.includeCookies).to.false;
+    expect(config.includePostData).to.false;
+    expect(config.includeQueryString).to.false;
+
+    config.includeMachineName = true;
+    expect(config.includePrivateInformation).to.false;
+    expect(config.includeUserName).to.false;
+    expect(config.includeMachineName).to.true;
+    expect(config.includeIpAddress).to.false;
+    expect(config.includeCookies).to.false;
+    expect(config.includePostData).to.false;
+    expect(config.includeQueryString).to.false;
   });
 
   it('should not add duplicate plugin', () => {