Repository navigation
test: cover capture invariants on both lanes - #1029
Merged
Merged
Conversation
This was referenced Oct 7, 2026
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 7, 2026 03:56
d5eb0bc to
ae647de
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
6 times, most recently
from
October 7, 2026 17:43
44df57f to
2011a51
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 7, 2026 17:43
42c1d07 to
691c8bd
Compare
Contributor
Author
|
@greptileai review |
eli-r-ph
marked this pull request as ready for review
October 7, 2026 20:44
dustinbyrne
approved these changes
Oct 7, 2026
dustinbyrne
left a comment
Contributor
There was a problem hiding this comment.
The test-only hardening looks sound, and existing CI passes for this snapshot. One non-blocking suggestion inline: assert exactly one aggregate loss record in the newly named single-loss-line test.
AI-assisted review.
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 8, 2026 16:25
aac8735 to
fd36bd2
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 8, 2026 16:25
691c8bd to
1f784bb
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 8, 2026 18:13
fd36bd2 to
d969e0f
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 8, 2026 18:13
1f784bb to
04507c0
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 8, 2026 21:12
d969e0f to
84e195d
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 8, 2026 21:12
04507c0 to
adf5b16
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 8, 2026 22:54
84e195d to
e542984
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 8, 2026 22:54
adf5b16 to
0cfa23c
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 8, 2026 23:40
e542984 to
e34c122
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 10, 2026 23:35
252b30d to
6e1a501
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 10, 2026 23:35
e34c122 to
5a36573
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
from
October 11, 2026 20:17
6e1a501 to
2765967
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 11, 2026 20:17
5a36573 to
2663fd0
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 14, 2026 03:11
2663fd0 to
87217bb
Compare
eli-r-ph
force-pushed
the
v1-capture-async-ai-lane
branch
2 times, most recently
from
October 14, 2026 03:47
4cd3a28 to
66b1e17
Compare
eli-r-ph
force-pushed
the
v1-capture-invariant-tests
branch
from
October 14, 2026 03:47
87217bb to
87ff592
Compare
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.
💡 Motivation and Context
Stacked on #1028. posthog-go and posthog-rs hit several capture bugs that tests did not catch, such as a race between enqueue and shutdown. This PR checks the Python stack for each of them and adds tests only where no existing test catches the bug.
Changes
Tests only:
before_sendthat changes nothing leaves the request body unchanged, on both clients and both lanes.💚 How did you test it?
🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Written with Cursor (Claude Opus) under the direction of the assignee.
🤖 Generated with Claude Code