• MoonMelon@lemmy.ml
    link
    fedilink
    English
    arrow-up
    27
    arrow-down
    5
    ·
    3 days ago

    I rewrote the rsync test suite in python from the old shell script design. I did the design for that myself (and I’m really quite pleased with it), but used claude with cross-checks from codex and gemini to do the grunt work. I did not just vibe-code “convert test suite to python”… I used AI tools to do the grunt work because they are good at that. I reviewed every part of it myself and ran through a huge amount of CI time getting it right

    If what he claims is true then he’s using LLMs for test coverage with significant editing by hand. I hate LLMs, but even I have to admit this seems like one of the few, valid use cases of LLM assisted coding. Unless “slop” has become one of those words that’s just lost all meaning.

    • diz@awful.systems
      link
      fedilink
      English
      arrow-up
      9
      ·
      edit-2
      2 days ago

      It’s a perfect example of how “using LLMs for test coverage” can also be harmful. He expected the tests to to prevent introduction of said regressions, probably based on a combination of the quantity of tests and their style (they look like what decent human written tests look like). But the tests are AI slop, and so they give a lot less value per line of code than he expects, hence a significant regression.

      It is literally useful to call these tests AI slop, and the problem is in part caused by not calling them AI slop, and having consequent inflated expectations. LLMs are not any better at writing tests than at writing other code! It is merely that the bar for tests can, legitimately, be a lot lower (in projects where there would otherwise be no tests at all). Making an exception to calling AI generated tests “slop” is thus counter productive, because it leads people to act as if LLMs are actually better at writing tests than at writing other code, and not just because the bar for tests is frequently very low.

      edit: actually scratch that I looked at the PR and those tests even look like dogshit and worse than the tests I seen claude write at a workplace that was into vibecoding (which i since quit).

      • MoonMelon@lemmy.ml
        link
        fedilink
        English
        arrow-up
        15
        ·
        3 days ago

        I don’t know anything about rsync aside from as a user, but I am pretty experienced with Python and I admit those tests look really bizarre. If he did “slot machine” code it (a term I wasn’t familiar with) then yeah, I agree that’s slop. If he didn’t, I don’t understand why he made these changes. OK yeah, that’s a bad sign.