revised testing critera

models.py

@@ -2,17 +2,30 @@ from flask_sqlalchemy import SQLAlchemy
 
 db = SQLAlchemy()
 
+
 class Feature(db.Model):
     id = db.Column(db.Integer, primary_key=True)
     title = db.Column(db.String(100), nullable=False)
     description = db.Column(db.String(200), nullable=False)
 
+    @staticmethod
     def seed_db():
-    if Feature.query.count() == 0:  # Seed only if empty
+        # Seed only if empty
+        if Feature.query.count() == 0:
             features = [
-            Feature(title="Responsive Design", description="Adapts seamlessly to mobile, tablet, and desktop devices."),
-            Feature(title="Modern UI", description="Uses Bootstrap 5 for clean, professional styling."),
-            Feature(title="Easy Deployment", description="Containerized with Docker for quick setup on any server.")
+                Feature(
+                    title="Responsive Design",
+                    description="Adapts seamlessly to mobile, tablet, and desktop devices.",
+                ),
+                Feature(
+                    title="Modern UI",
+                    description="Uses Bootstrap 5 for clean, professional styling.",
+                ),
+                Feature(
+                    title="Easy Deployment",
+                    description="Containerized with Docker for quick setup on any server.",
+                ),
             ]
             db.session.bulk_save_objects(features)
             db.session.commit()
+

test_app.py

@@ -1,27 +1,40 @@
-import pytest
 import os
+import pytest
+
+# Ensure tests always use the in-memory SQLite DB, even if CI sets DB_* secrets
+for var in ("DB_USER", "DB_PASS", "DB_NAME"):
+    os.environ.pop(var, None)
+
 from app import app
 from models import db, Feature
 
+
 @pytest.fixture
 def client():
-    app.config['TESTING'] = True
-    app.config['SQLALCHEMY_DATABASE_URI'] = 'sqlite:///:memory:'
-    app.config['SQLALCHEMY_TRACK_MODIFICATIONS'] = False
-    db.init_app(app)  # Re-init DB with test config
+    app.config["TESTING"] = True
+    # app was already initialised with SQLite because DB_USER is now unset
+
+    # Fresh DB per test
+    with app.app_context():
+        db.drop_all()
+        db.create_all()
 
     with app.test_client() as client:
-        with app.app_context():
-            db.create_all()
         yield client
+
+    # Clean up after test
+    with app.app_context():
         db.drop_all()
 
-def test_index(client):
-    response = client.get('/')
-    assert response.status_code == 200
-    assert b'Professional Demo Site' in response.data
 
-def test_seed_db(client):  # Add fixture here
+def test_index(client):
+    response = client.get("/")
+    assert response.status_code == 200
+    assert b"Professional Demo Site" in response.data
+
+
+def test_seed_db(client):
     with app.app_context():
         Feature.seed_db()
         assert Feature.query.count() == 3
+