Skip to content

Commit

Permalink
Simplify Test Runs
Browse files Browse the repository at this point in the history
This patch simplifies the  testing infrastructure by moving all
dependencies into a single requirements file instead of installing them
each separately.
  • Loading branch information
lkiesow committed Nov 7, 2021
1 parent 4adec70 commit ed7d260
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 10 deletions.
6 changes: 5 additions & 1 deletion .github/requirements.txt
@@ -1,2 +1,6 @@
bandit
coverage
coveralls
flake8
mock
selenium
urllib3
17 changes: 9 additions & 8 deletions .github/workflows/test.yml
Expand Up @@ -31,11 +31,14 @@ jobs:
with:
python-version: ${{ matrix.python-version }}

- name: install dependencies
run: |
pip install flake8 bandit mock coverage coveralls
pip install -r requirements.txt
npm ci
- name: install Python dependencies
run: pip install -r requirements.txt

- name: install JavaScript dependencies
run: npm ci

- name: install test dependencies
run: pip install -r .github/requirements.txt

- name: lint code
run: make lint
Expand All @@ -48,9 +51,7 @@ jobs:
gunzip .github/test.db.gz
mv .github/test.db pyca.db
./start.sh ui &
- name: install selenium
run: pip install -r .github/requirements.txt
sleep 2
- name: run user interface integration tests
run: ./.github/selenium-tests
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Expand Up @@ -3,7 +3,7 @@ all: lint test
export DOCKER_BUILDKIT=1

lint:
@flake8 $$(find pyca tests -name '*.py')
@flake8 $$(find pyca tests -name '*.py') .github/selenium-tests
@bandit pyca tests
@npm run eslint

Expand Down

0 comments on commit ed7d260

Please sign in to comment.