You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have always favored creativity over test coverage perfection. Putting coverage tests on any project is new to me. I will not encourage or discourage that. :)
You are mostly right about coverage, when it is your own code. When you try and write tests for code you didn't write, coverage is invaluable. It gives you a lot of guidance and areas of testing you are missing.
I know TDD mandates that all tests should be written before the code is written. In my world the code exists already and the tests usually come in months later.
Any way the mandate here is to enable the act, not do the act. (though I may add coverage to all the code in this repo)
Code cover in this project is slightly irrelevant. Consider removing.
When this code is being used to enable addon testing, it is the addon that needs the code coverage, not anything from here.
It's presence here might be confusing.
It should definitely be in the new form blender-fake-addon.
Thought?
The text was updated successfully, but these errors were encountered: