Add flat-param interface and disable/enable for SubscriptionEvent

Add SubscriptionEvent.destroy() and .modify() that accept flat params
(e.g. data={"id": 123}) and auto-wrap in the subscription_event
envelope. The old _with_params methods are preserved for backwards
compatibility.

Add SubscriptionEvent.disable() and .enable() convenience methods
for toggling the disabled state of subscription events.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: wscourge

chartmogul/api/subscription_event.py

@@ -46,3 +46,47 @@ def make(self, data, **kwargs):
 SubscriptionEvent.modify_with_params = SubscriptionEvent._method(
     "modify_with_params", "patch", "/subscription_events"
 )
+
+
+@classmethod
+def _destroy(cls, config, **kwargs):
+    """Accept flat params and wrap in subscription_event envelope for the API."""
+    data = kwargs.get("data", {})
+    if "subscription_event" not in data:
+        data = {"subscription_event": data}
+    return cls.destroy_with_params(config, data=data)
+
+
+@classmethod
+def _modify(cls, config, **kwargs):
+    """Accept flat params and wrap in subscription_event envelope for the API."""
+    data = kwargs.get("data", {})
+    if "subscription_event" not in data:
+        data = {"subscription_event": data}
+    return cls.modify_with_params(config, data=data)
+
+
+@classmethod
+def _disable(cls, config, **kwargs):
+    """Disable a subscription event by setting disabled to true."""
+    data = kwargs.get("data", {})
+    data["disabled"] = True
+    if "subscription_event" not in data:
+        data = {"subscription_event": data}
+    return cls.modify_with_params(config, data=data)
+
+
+@classmethod
+def _enable(cls, config, **kwargs):
+    """Enable a subscription event by setting disabled to false."""
+    data = kwargs.get("data", {})
+    data["disabled"] = False
+    if "subscription_event" not in data:
+        data = {"subscription_event": data}
+    return cls.modify_with_params(config, data=data)
+
+
+SubscriptionEvent.destroy = _destroy
+SubscriptionEvent.modify = _modify
+SubscriptionEvent.disable = _disable
+SubscriptionEvent.enable = _enable

test/api/test_subscription_event.py

@@ -224,6 +224,85 @@ def test_all_subscription_events(self, mock_requests):
         )
         self.assertTrue(isinstance(subscription_events.subscription_events[0], SubscriptionEvent))
 
+    @requests_mock.mock()
+    def test_destroy_flat_params(self, mock_requests):
+        mock_requests.register_uri(
+            "DELETE",
+            "https://api.chartmogul.com/v1/subscription_events",
+            request_headers={"Authorization": "Basic dG9rZW46"},
+            status_code=204,
+        )
+
+        config = Config("token")
+        result = SubscriptionEvent.destroy(config, data={"id": 7654321}).get()
+
+        self.assertEqual(mock_requests.call_count, 1, "expected call")
+        self.assertEqual(
+            mock_requests.last_request.json(),
+            {"subscription_event": {"id": 7654321}},
+        )
+        self.assertTrue(result is None)
+
+    @requests_mock.mock()
+    def test_modify_flat_params(self, mock_requests):
+        mock_requests.register_uri(
+            "PATCH",
+            "https://api.chartmogul.com/v1/subscription_events",
+            request_headers={"Authorization": "Basic dG9rZW46"},
+            status_code=200,
+            json=expected_sub_ev,
+        )
+
+        config = Config("token")
+        sub_ev = SubscriptionEvent.modify(
+            config, data={"id": 7654321, "amount_in_cents": 10}
+        ).get()
+
+        self.assertEqual(mock_requests.call_count, 1, "expected call")
+        self.assertEqual(
+            mock_requests.last_request.json(),
+            {"subscription_event": {"id": 7654321, "amount_in_cents": 10}},
+        )
+        self.assertTrue(isinstance(sub_ev, SubscriptionEvent))
+        self.assertEqual(sub_ev.id, 7654321)
+
+    @requests_mock.mock()
+    def test_disable_subscription_event(self, mock_requests):
+        mock_requests.register_uri(
+            "PATCH",
+            "https://api.chartmogul.com/v1/subscription_events",
+            request_headers={"Authorization": "Basic dG9rZW46"},
+            status_code=200,
+            json=expected_sub_ev,
+        )
+
+        config = Config("token")
+        sub_ev = SubscriptionEvent.disable(config, data={"id": 7654321}).get()
+
+        self.assertEqual(mock_requests.call_count, 1, "expected call")
+        body = mock_requests.last_request.json()
+        self.assertEqual(body["subscription_event"]["id"], 7654321)
+        self.assertTrue(body["subscription_event"]["disabled"])
+        self.assertTrue(isinstance(sub_ev, SubscriptionEvent))
+
+    @requests_mock.mock()
+    def test_enable_subscription_event(self, mock_requests):
+        mock_requests.register_uri(
+            "PATCH",
+            "https://api.chartmogul.com/v1/subscription_events",
+            request_headers={"Authorization": "Basic dG9rZW46"},
+            status_code=200,
+            json=expected_sub_ev,
+        )
+
+        config = Config("token")
+        sub_ev = SubscriptionEvent.enable(config, data={"id": 7654321}).get()
+
+        self.assertEqual(mock_requests.call_count, 1, "expected call")
+        body = mock_requests.last_request.json()
+        self.assertEqual(body["subscription_event"]["id"], 7654321)
+        self.assertFalse(body["subscription_event"]["disabled"])
+
     @requests_mock.mock()
     def test_all_subscription_events_with_filters(self, mock_requests):
         mock_requests.register_uri(