Skip to content

Commit 3a8c3ac

Browse files
committed
Update workflows, add summary
Remove --safe option from WPT workflows (it's no longer valid) Include a total test/case summary when --summary or --text (default) is used.
1 parent 1ccc879 commit 3a8c3ac

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

.github/workflows/wpt.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ jobs:
5555

5656
- uses: ./.github/actions/install
5757

58-
- run: zig build wpt -- --safe --summary
58+
- run: zig build wpt -- --summary
5959

6060
# For now WPT tests doesn't pass at all.
6161
# We accept then to continue the job on failure.
@@ -80,7 +80,7 @@ jobs:
8080
- uses: ./.github/actions/install
8181

8282
- name: json output
83-
run: zig build wpt -- --safe --json > wpt.json
83+
run: zig build wpt -- --json > wpt.json
8484

8585
- name: write commit
8686
run: |

src/main_wpt.zig

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,13 @@ const Writer = struct {
203203
fn finalize(self: *Writer) !void {
204204
if (self.format == .json) {
205205
try self.out.writeByte(']');
206+
} else {
207+
try self.out.print("\n==Summary==\nTests: {d}/{d}\nCases: {d}/{d}\n", .{
208+
self.pass_count,
209+
self.pass_count + self.fail_count,
210+
self.case_pass_count,
211+
self.case_pass_count + self.case_fail_count,
212+
});
206213
}
207214
}
208215

0 commit comments

Comments
 (0)