@bearmenu/ui 0.6.0 → 0.7.0

package/src/components/place-about-tab.test.tsx

@@ -1,20 +1,20 @@
 import { describe, it, expect, vi } from "vitest";
 import { render, screen, fireEvent } from "@testing-library/react";
 import { PlaceAboutTab } from "./place-about-tab";
-import type { PlaceAboutTabLabels, PlacePreview } from "./place-types";
+import type {
+  PlaceAboutTabLabels,
+  PlacePreview,
+  PlaceSchedule,
+} from "./place-types";
 
 // ─── Fixtures ─────────────────────────────────────────────────────────────────
 
 const LABELS: PlaceAboutTabLabels = {
-  overview: "Overview",
   description: "Description",
-  whatPeopleSay: "What people say",
   contactAndLocation: "Contact & Location",
-  address: "Address",
   instagram: "Instagram",
   facebook: "Facebook",
-  todayHours: "Today's hours",
-  statusOpen: "Open now",
+  todayHours: "Hours",
   statusClosed: "Closed",
   typicalBusyness: "Typical busyness",
   cuisines: "Cuisines",
@@ -23,10 +23,16 @@ const LABELS: PlaceAboutTabLabels = {
   accessibility: "Accessibility",
   parking: "Parking",
   nearbyLandmarks: "Nearby landmarks",
-  neighborhood: "Neighborhood",
-  businessInformation: "Business information",
-  type: "Type",
-  status: "Status",
+  scheduleDays: {
+    monday: "Monday",
+    tuesday: "Tuesday",
+    wednesday: "Wednesday",
+    thursday: "Thursday",
+    friday: "Friday",
+    saturday: "Saturday",
+    sunday: "Sunday",
+  },
+  noHoursAvailable: "Hours not available",
   amenityLabels: { wifi: "Free Wi-Fi" },
   paymentLabels: {},
   accessibilityLabels: {},
@@ -35,6 +41,19 @@ const LABELS: PlaceAboutTabLabels = {
   statusLabels: {},
 };
 
+function makeSchedule(overrides: Partial<PlaceSchedule> = {}): PlaceSchedule {
+  return {
+    monday: [],
+    tuesday: [],
+    wednesday: [],
+    thursday: [],
+    friday: [],
+    saturday: [],
+    sunday: [],
+    ...overrides,
+  };
+}
+
 function makePlace(overrides: Partial<PlacePreview> = {}): PlacePreview {
   return {
     id: "1",
@@ -104,6 +123,33 @@ describe("PlaceAboutTab", () => {
     expect(screen.queryByText("Facebook")).not.toBeInTheDocument();
   });
 
+  // ─── Address + neighborhood subline ─────────────────────────────────────────
+
+  it("renders address as primary text and neighborhood as muted subline", () => {
+    render(
+      <PlaceAboutTab
+        place={makePlace({
+          location: { address: "Strada Napoca 12" },
+          areas: [{ name: "Centru" }, { name: "Piata Unirii" }],
+        })}
+        labels={LABELS}
+      />
+    );
+    expect(screen.getByText("Strada Napoca 12")).toBeInTheDocument();
+    expect(screen.getByText("Centru · Piata Unirii")).toBeInTheDocument();
+  });
+
+  it("does not render neighborhood subline when areas is empty", () => {
+    render(
+      <PlaceAboutTab
+        place={makePlace({ location: { address: "Strada Napoca 12" } })}
+        labels={LABELS}
+      />
+    );
+    expect(screen.getByText("Strada Napoca 12")).toBeInTheDocument();
+    expect(screen.queryByText(/·/)).not.toBeInTheDocument();
+  });
+
   // ─── mapSlot ────────────────────────────────────────────────────────────────
 
   it("renders mapSlot when prop is passed (regardless of busyness data)", () => {
@@ -227,29 +273,49 @@ describe("PlaceAboutTab", () => {
     );
   });
 
-  // ─── Today's hours card ──────────────────────────────────────────────────────
+  // ─── Hours section (full week, schedule-driven) ─────────────────────────────
 
-  it("renders today's hours card when place.hours has entries", () => {
+  it("renders the hours section heading and a slot row when place.schedule is provided", () => {
     render(
       <PlaceAboutTab
-        place={makePlace({ hours: [{ opens_at: 9, closes_at: 22 }] })}
+        place={makePlace({
+          schedule: makeSchedule({ monday: [{ opens_at: 9, closes_at: 22 }] }),
+        })}
         labels={LABELS}
       />
     );
-    expect(screen.getByText("Today's hours")).toBeInTheDocument();
-    expect(screen.getByText("09:00 - 22:00")).toBeInTheDocument();
+    expect(screen.getByText("Hours")).toBeInTheDocument();
+    expect(screen.getByText("09:00 – 22:00")).toBeInTheDocument();
+    expect(screen.getByText("Monday")).toBeInTheDocument();
   });
 
-  it("does NOT render today's hours card when place.hours is empty", () => {
+  it("renders all 7 weekday labels when schedule is present", () => {
     render(
-      <PlaceAboutTab place={makePlace({ hours: [] })} labels={LABELS} />
+      <PlaceAboutTab
+        place={makePlace({ schedule: makeSchedule() })}
+        labels={LABELS}
+      />
+    );
+    expect(screen.getByText("Monday")).toBeInTheDocument();
+    expect(screen.getByText("Sunday")).toBeInTheDocument();
+  });
+
+  it("renders 'Closed' (line-through) for days with no slots", () => {
+    render(
+      <PlaceAboutTab
+        place={makePlace({
+          schedule: makeSchedule({ monday: [{ opens_at: 9, closes_at: 22 }] }),
+        })}
+        labels={LABELS}
+      />
     );
-    expect(screen.queryByText("Today's hours")).not.toBeInTheDocument();
+    // Six remaining days are closed → six "Closed" line-through entries.
+    expect(screen.getAllByText("Closed")).toHaveLength(6);
   });
 
-  it("does NOT render today's hours card when place.hours is absent", () => {
+  it("does NOT render the hours section when place.schedule is absent", () => {
     render(<PlaceAboutTab place={makePlace()} labels={LABELS} />);
-    expect(screen.queryByText("Today's hours")).not.toBeInTheDocument();
+    expect(screen.queryByText("Hours")).not.toBeInTheDocument();
   });
 
   // ─── amenityLabels fallback to humanizeEnum ──────────────────────────────────
@@ -261,7 +327,6 @@ describe("PlaceAboutTab", () => {
         labels={LABELS}
       />
     );
-    // LABELS.amenityLabels.wifi = "Free Wi-Fi"
     expect(screen.getByText("Free Wi-Fi")).toBeInTheDocument();
   });
 
@@ -272,13 +337,12 @@ describe("PlaceAboutTab", () => {
         labels={LABELS}
       />
     );
-    // humanizeEnum("outdoor_seating") → "outdoor seating"
     expect(screen.getByText("outdoor seating")).toBeInTheDocument();
   });
 
-  // ─── Description card visibility ────────────────────────────────────────────
+  // ─── Description visibility ────────────────────────────────────────────────
 
-  it("renders description card when description is set", () => {
+  it("renders description prose when description is set", () => {
     render(
       <PlaceAboutTab
         place={makePlace({ description: "A great place to eat." })}
@@ -288,7 +352,7 @@ describe("PlaceAboutTab", () => {
     expect(screen.getByText("A great place to eat.")).toBeInTheDocument();
   });
 
-  it("renders description card when generative_summary is set", () => {
+  it("renders description prose when generative_summary is set", () => {
     render(
       <PlaceAboutTab
         place={makePlace({ generative_summary: "AI summary here." })}
@@ -298,7 +362,7 @@ describe("PlaceAboutTab", () => {
     expect(screen.getByText("AI summary here.")).toBeInTheDocument();
   });
 
-  it("renders description card when review_summary is set", () => {
+  it("renders description prose when review_summary is set", () => {
     render(
       <PlaceAboutTab
         place={makePlace({ review_summary: "People love it." })}
@@ -308,18 +372,8 @@ describe("PlaceAboutTab", () => {
     expect(screen.getByText("People love it.")).toBeInTheDocument();
   });
 
-  it("hides description card when none of description/generative_summary/review_summary are set", () => {
+  it("hides the description section heading when no prose and no business chips are present", () => {
     render(<PlaceAboutTab place={makePlace()} labels={LABELS} />);
-    // The card header title is "Description" which is the same as labels.description.
-    // The description card renders labels.description as its CardTitle.
-    // Use a more specific query: look for the card header content.
-    // Since labels.contactAndLocation is always rendered, we need to count occurrences.
-    // "Description" appears in labels.description; if no card exists it should not appear at all.
-    expect(screen.queryByText("A great place to eat.")).not.toBeInTheDocument();
-    // Verify no card title "Description" is rendered (description card uses labels.description as title)
-    // Note: "Description" also equals labels.description label inside the card — use getAllByText and check count
-    const descTexts = screen.queryAllByText("Description");
-    // When description card is hidden, "Description" label should not appear
-    expect(descTexts).toHaveLength(0);
+    expect(screen.queryByText("Description")).not.toBeInTheDocument();
   });
 });