Skip to content

Commit

Permalink
Exclude performance test(s) from expected failures
Browse files Browse the repository at this point in the history
This omits tests containing `performance` (and not as part of a
larger "word", not even with `_`) from being expected to fail on CI
with `GIX_TEST_IGNORE_ARCHIVES=1` on Windows.

Currently there is one such test listed in GitoxideLabs#1358,
`gix-ref-tests::refs packed::iter::performance`.
  • Loading branch information
EliahKagan committed Nov 8, 2024
1 parent 63473bc commit 0a3f3af
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -119,9 +119,8 @@ jobs:
Out-String |
Select-String -Pattern '(?s)```text\r?\n(.*?)```'
# FIXME: Check that the diff can fail, then filter out performance tests in Where-Object.
$expected_failures = $match_info.Matches.Groups[1].Value -split "`n" |
Where-Object { $_ -match '^\s*FAIL \[' } |
Where-Object { ($_ -match '^\s*FAIL \[') -and ($_ -notmatch '\bperformance\b') } |
ForEach-Object { $_ -replace '^\s*FAIL \[\s*\d+\.\d+s\]\s*', '' -replace '\s+$', '' } |
Sort-Object
Expand Down
3 changes: 1 addition & 2 deletions scratchwork.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,8 @@ $match_info = gh issue --repo GitoxideLabs/gitoxide view $issue --json body --jq
Out-String |
Select-String -Pattern '(?s)```text\r?\n(.*?)```'

# FIXME: Check that the diff can fail, then filter out performance tests in Where-Object.
$expected_failures = $match_info.Matches.Groups[1].Value -split "`n" |
Where-Object { $_ -match '^\s*FAIL \[' } |
Where-Object { ($_ -match '^\s*FAIL \[') -and ($_ -notmatch '\bperformance\b') } |
ForEach-Object { $_ -replace '^\s*FAIL \[\s*\d+\.\d+s\]\s*', '' -replace '\s+$', '' } |
Sort-Object

Expand Down

0 comments on commit 0a3f3af

Please sign in to comment.