Skip to content

cleanup tests - #35

Merged
phiryll merged 11 commits into
mainfrom
cleanup-tests
Jul 5, 2025
Merged

cleanup tests#35
phiryll merged 11 commits into
mainfrom
cleanup-tests

Conversation

@phiryll

@phiryll phiryll commented Jul 5, 2025

Copy link
Copy Markdown
Owner

Still much more to do here. Not testing by key length is next.

  • Clean up one-off unit tests.
  • Move traverser-specific test functions to traversers_test.go.
  • Inline some helper functions that are only used once.
  • Move randomization functions to bench_test.go.
  • Add helper function for testing an early yield().
  • Compare iterators to avoid building slices.
  • Structure bench and test names as config first, impl second.
  • Have reference cache sorted keys.
  • No need to clone the store in BenchmarkGet.
  • Remove some benchmarks for ChildBounds.
  • Remove some unneeded unit tests.

@phiryll
phiryll merged commit 0cc5129 into main Jul 5, 2025
6 checks passed
@phiryll
phiryll deleted the cleanup-tests branch July 5, 2025 06:35
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.

1 participant