fix: 兼容提交提示型非滚动规格面板 (#175)

client/test/test_pdd_collect_service.py

@@ -609,6 +609,9 @@ class PddCollectParserTest(unittest.TestCase):
         cls.non_scrollable_spec_xml = (
             FIXTURES / "pdd_spec_panel_non_scrollable.xml"
         ).read_text(encoding="utf-8")
+        cls.submit_hint_spec_xml = (
+            FIXTURES / "pdd_spec_panel_submit_hint.xml"
+        ).read_text(encoding="utf-8")
         cls.sold_out_xml = (
             FIXTURES / "pdd_sold_out_recommendations.xml"
         ).read_text(encoding="utf-8")
@@ -789,6 +792,41 @@ class PddCollectParserTest(unittest.TestCase):
         self.assertEqual(result.raw_price, "¥8.17")
         self.assertNotIn("确认款式", [item.name for item in result.dimensions])
 
+    def test_parse_non_scrollable_submit_hint_spec_panel(self):
+        result = parse_spec_panel(self.submit_hint_spec_xml)
+
+        self.assertTrue(_is_spec_panel_open(self.submit_hint_spec_xml))
+        self.assertEqual(
+            [(item.key, item.name) for item in result.dimensions],
+            [("color", "颜色分类"), ("size", "尺码")],
+        )
+        self.assertEqual(
+            [value.text for value in result.dimensions[0].values],
+            ["测试黑色", "测试红色"],
+        )
+        self.assertEqual(
+            [value.text for value in result.dimensions[1].values],
+            ["M", "L"],
+        )
+        self.assertNotIn(
+            "请选择： 颜色分类 尺码",
+            [item.name for item in result.dimensions],
+        )
+
+    def test_isolated_submit_hint_is_not_a_spec_panel(self):
+        xml_data = """<hierarchy>
+          <node package="com.xunmeng.pinduoduo"
+                class="android.widget.FrameLayout" bounds="[0,0][1080,2376]">
+            <node package="com.xunmeng.pinduoduo" content-desc="关闭"
+                  clickable="true" bounds="[975,273][1050,348]" />
+            <node package="com.xunmeng.pinduoduo"
+                  text="选择规格后，提交订单" clickable="true"
+                  bounds="[0,2181][1080,2328]" />
+          </node>
+        </hierarchy>"""
+
+        self.assertFalse(_is_spec_panel_open(xml_data))
+
     def test_non_scrollable_spec_panel_exposes_three_color_rows(self):
         service = PddCollectService(
             PddDeviceService(lambda _serial: object()),
@@ -1790,6 +1828,32 @@ class PddCollectParserTest(unittest.TestCase):
             artifacts = raised.exception.diagnostics["artifacts"]
             self.assertTrue(Path(artifacts[0]["path"]).is_file())
 
+    def test_open_confirmation_with_unrecognized_structure_is_not_timeout(self):
+        malformed_panel = self.submit_hint_spec_xml.replace(
+            '<node package="com.xunmeng.pinduoduo" class="android.widget.EditText"\n'
+            '            text="1" clickable="true" bounds="[480,829][561,904]"\n'
+            '            visible-to-user="true" enabled="true" />',
+            "",
+        )
+        device = FakeCollectDevice(self.home_xml, malformed_panel)
+        device.opened_url = "https://mobile.yangkeduo.com/goods.html?goods_id=123"
+        device.panel_open = True
+        clock = FakeClock()
+        service = PddCollectService(
+            PddDeviceService(lambda _serial: device),
+            "USB-001",
+            "client-001",
+            sleeper=clock.sleep,
+            monotonic=clock.monotonic,
+            spec_panel_timeout=1.0,
+        )
+
+        with self.assertRaises(PddCollectError) as raised:
+            service._wait_spec_panel(device)
+
+        self.assertEqual(raised.exception.code, "PDD_DATA_SPEC_INCOMPLETE")
+        self.assertIn("已经打开", raised.exception.message)
+
     def test_transient_empty_spec_trees_are_skipped_during_full_collection(self):
         device = TransientSpecTreeDevice(
             self.home_xml, keep_only_one_sku(self.spec_xml)