Skip to content

fix: failing CI-Analysis build when BuildScope is not default#742

Merged
vbreuss merged 1 commit intomainfrom
topic/fix-failing-cianalysis
Aug 26, 2025
Merged

fix: failing CI-Analysis build when BuildScope is not default#742
vbreuss merged 1 commit intomainfrom
topic/fix-failing-cianalysis

Conversation

@vbreuss
Copy link
Copy Markdown
Member

@vbreuss vbreuss commented Aug 26, 2025

This PR fixes a CI-Analysis build failure by adding a conditional execution guard to the mutation testing target. The change ensures mutation tests only run when the BuildScope is set to Default, preventing failures in other build scopes.

Key Changes

  • Added conditional execution logic to the MutationTests target to restrict execution based on BuildScope

@vbreuss vbreuss self-assigned this Aug 26, 2025
Copilot AI review requested due to automatic review settings August 26, 2025 19:27
@vbreuss vbreuss added the bug Something isn't working label Aug 26, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a CI-Analysis build failure by adding a conditional execution guard to the mutation testing target. The change ensures mutation tests only run when the BuildScope is set to Default, preventing failures in other build scopes.

Key Changes

  • Added conditional execution logic to the MutationTests target to restrict execution based on BuildScope

@vbreuss vbreuss merged commit deb356b into main Aug 26, 2025
9 checks passed
@vbreuss vbreuss deleted the topic/fix-failing-cianalysis branch August 26, 2025 19:30
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

🚀 Benchmark Results

Details

BenchmarkDotNet v0.14.0, Ubuntu 24.04.2 LTS (Noble Numbat)
AMD EPYC 7763, 1 CPU, 4 logical and 2 physical cores
.NET SDK 8.0.413
[Host] : .NET 8.0.19 (8.0.1925.36514), X64 RyuJIT AVX2

Job=InProcess Toolchain=InProcessEmitToolchain IterationCount=15
LaunchCount=1 WarmupCount=10

Method Mean Error StdDev Gen0 Gen1 Allocated
Bool_aweXpect 205.8 ns 1.55 ns 1.45 ns 0.0300 - 504 B
Bool_FluentAssertions 246.8 ns 0.75 ns 0.58 ns 0.0567 - 952 B
Bool_TUnit 851.9 ns 4.16 ns 3.69 ns 0.1431 - 2408 B
Equivalency_aweXpect 285,533.7 ns 1,447.21 ns 1,353.72 ns 16.6016 0.4883 284972 B
Equivalency_FluentAssertions 2,201,338.3 ns 11,662.92 ns 9,739.06 ns 273.4375 46.8750 4584416 B
Equivalency_TUnit 674,208.4 ns 2,006.15 ns 1,675.23 ns 51.7578 2.9297 880545 B
Int_GreaterThan_aweXpect 235.8 ns 1.42 ns 1.18 ns 0.0486 - 816 B
Int_GreaterThan_FluentAssertions 241.3 ns 1.53 ns 1.43 ns 0.0730 - 1224 B
Int_GreaterThan_TUnit 1,188.1 ns 15.14 ns 14.16 ns 0.1774 - 2984 B
ItemsCount_AtLeast_aweXpect 490.2 ns 6.03 ns 5.64 ns 0.0868 - 1464 B
ItemsCount_AtLeast_FluentAssertions 498.6 ns 6.58 ns 6.15 ns 0.1192 - 2008 B
ItemsCount_AtLeast_TUnit 17,947.2 ns 297.11 ns 277.92 ns 1.6174 - 27480 B
String_aweXpect 378.2 ns 2.30 ns 2.15 ns 0.0691 - 1160 B
String_FluentAssertions 465.1 ns 5.38 ns 5.03 ns 0.1292 - 2168 B
String_TUnit 1,210.1 ns 6.45 ns 5.72 ns 0.1831 - 3072 B
StringArray_aweXpect 1,390.6 ns 12.25 ns 10.86 ns 0.1659 - 2776 B
StringArray_FluentAssertions 1,257.6 ns 9.09 ns 8.06 ns 0.2480 - 4152 B
StringArray_TUnit 3,243.2 ns 11.24 ns 9.38 ns 0.3128 - 5296 B
StringArrayInAnyOrder_aweXpect 1,640.0 ns 7.56 ns 6.70 ns 0.1755 - 2952 B
StringArrayInAnyOrder_FluentAssertions 149,626.3 ns 612.30 ns 542.78 ns 3.4180 - 63787 B
StringArrayInAnyOrder_TUnit 3,849.7 ns 13.64 ns 11.39 ns 0.3815 - 6488 B

github-actions Bot added a commit that referenced this pull request Aug 26, 2025
github-actions Bot added a commit that referenced this pull request Aug 26, 2025
@github-actions
Copy link
Copy Markdown
Contributor

👽 Mutation Results

Mutation testing badge

aweXpect

Details
File Score Killed Survived Timeout No Coverage Ignored Compile Errors Total Detected Total Undetected Total Mutants

The final mutation score is NaN%

Coverage Thresholds: high:80 low:60 break:0

aweXpect.Core

Details
File Score Killed Survived Timeout No Coverage Ignored Compile Errors Total Detected Total Undetected Total Mutants

The final mutation score is NaN%

Coverage Thresholds: high:80 low:60 break:0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 2, 2025

This is addressed in release v2.22.0.

@github-actions github-actions Bot added the state: released The issue is released label Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working state: released The issue is released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants