Skip to content

Commit

Permalink
Advanced test setup (#431)
Browse files Browse the repository at this point in the history
  • Loading branch information
rubenthoms authored Nov 16, 2023
1 parent a2cbcc1 commit 7da32a3
Show file tree
Hide file tree
Showing 24 changed files with 6,129 additions and 8,010 deletions.
239 changes: 128 additions & 111 deletions .github/workflows/webviz.yml
Original file line number Diff line number Diff line change
@@ -1,116 +1,133 @@
name: webviz

on:
push:
pull_request:
branches:
- main
release:
types:
- published
push:
pull_request:
branches:
- main
release:
types:
- published

jobs:
frontend:
runs-on: ubuntu-latest

steps:
- name: 📖 Checkout commit locally
uses: actions/checkout@v3

- uses: actions/setup-node@v3
with:
node-version: 18
cache: npm
cache-dependency-path: frontend/package-lock.json

- name: ℹ️ Node and npm versions
run: |
node -v
npm -v
- name: 📦 Install build dependencies
working-directory: ./frontend
run: |
npm ci --include=dev
- name: ℹ️ Installed npm packages (depth = 0)
working-directory: ./frontend
run: npm list --depth=0

- name: 🏗️ Build JavaScript bundle
working-directory: ./frontend
run: npm run build

- name: 🕵️ Check code style, linting & dependencies
working-directory: ./frontend
run: |
npm run validate
- name: 🤖 Run tests
working-directory: ./frontend
run: |
npm run test
- name: 🕵️ Check auto-generated frontend code is in sync with backend
run: |
docker build -f backend.Dockerfile -t backend:latest .
CONTAINER_ID=$(docker run --detach -p 5000:5000 --env UVICORN_PORT=5000 --env UVICORN_ENTRYPOINT=src.backend.primary.main:app --env WEBVIZ_CLIENT_SECRET=0 --env WEBVIZ_SMDA_SUBSCRIPTION_KEY=0 --env WEBVIZ_SMDA_RESOURCE_SCOPE=0 backend:latest)
sleep 5 # Ensure the backend server is up and running exposing /openapi.json
npm run generate-api --prefix ./frontend
docker stop $CONTAINER_ID
git diff --exit-code ./frontend/src/api || exit 1
backend:
runs-on: ubuntu-latest

steps:
- name: 📖 Checkout commit locally
uses: actions/checkout@v3

- name: 🐍 Set up Python
uses: actions/setup-python@v4
with:
python-version: "3.11"
cache: pip

- name: 📦 Install poetry and dependencies
working-directory: ./backend
run: |
pip install --upgrade pip
pip install poetry
poetry config virtualenvs.create false
poetry lock --check --no-update # Check lock file is consistent with pyproject.toml
poetry install --with dev
- name: 🕵️ Check code style & linting
working-directory: ./backend
run: |
black --check src/ tests/
pylint src/ tests/
bandit --recursive src/
mypy src/ tests/
- name: 🤖 Run tests
working-directory: ./backend
env:
WEBVIZ_CLIENT_SECRET: 0
WEBVIZ_SMDA_SUBSCRIPTION_KEY: 0
WEBVIZ_SMDA_RESOURCE_SCOPE: 0
run: |
pytest ./tests/unit
build_docker_images:
runs-on: ubuntu-latest

steps:
- name: 📖 Checkout commit locally
uses: actions/checkout@v3

- uses: actions/setup-node@v3
with:
node-version: 18

- name: 🐳 Verify Docker images build
run: |
docker build -f frontend-prod.Dockerfile .
docker build -f backend.Dockerfile .
frontend:
runs-on: ubuntu-latest

steps:
- name: 📖 Checkout commit locally
uses: actions/checkout@v3

- uses: actions/setup-node@v3
with:
node-version: 18
cache: npm
cache-dependency-path: frontend/package-lock.json

- name: ℹ️ Node and npm versions
run: |
node -v
npm -v
- name: 📦 Install build dependencies
working-directory: ./frontend
run: |
npm ci --include=dev
- name: ℹ️ Installed npm packages (depth = 0)
working-directory: ./frontend
run: npm list --depth=0

- name: 🏗️ Build JavaScript bundle
working-directory: ./frontend
run: npm run build

- name: 🕵️ Check code style, linting & dependencies
working-directory: ./frontend
run: |
npm run validate
- name: Install Playwright Browsers
working-directory: ./frontend
run: |
npx playwright install --with-deps
- name: 🤖 Run unit tests
working-directory: ./frontend
run: |
npm run test:unit
- name: 🤖 Run component tests
working-directory: ./frontend
run: |
npm run test:ct
- uses: actions/upload-artifact@v3
if: always()
with:
name: playwright-report
path: playwright-report/
retention-days: 30

- name: 🕵️ Check auto-generated frontend code is in sync with backend
run: |
docker build -f backend.Dockerfile -t backend:latest .
CONTAINER_ID=$(docker run --detach -p 5000:5000 --env UVICORN_PORT=5000 --env UVICORN_ENTRYPOINT=src.backend.primary.main:app --env WEBVIZ_CLIENT_SECRET=0 --env WEBVIZ_SMDA_SUBSCRIPTION_KEY=0 --env WEBVIZ_SMDA_RESOURCE_SCOPE=0 backend:latest)
sleep 5 # Ensure the backend server is up and running exposing /openapi.json
npm run generate-api --prefix ./frontend
docker stop $CONTAINER_ID
git diff --exit-code ./frontend/src/api || exit 1
backend:
runs-on: ubuntu-latest

steps:
- name: 📖 Checkout commit locally
uses: actions/checkout@v3

- name: 🐍 Set up Python
uses: actions/setup-python@v4
with:
python-version: "3.11"
cache: pip

- name: 📦 Install poetry and dependencies
working-directory: ./backend
run: |
pip install --upgrade pip
pip install poetry
poetry config virtualenvs.create false
poetry lock --check --no-update # Check lock file is consistent with pyproject.toml
poetry install --with dev
- name: 🕵️ Check code style & linting
working-directory: ./backend
run: |
black --check src/ tests/
pylint src/ tests/
bandit --recursive src/
mypy src/ tests/
- name: 🤖 Run tests
working-directory: ./backend
env:
WEBVIZ_CLIENT_SECRET: 0
WEBVIZ_SMDA_SUBSCRIPTION_KEY: 0
WEBVIZ_SMDA_RESOURCE_SCOPE: 0
run: |
pytest ./tests/unit
build_docker_images:
runs-on: ubuntu-latest

steps:
- name: 📖 Checkout commit locally
uses: actions/checkout@v3

- uses: actions/setup-node@v3
with:
node-version: 18

- name: 🐳 Verify Docker images build
run: |
docker build -f frontend-prod.Dockerfile .
docker build -f backend.Dockerfile .
5 changes: 5 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,11 @@ dist
!.vscode/tasks.json
!.vscode/launch.json

# playwright results
playwright-report/
/test-results/
/playwright/.cache/

# Ignore Jupyter Notebook files
*.ipynb

Expand Down
1 change: 1 addition & 0 deletions frontend/aliases.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
{
"compilerOptions": {
"baseUrl": ".",
"paths": {
"@api": ["./src/api"],
"@assets/*": ["./src/assets/*"],
Expand Down
10 changes: 0 additions & 10 deletions frontend/jest.config.cjs

This file was deleted.

Loading

0 comments on commit 7da32a3

Please sign in to comment.