Dogfood @remix-run/testing on internal tests

[brophdawg11]

Updated implementation of #5529 to dog food the @remix-run/testing package so we can stabilize it. This PR moves integration/meta-test.tsx to packages/remix-react/__tests__/integration/meta-test.tsx and changes it from playwright tests to createRemixStub tests

yarn test:primary packages/remix-react/__tests__/integration/meta-test.tsx

This makes some changes to createRemixStub to permit meta/links implementations on routes. It also combines the manifest/routeModules generation into the patchRoutesWithContext function (renamed processRoutes) so we just walk the route tree one time and generate everything.

I think this is a good starting point to get merged as an example, and then we can keep converting other modules over as we find time.

Notes:

• Internal testing via @remix-run/testing should be mostly (entirely?) for packages/remix-react so I'm just putting tests in packages/remix-react/__tests__/integration for now. This allows our current jest setup to pick them up - simpler is better at this stage IMO.
• No renaming yet of integration -> e2e but once we're comfortable I think it's worth doing that and moving __tests__/integration -> __integration__ in a follow up PR.

Replaces #5529
Closes #5528

[changeset-bot]

🦋 Changeset detected

Latest commit: cd0a518

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 16 packages

Name	Type
@remix-run/testing	Minor
create-remix	Minor
remix	Minor
@remix-run/architect	Minor
@remix-run/cloudflare	Minor
@remix-run/cloudflare-pages	Minor
@remix-run/cloudflare-workers	Minor
@remix-run/css-bundle	Minor
@remix-run/deno	Minor
@remix-run/dev	Minor
@remix-run/eslint-config	Minor
@remix-run/express	Minor
@remix-run/node	Minor
@remix-run/react	Minor
@remix-run/serve	Minor
@remix-run/server-runtime	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[markdalgleish]

Why don't they match?

[brophdawg11]

Remix forces a key/value context object in TS:

interface AppLoadContext {
  [key: string]: unknown;
}

interface DataFunctionArgs {
  request: Request;
  context: AppLoadContext;
  params: Params;
}

But React Router is more lenient since you can realistically pass anything through, it doesn't need to be key/value:

interface DataFunctionArgs {
  request: Request;
  params: Params;
  context?: any;
}

[github-actions]

🤖 Hello there,

We just published version v0.0.0-nightly-49e8da1-20230823 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

[github-actions]

🤖 Hello there,

We just published version 2.0.0-pre.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!