MSTest attributes should be sealed in most cases.
Nobody has claimed this yet.
- Dominant language
- C#
- Stars
- 1k
- Forks
- 312
- Avg merge
- 8h 30m
- Merged PRs (30d)
- 469
Description
There is no point in inheriting many of the MSTest attributes and they should be made sealed.
- `DiscoverInternalsAttribute`
- `DoNotParallelizeAttribute`
- `ParallelizeAttribute`
- `TestDataSourceDiscoveryAttribute`
- `WinUITestTargetAttribute`
- `STATestClassAttribute`
- `DataRowAttribute`
- `DataTestMethodAttribute` (attribute should be removed altogether)
- `UITestMethodAttribute` (maybe?)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the declarations for DiscoverInternalsAttribute, DoNotParallelizeAttribute, ParallelizeAttribute, TestDataSourceDiscoveryAttribute, WinUITestTargetAttribute, STATestClassAttribute, DataRowAttribute, DataTestMethodAttribute, and UITestMethodAttribute. Review their inheritance usage and existing tests, then make the requested sealing or removal changes and verify the MSTest test suite passes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100