Improve test reproducibility for price_lists

Author: albertsola

tests/e2e/catalog/price_lists/items/conftest.py

@@ -21,6 +21,5 @@ def price_list_item_data(short_uuid):
 
 
 @pytest.fixture
-def price_list_item(price_list_items_service):
-    price_list_items = price_list_items_service.fetch_page(1)
-    return price_list_items[0]
+def price_list_id(e2e_config):
+    return e2e_config["catalog.price_list.id"]

tests/e2e/catalog/price_lists/items/test_async_price_list_items.py

@@ -8,22 +8,22 @@
 pytestmark = [pytest.mark.flaky]
 
 
-async def test_get_price_list_item(async_price_list_items_service, price_list_item):
-    result = await async_price_list_items_service.get(price_list_item.id)
+async def test_get_price_list_item(async_price_list_items_service, price_list_id):
+    result = await async_price_list_items_service.get(price_list_id)
 
-    assert result.id == price_list_item.id
+    assert result.id == price_list_id
 
 
-async def test_filter_price_list_items(async_price_list_items_service, price_list_item):
+async def test_filter_price_list_items(async_price_list_items_service, price_list_id):
     await assert_async_service_filter_with_iterate(
-        async_price_list_items_service, price_list_item.id, ["-notes"]
+        async_price_list_items_service, price_list_id, ["-notes"]
     )  # act
 
 
 async def test_update_price_list_item(
-    async_price_list_items_service, price_list_item, price_list_item_data
+    async_price_list_items_service, price_list_id, price_list_item_data
 ):
-    result = await async_price_list_items_service.update(price_list_item.id, price_list_item_data)
+    result = await async_price_list_items_service.update(price_list_id, price_list_item_data)
 
     assert result.reason_for_change == price_list_item_data["reasonForChange"]
 
@@ -35,8 +35,8 @@ async def test_get_price_list_item_not_found(async_price_list_items_service):
         await async_price_list_items_service.get(bogus_id)
 
 
-async def test_create_price_list_item_invalid_data(async_price_list_items_service, price_list_item):
+async def test_create_price_list_item_invalid_data(async_price_list_items_service, price_list_id):
     invalid_data = {"unitPP": "0"}
 
     with pytest.raises(MPTAPIError, match=r"400 Bad Request"):
-        await async_price_list_items_service.update(price_list_item.id, invalid_data)
+        await async_price_list_items_service.update(price_list_id, invalid_data)

tests/e2e/catalog/price_lists/items/test_sync_price_list_items.py

@@ -8,20 +8,18 @@
 pytestmark = [pytest.mark.flaky]
 
 
-def test_get_price_list_item(price_list_items_service, price_list_item):
-    result = price_list_items_service.get(price_list_item.id)
+def test_get_price_list_item(price_list_items_service, price_list_id):
+    result = price_list_items_service.get(price_list_id)
 
-    assert result.id == price_list_item.id
+    assert result.id == price_list_id
 
 
-def test_filter_price_list_items(price_list_items_service, price_list_item):
-    assert_service_filter_with_iterate(
-        price_list_items_service, price_list_item.id, ["-notes"]
-    )  # act
+def test_filter_price_list_items(price_list_items_service, price_list_id):
+    assert_service_filter_with_iterate(price_list_items_service, price_list_id, ["-notes"])  # act
 
 
-def test_update_price_list_item(price_list_items_service, price_list_item, price_list_item_data):
-    result = price_list_items_service.update(price_list_item.id, price_list_item_data)
+def test_update_price_list_item(price_list_items_service, price_list_id, price_list_item_data):
+    result = price_list_items_service.update(price_list_id, price_list_item_data)
 
     assert result.reason_for_change == price_list_item_data["reasonForChange"]
 
@@ -33,8 +31,8 @@ def test_get_price_list_item_not_found(price_list_items_service):
         price_list_items_service.get(bogus_id)
 
 
-def test_create_price_list_item_invalid_data(price_list_items_service, price_list_item):
+def test_create_price_list_item_invalid_data(price_list_items_service, price_list_id):
     invalid_data = {"unitPP": "0"}
 
     with pytest.raises(MPTAPIError, match=r"400 Bad Request"):
-        price_list_items_service.update(price_list_item.id, invalid_data)
+        price_list_items_service.update(price_list_id, invalid_data)