tarantool / go-storage / 26221122596

90%

Build Type

push

github

Committed by bigbes

Commit Message

integrity: add WithKeyPrefix option

Add WithKeyPrefix option to namer.LayeredNamer and thread it through
integrity.CodecBuilder so codecs can be built with a fixed path prefix
prepended to every emitted/parsed key.

- Let two codecs sit in disjoint sub-trees of one storage.Storage so
  they can be committed atomically in a single integrity.Tx, which
  cannot span multiple storage handles the way storage.Prefixed wrappers
  can.
- Validate the prefix the same way storage.Prefixed does: must start
  with '/', must not end with '/'; empty is a no-op.
- Require the prefix to be followed by '/' (or the entire key) on parse,
  so a prefix like "/p" does not silently match sibling sub-trees like
  "/pp/...", "/p2/..." or "/papa/...". Such keys consistently return
  namer.ErrKeyPrefixMissing; ErrKeyPrefixNoLeadingSlash /
  ErrKeyPrefixTrailingSlash cover malformed prefixes at construction.
- Add namer tests for validation, generation, parse round-trip, sibling
  prefix rejection, multi-segment prefix, unnamed/legacy/compact layouts,
  multiple hashers and signers, empty-is-noop, and ParseKeys grouping.
- Add integrity tests covering atomic two-codec commit, predicate rollback
  across prefixes, disjoint key spaces, TxRange isolation, and the
  documented doubled-prefix behaviour when stacking WithKeyPrefix on top
  of storage.Prefixed.

Coverage Stats

54 of 54 new or added lines in 2 files covered. (100.0%)

2 existing lines in 1 file now uncovered.

3650 of 4056 relevant lines covered (89.99%)

533.3 hits per line