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
test_stars currently tries to avoid test duplication by abstracting a lot away in fixtures and parametrizing the tests. The intention behind this certainly was good!
Unfortunately, it leads to the tests being very convoluted, hard to navigate and it being easy to forget updating everything on changes.
I strongly suggest that we instead use the boilerplate approach. It's more manuel, sure, but GitHub Copilot will take care of most of that and the result will be that we now again what we're testing :D
The text was updated successfully, but these errors were encountered:
test_stars
currently tries to avoid test duplication by abstracting a lot away in fixtures and parametrizing the tests. The intention behind this certainly was good!Unfortunately, it leads to the tests being very convoluted, hard to navigate and it being easy to forget updating everything on changes.
I strongly suggest that we instead use the boilerplate approach. It's more manuel, sure, but GitHub Copilot will take care of most of that and the result will be that we now again what we're testing :D
The text was updated successfully, but these errors were encountered: