tests: completely remove flaky tests

It doesn't seem to be possible to set reasonable fudge factors
that will make this tests pass every time.

Because these tests don't make any sense anyway we just drop them.

Signed-off-by: Salvatore Mesoraca <[email protected]>

Author: aiven-sal

tests/test_asyncio/test_lock.py

@@ -107,13 +107,12 @@ async def test_blocking(self, r):
     async def test_blocking_timeout(self, r):
         lock1 = self.get_lock(r, "foo")
         assert await lock1.acquire(blocking=False)
-        bt = 0.9
+        bt = 0.2
         sleep = 0.05
         lock2 = self.get_lock(r, "foo", sleep=sleep, blocking_timeout=bt)
         start = asyncio.get_running_loop().time()
         assert not await lock2.acquire()
-        # The elapsed duration should be less than the total blocking_timeout
-        assert bt >= (asyncio.get_running_loop().time() - start) > bt - sleep
+        assert (asyncio.get_running_loop().time() - start) > (bt - sleep)
         await lock1.release()
 
     async def test_context_manager(self, r):

tests/test_lock.py

@@ -101,12 +101,10 @@ def test_blocking_timeout(self, r):
         assert lock1.acquire(blocking=False)
         bt = 0.4
         sleep = 0.05
-        fudge_factor = 0.5
         lock2 = self.get_lock(r, "foo", sleep=sleep, blocking_timeout=bt)
         start = time.monotonic()
         assert not lock2.acquire()
-        # The elapsed duration should be less than the total blocking_timeout
-        assert (bt + fudge_factor) > (time.monotonic() - start) > bt - sleep
+        assert (time.monotonic() - start) > (bt - sleep)
         lock1.release()
 
     def test_context_manager(self, r):
@@ -120,12 +118,10 @@ def test_context_manager_blocking_timeout(self, r):
         with self.get_lock(r, "foo", blocking=False):
             bt = 0.4
             sleep = 0.05
-            fudge_factor = 0.5
             lock2 = self.get_lock(r, "foo", sleep=sleep, blocking_timeout=bt)
             start = time.monotonic()
             assert not lock2.acquire()
-            # The elapsed duration should be less than the total blocking_timeout
-            assert (bt + fudge_factor) > (time.monotonic() - start) > bt - sleep
+            assert (time.monotonic() - start) > (bt - sleep)
 
     def test_context_manager_raises_when_locked_not_acquired(self, r):
         r.set("foo", "bar")