Update unit listing page tests

Author: jonkafton

frontends/api/src/test-utils/factories/channels.ts

@@ -148,6 +148,7 @@ const _channelShared = (): Partial<Omit<Channel, "channel_type">> => {
       key: faker.lorem.slug(),
       value: faker.lorem.slug(),
     }),
+    channel_url: `${faker.internet.url({ appendSlash: false })}${faker.system.directoryPath()}`,
   }
 }
 

frontends/api/src/test-utils/urls.ts

@@ -179,6 +179,8 @@ const channels = {
   details: (channelType: string, name: string) =>
     `${API_BASE_URL}/api/v0/channels/type/${channelType}/${name}/`,
   patch: (id: number) => `${API_BASE_URL}/api/v0/channels/${id}/`,
+  list: (params?: Params<ChannelsApi, "channelsList">) =>
+    `${API_BASE_URL}/api/v0/channels/${query(params)}`,
 }
 
 const widgetLists = {

frontends/main/src/app-pages/UnitsListingPage/UnitsListingPage.test.tsx

@@ -2,44 +2,71 @@ import React from "react"
 import { renderWithProviders, screen, waitFor, within } from "@/test-utils"
 import UnitsListingPage from "./UnitsListingPage"
 import { factories, setMockResponse, urls } from "api/test-utils"
+import { ChannelTypeEnum } from "api/v0"
+import type { UnitChannel } from "api/v0"
 import { assertHeadings } from "ol-test-utilities"
 
-describe("DepartmentListingPage", () => {
+describe("UnitListingPage", () => {
   const setupApis = () => {
-    const make = factories.learningResources
-    const academicUnit1 = make.offeror({
-      code: "academicUnit1",
-      name: "Academic Unit 1",
-      value_prop: "Academic Unit 1 value prop",
-      professional: false,
+    const make = factories.channels
+    const academicUnit1 = make.channel({
+      channel_type: ChannelTypeEnum.Unit,
+      name: "academicUnit1",
+      title: "Academic Unit 1",
+      unit_detail: {
+        unit: {
+          value_prop: "Academic Unit 1 value prop",
+          professional: false,
+        },
+      },
     })
-    const academicUnit2 = make.offeror({
-      code: "academicUnit2",
-      name: "Academic Unit 2",
-      value_prop: "Academic Unit 2 value prop",
-      professional: false,
+    const academicUnit2 = make.channel({
+      channel_type: ChannelTypeEnum.Unit,
+      name: "academicUnit2",
+      title: "Academic Unit 2",
+      unit_detail: {
+        unit: {
+          value_prop: "Academic Unit 2 value prop",
+          professional: false,
+        },
+      },
     })
-    const academicUnit3 = make.offeror({
-      code: "academicUnit3",
-      name: "Academic Unit 3",
-      value_prop: "Academic Unit 3 value prop",
-      professional: false,
+    const academicUnit3 = make.channel({
+      channel_type: ChannelTypeEnum.Unit,
+      name: "academicUnit3",
+      title: "Academic Unit 3",
+      unit_detail: {
+        unit: {
+          value_prop: "Academic Unit 3 value prop",
+          professional: false,
+        },
+      },
     })
 
-    const professionalUnit1 = make.offeror({
-      code: "professionalUnit1",
-      name: "Professional Unit 1",
-      value_prop: "Professional Unit 1 value prop",
-      professional: true,
+    const professionalUnit1 = make.channel({
+      channel_type: ChannelTypeEnum.Unit,
+      name: "professionalUnit1",
+      title: "Professional Unit 1",
+      unit_detail: {
+        unit: {
+          value_prop: "Professional Unit 1 value prop",
+          professional: true,
+        },
+      },
     })
-    const professionalUnit2 = make.offeror({
-      code: "professionalUnit2",
-      name: "Professional Unit 2",
-      value_prop: "Professional Unit 2 value prop",
-      professional: true,
+    const professionalUnit2 = make.channel({
+      channel_type: ChannelTypeEnum.Unit,
+      name: "professionalUnit2",
+      title: "Professional Unit 2",
+      unit_detail: {
+        unit: {
+          value_prop: "Professional Unit 2 value prop",
+          professional: true,
+        },
+      },
     })
 
-    const units = [
+    const unitChannels = [
       academicUnit1,
       academicUnit2,
       academicUnit3,
@@ -63,29 +90,29 @@ describe("DepartmentListingPage", () => {
 
     setMockResponse.get(
       urls.channels.counts("unit"),
-      units.map((unit) => {
+      unitChannels.map((channel) => {
         return {
-          name: unit.code,
+          name: channel.name,
           counts: {
-            courses: courseCounts[unit.code],
-            programs: programCounts[unit.code],
+            courses: courseCounts[channel.name],
+            programs: programCounts[channel.name],
           },
         }
       }),
     )
-    setMockResponse.get(urls.offerors.list(), {
-      count: units.length,
-      results: units,
+    setMockResponse.get(urls.channels.list({ channel_type: "unit" }), {
+      count: unitChannels.length,
+      results: unitChannels,
     })
 
-    units.forEach((unit) => {
-      setMockResponse.get(urls.channels.details("unit", unit.code), {
-        channel_url: `${window.location.origin}/units/${unit.code}`,
-      })
-    })
+    // units.forEach((unit) => {
+    //   setMockResponse.get(urls.channels.details("unit", unit.code), {
+    //     channel_url: `${window.location.origin}/units/${unit.code}`,
+    //   })
+    // })
 
     return {
-      units,
+      unitChannels,
       courseCounts,
       programCounts,
     }
@@ -98,7 +125,7 @@ describe("DepartmentListingPage", () => {
   })
 
   it("Shows unit properties within the proper section", async () => {
-    const { units, courseCounts, programCounts } = setupApis()
+    const { unitChannels, courseCounts, programCounts } = setupApis()
 
     renderWithProviders(<UnitsListingPage />)
 
@@ -116,11 +143,15 @@ describe("DepartmentListingPage", () => {
       return links
     })
 
-    units.forEach((unit) => {
+    unitChannels.forEach((channel) => {
+      const { unit } = (channel as UnitChannel).unit_detail
       const section = unit.professional ? professionalSection : academicSection
       const card = within(section).getByTestId(`unit-card-${unit.code}`)
       const link = within(card).getByRole("link")
-      expect(link).toHaveAttribute("href", `/units/${unit.code}`)
+      expect(link).toHaveAttribute(
+        "href",
+        new URL(channel.channel_url!).pathname,
+      )
 
       const courseCount = courseCounts[unit.code]
       const programCount = programCounts[unit.code]