Skip to content

refactor(whatsrust): remove residual facade test copies - #335

Merged
Andiveli merged 1 commit into
betafrom
refactor/whatsrust-facade-final
Aug 27, 2026
Merged

refactor(whatsrust): remove residual facade test copies#335
Andiveli merged 1 commit into
betafrom
refactor/whatsrust-facade-final

Conversation

@Andiveli

@Andiveli Andiveli commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • Complete the Rust facade cleanup by removing duplicate tests from lib.rs.
  • Preserve the tests in facade_tests.rs and keep explicit public reexports.

Chain Context

Testing

  • cargo fmt --all -- --check
  • CARGO_TARGET_DIR=/home/samael/Escritorio/public/wptui-public/target cargo check --workspace --all-targets
  • CARGO_TARGET_DIR=/home/samael/Escritorio/public/wptui-public/target cargo test -p whatsrust --lib -- --test-threads=1
  • git diff --check

Review Budget and Rollback

  • Authored diff: 226 lines (4 additions + 222 deletions).
  • Rollback: revert commit 7a4d8b1.

Closes #334

@Andiveli Andiveli added the type:refactor Code refactoring label Aug 26, 2026
@Andiveli
Andiveli changed the base branch from refactor/whatsrust-facade-tests-stage to beta August 27, 2026 02:59
@Andiveli
Andiveli merged commit 9b75562 into beta Aug 27, 2026
1 check passed
@Andiveli
Andiveli deleted the refactor/whatsrust-facade-final branch August 31, 2026 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:refactor Code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant