Merge pull request #16 from Flagsmith/release/3.0.2

Release 3.0.2

Author: matthewelwell

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    flagsmith (3.0.1)
+    flagsmith (3.0.2)
       faraday
       faraday-retry
       faraday_middleware
@@ -40,7 +40,7 @@ GEM
       faraday (~> 1.0)
     gem-release (2.2.0)
     method_source (1.0.0)
-    multipart-post (2.2.0)
+    multipart-post (2.2.3)
     parallel (1.20.1)
     parser (3.0.0.0)
       ast (~> 2.4.1)
@@ -50,7 +50,7 @@ GEM
     rainbow (3.0.0)
     rake (13.0.3)
     regexp_parser (2.0.3)
-    rexml (3.2.4)
+    rexml (3.2.5)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)
       rspec-expectations (~> 3.10.0)

example/Gemfile.lock

@@ -172,7 +172,7 @@ GEM
     public_suffix (4.0.7)
     puma (5.6.4)
       nio4r (~> 2.0)
-    rack (2.2.3)
+    rack (2.2.3.1)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
     rainbow (3.1.1)

lib/flagsmith/sdk/analytics_processor.rb

@@ -31,8 +31,8 @@ def flush
       @last_flushed = Time.now
     end
 
-    def track_feature(feature_id)
-      @analytics_data[feature_id] = @analytics_data.fetch(feature_id, 0) + 1
+    def track_feature(feature_name)
+      @analytics_data[feature_name] = @analytics_data.fetch(feature_name, 0) + 1
       flush if (Time.now - @last_flushed) > TIMER * 1000
     end
   end

lib/flagsmith/sdk/models/flags.rb

@@ -122,7 +122,7 @@ def feature_value(feature_name)
       def get_flag(feature_name)
         key = Flagsmith::Flags::Collection.normalize_key(feature_name)
         flag = flags.fetch(key)
-        @analytics_processor.track_feature(flag.feature_id) if @analytics_processor && flag.feature_id
+        @analytics_processor.track_feature(flag.feature_name) if @analytics_processor && flag.feature_id
         flag
       rescue KeyError
         return @default_flag_handler.call(feature_name) if @default_flag_handler

lib/flagsmith/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Flagsmith
-  VERSION = '3.0.1'
+  VERSION = '3.0.2'
 end

spec/sdk/analytics_processor_spec.rb

@@ -19,25 +19,25 @@
   end
 
   it 'test_analytics_processor_track_feature_updates_analytics_data' do
-    subject.track_feature(1)
-    expect(subject.analytics_data[1]).to eq(1)
+    subject.track_feature("my_feature")
+    expect(subject.analytics_data["my_feature"]).to eq(1)
 
-    subject.track_feature(1)
-    expect(subject.analytics_data[1]).to eq(2)
+    subject.track_feature("my_feature")
+    expect(subject.analytics_data["my_feature"]).to eq(2)
   end
 
   it 'test_analytics_processor_flush_clears_analytics_data' do
-    subject.track_feature(1)
+    subject.track_feature("my_feature")
     subject.flush
     expect(subject.analytics_data).to eql({})
   end
 
   it 'test_analytics_processor_flush_post_request_data_match_ananlytics_data' do
-    subject.track_feature(1)
-    subject.track_feature(2)
+    subject.track_feature("my_feature_1")
+    subject.track_feature("my_feature_2")
     expect(flagsmith.api_client).to receive(:post).exactly(1).times do |uri_path, body|
       expect(uri_path).to eq('analytics/flags/')
-      expect(body).to eq({ 1 => 1, 2 => 1 }.to_json)
+      expect(body).to eq({ "my_feature_1" => 1, "my_feature_2" => 1 }.to_json)
     end
     subject.flush
   end