clean up from merge

src/utils/course.js

@@ -4,7 +4,6 @@ export const availableSeatsInCourse = (
   registrations
 ) => {
   if (!Number.isInteger(capacity)) {
-    console.warn("capacity was not an integer so default course capacity is 0");
     return 0;
   }
   if (Array.isArray(selectedMembers)) {

tests/unit/views/TrainingDetails.spec.js

@@ -37,6 +37,11 @@ describe("TrainingDetails", () => {
     TrainingDetails.fetchData = jest.fn();
     const wrapper = shallowMount(TrainingDetails, {
       mocks: {
+        $route: {
+          params: {
+            trainingId: 1,
+          },
+        },
         $store: {
           state: {
             error: {},
@@ -171,92 +176,6 @@ describe("TrainingDetails", () => {
       true
     );
   });
-  it("should add student", async () => {
-    TrainingService.getCourse = jest.fn().mockResolvedValue(mockCourse);
-    TrainingService.getCourseRegistrationsById = jest
-      .fn()
-      .mockResolvedValue({ registrations: [] });
-    TrainingService.addStudents = jest.fn().mockResolvedValue([]);
-    const wrapper = shallowMount(TrainingDetails, {
-      mocks: {
-        $store: {
-          state: {
-            error: {},
-          },
-          commit: jest.fn(),
-        },
-      },
-      data() {
-        return {
-          selectedUsers: [{ userId: 1 }, { userId: 2 }],
-          listItems: [{ userId: 1 }, { userId: 2 }, { userId: 3 }],
-        };
-      },
-      propsData,
-      localVue,
-      vuetify,
-    });
-    await flushPromises();
-
-    wrapper.vm.$refs.userSelect.clear = jest.fn();
-    await wrapper.vm.addStudent([{ id: 1 }]);
-    expect(TrainingService.addStudents).toHaveBeenCalledTimes(1);
-  });
-  it("addStudent should catch error", async () => {
-    console.error = jest.fn(); // mock console.error when expecting error messages
-    TrainingService.getCourse = jest.fn().mockResolvedValue(mockCourse);
-    TrainingService.addStudents = jest.fn().mockRejectedValue("error");
-    const wrapper = shallowMount(TrainingDetails, {
-      mocks: {
-        $store: {
-          state: {
-            error: {},
-          },
-          commit: jest.fn(),
-        },
-      },
-      propsData,
-      localVue,
-      vuetify,
-    });
-    wrapper.vm.$refs.userSelect.clear = jest.fn();
-    await wrapper.vm.addStudent();
-    expect(wrapper.vm.$store.commit).toHaveBeenCalledWith(
-      SET_ERROR_MESSAGE,
-      "error"
-    );
-  });
-  it("should cancel add student", async () => {
-    TrainingService.getCourse = jest.fn().mockResolvedValue(mockCourse);
-    TrainingService.getCourseRegistrationsById = jest
-      .fn()
-      .mockResolvedValue({ registrations: [] });
-    const wrapper = shallowMount(TrainingDetails, {
-      mocks: {
-        $store: {
-          state: {
-            error: {},
-          },
-          commit: jest.fn(),
-        },
-      },
-      data() {
-        return {
-          selectedUsers: [{ userId: 1 }, { userId: 2 }],
-          listItems: [{ userId: 1 }, { userId: 2 }, { userId: 3 }],
-        };
-      },
-      propsData,
-      localVue,
-      vuetify,
-    });
-    await flushPromises();
-
-    wrapper.vm.$refs.userSelect.clear = jest.fn();
-    wrapper.vm.cancelAddStudent();
-    expect(wrapper.vm.$refs.userSelect.clear).toHaveBeenCalledTimes(1);
-  });
-
   it("should not show error by default", () => {
     TrainingService.getCourse = jest.fn().mockResolvedValue(mockCourse);
     // render the component