new | ask | show | jobs
girvo 4 months ago  [ - ]

The “change capture”/straight jacket style tests LLMs like to output drive me nuts. But humans write those all the time too so I shouldn’t be that surprised either!

mulmboy 4 months ago  [ - ]

What do these look like?

pmg101 4 months ago  [ - ]

  1. Take every single function, even private ones.
  2. Mock every argument and collaborator.
  3. Call the function.
  4. Assert the mocks were  called in the expected way.
These tests help you find inadvertent changes, yes, but they also create constant noise about changes you intend.
senbrow 4 months ago  [ - ]

These tests also break encapsulation in many cases because they're not testing the interface contract, they're testing the implementation.

ornornor 4 months ago  [ - ]

Juniors on one of the teams I work with only write this kind of tests. It’s tiring, and I have to tell them to test the behaviour, not the implementation. And yet every time they do the same thing. Or rather their AI IDE spits these out.

girvo 4 months ago  [ - ]

You beat me to it, and yep these are exactly it.

“Mock the world then test your mocks”, I’m simply not convinced these have any value at all after my nearly two decades of doing this professionally