fix creating SubscriptionEvent through .create!(attrs) (#140)

Signed-off-by: Bruno Vieira <bruno@chartmogul.com>
chartmogul · Aug 14, 2023 · 89e78b9 · 89e78b9
1 parent 5a80acd
commit 89e78b9
Show file tree

Hide file tree

Showing 4 changed files with 383 additions and 2 deletions.
diff --git a/...riptionEvent/API_interactions/creates_a_new_subscription_event_through_initialization.yml b/...riptionEvent/API_interactions/creates_a_new_subscription_event_through_initialization.yml
diff --git a/lib/chartmogul/subscription_event.rb b/lib/chartmogul/subscription_event.rb
@@ -35,6 +35,22 @@ def create!
       custom!(:post, resource_path.path, subscription_event: instance_attributes)
     end
 
+    # This endpoint requires we send the attributes as:
+    # { subscription_event: { key: value }}
+    # So we do not include the API::Actions::Create module here and rather use a
+    # variation of the method there to accommodate this difference in behaviour.
+    def self.create!(attributes)
+      resp = handling_errors do
+        connection.post(resource_path.path) do |req|
+          req.headers['Content-Type'] = 'application/json'
+          req.body = JSON.dump({ subscription_event: attributes })
+        end
+      end
+      json = ChartMogul::Utils::JSONParser.parse(resp.body, immutable_keys: immutable_keys)
+
+      new_from_json(json)
+    end
+
     def update!(attrs)
       custom!(:patch, resource_path.path, subscription_event: attrs.merge(id: instance_attributes[:id]))
     end

diff --git a/lib/chartmogul/version.rb b/lib/chartmogul/version.rb
@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ChartMogul
-  VERSION = '3.2.0'
+  VERSION = '3.3.0'
 end