Commit a5d3698
committed
Use XML nextest output instead of ad-hoc parsing
This eliminates the piping, tricky parsing, and special-casing to
preserve colorization, and runs the tests and the XML parsing in
the default `pwsh` shell (since this is a Windows job), using
PowerShell facilities to parse the XML. This also checks that there
are no *errors*, in addition to (still) checking that there are no
more *failures* than expected.
In the preceding commit, five additional tests, not currently noted
in #1358, failed:
FAIL [ 0.010s] gix-credentials::credentials program::from_custom_definition::empty
FAIL [ 0.008s] gix-credentials::credentials program::from_custom_definition::name
FAIL [ 0.010s] gix-credentials::credentials program::from_custom_definition::name_with_args
FAIL [ 0.009s] gix-credentials::credentials program::from_custom_definition::name_with_special_args
FAIL [ 0.014s] gix-discover::discover upwards::from_dir_with_dot_dot
In addition, one test noted in #1358 does not always fail on CI,
because it is a performance test and the CI runner is fast enough
so that it usually passes:
FAIL [ 181.270s] gix-ref-tests::refs packed::iter::performance
Let's see if running the tests more similarly to the way they are
run on Windows without `GIX_TEST_IGNORE_ARCHIVES`, i.e. without
piping and with the Windows default of `pwsh` as the shell, affects
any of those new/CI-specific failures.1 parent 63655d8 commit a5d3698
2 files changed
+6
-11
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
82 | 82 | | |
83 | 83 | | |
84 | 84 | | |
85 | | - | |
86 | | - | |
87 | | - | |
88 | 85 | | |
89 | 86 | | |
90 | 87 | | |
| |||
96 | 93 | | |
97 | 94 | | |
98 | 95 | | |
99 | | - | |
100 | | - | |
| 96 | + | |
101 | 97 | | |
102 | 98 | | |
103 | 99 | | |
104 | 100 | | |
105 | | - | |
106 | 101 | | |
107 | 102 | | |
108 | 103 | | |
109 | 104 | | |
110 | | - | |
111 | | - | |
112 | | - | |
113 | | - | |
114 | | - | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
115 | 108 | | |
116 | 109 | | |
117 | 110 | | |
| |||
0 commit comments