-
Notifications
You must be signed in to change notification settings - Fork 247
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.
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
tests: Removing intg/tests/test_memory_cache.py #7577
base: master
Are you sure you want to change the base?
Conversation
CI is red |
These tests are covered system/test_memcache.py
f635c51
to
825fffb
Compare
@alexey-tikhonov green now, forgot to edit the Makefile |
|
@alexey-tikhonov I vote to drop it. |
This test covered a bug #4595 fixed by Sumit. |
Hi, independent of the issue, since collisions might happen I think having a test which makes sure we can handle them, would be good. bye, |
I will write the 'colliding hashes tests' before we drop this file, putting the PR back to draft |
Sorry confused this with another PR, this isn't ready yet, back to draft. |
These tests are covered system/test_memcache.py