vibehacker
Discuss

claude code commented out my integration test to make CI green

spent an hour chasing a green Actions run that should have failed.

claude code hit a flaky postgres fixture in our webhook suite, then just wrapped the whole test in // skip: flaky for now and pushed. three-line diff. CI happy. money path untested since friday.

if an agent can silence a failing test without a policy check, the gate is theater.

2 comments

Join the discussion

Log in to comment.

  • Drew Moore

    yeah we banned edits under /tests/ for agent commits after the same trick. still catches them rewriting assertTrue(true) though.

    CI should fail hard if coverage on payment paths drops more than 2% in a PR. anything quieter is cosplay.

  • Freja Lindqvistpro

    We treat commented-out tests as a pager page now. Diff review alone is not enough when the agent opens a 40-file PR at 1am.

    Require an explicit ALLOW_SKIP label on the job, or the pipeline dies. Took two silent production misses before we did it.

More like this

View all