test(rust): lock acquisition time boundaries and replay snapshot - #548
Merged
LIghtJUNction merged 1 commit intoSep 30, 2026
Conversation
LIghtJUNction
marked this pull request as ready for review
September 30, 2026 07:27
LIghtJUNction
merged commit Sep 30, 2026
4aa2577
into
test/rust-route-contract-coverage
6 of 14 checks passed
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This was referenced Sep 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Stacked follow-up to #542, based on its exact head
a480c2bfc3aa4f5516f25ccee844ed848cd0c676. This PR targetedtest/rust-route-contract-coverageto isolate the incremental test changes. It was subsequently merged into that prerequisite branch at4aa25772586a486a55909f94ad3f8e6e4a3d5ac6; #542 remains the path to main.now + 10 secondsdeadlines with explicit persisted fixture timestamps.No production behavior, API, schema, frontend or Go changes.
Upstream relationship
LMM API-specific Rust regression tests. Depends on #542; no issue closed.
Verification
Passed locally:
cargo fmt --all --check(repository-pinned Rust 1.91.0)python3 apps/api-rust/tests/scripts/test-new-integration-suites.py: 7 tests passedbash apps/api-rust/tests/scripts/check-real-integration-gates.sh: passed, 96 ignored tests across 10 modulesbash -n apps/api-rust/tests/scripts/run-real-integration-gates.shgit diff --checkAlso passed locally on the exact published tree:
cargo test --locked -p lmm-api-rs --test acquisition -- --skip postgres_: 2 passedcargo clippy --locked -p lmm-api-rs --test acquisition -- -D warningscargo test --locked -p lmm-api-rs --test acquisition -- --ignored --list: all 5 PostgreSQL tests compiled and listed (listing does not execute them)Real PostgreSQL 18 integration logs for #542 at 4aa2577 confirm all 5 acquisition tests executed and passed (0 failed, 0 ignored), including the new boundary and replay assertions. Integrated commit 4aa2577 has the identical tree ff91ef2 as this PR's published 2a1e301. Local PostgreSQL was not available; database execution evidence comes from CI. The earlier #548 run was cancelled during the subsequent PR state changes.
Review scope
Searched open acquisition PRs; #542 is the existing prerequisite. Changes are limited to its test gaps and preserve the existing license and source headers. No credentials or production data are included.