How to avoid brittle code

Re:How to avoid brittle code by Bahamut on Monday March 28, 2016 @09:48AM (#11374210)

The main benefit of unit tests is the speed one can run them at, and the feedback. Unit tests make it vastly faster to figure out the culprit in a broken sector of code almost all the time, since it narrows down what code you have to check for any logic errors. It also so happens that unit tests tend to encourage better writing of code into reuseable units.

Unit tests cannot catch everything though, and for stuff like how multiple units are wired together, that's where integration tests come into play.

A good codebase has both types - even combined they won't catch everything, but they should capture almost all business logic issues.

[ Parent | on HN ]

Re:How to avoid brittle code by crdoconnor on Monday March 28, 2016 @03:07PM (#11376538)

I've found that where code involves complex logic, unit tests often are more effective for the reasons you gave.

However, a lot (perhaps most) code is more about integrating disparate components (browsers, lots of different kinds of external APIs, databases, times, distributed task processors, etc.) rather than lots of complex logic and for that kind of code, unit tests are substantially less effective and more horrible to use whereas integration tests really shine.

I think ideally, you should keep the two kinds of code strongly segregated, with clean APIs between them, so you can easily use the appropriate kind of testing for either one.

I don't agree that a good code base necessarily has both types. A web app with relatively shallow logic and lots of integrations only really benefits from having integration tests. A library to convert human readable strings into dates likewise gains nothing from having integration tests.

Similarly, a messy spaghetti codebase with a disgusting mix of both types of code can only really benefit from integration testing until it's substantially refactored and the two types of code are segregated.

[ Parent | on HN ]

Re:How to avoid brittle code by dennisgorelik on Tuesday March 29, 2016 @12:14AM (#11379455)

> you should keep the two kinds of code strongly segregated, with clean APIs between them

Why _strongly_ segregated?

Nothing stops you to write unit tests for the methods that should be covered by unit tests and integration tests for the functionality that needs to be covered by integration tests.

How would code segregation help?

[ Parent | on HN ]

Re:How to avoid brittle code by crdoconnor on Tuesday March 29, 2016 @06:11AM (#11380501)

If the code is not well segregated you won't be able to write a clean unit test that tests the complex logic without also mocking a bunch of stuff.

That test will also often break as you refactor the various connecting APIs it surrounds.

Tests that break in the presence of changed API contracts (as opposed to bugs) are vastly less valuable than tests which only go red in the presence of a bug.

[ Parent | on HN ]