feat: add test for internally connected pins #325
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
| // @ts-nocheck | ||
| import { AutoroutingPipelineDebugger } from "lib/testing/AutoroutingPipelineDebugger" | ||
| import bugReportJson from "./internal-connected-pins.json" | ||
| export default () => { | ||
| return <AutoroutingPipelineDebugger srj={bugReportJson} /> | ||
| } |
There was a problem hiding this comment.
The file name 'Internal-connected-pins.fixture.tsx' violates the file naming consistency rule. The file uses PascalCase ('Internal-connected-pins') which is inconsistent with the kebab-case naming convention used by other files in the project (e.g., 'internal-connected-pins.json' and 'internally-connected-pins.test.tsx'). The file should be renamed to 'internal-connected-pins.fixture.tsx' to maintain consistency with the project's kebab-case naming convention.
Spotted by Graphite Agent (based on custom rule: Custom rule)
Is this helpful? React 👍 or 👎 to let us know.
|
The json doesnt contain offBoardConnectedPointIds |
|
Make sure to skip the test so we dont need to merge on red |
|
This is not the most simplified version i sent you a screenshot of |
|
This PR has been automatically marked as stale because it has had no recent activity. It will be closed if no further activity occurs. |
2 participants
No description provided.