iotaledger / iota / 27924667289
67%

DEFAULT BRANCH: develop
Ran
Jobs 1
Files 1107
Run time 2min
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

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

Coverage Regressions

Lines Coverage ∆ File
35
87.58
 0.0% crates/iota-types/src/move_authenticator.rs
26
79.74
 0.0% crates/iota-types/src/crypto.rs
4
91.06
 -0.17% crates/iota-core/src/authority/authority_per_epoch_store.rs
3
77.39
 -0.14% crates/iota-core/src/checkpoints/mod.rs
3
89.0
 -1.03% crates/iota-core/src/connection_monitor.rs
2
92.67
 -0.24% crates/iota-core/src/consensus_adapter.rs
2
96.49
 -3.51% crates/starfish/core/src/commit_consumer.rs
1
86.11
 -2.78% crates/iota-cluster-test/src/test_case/native_transfer_test.rs
1
93.43
 -0.04% crates/starfish/core/src/authority_service.rs
1
76.81
 -0.16% crates/starfish/core/src/commit_syncer/mod.rs
1
97.3
 -0.1% crates/starfish/core/src/cordial_knowledge.rs
1
93.06
 -0.18% crates/starfish/core/src/storage/rocksdb_store.rs
Jobs
ID Job ID Ran Files Coverage
1 nextest+simtest - 27924667289.1 1107
67.4
 GitHub Action Run
Source Files on build 27924667289