Opened at 2022-09-08T18:46:59Z
#3924 new enhancement
Test for allmydata.immutable.layout.WriteBucketProxy invariant
Reported by: | itamarst | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | undecided |
Component: | unknown | Version: | n/a |
Keywords: | Cc: | ||
Launchpad Bug: |
Description
#3915 adds an invariant that calculated length must match the written data; this is sensible (and required by HTTP!) but wasn't previously enforced, relying on Foolscap write's semantics that allowed creating holes with nulls.
It would be nice to be able to test this (and other invariants) but as is the code is factored in ways that make it very difficult to test without creating a whole pile of different objects. This may be the influence of Foolscap pushing for more RPC-ish structure?
Anyway writing this test is going to require some Refactoring... For now this is just enforced at runtime.
Note: See
TracTickets for help on using
tickets.