fix: error in build pipeline by vbreuss · Pull Request #720 · Testably/aweXpect

vbreuss · 2025-08-13T16:37:00Z

Only write PR.txt when the build runs as part of a pull request
Execute the MutationTestsComment in the CI-Analysis step
Extract artifacts to separate directories so as not to overwrite the mutation-report.json files

Copilot

Pull Request Overview

This PR fixes an error in the build pipeline by updating the condition for writing the PR.txt file to only occur during actual pull request builds, rather than checking for a non-null pull request number.

Updates the condition from checking if PullRequestNumber is not null to using the IsPullRequest property
Adds logging to indicate when the PR.txt file is being written
Uses null-safe string conversion for the pull request number

github-actions · 2025-08-13T16:43:29Z

Test Results

14 files - 24 14 suites - 24 2m 47s ⏱️ -14s
14 358 tests - 21 14 356 ✅ - 21 2 💤 ±0 0 ❌ ±0
39 756 runs - 72 39 754 ✅ - 72 2 💤 ±0 0 ❌ ±0

Results for commit d5d016b. ± Comparison against base commit 6c33916.

This pull request removes 1553 and adds 1532 tests. Note that renamed tests count towards both.

aweXpect.Core.Tests.Core.Exceptions.FailExceptionTests ‑ Message_ShouldBeSet(message: "message0ab3fe55-f191-4da8-aee4-13c1239dbe49")
aweXpect.Core.Tests.Core.Exceptions.FailExceptionTests ‑ Message_ShouldBeSet(message: "message15b3f039-856d-4df4-82df-a10eb9e652ed")
aweXpect.Core.Tests.Core.Exceptions.FailExceptionTests ‑ Message_ShouldBeSet(message: "message5d9da588-f415-48d6-bdea-a722243e3015")
aweXpect.Core.Tests.Core.Exceptions.SkipExceptionTests ‑ Message_ShouldBeSet(message: "message1976e2cd-f7fb-4d19-9927-56a8237021e5")
aweXpect.Core.Tests.Core.Exceptions.SkipExceptionTests ‑ Message_ShouldBeSet(message: "message9211ef99-e9f3-4de8-943a-8cc07f37ed13")
aweXpect.Core.Tests.Core.Exceptions.SkipExceptionTests ‑ Message_ShouldBeSet(message: "messagecb907f6b-4a24-45a1-9d3b-22300c391fe6")
aweXpect.Core.Tests.Equivalency.EquivalencyOptionsExtensionsTests ‑ Generic_For_IgnoringMember_ShouldSetOptionForType(memberToIgnore: "memberToIgnore05a8652d-e64a-4b48-85b1-eba1d051a0cc")
aweXpect.Core.Tests.Equivalency.EquivalencyOptionsExtensionsTests ‑ Generic_For_IgnoringMember_ShouldSetOptionForType(memberToIgnore: "memberToIgnore5914fa12-8623-4ab1-a425-69ad20aa1dbc")
aweXpect.Core.Tests.Equivalency.EquivalencyOptionsExtensionsTests ‑ Generic_For_IgnoringMember_ShouldSetOptionForType(memberToIgnore: "memberToIgnore78dcb92e-d58a-40a8-b90d-48bae1d77e8f")
aweXpect.Core.Tests.FailTests ‑ Test_ShouldThrowException(reason: "reason365735b3-470a-4a46-9e99-a37e2d4d6db4")
…

aweXpect.Core.Tests.Core.Exceptions.FailExceptionTests ‑ Message_ShouldBeSet(message: "message816237ba-be26-4cb1-ba98-08d813305179")
aweXpect.Core.Tests.Core.Exceptions.FailExceptionTests ‑ Message_ShouldBeSet(message: "messageadb6e52c-9072-4bbf-9fc1-c45378f01e15")
aweXpect.Core.Tests.Core.Exceptions.FailExceptionTests ‑ Message_ShouldBeSet(message: "messaged98b8b22-065e-441e-83c6-30bb8ff25dbd")
aweXpect.Core.Tests.Core.Exceptions.SkipExceptionTests ‑ Message_ShouldBeSet(message: "message6978a21e-5ca0-4b28-b19e-a5788424862d")
aweXpect.Core.Tests.Core.Exceptions.SkipExceptionTests ‑ Message_ShouldBeSet(message: "messageb6b69f5b-0414-4850-869b-2ef01f06be51")
aweXpect.Core.Tests.Core.Exceptions.SkipExceptionTests ‑ Message_ShouldBeSet(message: "messagede3ffddd-7c21-42b1-8ff3-ae1c865f0cb6")
aweXpect.Core.Tests.Equivalency.EquivalencyOptionsExtensionsTests ‑ Generic_For_IgnoringMember_ShouldSetOptionForType(memberToIgnore: "memberToIgnore5868d19f-ae75-42ce-9b31-6acc5d11f757")
aweXpect.Core.Tests.Equivalency.EquivalencyOptionsExtensionsTests ‑ Generic_For_IgnoringMember_ShouldSetOptionForType(memberToIgnore: "memberToIgnore69d09761-2b3f-40d6-9462-a2b624e0ea99")
aweXpect.Core.Tests.Equivalency.EquivalencyOptionsExtensionsTests ‑ Generic_For_IgnoringMember_ShouldSetOptionForType(memberToIgnore: "memberToIgnorefc19f5d1-381a-4fad-9b19-67d15d652216")
aweXpect.Core.Tests.FailTests ‑ Test_ShouldThrowException(reason: "reason2e335c96-03a4-4cad-9ff7-3f50fac51de9")
…

♻️ This comment has been updated with latest results.

github-actions · 2025-08-13T16:47:34Z

🚀 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	183.9 ns	0.86 ns	0.76 ns	0.0281	-	472 B
Bool_FluentAssertions	244.7 ns	1.21 ns	1.07 ns	0.0567	-	952 B
Bool_TUnit	885.8 ns	3.58 ns	3.18 ns	0.1431	-	2408 B
Equivalency_aweXpect	288,209.6 ns	1,089.23 ns	965.58 ns	16.6016	0.4883	284940 B
Equivalency_FluentAssertions	2,228,961.6 ns	7,966.08 ns	7,451.47 ns	273.4375	46.8750	4584416 B
Equivalency_TUnit	676,448.0 ns	2,320.58 ns	2,057.13 ns	51.7578	2.9297	880545 B
Int_GreaterThan_aweXpect	209.0 ns	0.82 ns	0.72 ns	0.0467	-	784 B
Int_GreaterThan_FluentAssertions	242.4 ns	1.37 ns	1.28 ns	0.0730	-	1224 B
Int_GreaterThan_TUnit	1,174.3 ns	15.26 ns	12.74 ns	0.1774	-	2984 B
ItemsCount_AtLeast_aweXpect	501.4 ns	1.27 ns	1.12 ns	0.0849	-	1432 B
ItemsCount_AtLeast_FluentAssertions	481.2 ns	1.16 ns	0.97 ns	0.1192	-	2008 B
ItemsCount_AtLeast_TUnit	15,206.8 ns	79.68 ns	70.63 ns	1.6174	-	27480 B
String_aweXpect	344.9 ns	2.39 ns	2.12 ns	0.0672	-	1128 B
String_FluentAssertions	462.8 ns	2.04 ns	1.70 ns	0.1292	-	2168 B
String_TUnit	1,209.0 ns	6.50 ns	6.08 ns	0.1831	-	3072 B
StringArray_aweXpect	1,342.5 ns	7.09 ns	6.63 ns	0.1640	-	2744 B
StringArray_FluentAssertions	1,278.9 ns	5.48 ns	5.13 ns	0.2480	-	4152 B
StringArray_TUnit	3,172.4 ns	6.75 ns	5.98 ns	0.3128	-	5296 B
StringArrayInAnyOrder_aweXpect	1,513.8 ns	4.14 ns	3.67 ns	0.1736	-	2920 B
StringArrayInAnyOrder_FluentAssertions	154,346.4 ns	592.94 ns	554.63 ns	3.4180	-	63787 B
StringArrayInAnyOrder_TUnit	3,773.0 ns	14.93 ns	13.23 ns	0.3853	-	6488 B

- Only write PR.txt when the build runs as part of a pull request - Execute the MutationTestsComment in the CI-Analysis step - Extract artifacts to separate directories so as not to overwrite the mutation-report.json files

sonarqubecloud · 2025-08-13T17:25:57Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

…Valentin Breuß

github-actions · 2025-09-02T10:03:34Z

This is addressed in release v2.22.0.

vbreuss self-assigned this Aug 13, 2025

CopilotAI review requested due to automatic review settings August 13, 2025 16:37

vbreuss added the bug Something isn't working label Aug 13, 2025

CopilotAI reviewed Aug 13, 2025
View reviewed changes

Comment threadPipeline/Build.MutationTests.cs

fix: error in build pipeline
d5d016b
- Only write PR.txt when the build runs as part of a pull request - Execute the MutationTestsComment in the CI-Analysis step - Extract artifacts to separate directories so as not to overwrite the mutation-report.json files

vbreuss force-pushed the topic/fix-main-pipeline branch from ca37acc to d5d016bCompare August 13, 2025 17:21

vbreuss enabled auto-merge (squash) August 13, 2025 17:21

vbreuss merged commit e13592d into mainAug 13, 2025
14 checks passed

vbreuss deleted the topic/fix-main-pipeline branch August 13, 2025 17:30

github-actionsBot added a commit that referenced this pull request Aug 13, 2025
Update benchmark fore13592d: fix: error in build pipeline (#720) by …
9376e35
…Valentin Breuß

github-actionsBot added a commit that referenced this pull request Aug 13, 2025
Update benchmark fore13592d: fix: error in build pipeline (#720) by …
2e57ff8
…Valentin Breuß

github-actionsBot added the state: released The issue is released label Sep 2, 2025

This was referenced Sep 15, 2025
chore: Bump the awexpect group with 1 update Testably/aweXpect.Migration#74
Closed
chore: Bump aweXpect and aweXpect.Core Testably/aweXpect.Testably#94
Closed
chore: Bump the awexpect group with 1 update Testably/aweXpect.Migration#75
Closed

This was referenced Sep 29, 2025
chore: Bump the awexpect group with 2 updates Testably/aweXpect.Testably#96
Closed
chore: Bump aweXpect and aweXpect.Core Testably/aweXpect.Testably#97
Closed

This was referenced Oct 7, 2025
chore: Bump the awexpect group with 2 updates Testably/aweXpect.Mockolate#3
Closed
chore: Bump the awexpect group with 2 updates Testably/aweXpect.Testably#100
Closed
chore: Bump the awexpect group with 1 update Testably/aweXpect.Migration#77
Merged

dependabotBot mentioned this pull request Oct 27, 2025
chore: Bump aweXpect from 2.18.0 to 2.27.1 Testably/aweXpect.Chronology#76
Closed

dependabotBot mentioned this pull request Feb 9, 2026
chore: Bump aweXpect from 2.18.0 to 2.30.0 Testably/aweXpect.Chronology#84
Closed

This was referenced Mar 30, 2026
chore: Bump aweXpect from 2.18.0 to 2.31.0 Testably/aweXpect.Chronology#92
Merged
chore: Bump the awexpect group with 2 updates Testably/Mockolate.Migration#7
Closed

Navigation Menu

Uh oh!

Uh oh!

fix: error in build pipeline - #720

fix: error in build pipeline#720
vbreuss merged 1 commit into
mainfrom
topic/fix-main-pipeline

vbreuss commented Aug 13, 2025•
edited
Loading

Uh oh!

CopilotAI left a comment

Uh oh!

Uh oh!

github-actionsBot commented Aug 13, 2025•
edited
Loading

Uh oh!

github-actionsBot commented Aug 13, 2025•
edited
Loading

Uh oh!

sonarqubecloudBot commented Aug 13, 2025

Uh oh!

Uh oh!

github-actionsBot commented Sep 2, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Uh oh!

Conversation

vbreuss commented Aug 13, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

github-actionsBot commented Aug 13, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

github-actionsBot commented Aug 13, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🚀 Benchmark Results

Uh oh!

sonarqubecloudBot commented Aug 13, 2025

Quality Gate passed

Uh oh!

Uh oh!

github-actionsBot commented Sep 2, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vbreuss commented Aug 13, 2025•
edited
Loading

github-actionsBot commented Aug 13, 2025•
edited
Loading

github-actionsBot commented Aug 13, 2025•
edited
Loading