Add unit tests for book_review.py and app.py

tests/__init__.py

@@ -0,0 +1 @@
+

tests/test_app.py

@@ -0,0 +1,137 @@
+from unittest.mock import patch, MagicMock
+import json
+import pytest
+
+
+@pytest.fixture
+def client():
+    """Create a Flask test client with mocked book_review module."""
+    with patch("book_review.Api"), \
+         patch("book_review.api"), \
+         patch("book_review.table"):
+        from app import app
+        app.config["TESTING"] = True
+        with app.test_client() as client:
+            yield client
+
+
+class TestUppercaseEndpoint:
+    def test_converts_text_to_uppercase(self, client):
+        response = client.get("/uppercase?text=hello")
+        data = response.get_json()
+
+        assert response.status_code == 200
+        assert data["text"] == "HELLO"
+
+    def test_converts_mixed_case(self, client):
+        response = client.get("/uppercase?text=Hello+World")
+        data = response.get_json()
+
+        assert response.status_code == 200
+        assert data["text"] == "HELLO WORLD"
+
+    def test_already_uppercase(self, client):
+        response = client.get("/uppercase?text=ALREADY")
+        data = response.get_json()
+
+        assert response.status_code == 200
+        assert data["text"] == "ALREADY"
+
+    def test_empty_string(self, client):
+        response = client.get("/uppercase?text=")
+        data = response.get_json()
+
+        assert response.status_code == 200
+        assert data["text"] == ""
+
+    def test_special_characters(self, client):
+        response = client.get("/uppercase?text=hello%21%40%23")
+        data = response.get_json()
+
+        assert response.status_code == 200
+        assert data["text"] == "HELLO!@#"
+
+
+class TestRecordsEndpoint:
+    @patch("book_review.get_all_records")
+    def test_returns_books(self, mock_get_all, client):
+        mock_get_all.return_value = [
+            {"id": "rec1", "fields": {"Book": "Book A", "Rating": 8}},
+        ]
+
+        response = client.get("/records")
+        data = response.get_json()
+
+        assert response.status_code == 200
+        assert "books" in data
+        assert len(data["books"]) == 1
+
+    @patch("book_review.get_all_records")
+    def test_passes_count_and_sort(self, mock_get_all, client):
+        mock_get_all.return_value = []
+
+        client.get("/records?count=5&sort=ASC")
+
+        mock_get_all.assert_called_once_with(count="5", sort="ASC")
+
+    @patch("book_review.get_all_records")
+    def test_returns_empty_list(self, mock_get_all, client):
+        mock_get_all.return_value = []
+
+        response = client.get("/records")
+        data = response.get_json()
+
+        assert response.status_code == 200
+        assert data["books"] == []
+
+
+class TestAddRecordEndpoint:
+    @patch("book_review.add_record")
+    def test_adds_record_successfully(self, mock_add, client):
+        mock_add.return_value = True
+
+        response = client.post(
+            "/add-record",
+            data=json.dumps({"Book": "Test Book", "Rating": 8}),
+            content_type="application/json",
+        )
+        data = response.get_json()
+
+        assert response.status_code == 200
+        assert data["message"] == "Record added successfully"
+
+    @patch("book_review.add_record")
+    def test_returns_400_when_book_missing(self, mock_add, client):
+        response = client.post(
+            "/add-record",
+            data=json.dumps({"Rating": 8}),
+            content_type="application/json",
+        )
+        data = response.get_json()
+
+        assert response.status_code == 400
+
+    @patch("book_review.add_record")
+    def test_returns_400_when_rating_missing(self, mock_add, client):
+        response = client.post(
+            "/add-record",
+            data=json.dumps({"Book": "Test Book"}),
+            content_type="application/json",
+        )
+        data = response.get_json()
+
+        assert response.status_code == 400
+
+    @patch("book_review.add_record")
+    def test_returns_500_when_add_fails(self, mock_add, client):
+        mock_add.return_value = False
+
+        response = client.post(
+            "/add-record",
+            data=json.dumps({"Book": "Test Book", "Rating": 8}),
+            content_type="application/json",
+        )
+        data = response.get_json()
+
+        assert response.status_code == 500
+        assert data["message"] == "Failed to add record"

tests/test_book_review.py

@@ -0,0 +1,144 @@
+from unittest.mock import patch, MagicMock
+import pytest
+
+
+class TestGetAllRecords:
+    @patch("book_review.table")
+    def test_returns_all_records_no_params(self, mock_table):
+        import book_review
+
+        mock_table.all.return_value = [
+            {"id": "rec1", "fields": {"Book": "Book A", "Rating": 8}},
+            {"id": "rec2", "fields": {"Book": "Book B", "Rating": 5}},
+        ]
+
+        result = book_review.get_all_records()
+
+        mock_table.all.assert_called_once_with(max_records=None, sort=[])
+        assert len(result) == 2
+
+    @patch("book_review.table")
+    def test_returns_limited_records_with_count(self, mock_table):
+        import book_review
+
+        mock_table.all.return_value = [
+            {"id": "rec1", "fields": {"Book": "Book A", "Rating": 8}},
+        ]
+
+        result = book_review.get_all_records(count=1)
+
+        mock_table.all.assert_called_once_with(max_records=1, sort=[])
+        assert len(result) == 1
+
+    @patch("book_review.table")
+    def test_sorts_ascending(self, mock_table):
+        import book_review
+
+        mock_table.all.return_value = []
+
+        book_review.get_all_records(sort="ASC")
+
+        mock_table.all.assert_called_once_with(max_records=None, sort=["Rating"])
+
+    @patch("book_review.table")
+    def test_sorts_descending(self, mock_table):
+        import book_review
+
+        mock_table.all.return_value = []
+
+        book_review.get_all_records(sort="DESC")
+
+        mock_table.all.assert_called_once_with(max_records=None, sort=["-Rating"])
+
+    @patch("book_review.table")
+    def test_sort_case_insensitive(self, mock_table):
+        import book_review
+
+        mock_table.all.return_value = []
+
+        book_review.get_all_records(sort="desc")
+
+        mock_table.all.assert_called_once_with(max_records=None, sort=["-Rating"])
+
+    @patch("book_review.table")
+    def test_invalid_sort_ignored(self, mock_table):
+        import book_review
+
+        mock_table.all.return_value = []
+
+        book_review.get_all_records(sort="INVALID")
+
+        mock_table.all.assert_called_once_with(max_records=None, sort=[])
+
+    @patch("book_review.table")
+    def test_count_and_sort_together(self, mock_table):
+        import book_review
+
+        mock_table.all.return_value = []
+
+        book_review.get_all_records(count=5, sort="ASC")
+
+        mock_table.all.assert_called_once_with(max_records=5, sort=["Rating"])
+
+
+class TestGetRecordId:
+    @patch("book_review.table")
+    def test_returns_record_id(self, mock_table):
+        import book_review
+
+        mock_table.first.return_value = {"id": "rec123", "fields": {"Book": "Test"}}
+
+        result = book_review.get_record_id("Test")
+
+        mock_table.first.assert_called_once_with(formula="Book='Test'")
+        assert result == "rec123"
+
+
+class TestUpdateRecord:
+    @patch("book_review.table")
+    def test_updates_and_returns_true(self, mock_table):
+        import book_review
+
+        result = book_review.update_record("rec123", {"Rating": 9})
+
+        mock_table.update.assert_called_once_with("rec123", {"Rating": 9})
+        assert result is True
+
+
+class TestAddRecord:
+    @patch("book_review.table")
+    def test_adds_valid_record(self, mock_table):
+        import book_review
+
+        data = {"Book": "New Book", "Rating": 7}
+        result = book_review.add_record(data)
+
+        mock_table.create.assert_called_once_with(data)
+        assert result is True
+
+    @patch("book_review.table")
+    def test_rejects_missing_book(self, mock_table):
+        import book_review
+
+        result = book_review.add_record({"Rating": 7})
+
+        mock_table.create.assert_not_called()
+        assert result is False
+
+    @patch("book_review.table")
+    def test_rejects_missing_rating(self, mock_table):
+        import book_review
+
+        result = book_review.add_record({"Book": "Some Book"})
+
+        mock_table.create.assert_not_called()
+        assert result is False
+
+    @patch("book_review.table")
+    def test_rejects_empty_dict(self, mock_table):
+        import book_review
+
+        result = book_review.add_record({})
+
+        mock_table.create.assert_not_called()
+        assert result is False