Align patrol delete module naming
Rename the patrol deletion implementation and browser helper scripts away from the generic product name so the filesystem matches the queued patrol-delete task domain. Update imports, script loading, and tests to use the patrol_delete module path. Constraint: Existing task class remains ProductTask to avoid broad API churn beyond the requested file and script naming Rejected: Rename ProductTask class now | would expand the change into a larger API update across tests and task dispatch Confidence: high Scope-risk: narrow Directive: Keep patrol-delete script paths under app/amazon/scripts/patrol_delete when adding new helper scripts Tested: uv run pytest tests/test_product.py Not-tested: Remote push before this commit due previous SSH publickey failure
This commit is contained in:
@@ -10,7 +10,7 @@ from amazon.approve import ApproveTask
|
|||||||
from amazon.match_action import MatchTak
|
from amazon.match_action import MatchTak
|
||||||
from amazon.price_match import PriceTask
|
from amazon.price_match import PriceTask
|
||||||
from amazon.asin_status import StatusTask
|
from amazon.asin_status import StatusTask
|
||||||
from amazon.product import ProductTask
|
from amazon.patrol_delete import ProductTask
|
||||||
from amazon.detail_spider import SpiderTask
|
from amazon.detail_spider import SpiderTask
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -30,25 +30,25 @@ RECOMMENDED_OFFER_EXPAND_XPATHS = [
|
|||||||
]
|
]
|
||||||
|
|
||||||
|
|
||||||
SCRIPT_DIR = Path(__file__).with_name("scripts") / "product"
|
SCRIPT_DIR = Path(__file__).with_name("scripts") / "patrol_delete"
|
||||||
|
|
||||||
|
|
||||||
def _load_product_script(name: str) -> str:
|
def _load_patrol_delete_script(name: str) -> str:
|
||||||
return (SCRIPT_DIR / name).read_text(encoding="utf-8")
|
return (SCRIPT_DIR / name).read_text(encoding="utf-8")
|
||||||
|
|
||||||
|
|
||||||
LISTING_STATUS_DROPDOWN_DEBUG_SCRIPT = _load_product_script("listing_status_dropdown_debug.js")
|
LISTING_STATUS_DROPDOWN_DEBUG_SCRIPT = _load_patrol_delete_script("listing_status_dropdown_debug.js")
|
||||||
LISTING_STATUS_DROPDOWN_OPEN_SCRIPT = _load_product_script("listing_status_dropdown_open.js")
|
LISTING_STATUS_DROPDOWN_OPEN_SCRIPT = _load_patrol_delete_script("listing_status_dropdown_open.js")
|
||||||
LISTING_STATUS_DROPDOWN_OPTIONS_SCRIPT = _load_product_script("listing_status_dropdown_options.js")
|
LISTING_STATUS_DROPDOWN_OPTIONS_SCRIPT = _load_patrol_delete_script("listing_status_dropdown_options.js")
|
||||||
LISTING_STATUS_CLICK_VISIBLE_STATUS_SCRIPT = _load_product_script("listing_status_click_visible_status.js")
|
LISTING_STATUS_CLICK_VISIBLE_STATUS_SCRIPT = _load_patrol_delete_script("listing_status_click_visible_status.js")
|
||||||
LISTING_STATUS_SELECT_OPTION_SCRIPT = _load_product_script("listing_status_select_option.js")
|
LISTING_STATUS_SELECT_OPTION_SCRIPT = _load_patrol_delete_script("listing_status_select_option.js")
|
||||||
LISTING_STATUS_SECTION_SCRIPT = _load_product_script("listing_status_section.js")
|
LISTING_STATUS_SECTION_SCRIPT = _load_patrol_delete_script("listing_status_section.js")
|
||||||
COMPLETE_DRAFT_QUICK_VIEW_DEBUG_SCRIPT = _load_product_script("complete_draft_quick_view_debug.js")
|
COMPLETE_DRAFT_QUICK_VIEW_DEBUG_SCRIPT = _load_patrol_delete_script("complete_draft_quick_view_debug.js")
|
||||||
COMPLETE_DRAFT_QUICK_VIEW_TAGS_SCRIPT = _load_product_script("complete_draft_quick_view_tags.js")
|
COMPLETE_DRAFT_QUICK_VIEW_TAGS_SCRIPT = _load_patrol_delete_script("complete_draft_quick_view_tags.js")
|
||||||
COMPLETE_DRAFT_QUICK_VIEW_DIAGNOSTIC_SCRIPT = _load_product_script("complete_draft_quick_view_diagnostic.js")
|
COMPLETE_DRAFT_QUICK_VIEW_DIAGNOSTIC_SCRIPT = _load_patrol_delete_script("complete_draft_quick_view_diagnostic.js")
|
||||||
RECOMMENDED_OFFER_PERCENTAGE_OPEN_SCRIPT = _load_product_script("recommended_offer_percentage_open.js")
|
RECOMMENDED_OFFER_PERCENTAGE_OPEN_SCRIPT = _load_patrol_delete_script("recommended_offer_percentage_open.js")
|
||||||
RECOMMENDED_OFFER_PERCENTAGE_ROWS_SCRIPT = _load_product_script("recommended_offer_percentage_rows.js")
|
RECOMMENDED_OFFER_PERCENTAGE_ROWS_SCRIPT = _load_patrol_delete_script("recommended_offer_percentage_rows.js")
|
||||||
RECOMMENDED_OFFER_PERCENTAGE_DIAGNOSTIC_SCRIPT = _load_product_script("recommended_offer_percentage_diagnostic.js")
|
RECOMMENDED_OFFER_PERCENTAGE_DIAGNOSTIC_SCRIPT = _load_patrol_delete_script("recommended_offer_percentage_diagnostic.js")
|
||||||
|
|
||||||
LISTING_STATUS_SELECT_DELAY = 0.8
|
LISTING_STATUS_SELECT_DELAY = 0.8
|
||||||
INVENTORY_ROW_TIMEOUT = 10
|
INVENTORY_ROW_TIMEOUT = 10
|
||||||
@@ -1,4 +1,4 @@
|
|||||||
from amazon.product import InventoryManage, ProductTask
|
from amazon.patrol_delete import InventoryManage, ProductTask
|
||||||
|
|
||||||
|
|
||||||
def test_product_task_post_result_builds_patrol_delete_payload(monkeypatch):
|
def test_product_task_post_result_builds_patrol_delete_payload(monkeypatch):
|
||||||
@@ -14,7 +14,7 @@ def test_product_task_post_result_builds_patrol_delete_payload(monkeypatch):
|
|||||||
return Response()
|
return Response()
|
||||||
|
|
||||||
monkeypatch.setattr("config.DELETE_BRAND_API_BASE", "http://java.example")
|
monkeypatch.setattr("config.DELETE_BRAND_API_BASE", "http://java.example")
|
||||||
monkeypatch.setattr("amazon.product.requests.post", fake_post)
|
monkeypatch.setattr("amazon.patrol_delete.requests.post", fake_post)
|
||||||
|
|
||||||
country_sections = [
|
country_sections = [
|
||||||
{
|
{
|
||||||
|
|||||||
Reference in New Issue
Block a user