test(replace-remix-imports): fix shell.grep call

previously, `shell.grep` was being called with the `-r` option, which it does not recognize,
resulting in an exit ode of 1. we incorrectly interpreted that as `grep` finishing the search
without finding matches. instead, the `shell.grep` call was crashing. now we correctly pass a list
of files to `shell.grep` without the `-r` option.

packages/remix-dev/__tests__/replace-remix-imports-test.ts

@@ -4,8 +4,10 @@ import os from "os";
 import stripAnsi from "strip-ansi";
 import type { PackageJson } from "type-fest";
 import shell from "shelljs";
+import glob from "fast-glob";
 
 import { run } from "../cli/run";
+import { readConfig } from "../config";
 
 let output: string;
 const ORIGINAL_IO = {
@@ -72,6 +74,7 @@ const replaceRemixImports = async (projectDir: string) => {
 describe("`replace-remix-imports` migration", () => {
   it("runs successfully", async () => {
     let projectDir = makeApp();
+    let config = await readConfig(projectDir);
     await replaceRemixImports(projectDir);
 
     expect(output).toContain("detected `@remix-run/node`");
@@ -96,9 +99,15 @@ describe("`replace-remix-imports` migration", () => {
     expect(packageJson.scripts).not.toContain("postinstall");
 
     expect(output).toContain("✅ Your Remix imports look good!");
-    let { code } = shell.grep("-nri", 'from "remix"', projectDir);
-    // `grep` exits with status code `1` when no matches are found
-    expect(code).toBe(1);
+
+    let files = glob.sync("**/*.@(ts|tsx|js|jsx)", {
+      cwd: config.appDirectory,
+      absolute: true,
+    });
+    let result = shell.grep("-l", 'from "remix"', files);
+    expect(result.stdout.trim()).toBe("");
+    expect(result.stderr).toBeNull();
+    expect(result.code).toBe(0);
 
     expect(output).toContain("successfully migrated");
     expect(output).toContain("npm install");