scylla-rust-driver icon indicating copy to clipboard operation
scylla-rust-driver copied to clipboard

tests: introduce new test for compression

Open fruch opened this issue 9 months ago • 3 comments

test are trying both algorithms (snappy, lz4), while counting the size of the frames coming into scylla

for reference we also have one test doing the same without compression

test main structure was borrowed from java-driver, the usage of proxy to count the packets is unique to this change.

Pre-review checklist

  • [x] I have split my patch into logically separate commits.
  • [x] All commit messages clearly explain what they change and why.
  • [] I added relevant tests for new features and bug fixes.
  • [x] All commits compile, pass static checks and pass test.
  • [x] PR description sums up the changes and reasons why they should be introduced.
  • [ ] I have provided docstrings for the public items that I want to introduce.
  • [ ] I have adjusted the documentation in ./docs/source/.
  • [ ] I added appropriate Fixes: annotations to PR description.

fruch avatar Feb 10 '25 17:02 fruch

cargo semver-checks found no API-breaking changes in this PR! 🎉🥳 Checked commit: 58a623cced840a3503b1d9ae127bf5672cd7c1c1

github-actions[bot] avatar Feb 10 '25 17:02 github-actions[bot]

and now, with the file :)

fruch avatar Feb 10 '25 17:02 fruch

The test has failed for Cassandra: assertion failed: expected_frame_total_size_range.contains(&total_frame_size)

wprzytula avatar Feb 12 '25 16:02 wprzytula