Fix 'min' undefined error in /files view by adding min/max to Jinja2 globals

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>

app/views/base.py

@@ -15,6 +15,10 @@ from app.config import settings
 templates_dir = Path(__file__).parent.parent.parent / "frontend" / "templates"
 templates = Jinja2Templates(directory=str(templates_dir))
 
+# Add Python built-in functions to Jinja2 template globals
+templates.env.globals["min"] = min
+templates.env.globals["max"] = max
+
 # Customize Jinja2Templates to include app_version in all templates
 original_template_response = templates.TemplateResponse
 

tests/test_files_view.py

@@ -0,0 +1,69 @@
+"""
+Test the /files view UI endpoint to ensure template rendering works correctly.
+"""
+import pytest
+from fastapi.testclient import TestClient
+from app.models import FileRecord
+
+
+@pytest.mark.integration
+@pytest.mark.requires_db
+class TestFilesView:
+    """Tests for the /files UI view."""
+    
+    def test_files_view_renders_without_error(self, client: TestClient, db_session):
+        """Test that the /files view renders without 'min' undefined error."""
+        # Create some test files to ensure pagination works
+        for i in range(10):
+            file_record = FileRecord(
+                filehash=f"hash{i}",
+                original_filename=f"test{i}.pdf",
+                local_filename=f"/tmp/test{i}.pdf",
+                file_size=1024 * (i + 1),
+                mime_type="application/pdf"
+            )
+            db_session.add(file_record)
+        db_session.commit()
+        
+        # Access the /files view
+        response = client.get("/files")
+        assert response.status_code == 200
+        
+        # Check that the response is HTML
+        assert "text/html" in response.headers.get("content-type", "")
+        
+        # Check that the response contains expected content
+        content = response.text
+        assert "File Records" in content
+        
+        # Ensure no 'min' is undefined error - this is the key fix
+        assert "'min' is undefined" not in content.lower()
+        assert "'max' is undefined" not in content.lower()
+    
+    def test_files_view_pagination_with_many_pages(self, client: TestClient, db_session):
+        """Test that pagination works correctly with many pages."""
+        # Create enough files to span multiple pages (e.g., 150 files with 50 per page = 3 pages)
+        for i in range(150):
+            file_record = FileRecord(
+                filehash=f"hash{i}",
+                original_filename=f"test{i}.pdf",
+                local_filename=f"/tmp/test{i}.pdf",
+                file_size=1024,
+                mime_type="application/pdf"
+            )
+            db_session.add(file_record)
+        db_session.commit()
+        
+        # Access the /files view with pagination
+        response = client.get("/files?page=2&per_page=50")
+        assert response.status_code == 200
+        
+        content = response.text
+        # Check for pagination elements
+        assert "Showing" in content  # Pagination info
+        assert "of 150 files" in content
+        
+        # Ensure the min/max functions work in the template (they're used for pagination)
+        # The page should render successfully without JavaScript errors
+        assert "'min' is undefined" not in content.lower()
+        assert "'max' is undefined" not in content.lower()