Skip to content

Test-related improvements - #610

Merged
Giulia Stocco (gfs) merged 3 commits into
microsoft:mainfrom
Youssef1313:testing-improvements
May 29, 2025
Merged

Giulia Stocco (gfs) merged 3 commits into
microsoft:mainfrom
Youssef1313:testing-improvements

Conversation

@Youssef1313

@Youssef1313 Youssef Fahmy (Youssef1313) commented May 14, 2025 •

Copy link
Copy Markdown
Member
  • TrxReport and CodeCoverage extensions are already dependencies of MSTest metapackage. So, removed those unnecessary PackageReferences.
  • Using Combinatorial.MSTest to simplify some usages of DataRows (NOTE: it's mostly an adjusted copy of xunit.combinatorial done under the same license, MS-PL).
  • Fix analyzer warnings to use ThrowsExactly
  • Use TestMethodAttribute instead of DataTestMethodAttribute

Dan Fiedler (@danfiedler-msft) Giulia Stocco (@gfs) for review please.

FYI Amaury Levé (@Evangelink) as parts of this PR is a follow-up to #603

@gfs

Copy link
Copy Markdown
Contributor

/azp run

@gfs

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@gfs

Copy link
Copy Markdown
Contributor

SDL Sources failure expected on builds from fork.

@gfs
Giulia Stocco (gfs) merged commit 30c2b5a into microsoft:main May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants