Skip to content

Commit

Permalink
There's no need to use ansible-test for unit test
Browse files Browse the repository at this point in the history
It's annoying as fuck
  • Loading branch information
Vidimensional committed Mar 22, 2024
1 parent 9da9912 commit a643f2f
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,10 @@ jobs:
pip install -r requirements.txt
fi
if [ -f requirements-dev.txt ]; then
pip install -r requirements-dev.txt
fi
- name: Run Ansible sanity checks
run: ansible-test sanity -v --venv --redact --python 3.10 --color yes

Expand All @@ -54,6 +58,4 @@ jobs:
run: ansible-doc -t lookup github_release -M "${PWD}/plugins/lookup/" --json

- name: Run unit tests
run: |
cat requirements* > tests/unit/requirements.txt
ansible-test units -v --venv --python 3.10 --color yes --requirements
run: make unit
3 changes: 1 addition & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@ sanity:
unit: unittest
units: unittest
unittest:
cat requirements* > tests/unit/requirements.txt
ansible-test units -v --venv --python 3.10 --color yes --requirements
python -m unittest discover --pattern 'test_*.py' -s tests/unit

.PHONY: docs
docs:
Expand Down

0 comments on commit a643f2f

Please sign in to comment.