Khafra
Khafra
100% test coverage and validating logic are mutually exclusive issues, the latter one I have no problems with. I see it as not only a false sense of security, but...
I consider tests that exist for the sake of code coverage erroneous and obsolete 😉
we've talked about it in the past and there is no good way to support it
> Then we should document it properly. Definitely > Is it not a good way to support it, because it would be too complex or, or because it would eat...
If we can kill two birds with one stone, sounds great to me.
I won't implement it into fetch until it becomes standardized. I don't think node should land it either until then.
> almost always a good idea There won't be anything problematic with it 😄.
it did land in the spec already and I've opened a PR to add it to fetch
I took a look at as many uses of test_blob and test_blob_binary as I could find and there didn't seem to be anything special about the tests that isn't covered...
We should add in your test case since our tests did not catch the bug nor the fix.