Skip to content

fix: use "an" before vowel sounds in header and method expectations - #173

Merged
vbreuss merged 2 commits into
mainfrom
fix/indefinite-article-in-expectations
Oct 10, 2026
Merged

vbreuss merged 2 commits into
mainfrom
fix/indefinite-article-in-expectations

Conversation

@vbreuss

@vbreuss vbreuss commented Oct 10, 2026

Copy link
Copy Markdown
Member

Failure messages read "has a Accept header" or "has a OPTIONS method", because the article was hard-coded. It now follows the header name (reading a leading single letter such as in X-Request-Id by its name) and the HTTP method.

Failure messages read "has a `Accept` header" or "has a OPTIONS method", because the article was hard-coded. It now follows the header name (reading a leading single letter such as in `X-Request-Id` by its name) and the HTTP method.
@vbreuss vbreuss self-assigned this Oct 10, 2026
Copilot AI balanced review requested due to automatic review settings October 10, 2026 06:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vbreuss
vbreuss enabled auto-merge (squash) October 10, 2026 06:18
@vbreuss
vbreuss disabled auto-merge October 10, 2026 06:26
@vbreuss
vbreuss enabled auto-merge (squash) October 10, 2026 06:27
Resolves the Sonar readability finding; string.Contains(char) is not available on netstandard2.0, so a pattern match is used instead.
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

Test Results

   19 files  ±  0     19 suites  ±0   15s ⏱️ +2s
  652 tests + 26    651 ✅ + 26  1 💤 ±0  0 ❌ ±0 
3 880 runs  +156  3 879 ✅ +156  1 💤 ±0  0 ❌ ±0 

Results for commit a389481. ± Comparison against base commit 491112b.

@vbreuss
vbreuss merged commit 9a69f71 into main Oct 10, 2026
13 checks passed
@vbreuss
vbreuss deleted the fix/indefinite-article-in-expectations branch October 10, 2026 06:54
@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

🚀 Benchmark Results

Details

BenchmarkDotNet v0.15.8, Linux Ubuntu 24.04.5 LTS (Noble Numbat)
AMD EPYC 9V74 2.60GHz, 1 CPU, 4 logical and 2 physical cores
.NET SDK 10.0.303
[Host] : .NET 8.0.31 (8.0.31, 8.0.3126.42015), X64 RyuJIT x86-64-v4
DefaultJob : .NET 8.0.31 (8.0.31, 8.0.3126.42015), X64 RyuJIT x86-64-v4

Method Mean Error StdDev Gen0 Allocated
Content_aweXpect 382.6 ns 2.70 ns 2.39 ns 0.0634 1064 B
SuccessStatusCode_aweXpect 208.6 ns 2.27 ns 2.12 ns 0.0520 872 B

@github-actions

github-actions Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

👽 Mutation Results

Mutation testing badge

aweXpect.Web

Details
File Score Killed Survived Timeout No Coverage Ignored Compile Errors Runtime Errors Total Detected Total Undetected Total Mutants
Helpers/StringExtensions.cs 92.31% 24 2 0 0 2 1 0 24 2 29
ThatHttpRequestMessage.HasHeader.cs 100.00% 15 0 0 0 2 18 0 15 0 35
ThatHttpRequestMessage.HasMethod.cs 85.71% 18 2 0 1 2 15 0 18 3 38
ThatHttpResponseMessage.HasHeader.cs 100.00% 15 0 0 0 2 18 0 15 0 35

The final mutation score is 93.51%

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

@github-actions

Copy link
Copy Markdown

This is addressed in release v2.0.0.

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

Labels

state: released The issue is released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants