Merge branch 'main' into main

Author: Weng1002

.github/scripts/test_homework3.py

@@ -0,0 +1,102 @@
+# .github/scripts/test_homework3.py
+from fastapi.testclient import TestClient
+
+try:
+    from main import app  # Assuming the FastAPI application is in main.py
+except ImportError:
+    from app.main import app  # If unable to import from the main module, try importing from app.main
+
+client = TestClient(app)
+
+# Test the correctness of GET /items/{item_id}
+def test_get_item_valid():
+    response = client.get("/items/123?q=test_query&sort_order=asc")
+    assert response.status_code == 200
+    assert response.json() == {
+        "item_id": 123,
+        "description": "This is a sample item that matches the query test_query",
+        "sort_order": "asc"
+    }
+
+# Test GET /items/{item_id} with invalid path parameter
+def test_get_item_invalid_id():
+    response = client.get("/items/1001")
+    assert response.status_code == 422  # Should return 422 Unprocessable Entity
+    assert "detail" in response.json()
+
+# Test GET /items/{item_id} with invalid query parameter q
+def test_get_item_invalid_query():
+    response = client.get("/items/123?q=ab&sort_order=asc")
+    assert response.status_code == 422  # Should return 422 Unprocessable Entity
+    assert "detail" in response.json()
+
+# Test GET /items/{item_id} without query parameters
+def test_get_item_no_query():
+    response = client.get("/items/123")
+    assert response.status_code == 200
+    assert response.json() == {
+        "item_id": 123,
+        "description": "This is a sample item.",
+        "sort_order": "asc"
+    }
+
+# Test PUT /items/{item_id} to update item data
+def test_put_item_valid():
+    data = {
+        "name": "Updated Item",
+        "description": "Updated item description",
+        "price": 20.0,
+        "tax": 2.5
+    }
+    response = client.put("/items/123", json=data)
+    assert response.status_code == 200
+    assert response.json() == {
+        "item_id": 123,
+        "name": "Updated Item",
+        "description": "Updated item description",
+        "price": 20.0,
+        "tax": 2.5
+    }
+
+# Test PUT /items/{item_id} with valid query parameter q
+def test_put_item_with_query():
+    data = {
+        "name": "Updated Item",
+        "description": "Updated item description",
+        "price": 20.0,
+        "tax": 2.5
+    }
+    response = client.put("/items/123?q=test_query", json=data)
+    assert response.status_code == 200
+    assert response.json() == {
+        "item_id": 123,
+        "name": "Updated Item",
+        "description": "Updated item description",
+        "price": 20.0,
+        "tax": 2.5,
+        "q": "test_query"
+    }
+
+# Test PUT /items/{item_id} with invalid path parameter
+def test_put_item_invalid_id():
+    data = {
+        "name": "Updated Item",
+        "description": "Updated item description",
+        "price": 20.0,
+        "tax": 2.5
+    }
+    response = client.put("/items/1001", json=data)
+    assert response.status_code == 422
+    assert "detail" in response.json()
+
+# Test PUT /items/{item_id} with invalid query parameter q
+def test_put_item_invalid_query():
+    data = {
+        "name": "Updated Item",
+        "description": "Updated item description",
+        "price": 20.0,
+        "tax": 2.5
+    }
+    response = client.put("/items/123?q=ab", json=data)
+    assert response.status_code == 422
+    assert "detail" in response.json()

.github/workflows/test_homework3.yml

@@ -0,0 +1,78 @@
+name: Test Homework 3
+
+on:
+  pull_request:
+    paths:
+      - 'members/**'
+
+permissions:
+  contents: read
+  pull-requests: read
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+        with:
+          fetch-depth: 0
+
+      - name: Set up Python 3.10
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.10'
+
+      - name: Install global dependencies
+        run: |
+          pip install fastapi pytest uvicorn
+
+      - name: Get list of changed submodules
+        id: changed_submodules
+        uses: actions/github-script@v6
+        with:
+          script: |
+            const pull_number = context.payload.pull_request.number;
+            const owner = context.repo.owner;
+            const repo = context.repo.repo;
+            const { data: files } = await github.rest.pulls.listFiles({
+              owner,
+              repo,
+              pull_number,
+            });
+            const changedSubmodules = files
+              .map(file => file.filename)
+              .filter(filename => filename.startsWith('members/'))
+              .map(filename => filename.split('/')[1])
+              .filter((value, index, self) => self.indexOf(value) === index);
+            
+            if (changedSubmodules.length > 1) {
+              core.setFailed(`More than one submodule changed: ${changedSubmodules.join(', ')}`);
+            } else if (changedSubmodules.length === 0) {
+              core.setFailed('No submodules were changed.');
+            } else {
+              core.setOutput('submodule', changedSubmodules[0]);
+            }
+
+      - name: Install dependencies
+        if: steps.changed_submodules.outputs.submodule != ''
+        run: |
+          submodule=${{ steps.changed_submodules.outputs.submodule }}
+          echo "Testing $submodule"
+
+      - name: Install dependencies
+        if: steps.changed_submodules.outputs.submodule != ''
+        run: |
+          submodule=${{ steps.changed_submodules.outputs.submodule }}
+          git submodule update --init --recursive members/$submodule
+          pip install -r members/$submodule/requirements.txt
+
+      - name: Test submodule
+        if: steps.changed_submodules.outputs.submodule != ''
+        run: |
+          submodule=${{ steps.changed_submodules.outputs.submodule }}
+          cd members/$submodule
+          cp ../../.github/scripts/test_homework3.py .
+          pytest test_homework3.py
+

.gitmodules

@@ -61,7 +61,7 @@
 [submodule "members/Lyc1103"]
 	path = members/Lyc1103
 	url = https://github.com/Lyc1103/SDC-backend-training.git
-
 [submodule "members/weng1002"]
 	path = members/weng1002
 	url = https://github.com/Weng1002/sdc-backend-training.git
+