Closed tbauriedel closed 1 year ago
I will have a look at it this week. Not really sure what these test functions do.
Most (to my knowledge even all) just call their Collect()
function without using the t *testing.T
variable.
For the future, it would be appropriate to expand the tests somewhat at this point (as far as possible and necessary).
We should rather tackle the whole thing directly once cleanly instead of just using the variable somehow. I'll take care of the whole thing.
I updated the PR.
The Test functions now have more to do and use the testing.T object.
Plus some updates to the CI.
Remove unused variables in tests