-
Notifications
You must be signed in to change notification settings - Fork 10
Add Phase 5 cross-adapter verification suite (PR 18) #725
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: feature/edgezero-pr17-cloudflare-adapter
Are you sure you want to change the base?
Changes from 28 commits
a499f93
ede4e81
acd3f5d
7efa2ff
69d3693
75fac49
e891f5e
72be307
4f79021
040a77b
1e2979d
0e4d2ea
24f122f
99967c8
8a509ac
c114209
84cbdfe
95fde69
0b2228c
fdef759
5123bd4
641bfde
d7670aa
b6a8894
a521479
5f91da2
c0f2eec
161c41e
70def67
254bd2e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -74,6 +74,10 @@ jobs: | |
| - name: Run Axum adapter tests | ||
| run: cargo test-axum | ||
|
|
||
| - name: Run HTML processor benchmarks (smoke) | ||
| # -- --test runs each benchmark as a regular test (no timing harness) so CI stays fast | ||
| run: cargo bench -p trusted-server-core --bench html_processor_bench -- --test | ||
|
|
||
| test-cloudflare: | ||
| name: cargo check (cloudflare native + wasm32-unknown-unknown) | ||
| runs-on: ubuntu-latest | ||
|
|
@@ -101,6 +105,30 @@ jobs: | |
| - name: Run Cloudflare adapter tests (native host) | ||
| run: cargo test-cloudflare | ||
|
|
||
| test-parity: | ||
| name: cargo test (cross-adapter parity) | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Retrieve Rust version | ||
| id: rust-version | ||
| run: echo "rust-version=$(grep '^rust ' .tool-versions | awk '{print $2}')" >> $GITHUB_OUTPUT | ||
| shell: bash | ||
|
|
||
| - name: Set up Rust toolchain | ||
| uses: actions-rust-lang/setup-rust-toolchain@v1 | ||
| with: | ||
| toolchain: ${{ steps.rust-version.outputs.rust-version }} | ||
| components: clippy | ||
| cache-shared-key: cargo-${{ runner.os }} | ||
|
|
||
| - name: Run cross-adapter parity tests | ||
| run: cargo test --manifest-path crates/integration-tests/Cargo.toml --test parity | ||
|
|
||
| - name: Clippy (parity test crate) | ||
| run: cargo clippy --manifest-path crates/integration-tests/Cargo.toml -- -D warnings | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🔧 wrench — Clippy gate runs against nothing.
There is also a second issue: Suggested fix: run: cargo clippy --manifest-path crates/integration-tests/Cargo.toml --all-targets -- -D warningsAdditionally, add The prior reviewer flagged the |
||
|
|
||
| test-typescript: | ||
| name: vitest | ||
| runs-on: ubuntu-latest | ||
|
|
||
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Uh oh!
There was an error while loading. Please reload this page.