iotaledger / iota / 27924667289 / 1
67%
develop: 67%

DEFAULT BRANCH: develop
Ran
Files 1107
Run time 31s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 67.402%. Remained the same
27924667289.1

push

github

web-flow 
refactor(iota-types): remove unused SignableBytes trait (#11983)

# Description of change

The `SignableBytes` trait in `iota-types::crypto` is the BCS-decode
counterpart to `Signable`'s write/encode side, exposing
`from_signable_bytes(&[u8]) -> Result<Self>`. A search of the repo shows
it has **no production call sites** — `from_signable_bytes` was only
ever invoked from tests. The encode side (`Signable::write`) is used in
production (signing checkpoints, effects, transactions, …); the decode
side is not wired into any live code path.

This PR removes the dead trait and its only consumers:

- the `SignableBytes` trait definition,
- its blanket impl over `T: BcsSignable`,
- the manual `SignableBytes for MoveAuthenticator` impl,
- the now-orphaned `MoveAuthenticator::from_inner` helper (its sole
caller was the removed impl), and
- the test-only call sites in `move_authenticator.rs` and
`crypto_tests.rs`.

The `Signable` (write) machinery and the `bcs_signable_test` fixtures
(`Foo`/`Bar`) are still used elsewhere (e.g. `base_types_tests.rs`,
`messages_tests.rs`, and the verification-obligation tests) and are left
untouched. The remaining `MoveAuthenticator` `signable_bytes_*` tests,
which exercise the write-side format, are kept.

## Links to any relevant issues

N/A

## How the change has been tested

- [x] Basic tests (linting, compilation, formatting, unit/integration
tests)
- [x] Patch-specific tests (correctness, functionality coverage)
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [x] I have checked that new and existing unit tests pass locally with
my changes

`cargo check -p iota-types --tests` is clean (no new warnings), and
`cargo nextest run -p iota-types --lib move_authenticator crypto` passes
(21 tests). The only removed tests are those that exercised the deleted
`from_signable_bytes` path.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

157990 of 234401 relevant lines covered (67.4%)

472153.95 hits per line

Source Files on job nextest+simtest - 27924667289.1