7
kobenz
2d

I'm rewriting this horrendous piece of software that's like, real ugly, I mean, seriously ugly. A non stop mess of global variables and aliases and logical tantrums and fuck I know what else...

That's until today.

Today, after a ton of time spent clenching 2k LOC of ASS into readable 1.2k LOC, all tests passing, and passing faster if I may...

Then came a horrifying realization...

It downed on me, harder than a naked superman for a young Louis Lane

Tha maybe, just maybe, if them codes 's só shitty then probably so is them tests

So I tried deleting a obviously important conditional and to my dismay... oh boy, tests kept passing

...

Comments
  • 3
    Tests fail => something needs fixing

    Tests pass => { }
  • 5
    Lol. Typical placebo tests. That‘s why chasing a coverage % number is so idiotic.
  • 1
    all I hear is it's MAKEOVER TIME, GIRLIES
  • 0
    @Lensflare coverage % in JavaScript is based on lines that code visited. so if the test is empty it wouldn't have visited the lines

    asserts are another matter tho

    coverage I would assume everywhere would be based on the lines the code visited so other than asserts should work
Add Comment