#21 - Unit test checkpoint

Signed-off-by: Wright, Christopher R <Christopher.Wright@ca.com>
zowe · Sep 18, 2018 · f3b045d · f3b045d
1 parent e2d4236
commit f3b045d
Showing 1 changed file with 17 additions and 3 deletions.
diff --git a/packages/settings/__tests__/AppSettings.test.ts b/packages/settings/__tests__/AppSettings.test.ts
@@ -19,7 +19,6 @@ import { existsSync } from "fs";
 import { SettingsAlreadyInitialized, SettingsNotInitialized } from "../src/errors";
 import { readFileSync, writeFileSync } from "jsonfile";
 import { ISettingsFile } from "../src/doc/ISettingsFile";
-import over = require("lodash/fp/over");
 
 fdescribe("AppSettings", () => {
     const mocks = {
@@ -110,15 +109,30 @@ fdescribe("AppSettings", () => {
             const recoveryFcn = jest.fn((arg1, arg2) => {
                 expect(arg1).toEqual(fileName);
                 expect(arg2).toEqual(defaultSettings);
-                console.log("passed here");
+
                 return overwriteSettings;
             });
 
             AppSettings.initialize(fileName, recoveryFcn);
 
             expect(AppSettings.instance.settings).toEqual(overwriteSettings);
             expect(recoveryFcn).toHaveBeenCalledTimes(1);
-            // expect(recoveryFcn).toHaveBeenCalledWith(fileName, defaultSettings); // Jest has a bug :) @TODO
+
+            // The below check doesn't work because jest doesn't clone arugments.
+            // expect(recoveryFcn).toHaveBeenCalledWith(fileName, defaultSettings);
+        });
+
+        it("should merge settings provided from the file", () => {
+            const settings = [
+                {
+                    overrides: {
+                        CredentialManager: "some-plugin"
+                    }
+                },
+                {
+                    abcd: "here"
+                }
+            ];
         });
     });
 });