test: restore 100% coverage after search and web changes

- Add tests for /public-search edge cases (malformed, no condition_id, empty)
- Add tests for get_categories API and web route
- Add tests for web command --reload flag
- Add tests for _serialize list/dict branches and _trade_to_dict
- Mark app.py main() as pragma: no cover (standalone entry point)

Author: qhuang20

pm_trader/web/app.py

@@ -72,7 +72,7 @@ def health() -> dict:
     return app
 
 
-def main() -> None:
+def main() -> None:  # pragma: no cover
     import uvicorn
 
     uvicorn.run(create_app(), host="127.0.0.1", port=8000)

tests/test_api.py

@@ -695,6 +695,74 @@ def test_empty_events_returns_empty(
         result = client.search_markets("bitcoin")
         assert result == []
 
+    def test_malformed_market_skipped(
+        self, client: PolymarketClient, httpx_mock
+    ):
+        """Markets that fail to parse are skipped (KeyError/TypeError)."""
+        # outcomes as an int triggers TypeError in json.loads / iteration
+        bad_market = {"condition_id": "0xbad", "outcomes": 999, "outcomePrices": 999}
+        httpx_mock.add_response(json={
+            "events": [{"markets": [bad_market, SAMPLE_GAMMA_MARKET]}],
+            "pagination": {},
+        })
+        result = client.search_markets("bitcoin")
+        assert len(result) == 1
+        assert result[0].slug == "will-bitcoin-hit-100k"
+
+    def test_market_without_condition_id_skipped(
+        self, client: PolymarketClient, httpx_mock
+    ):
+        """Markets without condition_id are skipped."""
+        no_cid = {**SAMPLE_GAMMA_MARKET, "condition_id": "", "conditionId": ""}
+        httpx_mock.add_response(json={
+            "events": [{"markets": [no_cid]}],
+            "pagination": {},
+        })
+        result = client.search_markets("bitcoin")
+        assert result == []
+
+
+# ---------------------------------------------------------------------------
+# get_categories tests
+# ---------------------------------------------------------------------------
+
+class TestGetCategories:
+    def test_get_categories(self, client: PolymarketClient, httpx_mock):
+        httpx_mock.add_response(json=[
+            {"label": "Politics", "slug": "politics"},
+            {"label": "Sports", "slug": "sports", "parentCategory": "top"},
+        ])
+        result = client.get_categories(top_level_only=True)
+        assert len(result) == 1
+        assert result[0]["label"] == "Politics"
+
+    def test_get_categories_all(self, client: PolymarketClient, httpx_mock):
+        httpx_mock.add_response(json=[
+            {"label": "Politics", "slug": "politics"},
+            {"label": "Sports", "slug": "sports", "parentCategory": "top"},
+        ])
+        result = client.get_categories(top_level_only=False)
+        assert len(result) == 2
+
+    def test_get_categories_empty(self, client: PolymarketClient, httpx_mock):
+        httpx_mock.add_response(json=[])
+        result = client.get_categories()
+        assert result == []
+
+    def test_get_categories_non_list(self, client: PolymarketClient, httpx_mock):
+        httpx_mock.add_response(json={"error": "bad"})
+        result = client.get_categories()
+        assert result == []
+
+    def test_get_categories_cached(self, client: PolymarketClient, httpx_mock):
+        cats = [{"label": "Politics", "slug": "politics"}]
+        httpx_mock.add_response(json=cats)
+        result1 = client.get_categories()
+        assert len(result1) == 1
+        # Second call should use cache (no extra HTTP request)
+        result2 = client.get_categories()
+        assert len(result2) == 1
+
 
 # ---------------------------------------------------------------------------
 # Line 213: get_tick_size cache hit path

tests/test_cli.py

@@ -1077,3 +1077,24 @@ def test_web_command_exists(self):
         assert "Start the web dashboard server" in result.output
         assert "--port" in result.output
         assert "--host" in result.output
+        assert "--reload" in result.output
+
+    @patch("uvicorn.run")
+    def test_web_no_reload(self, mock_run):
+        runner = click.testing.CliRunner()
+        result = runner.invoke(main, ["web", "--port", "9999"])
+        assert result.exit_code == 0
+        assert "Starting dashboard" in result.output
+        mock_run.assert_called_once()
+        assert mock_run.call_args[1]["port"] == 9999
+        assert "reload" not in mock_run.call_args[1]
+
+    @patch("uvicorn.run")
+    def test_web_reload(self, mock_run):
+        runner = click.testing.CliRunner()
+        result = runner.invoke(main, ["web", "--port", "9999", "--reload"])
+        assert result.exit_code == 0
+        mock_run.assert_called_once_with(
+            "pm_trader.web.app:create_app", factory=True,
+            host="127.0.0.1", port=9999, reload=True,
+        )

tests/test_web/test_dashboard.py

@@ -77,6 +77,29 @@ def test_get_trades_with_filters(self, client, mock_engine):
             limit=10, offset=5, side="buy", market_slug="test-slug",
         )
 
+    def test_get_trades_with_data(self, client, mock_engine):
+        mock_trade = MagicMock()
+        mock_trade.id = 1
+        mock_trade.market_condition_id = "0xabc"
+        mock_trade.market_slug = "test-slug"
+        mock_trade.market_question = "Test?"
+        mock_trade.outcome = "yes"
+        mock_trade.side = "buy"
+        mock_trade.order_type = "fok"
+        mock_trade.avg_price = 0.65
+        mock_trade.amount_usd = 100.0
+        mock_trade.shares = 153.85
+        mock_trade.fee_rate_bps = 200
+        mock_trade.fee_paid = 1.30
+        mock_trade.timestamp = "2026-03-12T00:00:00"
+        mock_engine.db = MagicMock()
+        mock_engine.db.get_trades.return_value = [mock_trade]
+        resp = client.get("/api/trades")
+        assert resp.status_code == 200
+        data = resp.json()["data"]
+        assert len(data) == 1
+        assert data[0]["market_slug"] == "test-slug"
+
 
 class TestChartPnl:
     def test_chart_pnl(self, client, mock_engine):

tests/test_web/test_markets.py

@@ -4,6 +4,7 @@
 from fastapi.testclient import TestClient
 from pm_trader.web.app import create_app
 from pm_trader.models import OrderBook, TradeResult
+from pm_trader.web.routes.markets import _serialize
 
 @pytest.fixture
 def client():
@@ -22,6 +23,25 @@ def test_search(self, client, mock_engine):
         assert resp.status_code == 200
         assert resp.json()["ok"] is True
 
+    def test_list_by_tag(self, client, mock_engine):
+        mock_engine.api.get_markets_by_tag.return_value = []
+        resp = client.get("/api/markets?tag=politics")
+        assert resp.status_code == 200
+        assert resp.json()["ok"] is True
+        mock_engine.api.get_markets_by_tag.assert_called_once()
+
+class TestGetCategories:
+    def test_categories(self, client, mock_engine):
+        mock_engine.api.get_categories.return_value = [
+            {"label": "Politics", "slug": "politics"},
+            {"label": "Sports", "slug": "sports"},
+        ]
+        resp = client.get("/api/categories")
+        assert resp.status_code == 200
+        data = resp.json()["data"]
+        assert len(data) == 2
+        assert data[0]["label"] == "Politics"
+
 class TestGetTags:
     def test_tags(self, client, mock_engine):
         mock_engine.api.get_tags.return_value = [{"label": "Politics", "slug": "politics"}]
@@ -53,3 +73,14 @@ def test_sell(self, client, mock_engine):
         mock_engine.sell.return_value = TradeResult(trade=mock_trade, account=mock_account)
         resp = client.post("/api/sell", json={"slug": "test-market", "outcome": "yes", "shares": 100.0, "order_type": "fok"})
         assert resp.status_code == 200
+
+
+class TestSerialize:
+    def test_serialize_list(self):
+        assert _serialize([1, 2, 3]) == [1, 2, 3]
+
+    def test_serialize_dict(self):
+        assert _serialize({"a": 1}) == {"a": 1}
+
+    def test_serialize_nested(self):
+        assert _serialize({"a": [1, {"b": 2}]}) == {"a": [1, {"b": 2}]}