Scrub skipped test issue numbers

dotnet / efcore

EF Core is a modern object-database mapper for .NET. It supports LINQ queries, change tracking, updates, and schema migrations.

https://docs.microsoft.com/ef/

MIT License

13.66k stars 3.15k forks source link

Scrub skipped test issue numbers #17235

Open ajcvickers opened 5 years ago

ajcvickers commented 5 years ago

Scrub all skipped tests to:

Determine if it is useful to keep the test
If so, then make sure an open issue number is referenced, and the issue is appropriate for the test being disabled
Skip the test using a common pattern
- Add API consistency tests to ensure that tests are using the correct pattern
- Consider automating checks so that tests are flagged if they are disabled for a closed issue

smitpatel commented 3 years ago

I propose we do this exercise once we have good progress to zero in ZBB. Many of the tests are disabled with tracking bugs which are in 6.0. We could save some cycles if we just fixed bugs first.

AndriySvyryd commented 3 years ago

Agree, there are plenty of other MQ tasks to tackle