attish/heapkeeper — comment on commit 6b548ddf5161f2302f6a947dbb614fe3496f839e
File:  line 0
By attish:

> My suggestion though is to make get_web_args an almost pure function:

I agree that separating pure functions from parts with outside
depencies or side effects is a great thing.

> If you follow this, you can easily write the tests for 90% of the functionality of this function, and you need the mock trickery only to the test the first two lines.

But in this case, once the mock trickery is in place, it can be used
to easily perform any number of tests on the function. It seems to me
that the only benefit from re-writing the test case would be a shorter
testing function, but, as I understand, when writing test cases, long
functions do not matter at all.