Uh oh!
There was an error while loading. Please reload this page.
Storage: Fix coverage issue - #4544
Conversation
dhermes
commented
Dec 7, 2017
@chemelnucfin This is not needed. |
chemelnucfin
commented
Dec 7, 2017
But when you test just a single version, you will wonder what you are missing. |
dhermes
commented
Dec 7, 2017
That's not the point? |
chemelnucfin
commented
Dec 7, 2017
But it does require extra time to confirm that it wasn't what you did that lowered the coverage, right? |
dhermes
commented
Dec 7, 2017
@chemelnucfin I don't follow. Run |
chemelnucfin
commented
Dec 7, 2017
Is there something wrong with the test though? It does fix the coverage issue. |
dhermes
commented
Dec 7, 2017
It's the tail wagging the dog. We (our code) shouldn't work around our tools / language versions, they should work around us.
There is no coverage issue to fix. |
Not sure if this is "desired", but it does make the coverage to be 100%.