Merge pull request #1 from albipuliga/main

Added test template

Author: RestartDK

.github/CODE_OF_CONDUCT.md

@@ -25,15 +25,36 @@ jobs:
           template: ./main.bicep
           parameters: ./main.parameters.json
 
-  build-and-push-image:
+  run-tests:
+    runs-on: ubuntu-latest
     needs: deploy-infrastructure
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: "3.x"
+
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install -r requirements.txt
+          pip install pytest
+
+      - name: Run tests
+        run: |
+          PYTHONPATH=$PYTHONPATH:$(pwd) pytest tests/
+
+  build-and-push-image:
+    needs: [deploy-infrastructure, run-tests]
     runs-on: ubuntu-latest
     env:
-      KEY_VAULT_NAME_DEV: "dkumlin-demo-kvtest"
+      KEY_VAULT_NAME_DEV: "apuliga-demo-kvtest"
       KEY_VAULT_SECRET_ACR_USERNAME: "acr-username"
       KEY_VAULT_SECRET_ACR_PASSWORD: "acr-password0"
-      CONTAINER_REGISTRY_SERVER_URL_DEV: "dkumlinacrdemo.azurecr.io"
-      IMAGE_NAME_DEV: "dkumlin-demo-image"
+      CONTAINER_REGISTRY_SERVER_URL_DEV: "apuligaacrdemo.azurecr.io"
+      IMAGE_NAME_DEV: "apuliga-demo-image"
     steps:
       - uses: actions/checkout@v3
 
@@ -75,9 +96,9 @@ jobs:
     needs: build-and-push-image
     runs-on: ubuntu-latest
     env:
-      CONTAINER_REGISTRY_SERVER_URL_DEV: "dkumlinacrdemo.azurecr.io"
-      IMAGE_NAME_DEV: "dkumlin-demo-image"
-      WEB_APP: "dkumlin-demo-web-app"
+      CONTAINER_REGISTRY_SERVER_URL_DEV: "apuligaacrdemo.azurecr.io"
+      IMAGE_NAME_DEV: "apuliga-demo-image"
+      WEB_APP: "apuliga-demo-web-app"
     steps:
       - name: Login to Azure CLI
         uses: azure/login@v1

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,7 @@
+{
+    "python.testing.pytestArgs": [
+        "tests"
+    ],
+    "python.testing.unittestEnabled": false,
+    "python.testing.pytestEnabled": true
+}

.github/PULL_REQUEST_TEMPLATE.md

@@ -43,13 +43,19 @@ Created `.github/workflows/workflow.yaml` with three main stages:
    - Creates/updates Azure resources
    - Sets up RBAC permissions
 
-2. **Container Build & Push Stage**
+2. **Run Tests Stage**
+
+   - Builds the Docker image
+   - Runs tests on the container
+   - Exits with error if tests fail
+
+3. **Container Build & Push Stage**
 
    - Builds the Docker image
    - Tags with git SHA
    - Pushes to Azure Container Registry
 
-3. **Web App Deployment Stage**
+4. **Web App Deployment Stage**
    - Pulls latest image from ACR
    - Deploys to Azure Web App
    - Updates container configuration
@@ -68,7 +74,7 @@ Created `.github/workflows/workflow.yaml` with three main stages:
 1. **Fork/Clone the Repository**
 
 2. **Update Parameters**
-   Replace all instances of `dkumlin` with your identifier in:
+   Replace all instances of `yourname` with your identifier in:
 
    ```json
    // main.parameters.json

.github/workflows/workflow.yaml

@@ -6,22 +6,22 @@
       "value": "westeurope"
     },
     "acrName": {
-      "value": "dkumlinacrdemo"
+      "value": "apuligaacrdemo"
     },
     "appServicePlanName": {
-      "value": "dkumlin-demo-service-plan"
+      "value": "apuliga-demo-service-plan"
     },
     "webAppName": {
-      "value": "dkumlin-demo-web-app"
+      "value": "apuliga-demo-web-app"
     },
     "containerRegistryImageName": {
-      "value": "dkumlin-demo-image"
+      "value": "apuliga-demo-image"
     },
     "containerRegistryImageVersion": {
       "value": "latest"
     },
     "keyVaultName": {
-      "value": "dkumlin-demo-kvtest"
+      "value": "apuliga-demo-kvtest"
     },
     "keyVaultRoleAssignments": {
       "value": [
@@ -38,4 +38,4 @@
       ]
     }
   }
-}
+}

.vscode/settings.json

@@ -1,3 +1,3 @@
 Flask==2.2.2
 gunicorn
-Werkzeug==2.2.2
+Werkzeug==2.2.2

CHANGELOG.md

@@ -0,0 +1,40 @@
+import pytest
+
+from app import app
+
+
+@pytest.fixture
+def client():
+    app.config["TESTING"] = True
+    with app.test_client() as client:
+        yield client
+
+
+def test_index_route(client):
+    """Test the index route returns correct template"""
+    response = client.get("/")
+    assert response.status_code == 200
+    assert b"<title>Hello Azure - Python Quickstart</title>" in response.data
+
+
+def test_favicon_route(client):
+    """Test favicon.ico is served correctly"""
+    response = client.get("/favicon.ico")
+    assert response.status_code == 200
+    assert response.mimetype == "image/vnd.microsoft.icon"
+
+
+def test_hello_route_with_name(client):
+    """Test hello route with a name parameter"""
+    response = client.post("/hello", data={"name": "Test User"})
+    assert response.status_code == 200
+    assert (
+        b"Test User" in response.data
+    )  # Less strict assertion that checks if name appears anywhere
+
+
+def test_hello_route_without_name(client):
+    """Test hello route redirects when no name provided"""
+    response = client.post("/hello", data={})
+    assert response.status_code == 302
+    assert response.headers["Location"] == "/"