iotaledger / iota / 25029037219

67%

Build Type

push

github

Committed by web-flow

Commit Message

feat!: Replace `IotaAddress` with SDK `Address` (#11330)

This PR is part of the ongoing effort to consolidate blockchain types in
the
canonical `iota-rust-sdk`. Types that are needed by both clients/SDK
consumers
and the node should live in `iota-rust-sdk` and be imported from there,
instead
of being defined twice (once in `iota-types`, once in the SDK).

This PR replaces `IotaAddress` (previously defined in `iota-types`) with
the
SDK version from `iota-sdk-types`. To keep the diff small and avoid
touching
every call site, the SDK type is re-exported from `iota-types` under the
original name `IotaAddress`, so most existing imports continue to work
unchanged.

## What changed

- The local `pub struct IotaAddress([u8; IOTA_ADDRESS_LENGTH])`
definition
  and its inherent impls in

[crates/iota-types/src/base_types.rs](crates/iota-types/src/base_types.rs)
  are removed. `IotaAddress` is now a re-export:
  `pub use iota_sdk_types::Address as IotaAddress;`

## Call-site adjustments

Because the SDK `Address` has a slightly different surface than the old
`IotaAddress`, a few call sites had to be updated rather than relying on
the
re-export:

- `IotaAddress::random_for_testing_only()` is replaced by the SDK's
`IotaAddress::random()` everywhere (tests, fuzzers, sample data
builders).
- `IotaAddress::default()` is replaced by `IotaAddress::ZERO` in
[effects_v1.rs](crates/iota-types/src/effects/effects_v1.rs); the SDK
type
  no longer derives `Default`, but exposes the same zero-value via the
  `ZERO` associated constant.
- `IotaAddress::try_from(bytes)` / `IotaAddress::try_from(&bytes[..])`
are
  replaced by `IotaAddress::from_bytes(...)`. The SDK signature is
`from_bytes(&[u8])` rather than `from_bytes<T: AsRef<[u8]>>`, so a few
call
  sites pass `&vec` / `&arr` explicitly (e.g.
  [iota-json/src/lib.rs](crates/iota-json/src/lib.rs),

[base_types_tests.rs](crates/iota-types/src/unit_tests/base_types_tests.rs)).
- The blanket `impl<T: IotaPublicKey> From<&T> fo... (continued)

Coverage Stats

149 of 207 new or added lines in 55 files covered. (71.98%)

1319 existing lines in 66 files now uncovered.

156907 of 248513 relevant lines covered (63.14%)

447059.97 hits per line