Skip to content

Commit

Permalink
ci: fix most of the tests
Browse files Browse the repository at this point in the history
There's still an error with ci.
engineio/src/lib.rs/test/tls_connector doesn't function the same as native-tls one (as it properly validates hostnames)

Needs further investigation as to why the hostname shows as invalid in ci. If we no longer need that, then this PR can be merged.
  • Loading branch information
ctrlaltf24 committed Sep 20, 2024
1 parent 1802e35 commit bc8f6d3
Show file tree
Hide file tree
Showing 11 changed files with 128 additions and 242 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
run: cargo test-all-features --verbose --lib --examples

- name: Run doc tests
run: cargo test-all-features --verbose --lib --examples --doc
run: cargo test-all-features --verbose --doc

- name: Run integration tests
run: cargo test --verbose --features "async" --benches --tests
Loading

0 comments on commit bc8f6d3

Please sign in to comment.