fix robocop

Author: Daksh

lib/stream-chat/client.rb

@@ -441,7 +441,7 @@ def hard_delete_message(message_id)
     sig { params(message_id: String, user_id: String).returns(StreamChat::StreamResponse) }
     def delete_message_for_me(message_id, user_id)
       raise ArgumentError, 'user_id must not be empty for delete_for_me functionality' if user_id.to_s.empty?
-      
+
       delete_message(message_id, delete_for_me: true, deleted_by: user_id)
     end
 
@@ -450,13 +450,14 @@ def delete_message_for_me(message_id, user_id)
     def delete_message_with_options(message_id, hard: nil, delete_for_me: nil, user_id: nil)
       options = {}
       options[:hard] = true if hard
-      
+
       if delete_for_me
         raise ArgumentError, 'user_id must not be empty for delete_for_me functionality' if user_id.to_s.empty?
+
         options[:delete_for_me] = true
         options[:deleted_by] = user_id
       end
-      
+
       delete_message(message_id, **options)
     end
 
@@ -1102,7 +1103,7 @@ def make_http_request(method, relative_url, params: nil, data: nil)
       headers['Authorization'] = @auth_token
       headers['stream-auth-type'] = 'jwt'
       params = {} if params.nil?
-      params = (get_default_params.merge(params).sort_by { |k, _v| k.to_s }).to_h
+      params = get_default_params.merge(params).sort_by { |k, _v| k.to_s }.to_h
       url = "#{relative_url}?#{URI.encode_www_form(params)}"
 
       body = data.to_json if %w[patch post put].include? method.to_s

spec/client_spec.rb

@@ -425,7 +425,7 @@ def loop_times(times)
 
   it 'hard deletes a message using hard_delete_message method' do
     msg_id = SecureRandom.uuid
-    
+
     # Try to send message, but don't fail if channel doesn't exist
     begin
       @channel.send_message({
@@ -436,20 +436,20 @@ def loop_times(times)
       # Skip test if channel doesn't exist in CI environment
       skip "Channel not available in CI environment: #{e.message}"
     end
-    
+
     # Test the delete method - it may fail if message doesn't exist, but that's expected
     begin
       @client.hard_delete_message(msg_id)
     rescue StreamChat::StreamAPIException => e
       # This is expected if the message doesn't exist
-      expect(e.message).to include("DeleteMessage failed")
+      expect(e.message).to include('DeleteMessage failed')
     end
   end
 
   it 'deletes a message for me using delete_message_for_me method' do
     msg_id = SecureRandom.uuid
     user_id = @random_user[:id]
-    
+
     # Try to send message, but don't fail if channel doesn't exist
     begin
       @channel.send_message({
@@ -460,20 +460,20 @@ def loop_times(times)
       # Skip test if channel doesn't exist in CI environment
       skip "Channel not available in CI environment: #{e.message}"
     end
-    
+
     # Test the delete method - it may fail if message doesn't exist, but that's expected
     begin
       @client.delete_message_for_me(msg_id, user_id)
     rescue StreamChat::StreamAPIException => e
       # This is expected if the message doesn't exist
-      expect(e.message).to include("DeleteMessage failed")
+      expect(e.message).to include('DeleteMessage failed')
     end
   end
 
   it 'deletes a message with options using delete_message_with_options method' do
     msg_id = SecureRandom.uuid
     user_id = @random_user[:id]
-    
+
     # Try to send message, but don't fail if channel doesn't exist
     begin
       @channel.send_message({
@@ -484,19 +484,19 @@ def loop_times(times)
       # Skip test if channel doesn't exist in CI environment
       skip "Channel not available in CI environment: #{e.message}"
     end
-    
+
     # Test the delete method - it may fail if message doesn't exist, but that's expected
     begin
       @client.delete_message_with_options(msg_id, delete_for_me: true, user_id: user_id)
     rescue StreamChat::StreamAPIException => e
       # This is expected if the message doesn't exist
-      expect(e.message).to include("DeleteMessage failed")
+      expect(e.message).to include('DeleteMessage failed')
     end
   end
 
   it 'hard deletes a message with options using delete_message_with_options method' do
     msg_id = SecureRandom.uuid
-    
+
     # Try to send message, but don't fail if channel doesn't exist
     begin
       @channel.send_message({
@@ -507,19 +507,19 @@ def loop_times(times)
       # Skip test if channel doesn't exist in CI environment
       skip "Channel not available in CI environment: #{e.message}"
     end
-    
+
     # Test the delete method - it may fail if message doesn't exist, but that's expected
     begin
       @client.delete_message_with_options(msg_id, hard: true)
     rescue StreamChat::StreamAPIException => e
       # This is expected if the message doesn't exist
-      expect(e.message).to include("DeleteMessage failed")
+      expect(e.message).to include('DeleteMessage failed')
     end
   end
 
   it 'raises error when delete_for_me is true but user_id is empty' do
     msg_id = SecureRandom.uuid
-    
+
     # Try to send message, but don't fail if channel doesn't exist
     begin
       @channel.send_message({
@@ -530,15 +530,15 @@ def loop_times(times)
       # Skip test if channel doesn't exist in CI environment
       skip "Channel not available in CI environment: #{e.message}"
     end
-    
-    expect {
+
+    expect do
       @client.delete_message_for_me(msg_id, '')
-    }.to raise_error(ArgumentError, 'user_id must not be empty for delete_for_me functionality')
+    end.to raise_error(ArgumentError, 'user_id must not be empty for delete_for_me functionality')
   end
 
   it 'raises error when delete_for_me is true but user_id is nil in delete_message_with_options' do
     msg_id = SecureRandom.uuid
-    
+
     # Try to send message, but don't fail if channel doesn't exist
     begin
       @channel.send_message({
@@ -549,10 +549,10 @@ def loop_times(times)
       # Skip test if channel doesn't exist in CI environment
       skip "Channel not available in CI environment: #{e.message}"
     end
-    
-    expect {
+
+    expect do
       @client.delete_message_with_options(msg_id, delete_for_me: true, user_id: nil)
-    }.to raise_error(ArgumentError, 'user_id must not be empty for delete_for_me functionality')
+    end.to raise_error(ArgumentError, 'user_id must not be empty for delete_for_me functionality')
   end
 
   it 'undeletes a message' do