perf: Update Python to stable v3.11 & add smoke test (#90)

* Add smoke test to the separate directory * Enable smoke-test on the pipeline, like the last step
PacketHelper · Oct 25, 2022 · 120707b · 120707b
1 parent 70f2697
commit 120707b
Show file tree

Hide file tree

Showing 14 changed files with 38 additions and 4 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -9,7 +9,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-python@v4
         with:
-          python-version: '3.11.0-rc.2'
+          python-version: '3.11'
           cache: 'pip'
       - name: Install Python dependencies (pip)
         run: |
@@ -153,3 +153,7 @@ jobs:
           export GIT_VERSION_TAG=`git describe --tags --abbrev=0`
           export SHORT_SHA=`echo ${GITHUB_SHA} | cut -c1-8`
           python check_deployment.py https://www.packethelper.com $GIT_VERSION_TAG $SHORT_SHA
+      - name: Run Smoke tests
+        run: |
+          pip install requests pytest
+          make test-smoke
diff --git a/Dockerfile b/Dockerfile
@@ -8,7 +8,7 @@ COPY . .
 RUN yarn build
 
 # build stage (backend & package)
-FROM python:3.11-rc-buster
+FROM python:3.11-buster
 
 # Install tshark/wireshark dependecies
 RUN echo "wireshark-common wireshark-common/install-setuid boolean true" | debconf-set-selections

diff --git a/Makefile b/Makefile
@@ -37,17 +37,22 @@ format-ui:
 format:
 	@echo "Formatting..."
 	python3 -m isort ph/ tests/ --profile black 
-	python3 -m black -t py310 ph/ tests/
+	python3 -m black -t py311 ph/ tests/
 	prettier --write src/
 	@echo "Formatting... Done"
 
 .PHONY: run-back
 run-back:
 	uvicorn --port 8080 ph.main:app 
 
+.PHONY: test
 test:
 	mkdir -p static && touch static/index.html
-	PYTHONPATH=${PWD} pytest
+	PYTHONPATH=${PWD} pytest tests/integration
+
+.PHONY: test-smoke
+test-smoke:
+	PYTHONPATH=${PWD} pytest tests/smoke
 
 .PHONY: init
 init:

diff --git a/tests/routers/api/__init__.py → tests/__init__.py b/tests/routers/api/__init__.py → tests/__init__.py
diff --git a/tests/integration/__init__.py b/tests/integration/__init__.py
diff --git a/tests/integration/routers/__init__.py b/tests/integration/routers/__init__.py
diff --git a/tests/integration/routers/api/__init__.py b/tests/integration/routers/api/__init__.py
diff --git a/tests/routers/api/test_create.py → tests/integration/routers/api/test_create.py b/tests/routers/api/test_create.py → tests/integration/routers/api/test_create.py
diff --git a/tests/routers/api/test_hex.py → tests/integration/routers/api/test_hex.py b/tests/routers/api/test_hex.py → tests/integration/routers/api/test_hex.py
diff --git a/tests/routers/api/test_packets.py → ...s/integration/routers/api/test_packets.py b/tests/routers/api/test_packets.py → ...s/integration/routers/api/test_packets.py
diff --git a/tests/test_main.py → tests/integration/test_main.py b/tests/test_main.py → tests/integration/test_main.py
diff --git a/tests/smoke/__init__.py b/tests/smoke/__init__.py
diff --git a/tests/smoke/conftest.py b/tests/smoke/conftest.py
@@ -0,0 +1,13 @@
+import os
+
+import pytest
+
+
+@pytest.fixture
+def instance_uri() -> str:
+    return os.getenv("PACKET_HELPER_URI", "https://www.packethelper.com")
+
+
+@pytest.fixture
+def api_uri(instance_uri: str) -> str:
+    return f"{instance_uri}/api"
diff --git a/tests/smoke/test_smoke_prod.py b/tests/smoke/test_smoke_prod.py
@@ -0,0 +1,12 @@
+from http import HTTPStatus
+
+import requests
+
+
+def test_smoke(api_uri: str) -> None:
+    simple_packet = (
+        "00001Cffffff0000000000000800450000340001000040047cc37f0000017f00000"
+        "14500002000010000402f7cac7f0000017f00000100000000"
+    )  # Ethernet / IP / IPv6 / GRE
+    response = requests.get(f"{api_uri}/hex/{simple_packet}")
+    assert response.status_code == HTTPStatus.OK