Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add tests for homomorphic hashing disabling #603

Merged
merged 5 commits into from
Aug 23, 2023
Merged

Add tests for homomorphic hashing disabling #603

merged 5 commits into from
Aug 23, 2023

Conversation

evgeniiz321
Copy link
Contributor

@evgeniiz321 evgeniiz321 commented Aug 18, 2023 •
edited
Loading

https://github.com/evgeniiz321/neofs-node/actions/runs/5897635089/job/15997535744?pr=1 - sanity run, no related failures to these changes
https://github.com/evgeniiz321/neofs-node/actions/runs/5899012614/job/16000920036?pr=1 - these tests run, no failures:

pytest_tests/testsuites/network/test_homomorphic_hash.py::TestHomomorphicHash::test_homomorphic_hash_enable_disable PASSED
pytest_tests/testsuites/network/test_homomorphic_hash.py::TestHomomorphicHash::test_new_containers_created_with_specified_homomorphic_hash_value PASSED
pytest_tests/testsuites/network/test_homomorphic_hash.py::TestHomomorphicHash::test_old_containers_have_old_homomorphic_hash_value PASSED

@evgeniiz321 evgeniiz321 force-pushed the ISSUE-549 branch 2 times, most recently from dd8a598 to 0e59f49 Compare August 18, 2023 05:06
@roman-khimov roman-khimov mentioned this pull request Aug 18, 2023
Open
@evgeniiz321
Copy link
Contributor Author

Review once again, please. Ready to merge as for me.

@roman-khimov roman-khimov merged commit c440d24 into nspcc-dev:master Aug 23, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vvarg229 vvarg229 vvarg229 approved these changes

@roman-khimov roman-khimov roman-khimov approved these changes

@cthulhu-rider cthulhu-rider Awaiting requested review from cthulhu-rider cthulhu-rider is a code owner

Assignees

@roman-khimov roman-khimov

@vvarg229 vvarg229

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@evgeniiz321 @roman-khimov @vvarg229