Merge pull request #196 from permitio/roe/per-10888-fix-pdp-setup-in-…

…ci-and-in-general

Fix PDP Setup (in CI and in general)

.github/workflows/pdp_cicd.yml

@@ -58,11 +58,8 @@ jobs:
 
     - name: Run Pytests
       run: |
-        export PYTHONPATH=$(pwd)
         python -m pip install --upgrade pip
-        python setup.py install
-        if [ -f requirements-dev.txt ]; then pip install -r requirements-dev.txt; fi
-        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
+        pip install ".[dev]"
         pytest -s --cache-clear horizon/tests/
 
     - name: Run E2E tests

.github/workflows/pdp_tests.yml

@@ -0,0 +1,31 @@
+name: PDP Sanity Tests
+
+on:
+  pull_request:
+  push:
+    branches: [master, main, v*]
+
+jobs:
+  pre-commit:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - uses: actions/setup-python@v3
+    - uses: pre-commit/[email protected]
+
+  pytests:
+    runs-on: ubuntu-latest
+    steps:
+    - name: Python setup
+      uses: actions/setup-python@v5
+      with:
+        python-version: '3.11.8'
+
+    - name: Checkout code
+      uses: actions/checkout@v4
+
+    - name: Run Pytests
+      run: |
+        python -m pip install --upgrade pip
+        pip install ".[dev]"
+        pytest -s --cache-clear horizon/tests/

requirements-dev.txt

@@ -1,4 +1,3 @@
 pytest
-aioresponses
 pytest-asyncio
-aiohttp>=3.9.4,<4
+aioresponses

setup.py

@@ -33,5 +33,7 @@ def get_data_files(root_directory: str):
     include_package_data=True,
     data_files=get_data_files("horizon/static"),
     install_requires=get_requirements(),
-    # dev_requires=get_requirements("dev"),
+    extras_require={
+        "dev": get_requirements("dev"),
+    },
 )