Skip to content

Commit 5692fdc

Browse files
committed
fix(smoke): drop stale assertion that first CPR half has no position
v3.3 retro-fills both halves of a resolved CPR pair when the bootstrap cluster locks, so results[0] now legitimately carries lat/lon. Pin only the post-pair invariants (results[1] is filled, coordinates in the Luxembourg ballpark).
1 parent 2fd6222 commit 5692fdc

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

scripts/smoke_test.sh

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,10 @@ results = pyModeS.decode(
7474
timestamps=[1.0, 2.0],
7575
)
7676
assert len(results) == 2
77-
# First frame yields no position yet (no pair partner).
78-
assert results[0].get("latitude") is None, results[0]
7977
# Second frame completes the pair — latitude/longitude filled.
78+
# (v3.3+ also retro-fills the first half when the bootstrap cluster
79+
# locks, so results[0] may have a position too — we don't pin either
80+
# way here.)
8081
assert results[1]["latitude"] is not None, results[1]
8182
assert results[1]["longitude"] is not None, results[1]
8283
# Sanity-check the decoded coordinates land in the expected

0 commit comments

Comments
 (0)