Skip to content

fix(client): gate Docker-only test helpers - #245

Merged
mkatychev merged 1 commit into
bytecodealliance:mainfrom
subotac:fix/no-default-features-test-warnings
Jul 30, 2026
Merged

fix(client): gate Docker-only test helpers#245
mkatychev merged 1 commit into
bytecodealliance:mainfrom
subotac:fix/no-default-features-test-warnings

Conversation

@subotac

@subotac subotac commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Fixes #224.

Gate the Docker-only e2e imports, fixture constant, and TLS helper behind the existing docker-tests feature. This removes warnings from no-default-features test builds without changing Docker-enabled tests.

Testing

  • cargo test -p wasm-pkg-client --no-default-features
  • cargo fmt --all -- --check

@subotac
subotac requested a review from mkatychev as a code owner July 30, 2026 17:59

@mkatychev mkatychev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for fix!

@mkatychev
mkatychev merged commit 3d79978 into bytecodealliance:main Jul 30, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle cargo test -p wasm-pkg-client --no-default-features lint warnings

2 participants