Set timeout for tests job on CI (#27)

- Remove timeouts from tests
- Remove pytest-timeout

Author: tamirOK

.github/workflows/ci.yml

@@ -62,6 +62,7 @@ jobs:
         run: make bootstrap
 
       - name: Run tests on Python ${{ matrix.python }}
+        timeout-minutes: 10
         run: make testcluster
 
       - name: Run genproto

Makefile

@@ -44,7 +44,7 @@ test: bootstrap
 	$(PYTHON) -m pytest
 
 testcluster: bootstrap
-	$(PYTHON) -m pifpaf -e TEST --debug run etcd --cluster -- $(PYTHON) -m pytest --timeout=60 --cov-report=xml
+	$(PYTHON) -m pifpaf -e TEST --debug run etcd --cluster -- $(PYTHON) -m pytest --with-cluster --cov-report=xml
 
 testreport: bootstrap
 	$(PYTHON) -m pytest --cov-report=html

setup.cfg

@@ -9,7 +9,6 @@ application-import-names = aetcd
 [tool:pytest]
 addopts =
     --cov=aetcd
-    --timeout=30
     tests
 asyncio_mode = auto
 

setup.py

@@ -70,7 +70,6 @@ def _get_long_description():
             'pytest-asyncio==0.20.3',
             'pytest-cov==4.0.0',
             'pytest-mock==3.10.0',
-            'pytest-timeout==2.1.0',
             'pytest==7.2.1',
         ],
     },

tests/integration/conftest.py

@@ -30,13 +30,7 @@ async def _etcdctl(*args, ignore_result=False):
             stderr=asyncio.subprocess.PIPE,
         )
 
-        try:
-            stdout, stderr = await asyncio.wait_for(
-                proc.communicate(),
-                timeout=15,
-            )
-        except asyncio.TimeoutError:
-            raise Exception(f'Timeout during awaiting process with args {args!r}')
+        stdout, stderr = await proc.communicate()
 
         if proc.returncode != 0:
             raise Exception(f'Error during awaiting process with args {args!r}: {stderr}')