Skip to content

Tests extraction#410

Open
gshigin wants to merge 6 commits into
ppfrom
test-extraction
Open

Tests extraction#410
gshigin wants to merge 6 commits into
ppfrom
test-extraction

Conversation

@gshigin

@gshigin gshigin commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Tests were moved away from test directories and placed near respective header files.

Some tests were lest in test directories due to the fact, that they require rework to be addressed to specific header files. So they left in place for later pr, this pr being only file moving and no behaviour changed

Whats left:

  • bare_bones/tests/container_tests.cpp - shared typed fixture for stream_v_byte.h and vector.h.
  • bare_bones/tests/coredump_test_separate.cpp - separate crash-process test for exception.h and vector bounds failures. Can be removed as core-dumping will be removed.
  • series_data/tests/serialization/serializer_deserializer_tests.cpp - serialization round-trip across data storage, encoding, deserialization, and serialized data. More like integration test to be placed closer to entrypoint.
  • series_data/tests/unloading/loader_unloader_tests.cpp - end-to-end unload/load state transition. Same.
  • series_index/tests/trie/cedarpp_trie_tests.cpp - covers both cedarpp_tree.h and its regexp-searcher integration.
  • series_index/tests/trie/regexp_searcher_test_cases.h - header with test data. Can be inlined into test file.

@gshigin
gshigin requested review from cherep58 and vporoshok July 15, 2026 12:23
@gshigin gshigin self-assigned this Jul 15, 2026
@gshigin gshigin mentioned this pull request Jul 16, 2026

@vporoshok vporoshok left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, resolve conflict

@vporoshok vporoshok added this to the v0.9.0 milestone Jul 22, 2026
@vporoshok
vporoshok enabled auto-merge (squash) July 22, 2026 16:43
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.

3 participants