Skip to content

Add bugreport46-ac4337 Arduino Uno fixture and skipped test#646

Merged
seveibar merged 1 commit intomainfrom
codex/create-fixture-and-skip-test-for-bug-report
Mar 11, 2026
Merged

Add bugreport46-ac4337 Arduino Uno fixture and skipped test#646
seveibar merged 1 commit intomainfrom
codex/create-fixture-and-skip-test-for-bug-report

Conversation

@seveibar
Copy link
Copy Markdown
Contributor

Motivation

  • Provide a concrete autorouting bug report and a visual fixture to reproduce and debug an edge-case routing failure for the Arduino Uno layout.

Description

  • Add fixtures/bug-reports/bugreport46-ac4337/bugreport46-ac4337-arduino-uno.json containing the simple_route_json for the bug report payload.
  • Add fixtures/bug-reports/bugreport46-ac4337/bugreport46-ac4337-arduino-uno.fixture.tsx that renders <AutoroutingPipelineDebugger srj={bugReportJson.simple_route_json} /> (with // @ts-nocheck).
  • Add tests/bugs/bugreport46-ac4337-arduino-uno.test.ts which imports the fixture JSON and constructs an AutoroutingPipelineSolver to run solver.solve() and compare the final SVG via getLastStepSvg, with the test marked test.skip to avoid breaking CI.

Testing

  • Added a bun:test file at tests/bugs/bugreport46-ac4337-arduino-uno.test.ts but it is test.skip, so it did not run in automated test runs.
  • No other automated tests were modified and no test failures were introduced by these additions.

Codex Task

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
capacity-node-autorouter Ready Ready Preview, Comment Mar 11, 2026 5:51pm

Request Review

@tscircuitbot
Copy link
Copy Markdown

🏃 Benchmark This PR

Run benchmarks by commenting on this PR:

/benchmark [solver-name|all] [scenario-limit] --concurrency <n> --effort <n>

Examples:

  • /benchmark -> AutoroutingPipelineSolver, all scenarios (default concurrency uses the benchmark runner CPU count)
  • /benchmark AutoroutingPipelineSolver -> one solver, all scenarios
  • /benchmark all 20 -> all solvers, first 20 scenarios
  • /benchmark AutoroutingPipelineSolver 20 --concurrency 8 -> one solver, 20 scenarios, 8 workers
  • /benchmark AutoroutingPipelineSolver 20 --effort 2 -> one solver, 20 scenarios, 2x effort

Any PR whose title contains [BENCHMARK TEST] will automatically run the benchmark workflow on PR updates.

@seveibar seveibar force-pushed the codex/create-fixture-and-skip-test-for-bug-report branch from e8f8f22 to f7374ab Compare March 11, 2026 17:49
@seveibar seveibar merged commit 14ec1de into main Mar 11, 2026
13 checks passed
@tscircuitbot
Copy link
Copy Markdown


Thank you for your contribution! 🎉

PR Rating:
Impact: Tiny

Track your contributions and see the leaderboard at: tscircuit Contribution Tracker


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants