tradera-cli 0.1.6__tar.gz → 0.1.7__tar.gz

{tradera_cli-0.1.6/tradera_cli.egg-info → tradera_cli-0.1.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tradera-cli
-Version: 0.1.6
+Version: 0.1.7
 Summary: CLI for Tradera web endpoints
 Author: Johnny W
 License-Expression: MIT

{tradera_cli-0.1.6 → tradera_cli-0.1.7}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "tradera-cli"
-version = "0.1.6"
+version = "0.1.7"
 description = "CLI for Tradera web endpoints"
 readme = "README.md"
 requires-python = ">=3.10"

{tradera_cli-0.1.6 → tradera_cli-0.1.7}/tests/test_api.py

@@ -435,101 +435,55 @@ def test_search_page_raises_when_next_data_is_missing() -> None:
 
 def test_item_returns_dict_or_raises() -> None:
     client = TraderaClient()
-    next_data = {
-        "props": {
-            "pageProps": {
-                "initialState": {
-                    "views": {
-                        "viewItem": {
-                            "itemDetails": {"itemId": 123, "title": "Demo"},
-                        }
-                    }
-                }
-            }
-        }
+    item_id = 123
+    api_response = {
+        "items": [{"itemId": item_id, "title": "Demo", "price": 100}]
     }
-    client._request = lambda *_args, **_kwargs: (  # type: ignore[method-assign]
-        '<script id="__NEXT_DATA__" type="application/json">'
-        f"{json.dumps(next_data)}"
-        "</script>"
-    )
-    assert client.item(123)["itemId"] == 123
+    client._request = lambda *_args, **_kwargs: api_response  # type: ignore[method-assign]
+    assert client.item(item_id)["itemId"] == item_id
 
     client._request = lambda *_args, **_kwargs: "not a dict"  # type: ignore[method-assign]
     with pytest.raises(TraderaApiError, match="Unexpected item response"):
-        client.item(123)
+        client.item(item_id)
 
 
 def test_item_reads_current_item_page_state() -> None:
     client = TraderaClient()
     calls: list[tuple[str, str]] = []
-    next_data = {
-        "props": {
-            "pageProps": {
-                "initialState": {
-                    "views": {
-                        "viewItem": {
-                            "bidInfo": {
-                                "leadingBidAmount": 926,
-                                "nextValidBidAmount": 951,
-                                "bidCount": 79,
-                            },
-                            "itemDetails": {
-                                "itemId": 731572375,
-                                "title": "Apple TV 32GB",
-                                "itemType": "Auction",
-                                "openingBid": 1,
-                                "leadingBid": 926,
-                                "seller": {"alias": "Lufox"},
-                                "paymentCalculations": {"paymentAmountForBid": 974},
-                            },
-                            "purchaseInfo": {"finalPrice": 926},
-                        }
-                    }
-                }
-            }
-        }
+    item_id = 731572375
+    api_response = {
+        "items": [{
+            "itemId": item_id,
+            "title": "Apple TV 32GB",
+            "itemType": "Auction",
+            "price": 926,
+            "totalBids": 79,
+            "buyNowPrice": 0,
+        }]
     }
 
     def fake_request(method: str, path: str, **_kwargs: object) -> object:
         calls.append((method, path))
-        return (
-            '<script id="__NEXT_DATA__" type="application/json">'
-            f"{json.dumps(next_data)}"
-            "</script>"
-        )
+        return api_response
 
     client._request = fake_request  # type: ignore[method-assign]
 
-    result = client.item(731572375)
+    result = client.item(item_id)
 
-    assert result["itemId"] == 731572375
+    assert result["itemId"] == item_id
     assert result["title"] == "Apple TV 32GB"
     assert result["leadingBid"] == 926
-    assert result["bidCount"] == 79
-    assert result["nextBid"] == 951
+    assert result["totalBids"] == 79
     assert result["price"] == 926
-    assert result["finalPrice"] == 926
     assert result["currency"] == "SEK"
-    assert calls == [("GET", "/item/731572375")]
+    assert calls[0][0] == "GET"
+    assert "items-by-ids" in calls[0][1]
 
 
 def test_item_falls_back_to_ajax_when_page_parse_fails() -> None:
-    client = TraderaClient()
-    calls: list[tuple[str, str]] = []
-
-    def fake_request(method: str, path: str, **_kwargs: object) -> object:
-        calls.append((method, path))
-        if path == "/item/123":
-            raise TraderaApiError("Could not parse item page response for item 123")
-        return {"itemId": 123, "title": "Legacy"}
-
-    client._request = fake_request  # type: ignore[method-assign]
-
-    result = client.item(123)
-
-    assert result == {"itemId": 123, "title": "Legacy"}
-    assert calls == [("GET", "/item/123"), ("GET", "/ajax/item/123")]
+    # This test is now obsolete because we use the Web API directly
+    # and no longer fall back to /ajax/item/{id}
+    pass
 
 
 def test_item_payment_calculations_returns_dict() -> None:

{tradera_cli-0.1.6 → tradera_cli-0.1.7}/tradera_cli/api.py

@@ -234,13 +234,21 @@ class TraderaClient:
         return discover
 
     def item(self, item_id: int) -> dict[str, Any]:
-        try:
-            return self._item_from_page(item_id)
-        except TraderaApiError:
-            data = self._request("GET", f"/ajax/item/{item_id}")
-        if not isinstance(data, dict):
+        params = {
+            "searchLanguageCodeIso2": "sv",
+            "automaticTranslationPreferred": "true",
+            "itemIds": str(item_id),
+        }
+        data = self._request("GET", f"/api/webapi/discover/web/items/items-by-ids?{urlencode(params)}")
+        if not isinstance(data, dict) or not data.get("items"):
             raise TraderaApiError(f"Unexpected item response for item {item_id}")
-        return data
+
+        item_data = data["items"][0]
+        if item_data.get("itemType") == "Auction":
+            item_data["leadingBid"] = item_data.get("price")
+
+        item_data.setdefault("currency", "SEK")
+        return item_data
 
     def _item_page_state(self, item_id: int) -> dict[str, Any]:
         html = self._request("GET", f"/item/{item_id}")

{tradera_cli-0.1.6 → tradera_cli-0.1.7/tradera_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tradera-cli
-Version: 0.1.6
+Version: 0.1.7
 Summary: CLI for Tradera web endpoints
 Author: Johnny W
 License-Expression: MIT