polish

Author: Bilal Al

lib/splitclient-rb/clients/split_client.rb

@@ -99,9 +99,11 @@ def destroy
       if !@config.cache_adapter.is_a?(SplitIoClient::Cache::Adapters::RedisAdapter) && @config.impressions_mode != :none &&
           (!@impressions_repository.empty? || !@events_repository.empty?)
         @config.logger.debug("Impressions and/or Events cache is not empty")
+        # Adding small delay to ensure sender threads are fully running
+        sleep(0.1)
         if !@config.threads.key?(:impressions_sender) || !@config.threads.key?(:events_sender)
           @config.logger.debug("Periodic data recording thread has not started yet, waiting for service startup.")
-          @config.threads[:start_sdk].join
+          @config.threads[:start_sdk].join if @config.threads.key?(:start_sdk)
         end
       end
       @config.threads.select { |name, thread| name.to_s.end_with? 'sender' }.values.each do |thread|

spec/splitclient/split_client_spec.rb

@@ -72,8 +72,6 @@
 
       factory5 = SplitIoClient::SplitFactory.new('test_api_key',
           features_refresh_rate: 9999,
-          telemetry_refresh_rate: 99999,
-          impressions_refresh_rate: 99999,
           streaming_enabled: false)
       client5 = factory5.client
       client5.block_until_ready