3rd

Author: wahidpanda

.github/workflows/cicd.yml

@@ -23,8 +23,11 @@ jobs:
           python -m pip install --upgrade pip
           pip install flask pytest
 
+      - name: Debug: List Files
+        run: ls -R  # Helps debug missing files issue
+
       - name: Run tests
-        run: pytest
+        run: pytest -v --tb=short  # Verbose mode for better debugging
 
   build-and-publish:
     needs: build-and-test
@@ -51,6 +54,4 @@ jobs:
           tags: ${{ secrets.DOCKER_USERNAME }}/flasktest-app:latest
 
       - name: Image digest
-        run: echo ${{ steps.docker_build.outputs.digest }}
-
-## modified
+        run: echo "Docker Image Digest: ${{ steps.docker_build.outputs.digest }}"

DockerFile

@@ -1,11 +1,17 @@
+# Use slim Python image
 FROM python:3.9-slim
 
+# Set working directory
 WORKDIR /app
 
+# Copy application files
 COPY . /app
 
-RUN pip install flask
+# Install dependencies
+RUN pip install flask pytest
 
+# Expose port 5000 for Flask
 EXPOSE 5000
 
+# Start Flask app
 CMD ["python", "app.py"]

README.md

@@ -1 +1,5 @@
-## Create an End-to-End MLops project with GitHub action workflow and DockerHub integration.
+<<<<<<< HEAD
+## Create an End-to-End MLops project with GitHub action workflow and DockerHub integration.
+=======
+## Create an End-to-End MLops project with GitHub action workflow and DockerHub integration.
+>>>>>>> c234883201868eb611a71cc098f9a947d9c6771c

app.py

@@ -1,12 +1,10 @@
 from flask import Flask
 
-
-app= Flask(__name__)
+app = Flask(__name__)
 
 @app.route("/")
 def home():
-    return "hello world"    
+    return "Hello, World!"  # Fix to match test case
 
-if __name__=="__main__":
-    app.run(debug=True)
-    
+if __name__ == "__main__":
+    app.run(debug=True, host="0.0.0.0", port=5000)  # Ensure it's accessible in Docker

test_app.py

@@ -0,0 +1,7 @@
+from app import app
+
+def test_home():
+    client = app.test_client()
+    response = client.get('/')
+    assert response.status_code == 200
+    assert response.data == b"Hello, World!"  # Match Flask response