unit tests

src/components/AddTeam.vue

@@ -138,18 +138,21 @@ export default {
     edit() {
       // merge members and leads
       const members = new Map();
-      this.toAdd.members.forEach((member) => {
-        member.TeamMember = { isTeamLead: false };
-        members.set(member.id, member);
-      });
-      console.log(this.toAdd);
-      this.leads.forEach((lead) => {
-        lead.TeamMember = { isTeamLead: true };
-        members.set(lead.id, lead);
-      });
+      if (this.toAdd.members) {
+        this.toAdd.members.forEach((member) => {
+          member.TeamMember = { isTeamLead: false };
+          members.set(member.id, member);
+        });
+      }
+      if (this.leads) {
+        this.leads.forEach((lead) => {
+          lead.TeamMember = { isTeamLead: true };
+          members.set(lead.id, lead);
+        });
+      }
       this.toAdd.members = [...members.values()];
 
-      this.$emit("edit", this.toAdd, this.leads);
+      this.$emit("edit", this.toAdd);
       this.init();
     },
   },

src/components/UserSelect.vue

@@ -154,7 +154,6 @@ export default {
     async query(val) {
       try {
         const newItems = await UserService.search({ q: val });
-        console.log(newItems);
         if (this.model) {
           if (Array.isArray(this.model)) {
             this.items = uniq([...newItems.users, ...this.model]);

tests/unit/components/AddTeam.spec.js

@@ -29,7 +29,7 @@ describe("AddTeam", () => {
     wrapper.vm.$emit = jest.fn();
     wrapper.vm.edit();
     expect(wrapper.vm.$emit).toHaveBeenCalledWith("edit", {
-      memberCount: { count: 0 },
+      members: [],
     });
   });
   it("should trigger cancel", async () => {
@@ -68,8 +68,6 @@ describe("AddTeam", () => {
 
     wrapper.vm.$emit = jest.fn();
     wrapper.vm.add();
-    expect(wrapper.vm.$emit).toHaveBeenCalledWith("add", {
-      memberCount: { count: 0 },
-    });
+    expect(wrapper.vm.$emit).toHaveBeenCalledWith("add", {});
   });
 });

tests/unit/components/EditableInput.spec.js

@@ -118,15 +118,13 @@ describe("components/EditableInput", () => {
           },
         },
       },
-      props: {
-        value: "mockValue",
-      },
       localVue,
       vuetify,
     });
 
     wrapper.vm.$emit = jest.fn();
     await wrapper.setProps({ value: "newMockValue" });
+    await wrapper.setData({ model: "MockValue" });
     wrapper.vm.updateValue();
     expect(wrapper.vm.$emit).toHaveBeenCalled();
   });

tests/unit/components/UserSelect.spec.js

@@ -3,7 +3,7 @@ import Vuetify from "vuetify";
 import UserSelect from "@/components/UserSelect";
 import UserService from "@/api/services/user";
 
-describe("LaunchboardAdmin", () => {
+describe("UserSelect", () => {
   // vuetify has to be mocked for the v-autocomplete to not crash
   let vuetify;
   beforeEach(() => {
@@ -119,7 +119,7 @@ describe("LaunchboardAdmin", () => {
       vuetify,
     });
 
-    UserService.search = jest.fn().mockResolvedValue(["a"]);
+    UserService.search = jest.fn().mockResolvedValue({ users: ["a"] });
 
     // multiple model
     wrapper.vm.model = ["b", "c"];