fix: validate payment handler structure from profile instead of hardcoded list

Replace hardcoded expected handler IDs (google_pay, mock_payment_handler,
shop_pay) with structural validation that:
- Discovers handlers from the business profile dynamically
- Validates required fields (id, version) are present
- Validates handler group names follow reverse-DNS convention
- Works against any UCP merchant, not just the Flower Shop

This makes the protocol conformance tests server-agnostic.

Addresses #13

Author: sbeashwar

protocol_test.py

@@ -180,32 +180,39 @@ def test_discovery(self):
       f"Missing expected capabilities in discovery: {missing_caps}",
     )
 
-    # Verify Payment Handlers
-    handlers = {
-      h.get("id")
-      for handlers in data.get("payment_handlers", {}).values()
-      for h in (handlers if isinstance(handlers, list) else [handlers])
-    }
-    expected_handlers = {"google_pay", "mock_payment_handler", "shop_pay"}
-    missing_handlers = expected_handlers - handlers
-    self.assertFalse(
-      missing_handlers,
-      f"Missing expected payment handlers: {missing_handlers}",
-    )
-
-    # Specific check for Shop Pay config
-    shop_pay = next(
-      (
-        h
-        for handlers in data.get("payment_handlers", {}).values()
-        for h in (handlers if isinstance(handlers, list) else [handlers])
-        if h.get("id") == "shop_pay"
-      ),
-      None,
-    )
-    self.assertIsNotNone(shop_pay, "Shop Pay handler not found")
-    self.assertEqual(shop_pay.get("name"), "com.shopify.shop_pay")
-    self.assertIn("shop_id", shop_pay.get("config"))
+    # Verify Payment Handlers - structural validation (server-agnostic)
+    if data.get("payment_handlers"):
+      handler_count = 0
+      for handler_name, handler_list in data.get(
+        "payment_handlers", {}
+      ).items():
+        # Validate handler group name follows reverse-DNS convention
+        self.assertRegex(
+          str(handler_name),
+          r"^[a-z][a-z0-9]*(\.[a-z][a-z0-9_]*)+$",
+          f"Payment handler group name '{handler_name}' "
+          "does not follow reverse-DNS convention",
+        )
+        for h in (
+          handler_list
+          if isinstance(handler_list, list)
+          else [handler_list]
+        ):
+          handler_count += 1
+          # Validate required fields are present and non-empty
+          self.assertTrue(
+            h.get("id"),
+            "Payment handler missing 'id'",
+          )
+          self.assertIsNotNone(
+            h.get("version"),
+            f"Payment handler '{h.get('id')}' missing 'version'",
+          )
+      self.assertGreater(
+        handler_count,
+        0,
+        "payment_handlers is present but contains no handlers",
+      )
 
     # Verify shopping capability
     shopping_services = data.get("services", {}).get("dev.ucp.shopping")