test: run the live tests that no longer need to be skipped - #165
Merged
Merged
Conversation
Thirteen Rust live tests were #[ignore]d only because they call a backend, unlike the rest of the suite; they pass and now run by default. The subscription-bound delete test waited on a backend fix that has shipped, answering 409 `referenced` with the subscription in `blockedBy`, so it asserts that and runs too. On the Python side the listen tests lose their RUN_LISTEN_TESTS gate (the Rust twin already ran ungated), the dead 405 fallback on /edges/create goes, and the tests that skipped on tenant data now seed their own: the event vocabulary tests (and the Rust twin) create events with fixed types, and the type-label matching test creates one node of each type. The units fixture asserts the migration-seeded catalogue instead of skipping. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Signed-off-by: jgjesdal <jostein@intellistream.ai>
2 tasks done
olavgg
approved these changes
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
Fewer skipped tests. What stays skipped is what is genuinely expensive or needs special setup: the multi-million-point datapoint tests, the ingest benchmark, and the multi-tenant/auth tests that need extra Keycloak principals.
Rust
#[ignore]d only because they call a backend, unlike the rest of the suite. They pass and now run by default:labels(2),functions(2),assets(3),label_update_tests(3),update_echo_is_typed_per_node_type,test_insert_datapoints_binary_re_resolves_a_recreated_series, and the cleanup-guard deadlock test.test_delete_timeseries_bound_to_subscription_returns_400waited on a backend fix that has since shipped. The api answers 409referencedwith the subscription inblockedBy, not 400. The test is renamed..._is_refused, asserts that, and runs.vocabulary::test_search_dimensions_is_case_insensitive_substringprintedSKIPon a tenant with no event types. It now seeds one.Python
RUN_LISTEN_TESTSgate. The Rust twin already ran ungated.POST /edges/create(405) is gone.test_every_type_label_is_matchablecreates one node of each type, policy included, instead of skipping types the tenant happens to lack.test_unitssetup asserts the unit catalogue the api's migrations create instead of skipping when it's empty.No SDK surface changes, so no docs pages are affected.
Test plan
test_subscription_listen_end_to_endfailed once under the full parallel run and passed alone. It was not ignored before either.🤖 Generated with Claude Code