Skip to content

Commit 1304fcc

Browse files
justin808claude
andcommitted
Fix RSC test infrastructure and add TODO explaining temporary disable
- Add TODO message explaining why RSC tests are temporarily disabled (jest.setup.js uses require() incompatible with ESM mode) - Fix utils.test.js import path: testUtils.js -> testUtils.ts (was causing "Cannot find module" error) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent c1f0d54 commit 1304fcc

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

packages/react-on-rails-pro/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"clean": "rm -rf ./lib",
1010
"test": "yarn test:non-rsc && yarn test:rsc",
1111
"test:non-rsc": "jest tests --testPathIgnorePatterns=\".*(RSC|stream|registerServerComponent|serverRenderReactComponent|SuspenseHydration).*\"",
12-
"test:rsc": "exit 0",
12+
"test:rsc": "echo 'TODO: RSC tests disabled - jest.setup.js uses require() incompatible with ESM mode. Re-enable after converting setup to ESM.' && exit 0",
1313
"type-check": "yarn run tsc --noEmit --noErrorTruncation",
1414
"prepare": "[ -f lib/ReactOnRails.full.js ] || (rm -rf ./lib && tsc)",
1515
"prepublishOnly": "yarn run build",

packages/react-on-rails-pro/tests/utils.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { enableFetchMocks } from 'jest-fetch-mock';
22

33
import { fetch } from '../src/utils.ts';
4-
import { createNodeReadableStream, getNodeVersion } from './testUtils.js';
4+
import { createNodeReadableStream, getNodeVersion } from './testUtils.ts';
55

66
enableFetchMocks();
77

0 commit comments

Comments
 (0)