fix(identifier): Escape identifier in put and destroy methods (#165)

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    lago-ruby-client (0.52.1.pre.beta)
+    lago-ruby-client (0.52.2.pre.beta)
       jwt
       openssl
 

lib/lago/api/connection.rb

@@ -22,7 +22,7 @@ def post(body, path = uri.path)
       end
 
       def put(path = uri.path, identifier:, body:)
-        uri_path = identifier.nil? ? path : "#{path}/#{identifier}"
+        uri_path = identifier.nil? ? path : "#{path}/#{URI.encode_www_form_component(identifier)}"
         response = http_client.send_request(
           'PUT',
           uri_path,
@@ -47,7 +47,7 @@ def get(path = uri.path, identifier:)
 
       def destroy(path = uri.path, identifier:, options: nil)
         uri_path = path
-        uri_path += "/#{identifier}" if identifier
+        uri_path += "/#{URI.encode_www_form_component(identifier)}" if identifier
         uri_path += "?#{URI.encode_www_form(options)}" unless options.nil?
         response = http_client.send_request(
           'DELETE',

spec/lago/api/connection_spec.rb

@@ -43,4 +43,40 @@
       expect(URI).to have_received(:encode_www_form_component).with(identifier)
     end
   end
+
+  describe '#put' do
+    let(:identifier) { 'gid://app/Customer/1234' }
+
+    before do
+      stub_request(:put, 'https://testapi.example.org:443/gid:%2F%2Fapp%2FCustomer%2F1234')
+
+      allow(URI).to receive(:encode_www_form_component)
+        .with(identifier)
+        .and_return('gid:%2F%2Fapp%2FCustomer%2F1234')
+
+      connection.put(identifier: identifier, body: nil)
+    end
+
+    it 'encodes the identifier' do
+      expect(URI).to have_received(:encode_www_form_component).with(identifier)
+    end
+  end
+
+  describe '#destroy' do
+    let(:identifier) { 'gid://app/Customer/1234' }
+
+    before do
+      stub_request(:delete, 'https://testapi.example.org:443/gid:%2F%2Fapp%2FCustomer%2F1234')
+
+      allow(URI).to receive(:encode_www_form_component)
+        .with(identifier)
+        .and_return('gid:%2F%2Fapp%2FCustomer%2F1234')
+
+      connection.destroy(identifier: identifier)
+    end
+
+    it 'encodes the identifier' do
+      expect(URI).to have_received(:encode_www_form_component).with(identifier)
+    end
+  end
 end