Skip to content

Ensure MSTest.GlobalConfigsGenerator is built before MSTest.Analyzers.Package #5807

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Youssef1313
Copy link
Member

I think dotnet run could be invoking a build which is racing with the main build of GlobalConfigsGenerator. Adding a ProjectReference to force GlobalConfigsGenerator to be built before MSTest.Analyzers.Package. Then invoking dotnet run with --no-build.

Comment on lines +166 to +168
ProjectSection(ProjectDependencies) = postProject
{A85AA656-6DB6-4A0B-AA80-CBB4058B3DDB} = {A85AA656-6DB6-4A0B-AA80-CBB4058B3DDB}
EndProjectSection
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We only use .sln on Windows currently. So this won't work for Linux/macOS.

testfx/eng/Build.props

Lines 8 to 10 in 7bf53d1

<ItemGroup Condition=" '$(ProductsToBuild)' == 'all' AND '$(OS)' == 'Windows_NT' ">
<ProjectToBuild Include="$(RepoRoot)TestFx.sln" />
</ItemGroup>

@Youssef1313 Youssef1313 marked this pull request as draft June 18, 2025 06:22
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.34%. Comparing base (166a1d1) to head (f229cbd).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5807      +/-   ##
==========================================
- Coverage   73.41%   73.34%   -0.07%     
==========================================
  Files         605      605              
  Lines       37921    37921              
==========================================
- Hits        27839    27814      -25     
- Misses      10082    10107      +25     
Flag Coverage Δ
Debug 73.34% <ø> (-0.07%) ⬇️
integration 73.76% <ø> (+0.35%) ⬆️
production 73.34% <ø> (-0.07%) ⬇️
unit 69.24% <ø> (-0.09%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants