Detected duplicate tests

cursorless-dev / cursorless

Don't let the cursor slow you down

https://www.cursorless.org/

MIT License

1.13k stars 79 forks source link

Open AndreasArvidsson opened 3 months ago

AndreasArvidsson commented 3 months ago

Sometimes I'm unsure if it tests already exist and I add another one. It would be beneficial if we could detect this duplication.

For our yaml tests we could the check that the command and initial state were the same

Bonus points if we could normalize setSelection and clearAndSetSelection actions to see we have a cross duplication here

For our scope tests just check that we are testing the same scope facet with the same document content

pokey commented 3 months ago

Sorry can you elaborate? Not sure I'm following

AndreasArvidsson commented 3 months ago

I elaborated on my description in the original post. Better?

pokey commented 3 months ago

Makes more sense. Seems unlikely exact duplicate check would catch much tho?

AndreasArvidsson commented 3 months ago

Probably not, but would be nice to catch them if we have duplication.