Skip to content

test: no fixture for a queue backlog or a belt shared with a queue #954

Closed as not planned

Description

@bpowers

Two queue and conveyor behaviours have no model file in test/conveyors/ or test/queues/:

  • a belt with both a queue-fed inflow and an ordinary equation-driven inflow, where the queue's volume uses up capacity the equation inflow then competes for;
  • a queue that builds a backlog of several batches, served under each of the four one_at_a_time / batch_integrity combinations.

queue_coupled_conveyor.xmile has neither: its belt has only the queue inflow, and its queue has an <overflow/>, so it never holds a second batch.
docs/design/queues.md section 12 asks for a backlog fixture, test/queues/queue_wait.xmile, which was never added.

What already covers it

  • Shared capacity: conv_vol_charges_capacity_room in src/simlin-engine/src/wasmgen/belt_tests.rs (VM against wasm). It uses a conveyor-fed inflow, not a queue-fed one.
  • Batch rules with a backlog: every_coupled_batch_rule_matches_vm and both_batch_integrity_takes_are_inclusive_at_the_exact_budget in the same file, and VM expected values in coupling_one_at_a_time_takes_at_most_front_batch and coupling_batch_integrity_never_splits_the_front_batch in src/simlin-engine/src/queue_compile.rs.
  • XMILE round trip of the batch attributes: conveyor_block_all_fields_roundtrip in src/simlin-engine/src/xmile/variables.rs.

Fix

If still wanted, add test/queues/queue_wait.xmile (no <overflow/>, arrivals faster than the belt takes them) and wire it into QUEUE_CORPUS_FIXTURES in tests/integration/simulate.rs.

Pointers

Activity

  1. added
    engineIssues with the rust-based simulation engine
    hygieneToil, but its useful to get get too behind on it
    on Oct 12, 2026
  2. changed the title [-]test: no corpus fixture for the belt apportion loop or for FIFO backlog / the four batch rules (README says #949 covers them; it does not)[/-] [+]test: no fixture for a queue backlog or a belt shared with a queue[/+] on Oct 12, 2026
  3. bpowers commented on Oct 12, 2026

    @bpowers
    OwnerAuthor

    Closing as won't fix.

    belt_tests.rs already covers these. conv_vol_charges_capacity_room is the shared-capacity case. every_coupled_batch_rule_matches_vm builds a backlog and checks all four rules VM against wasm. queue_compile.rs:3101/3152 pin VM values, and the xmile round-trip test covers the batch attributes. The README fix it asked for is already done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    engineIssues with the rust-based simulation enginehygieneToil, but its useful to get get too behind on itwontfixThis will not be worked on

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions