fix: use screen whereever possible 🔥

Author: tigerabrodi

package-lock.json

@@ -1,6 +1,6 @@
 import "@testing-library/jest-dom/extend-expect";
 import { createSignal, createEffect } from "solid-js";
-import { fireEvent, render } from "..";
+import { fireEvent, render, screen } from "..";
 
 declare global {
   var _$HYDRATION: Record<string, any>;
@@ -24,9 +24,9 @@ test("render calls createEffect immediately", () => {
 test("findByTestId returns the element", async () => {
   let ref!: HTMLDivElement;
 
-  const { findByTestId } = render(() => <div ref={ref} data-testid="foo" />);
+  render(() => <div ref={ref} data-testid="foo" />);
 
-  expect(await findByTestId("foo")).toBe(ref);
+  expect(await screen.findByTestId("foo")).toBe(ref);
 });
 
 test("fireEvent triggers createEffect calls", () => {

src/__tests__/basic.tsx

@@ -13,9 +13,9 @@ afterEach(() => {
 test("debug pretty prints the container", () => {
   const HelloWorld = () => <h1>Hello World</h1>;
 
-  const { debug } = render(() => <HelloWorld />);
+  render(() => <HelloWorld />);
 
-  debug();
+  screen.debug();
 
   expect(console.log).toHaveBeenCalledTimes(1);
   expect(console.log).toHaveBeenCalledWith(expect.stringContaining("Hello World"));
@@ -29,9 +29,9 @@ test("debug pretty prints multiple containers", () => {
     </>
   );
 
-  const { debug } = render(() => <HelloWorld />);
+  render(() => <HelloWorld />);
   const multipleElements = screen.getAllByTestId("testId");
-  debug(multipleElements);
+  screen.debug(multipleElements);
   expect(console.log).toHaveBeenCalledTimes(2);
   expect(console.log).toHaveBeenCalledWith(expect.stringContaining("Hello World"));
 });

src/__tests__/debug.tsx

@@ -1,5 +1,5 @@
 import { createSignal, onCleanup } from "solid-js";
-import { fireEvent, render } from "..";
+import { fireEvent, screen, render } from "..";
 
 function StopWatch() {
   const [lapse, setLapse] = createSignal(0);
@@ -38,9 +38,9 @@ const wait = (time: number) => new Promise(resolve => setTimeout(resolve, time))
 test("unmounts a component", async () => {
   jest.spyOn(console, "error").mockImplementation(() => {});
 
-  const { unmount, getByText, container } = render(() => <StopWatch />);
+  const { unmount, container } = render(() => <StopWatch />);
 
-  fireEvent.click(getByText("Start") as Element);
+  fireEvent.click(screen.getByText("Start") as Element);
 
   unmount();