Skip to content

test: run the live tests that no longer need to be skipped - #165

Merged
olavgg merged 1 commit into
mainfrom
test/fewer-skipped-tests
Sep 30, 2026
Merged

olavgg merged 1 commit into
mainfrom
test/fewer-skipped-tests

Conversation

@JosteinGj

@JosteinGj JosteinGj commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

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

  • 13 live tests were #[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_400 waited on a backend fix that has since shipped. The api answers 409 referenced with the subscription in blockedBy, not 400. The test is renamed ..._is_refused, asserts that, and runs.
  • vocabulary::test_search_dimensions_is_case_insensitive_substring printed SKIP on a tenant with no event types. It now seeds one.

Python

  • The 5 listen tests lose their RUN_LISTEN_TESTS gate. The Rust twin already ran ungated.
  • The skip for a backend without POST /edges/create (405) is gone.
  • The event vocabulary tests seed two events with fixed types instead of skipping. The type names are fixed, not unique, because a type stays in the vocabulary after its events are deleted.
  • test_every_type_label_is_matchable creates one node of each type, policy included, instead of skipping types the tenant happens to lack.
  • The test_units setup 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

  • Every test this PR stops skipping, run against a local backend: all pass
  • Full Python suite: 656 passed, 1 failed, 3 skipped
  • Full Rust suite: 292 passed, 2 failed
  • The file-lifecycle failure (Rust and Python) is not from this PR: the current api no longer renames a deleted file's external id. fix(files)!: follow the api's verbatim file external ids and deletedAt trash #164 fixes it.
  • test_subscription_listen_end_to_end failed once under the full parallel run and passed alone. It was not ignored before either.

🤖 Generated with Claude Code

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>
@olavgg
olavgg merged commit 1011647 into main Sep 30, 2026
18 checks passed
@olavgg
olavgg deleted the test/fewer-skipped-tests branch September 30, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants