Skip to content

Commit

Permalink
[qa] Updated CI, caching of pip deps, minor QA improvements
Browse files Browse the repository at this point in the history
  • Loading branch information
nemesifier committed Aug 5, 2024
1 parent c7d1911 commit 5b82ab6
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 7 deletions.
22 changes: 17 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,11 +35,13 @@ jobs:
uses: actions/setup-python@v5
with:
python-version: "3.10"
cache: "pip"
cache-dependency-path: "**/requirements*.txt"

- name: Installing dependencies
run: |
yarn install
pip install openwisp-utils[qa]
pip install -r requirements-qa.txt
- name: QA checks
run: ./run-qa-checks
Expand Down Expand Up @@ -67,6 +69,8 @@ jobs:
run: |
yarn install
yarn setup
sudo apt update -qq
sudo apt install -qq -y libssl-dev libffi-dev
- name: Tests
run: yarn coverage
Expand All @@ -75,10 +79,13 @@ jobs:
uses: actions/setup-python@v5
with:
python-version: "3.10"
cache: "pip"
cache-dependency-path: "**/requirements*.txt"

- name: Get openwisp-radius
run: |
curl -L https://github.com/openwisp/openwisp-radius/tarball/master -o openwisp-radius.tar.gz
curl -L https://github.com/openwisp/openwisp-radius/tarball/master \
-o openwisp-radius.tar.gz
tar -xvzf openwisp-radius.tar.gz && mkdir openwisp-radius
mv openwisp-*/* openwisp-radius
Expand All @@ -90,7 +97,10 @@ jobs:

- name: Installing OpenWISP Radius
run: |
cd openwisp-radius && pip install -U -e ".[saml]" "Django~=4.2.0"
cd openwisp-radius
pip install -U pip setuptools wheel
pip install -U -e ".[saml]"
pip install -U -r requirements-test.txt
./tests/manage.py migrate
- name: Creating configuration of organization (supports mobile verification)
Expand All @@ -104,8 +114,10 @@ jobs:

- name: Running OpenWISP Radius
run: |
cp browser-test/local_settings.py openwisp-radius/tests/openwisp2/local_settings.py \
&& cd openwisp-radius && ./tests/manage.py runserver &
cp browser-test/local_settings.py \
openwisp-radius/tests/openwisp2/local_settings.py
cd openwisp-radius
./tests/manage.py runserver &
- name: geckodriver/firefox
run: |
Expand Down
3 changes: 1 addition & 2 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ OpenWISP documentation for a complete overview of features.
Documentation
-------------

- `Usage documentation
<https://openwisp.io/docs/dev/wifi-login-pages/>`_
- `Usage documentation <https://openwisp.io/docs/dev/wifi-login-pages/>`_
- `Developer documentation
<https://openwisp.io/docs/dev/wifi-login-pages/developer/index.html>`_

Expand Down
1 change: 1 addition & 0 deletions requirements-qa.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
openwisp-utils[qa] @ https://github.com/openwisp/openwisp-utils/tarball/master
1 change: 1 addition & 0 deletions requirements-tests.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Django~=4.2.0

0 comments on commit 5b82ab6

Please sign in to comment.