339
you are viewing a single comment's thread
view the rest of the comments
[-] willia4@lemmy.world 18 points 1 year ago

TDD is overrated. Code coverage is extremely overrated. Both of these tend to lead to a morass of tests that prove the compiler works at its most basic level while simultaneously generating a surplus of smugness about the whole situation.

Tests have their place. Tests can be, and often are, valuable. But the easier the test is to write, the easier it would've been to just encode it into the type system to begin with.

[-] wolf@lemmy.zip 6 points 1 year ago

TDD as in religion is overrated. TDD done right is IMHO extremely effective.

The problem is, writing good tests is really hard, and I have seen/committed/experienced a lot of bad tests... just the top of my mind problems with TDD done wrong:

  • testing the implementation instead the interface
  • creating a change detector
  • not writing / factoring the tests in a good way
  • writing tests / TDD w/o having an overall design for the software

For every non trivial piece of software written w/o TDD, I always saw the same pattern: First few hours/days/weeks, rapid progress compared to TDD, afterwards: hours/days/weeks wasted in debugging, bug fixing etc... and the people can not even catch up with tests if they wanted.

Is TDD always the answer? Of course not, it is a tradeoff like everything else in technology. OTOH I have yet to see a project which benefited from not using TDD by any metric after a few days in.

[-] bamboo@lemm.ee 4 points 1 year ago

As someone who works on python code and kinda hates it, type annotations and a CI pass for mypy would catch the majority of our bugs. It's painful

[-] SpaceCowboy@lemmy.ca 3 points 1 year ago

Yeah creating tests for every single method is insane. If a feature changes it's more difficult you either have to figure out how to implement the change without changing the method, or you change the method and have update the unit test. But if you're constantly updating the unit tests, how do you know if you might've broken something else that the test was intended for.

It's way better just to do integration tests that match the feature request. That way the feature that someone asked for will continue to work even if you decide to refactor the code.

[-] Alexstarfire@lemmy.world 3 points 1 year ago

Unit tests are only worthwhile if you refractor code or write the unit tests before writing the code. We started adding unit test for most everything where I work and I think it's far more effort than it's worth. It's not that it catches nothing but it catches so little I don't think it's worth the time spent writing them.

[-] Double_A@discuss.tchncs.de 1 points 1 year ago

Code changes that could affect tests happen all the time. It doesn't need to be a specific refactoring of that unit.

[-] Alexstarfire@lemmy.world 1 points 1 year ago

I don't think you understood my point. That's exactly why I think unit tests aren't all that useful. Most code changes require updating the unit tests so unless you change the unit tests first all that's being done is saying, yep this works how I programmed it to work.

[-] Double_A@discuss.tchncs.de 1 points 1 year ago* (last edited 1 year ago)

But if unit tests that other people wrote unexpectedly break, you know that you changed things that you maybe didn't mean to change.

[-] Alexstarfire@lemmy.world 2 points 1 year ago

Ideally. It's just that more often than not it means you need to update the unit tests. I can still use my fingers to count how many times a unit test has caught a mistake I made, and I've been at my job for 10 years.

I'm curious if others have a different experience.

this post was submitted on 01 Sep 2023
339 points (96.2% liked)

Programming

17691 readers
146 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities !webdev@programming.dev



founded 2 years ago
MODERATORS