minor fixes

Author: Rafael Marinho

lib/stream-chat/client.rb

@@ -824,16 +824,13 @@ def get_active_live_locations(user_id)
     # @param [Float] longitude Optional longitude coordinate
     # @param [String] end_at Optional end time for the location sharing
     # @return [StreamChat::StreamResponse]
-    sig { params(user_id: String, created_by_device_id: String, message_id: String, latitude: T.nilable(Float), longitude: T.nilable(Float), end_at: T.nilable(String)).returns(StreamChat::StreamResponse) }
-    def update_location(user_id, created_by_device_id:, message_id:, latitude: nil, longitude: nil, end_at: nil)
+    sig { params(user_id: String, created_by_device_id: String, message_id: String, options: T.untyped).returns(StreamChat::StreamResponse) }
+    def update_location(user_id, created_by_device_id:, message_id:, **options)
       data = {
         created_by_device_id: created_by_device_id,
         message_id: message_id
       }
-      data[:latitude] = latitude if latitude
-      data[:longitude] = longitude if longitude
-      data[:end_at] = end_at if end_at
-
+      data.merge!(options) if options
       put('users/location', data: data, params: { user_id: user_id })
     end
 

spec/client_spec.rb

@@ -1038,17 +1038,19 @@ def loop_times(times)
     end
 
     it 'can create and update a location' do
-      location = { 
+      location = {
         created_by_device_id: SecureRandom.uuid,
         latitude: 40.7128,
         longitude: -74.0060,
         end_at: (Time.now + 3600).iso8601
       }
 
-      response = @channel.send_message({ 
-        text: 'Location sharing message',
-        shared_location: location,
-      }, @location_test_user[:id])
+      response = @channel.send_message(
+        { 
+          text: 'Location sharing message',
+          shared_location: location
+        }, @location_test_user[:id]
+      )
 
       expect(response['message']).to include 'shared_location'
       expect(response['message']['shared_location']['created_by_device_id']).to eq(location[:created_by_device_id])
@@ -1076,21 +1078,22 @@ def loop_times(times)
       longitude = -74.0060
       end_at = (Time.now + 3600).iso8601
 
-      response = @location_channel.send_message({ 
-        text: 'Location sharing message',
-        shared_location: {
-          created_by_device_id: device_id,
-          latitude: latitude,
-          longitude: longitude,
-          end_at: end_at
-        }
-      }, @location_test_user[:id])
+      response = @location_channel.send_message(
+        {
+          text: 'Location sharing message',
+          shared_location: {
+            created_by_device_id: device_id,
+            latitude: latitude,
+            longitude: longitude,
+            end_at: end_at
+          }
+        }, @location_test_user[:id]
+      )
 
       response = @client.get_active_live_locations(@location_test_user[:id])
       expect(response).to include 'active_live_locations'
       expect(response['active_live_locations']).to be_an(Array)
       expect(response['active_live_locations'].length).to be >= 1
-      
       location = response['active_live_locations'].find { |loc| loc['created_by_device_id'] == device_id }
       expect(location).not_to be_nil
       expect(location['latitude']).to eq(latitude)
@@ -1099,7 +1102,7 @@ def loop_times(times)
     end
 
     it 'should have active live locations on the channel' do
-      response = @channel.query()
+      response = @channel.query
       expect(response).to include 'active_live_locations'
       expect(response['active_live_locations'].length).to be >= 1
     end