Replaced getByTestId with getByAltText

hackforla · Mar 28, 2024 · 590ac43 · 590ac43
1 parent 676607f
commit 590ac43
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/website/src/entities/branding/ui/logo.test.tsx b/packages/website/src/entities/branding/ui/logo.test.tsx
@@ -10,7 +10,7 @@ test("clicking on the logo redirects to the home page", () => {
     </MemoryRouter>,
   );
 
-  const logo = screen.getByTestId("logo");
+  const logo = screen.getByAltText("Parking Insights brand mark");
   userEvent.click(logo);
 
   expect(window.location.pathname).toBe("/");

diff --git a/packages/website/src/entities/branding/ui/logo.tsx b/packages/website/src/entities/branding/ui/logo.tsx
@@ -7,7 +7,7 @@ export default function Logo() {
      * reloadDocument - A property used in the Link component from React Router to skip client side routing and let the browser handle the transition normally (as if it were an <a href>).
      * @typedef {boolean} reloadDocument
      */
-    <Link to="/" reloadDocument={true} data-testid="logo" className="flex items-center space-x-2">
+    <Link to="/" reloadDocument={true} className="flex items-center space-x-2">
       <img src={BrandMark} alt="Parking Insights brand mark" />
 
       <div className="select-none -space-y-1 font-semibold uppercase text-blue-600">