@snapdragonsnursery/react-components 1.2.0 → 1.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@snapdragonsnursery/react-components",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "description": "",
   "main": "src/index.js",
   "scripts": {

package/src/components/ChildSearchFilters.test.jsx

@@ -1,5 +1,5 @@
 // ChildSearchFilters Component Tests
-// Tests the filter functionality including status, site, date of birth, age, and sorting options
+// Tests the filter functionality including status, site, date of birth, and age options
 
 import React from "react";
 import { render, screen, fireEvent, waitFor } from "@testing-library/react";
@@ -68,8 +68,6 @@ describe("ChildSearchFilters", () => {
     dobTo: "",
     ageFrom: "",
     ageTo: "",
-    sortBy: "last_name",
-    sortOrder: "asc",
   };
 
   const mockSites = [
@@ -107,8 +105,6 @@ describe("ChildSearchFilters", () => {
     expect(screen.getByText("Date of Birth Range")).toBeInTheDocument();
     expect(screen.getByText("Age From (months)")).toBeInTheDocument();
     expect(screen.getByText("Age To (months)")).toBeInTheDocument();
-    expect(screen.getByText("Sort By")).toBeInTheDocument();
-    expect(screen.getByText("Sort Order")).toBeInTheDocument();
   });
 
   it("hides advanced filters when isAdvancedFiltersOpen is false", () => {
@@ -157,7 +153,7 @@ describe("ChildSearchFilters", () => {
     );
     
     const selects = screen.getAllByTestId("select");
-    expect(selects).toHaveLength(3); // Status, Sort By, Sort Order (no site filter when sites=null)
+    expect(selects).toHaveLength(1); // Only Status filter (no site filter when sites=null)
   });
 
   it("shows apply button when filters are changed", () => {
@@ -220,22 +216,6 @@ describe("ChildSearchFilters", () => {
     );
   });
 
-  it("calls onApplyFilters when apply button is clicked with sort filter", () => {
-    render(<ChildSearchFilters {...defaultProps} isAdvancedFiltersOpen={true} />);
-    
-    const sortBySelect = screen.getAllByTestId("select")[2];
-    fireEvent.change(sortBySelect, { target: { value: "first_name" } });
-    
-    const applyButton = screen.getByText("Apply Filters");
-    fireEvent.click(applyButton);
-    
-    expect(defaultProps.onApplyFilters).toHaveBeenCalledWith(
-      expect.objectContaining({
-        sortBy: "first_name"
-      })
-    );
-  });
-
   it("displays correct filter values", () => {
     const filtersWithValues = {
       ...mockFilters,
@@ -245,8 +225,6 @@ describe("ChildSearchFilters", () => {
       dobTo: "2020-12-31",
       ageFrom: "12",
       ageTo: "60",
-      sortBy: "first_name",
-      sortOrder: "desc",
     };
 
     render(
@@ -262,16 +240,12 @@ describe("ChildSearchFilters", () => {
     const dateRangePicker = screen.getByTestId("date-range-picker");
     const ageFromInput = screen.getAllByTestId("input")[0];
     const ageToInput = screen.getAllByTestId("input")[1];
-    const sortBySelect = screen.getAllByTestId("select")[2];
-    const sortOrderSelect = screen.getAllByTestId("select")[3];
     
     expect(statusSelect).toHaveValue("inactive");
     expect(siteSelect).toHaveValue("1");
     expect(dateRangePicker).toBeInTheDocument();
     expect(ageFromInput).toHaveValue(12);
     expect(ageToInput).toHaveValue(60);
-    expect(sortBySelect).toHaveValue("first_name");
-    expect(sortOrderSelect).toHaveValue("desc");
   });
 
   it("has correct accessibility attributes", () => {
@@ -283,8 +257,6 @@ describe("ChildSearchFilters", () => {
     expect(screen.getByText("Date of Birth Range")).toBeInTheDocument();
     expect(screen.getByText("Age From (months)")).toBeInTheDocument();
     expect(screen.getByText("Age To (months)")).toBeInTheDocument();
-    expect(screen.getByText("Sort By")).toBeInTheDocument();
-    expect(screen.getByText("Sort Order")).toBeInTheDocument();
   });
 
   it("applies correct CSS classes for dark mode support", () => {