fix: do not call to set config when env vars are not defined

Author: javierbrea

src/commands.js

@@ -68,10 +68,12 @@ function commands(Cypress) {
     return apiClient.config(customConfig);
   };
 
-  config({
-    adminApiPath: Cypress.env(ADMIN_API_PATH_ENVIRONMENT_VAR),
-    baseUrl: Cypress.env(BASE_URL_ENVIRONMENT_VAR),
-  });
+  if (Cypress.env(ADMIN_API_PATH_ENVIRONMENT_VAR)) {
+    config({ adminApiPath: Cypress.env(ADMIN_API_PATH_ENVIRONMENT_VAR) });
+  }
+  if (Cypress.env(BASE_URL_ENVIRONMENT_VAR)) {
+    config({ baseUrl: Cypress.env(BASE_URL_ENVIRONMENT_VAR) });
+  }
 
   return {
     setBehavior,

test/commands.spec.js

@@ -32,6 +32,28 @@ describe("commands", () => {
     sandbox.restore();
   });
 
+  describe("when initializing", () => {
+    it("should call to set adminApiPath config if env var is defined", () => {
+      cypressMock.stubs.env.withArgs("MOCKS_SERVER_ADMIN_API_PATH").returns("foo");
+      commands(cypressMock.stubs);
+      expect(
+        apiClient.config.calledWith({
+          adminApiPath: "foo",
+        })
+      ).toBe(true);
+    });
+
+    it("should call to set baseUrl config if env var is defined", () => {
+      cypressMock.stubs.env.withArgs("MOCKS_SERVER_BASE_URL").returns("foo");
+      commands(cypressMock.stubs);
+      expect(
+        apiClient.config.calledWith({
+          baseUrl: "foo",
+        })
+      ).toBe(true);
+    });
+  });
+
   describe("setMock command", () => {
     it("should call to update delay", () => {
       setMock("foo");