Add unit test for config method

Author: javierbrea

src/commands.js

@@ -1,28 +1,31 @@
-const { settings, config } = require("@mocks-server/admin-api-client");
+const adminApiClient = require("@mocks-server/admin-api-client");
 
 const setBehavior = behavior => {
-  return settings.update({
+  return adminApiClient.settings.update({
     behavior
   });
 };
 
 const setDelay = delay => {
-  return settings.update({
+  return adminApiClient.settings.update({
     delay
   });
 };
 
 const setSettings = newSettings => {
-  return settings.update(newSettings);
+  return adminApiClient.settings.update(newSettings);
 };
 
 // TODO, remove when admin-api-client supports adminApiPath option
 const mapConfig = customConfig => {
-  const configToSet = customConfig;
+  const configToSet = {};
   if (customConfig.adminApiPath) {
     configToSet.apiPath = customConfig.adminApiPath;
   }
-  return config(configToSet);
+  if (customConfig.baseUrl) {
+    configToSet.baseUrl = customConfig.baseUrl;
+  }
+  return adminApiClient.config(configToSet);
 };
 
 module.exports = {

test-e2e/app-data-provider/package.json

@@ -1,5 +1,5 @@
 {
-  "name": "app-fetch",
+  "name": "app-data-provider",
   "version": "1.0.0",
   "private": true,
   "dependencies": {

test/commands.spec.js

@@ -1,48 +1,71 @@
 const sinon = require("sinon");
-const { settings } = require("@mocks-server/admin-api-client");
+const adminApiClient = require("@mocks-server/admin-api-client");
 
-const { setBehavior, setDelay, setSettings } = require("../src/commands");
+const { setBehavior, setDelay, setSettings, config } = require("../src/commands");
 
-describe("register", () => {
+describe("commands", () => {
   let sandbox;
 
   beforeEach(() => {
     sandbox = sinon.createSandbox();
-    sandbox.stub(settings, "update");
+    sandbox.stub(adminApiClient.settings, "update");
+    sandbox.stub(adminApiClient, "config");
   });
 
   afterEach(() => {
     sandbox.restore();
   });
 
-  describe("commands", () => {
-    describe("setBehavior command", () => {
-      it("should call to update behavior", () => {
-        setBehavior("foo-behavior");
-        expect(
-          settings.update.calledWith({
-            behavior: "foo-behavior"
-          })
-        ).toBe(true);
-      });
+  describe("setBehavior command", () => {
+    it("should call to update behavior", () => {
+      setBehavior("foo-behavior");
+      expect(
+        adminApiClient.settings.update.calledWith({
+          behavior: "foo-behavior"
+        })
+      ).toBe(true);
+    });
+  });
+
+  describe("setDelay command", () => {
+    it("should call to update delay", () => {
+      setDelay(3000);
+      expect(
+        adminApiClient.settings.update.calledWith({
+          delay: 3000
+        })
+      ).toBe(true);
     });
+  });
+
+  describe("setSettings command", () => {
+    it("should call to update delay", () => {
+      setSettings("foo");
+      expect(adminApiClient.settings.update.calledWith("foo")).toBe(true);
+    });
+  });
 
-    describe("setDelay command", () => {
-      it("should call to update delay", () => {
-        setDelay(3000);
-        expect(
-          settings.update.calledWith({
-            delay: 3000
-          })
-        ).toBe(true);
+  describe("config method", () => {
+    it("should call to config admin-api-client baseUrl", () => {
+      config({
+        baseUrl: "foo"
       });
+      expect(
+        adminApiClient.config.calledWith({
+          baseUrl: "foo"
+        })
+      ).toBe(true);
     });
 
-    describe("setSettings command", () => {
-      it("should call to update delay", () => {
-        setSettings("foo");
-        expect(settings.update.calledWith("foo")).toBe(true);
+    it("should call to config admin-api-client apiPath with adminApiPath value", () => {
+      config({
+        adminApiPath: "foo"
       });
+      expect(
+        adminApiClient.config.calledWith({
+          apiPath: "foo"
+        })
+      ).toBe(true);
     });
   });
 });