search

thomhurst / TUnit

A modern, fast and flexible .NET testing framework
MIT License
1.02k stars 8 forks source link

remove ArgumentsAttribute constructor that always throws #542

Open SimonCropp opened 1 day ago

thomhurst commented 1 day ago

Thanks @SimonCropp - looks like there was an analyzer for this, and now a failing test so that might need to go too

SimonCropp commented 3 hours ago

@thomhurst done

SimonCropp commented 3 hours ago

well i remove the test. do i remove the analyser as well?

thomhurst commented 3 hours ago

well i remove the test. do i remove the analyser as well?

Tbh I wrote that analyser very early on. If it's basically redundant (because the attribute constructor is redundant) then yeah delete it.

I basically want it to throw if no arguments were provided, and it still will do that right?