Merge branch 'feat/evaluation-get-result' of github.com:Flagsmith/flagsmith-ruby-client into feat/sdk-consumes-context-engine

Author: Zaimwa9

lib/flagsmith/engine/mappers/environment.rb

@@ -20,10 +20,9 @@ def self.build_features_context(feature_states)
           features
         end
 
-        def self.build_feature_hash(feature_state) # rubocop:disable Metrics/MethodLength
+        def self.build_feature_hash(feature_state)
           feature_hash = {
             key: feature_state.django_id&.to_s || feature_state.uuid,
-            feature_key: feature_state.feature.id.to_s,
             name: feature_state.feature.name,
             enabled: feature_state.enabled,
             value: feature_state.get_value,

lib/flagsmith/engine/mappers/identity.rb

@@ -44,11 +44,10 @@ def self.group_by_overrides(identity_overrides)
           features_to_identifiers
         end
 
-        def self.build_overrides_key(identity_features) # rubocop:disable Metrics/MethodLength
+        def self.build_overrides_key(identity_features)
           sorted_features = identity_features.to_a.sort_by { |fs| fs.feature.name }
           sorted_features.map do |feature_state|
             {
-              feature_key: feature_state.feature.id.to_s,
               name: feature_state.feature.name,
               enabled: feature_state.enabled,
               value: feature_state.get_value,

lib/flagsmith/engine/mappers/segments.rb

@@ -30,7 +30,6 @@ def self.build_overrides(feature_states) # rubocop:disable Metrics/MethodLength
           feature_states.map do |feature_state|
             override_hash = {
               key: feature_state.django_id&.to_s || feature_state.uuid,
-              feature_key: feature_state.feature.id.to_s,
               name: feature_state.feature.name,
               enabled: feature_state.enabled,
               value: feature_state.get_value,

spec/engine/unit/evaluation_context_mappers_spec.rb

@@ -68,7 +68,6 @@
       expect(override_condition[:value]).to include('overridden-id')
 
       override = identity_override_segment[:overrides][0]
-      expect(override[:feature_key]).to eq('1')
       expect(override[:name]).to eq('some_feature')
       expect(override[:enabled]).to be false
       expect(override[:value]).to eq('some-overridden-value')
@@ -80,7 +79,6 @@
       expect(context[:features]).to have_key('some_feature')
 
       some_feature = context[:features]['some_feature']
-      expect(some_feature[:feature_key]).to eq('1')
       expect(some_feature[:name]).to eq('some_feature')
       expect(some_feature[:enabled]).to be true
       expect(some_feature[:value]).to eq('some-value')
@@ -90,7 +88,6 @@
       # Verify multivariate feature
       expect(context[:features]).to have_key('test_mv')
       test_mv = context[:features]['test_mv']
-      expect(test_mv[:feature_key]).to eq('83755')
       expect(test_mv[:name]).to eq('test_mv')
       expect(test_mv[:enabled]).to be false
       expect(test_mv[:value]).to eq('1111')