Skip to content

Commit 816a13d

Browse files
committed
Disable WPT workflow
1 parent 3edd11c commit 816a13d

2 files changed

Lines changed: 0 additions & 256 deletions

File tree

.github/workflows/wpt.yml

Lines changed: 0 additions & 153 deletions
This file was deleted.

CLAUDE.md

Lines changed: 0 additions & 103 deletions
Original file line numberDiff line numberDiff line change
@@ -119,109 +119,6 @@ ctest --test-dir _build
119119
endforeach ()
120120
```
121121

122-
## Web Platform Tests (WPT)
123-
124-
**Web Platform Tests** provide conformance testing against the official WhatWG URL specification test suite. Unlike unit tests, WPT generates a **report** showing which edge cases are handled and which are not.
125-
126-
### Philosophy
127-
128-
- **Not pass/fail tests** - WPT is a reporting tool, not a quality gate
129-
- **Edge case discovery** - Shows which obscure URL patterns are supported
130-
- **100% is not the goal** - Some edge cases may be intentionally unsupported if they add complexity or hurt performance
131-
- **Separate from unit tests** - Does not affect CI/PR status
132-
133-
### Building WPT Tests
134-
135-
```bash
136-
cmake \
137-
-B _build \
138-
-G "Ninja" \
139-
-DCMAKE_TOOLCHAIN_FILE=${VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake \
140-
-Dskyr_BUILD_WPT=ON \
141-
.
142-
cmake --build _build
143-
```
144-
145-
**Dependencies**: Requires `nlohmann-json` for parsing test data.
146-
147-
### Running WPT Tests
148-
149-
```bash
150-
# Run with latest test data (downloads from WPT repository)
151-
./_build/tests/wpt/wpt_runner --force-download
152-
153-
# Run with cached data (if less than 24 hours old)
154-
./_build/tests/wpt/wpt_runner
155-
156-
# Specify custom cache location
157-
./_build/tests/wpt/wpt_runner --cache /path/to/urltestdata.json
158-
```
159-
160-
### Report Format
161-
162-
The runner outputs a comprehensive report:
163-
164-
```
165-
=================================================
166-
WPT URL Test Suite Report
167-
=================================================
168-
Test Data: https://github.com/web-platform-tests/...
169-
Generated: 2025-12-30 10:30:00 UTC
170-
171-
SUMMARY:
172-
Total Tests: 1,234
173-
Successful: 1,150 (93.2%)
174-
Failed: 84 (6.8%)
175-
176-
FAILURE BREAKDOWN:
177-
Parse Errors: 45 (53.6% of failures)
178-
Component Mismatch: 39 (46.4% of failures)
179-
180-
SAMPLE FAILURES (first 20):
181-
[1] Input: "http://example.org/foo%2€bar"
182-
Expected: parse failure
183-
Actual: parsed successfully
184-
185-
[2] Input: "http://[::1]:65536/"
186-
Expected: parsed successfully
187-
Actual: parse failure (invalid port)
188-
...
189-
=================================================
190-
```
191-
192-
### Implementation Details
193-
194-
- **Parallel execution**: Uses `std::execution::par_unseq` to process tests in parallel
195-
- **Thread-safe aggregation**: Atomic counters and mutex-protected failure collection
196-
- **Automatic downloads**: Fetches latest `urltestdata.json` from WPT repository
197-
- **Smart caching**: Reuses cached data if less than 24 hours old
198-
199-
### GitHub Actions
200-
201-
WPT runs automatically via `.github/workflows/wpt-integration.yml`:
202-
203-
**Triggers:**
204-
- Push to `main` branch
205-
- Weekly schedule (Mondays at 00:00 UTC)
206-
- Manual workflow dispatch
207-
208-
**Output:**
209-
- Report displayed in workflow log
210-
- Full report uploaded as artifact (90-day retention)
211-
- Does not fail CI - informational only
212-
213-
**Configuration:**
214-
- Single build: Linux GCC 14 Release
215-
- Fast execution: Parallel test processing
216-
- Always downloads latest test data
217-
218-
### Test Data Source
219-
220-
Test data comes from the official WPT repository:
221-
`https://github.com/web-platform-tests/wpt/blob/master/url/resources/urltestdata.json`
222-
223-
This ensures compliance testing against the latest WhatWG URL specification test cases.
224-
225122
## Benchmarks
226123

227124
**Performance benchmarks** measure runtime URL parsing speed to identify optimization opportunities and track performance regressions.

0 commit comments

Comments
 (0)