Skip to content

tests/server: replace atoi() and atol() with curlx_str_number()#19510

Closed
vszakats wants to merge 12 commits into
curl:masterfrom
vszakats:satoil
Closed

tests/server: replace atoi() and atol() with curlx_str_number()#19510
vszakats wants to merge 12 commits into
curl:masterfrom
vszakats:satoil

Conversation

@vszakats

@vszakats vszakats commented Nov 13, 2025

Copy link
Copy Markdown
Member

@vszakats vszakats added the tests label Nov 13, 2025
@vszakats vszakats changed the title tests/server: replace atoi() with curlx_str_number() tests/server: replace atoi() and atol() with curlx_str_number() Nov 13, 2025
@vszakats vszakats closed this in bb1391f Nov 13, 2025
@vszakats
vszakats deleted the satoil branch November 13, 2025 13:09
vszakats added a commit that referenced this pull request Dec 12, 2025
To detect mistakes made in the runtests framework that reduce
the number of test runs. Before this patch it could go undetected with
a green CI.

The minimum thresholds will need light maintenance going forward (either
bumping them periodically, or adjust if some may fell below minimums for
justified reasons). We may also make minimums tighter or looser, or more
job-specific.

Latest number of test runs for each job can be seen at Test Clutch:
https://testclutch.curl.se/static/reports/feature-matrix.html

Also:
- GHA: set minimums.

Assisted-by: Dan Fandrich

Follow-up to f2a75a1
Follow-up to bb1391f #19510

Closes #19942
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant