flipper-api 0.20.0 → 0.21.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36b908794b663b31d3c33518c1719eeac908c0d79c4d756dd887e9670b92648b
-  data.tar.gz: 9952c9259f9a0d0bbae6e7c87a3244f14fa4fee149d89231ad5009f17109abd6
+  metadata.gz: a886dcc779b4111853197cbefac1f3a6c055c3ea746d9ca8fb33d8b31833d488
+  data.tar.gz: f1ea8a89c6d37b89f1cac3b1e3c6548c6479f450e9639826b455c8de875eaacc
 SHA512:
-  metadata.gz: 5b407a469637bedbe0d5094a62e6310041a2e16c5b6c042f1987f2c7f4632dbcabea242e0c696a421f9c9b1eb02f8bcba8b6187e1fb306b0c28ae348508c664e
-  data.tar.gz: 57602b98343f6be918be82ac4b0cd33275f940c7a9a0b655c05fb585553b011627b2691d56c58a86ce3fa8fa0f28d586696d7045bf79598b66de99c34be3773a
+  metadata.gz: 4d932b037e0160c8ecd49ef1b103ea4df2385f4c5c9f9b166a4fad67d76af82021c20ebc16136046a71be5e8a418496b4c636bde99d88f1854d31eb7289eb739
+  data.tar.gz: a240a8e491227c12b12017fea85374745169a60ddb4c0f30b32f05782b36944b360804476462efcd517c8f1fe99ec095fddfd121ae1dd7edeb458c067241bb1f

data/lib/flipper/api/v1/actions/percentage_of_actors_gate.rb

@@ -30,9 +30,17 @@ module Flipper
 
           private
 
+          def percentage_param
+            @percentage_param ||= params['percentage'].to_s
+          end
+
           def percentage
             @percentage ||= begin
-              Integer(params['percentage'])
+              unless percentage_param.match(/\d/)
+                raise ArgumentError, "invalid numeric value: #{percentage_param}"
+              end
+
+              Flipper::Types::Percentage.new(percentage_param).value
             rescue ArgumentError, TypeError
               -1
             end

data/lib/flipper/api/v1/actions/percentage_of_time_gate.rb

@@ -31,9 +31,17 @@ module Flipper
 
           private
 
+          def percentage_param
+            @percentage_param ||= params['percentage'].to_s
+          end
+
           def percentage
             @percentage ||= begin
-              Integer(params['percentage'])
+              unless percentage_param.match(/\d/)
+                raise ArgumentError, "invalid numeric value: #{percentage_param}"
+              end
+
+              Flipper::Types::Percentage.new(percentage_param).value
             rescue ArgumentError, TypeError
               -1
             end

data/lib/flipper/version.rb

@@ -1,3 +1,3 @@
 module Flipper
-  VERSION = '0.20.0'.freeze
+  VERSION = '0.21.0.rc1'.freeze
 end

data/spec/flipper/api/v1/actions/percentage_of_actors_gate_spec.rb

@@ -4,53 +4,79 @@ RSpec.describe Flipper::Api::V1::Actions::PercentageOfActorsGate do
   let(:app) { build_api(flipper) }
 
   describe 'enable' do
+    shared_examples 'gates with percentage' do
+      it 'enables gate for feature' do
+        expect(flipper[path].enabled_gate_names).to include(:percentage_of_actors)
+      end
+
+      it 'returns decorated feature with gate enabled for a percent of actors' do
+        gate = json_response['gates'].find { |gate| gate['name'] == 'percentage_of_actors' }
+        expect(gate['value']).to eq(percentage)
+      end
+    end
+
     context 'for feature with slash in name' do
+      let(:path) { 'my/feature' }
+
       before do
-        flipper["my/feature"].disable
-        post '/features/my/feature/percentage_of_actors', percentage: '10'
+        flipper[path].disable
+        post "/features/#{path}/percentage_of_actors", percentage: percentage
       end
 
-      it 'enables gate for feature' do
-        expect(flipper["my/feature"].enabled_gate_names).to include(:percentage_of_actors)
+      context 'with integer percentage' do
+        let(:percentage) { '10' }
+
+        it_behaves_like 'gates with percentage'
       end
 
-      it 'returns decorated feature with gate enabled for 10 percent of actors' do
-        gate = json_response['gates'].find { |gate| gate['name'] == 'percentage_of_actors' }
-        expect(gate['value']).to eq('10')
+      context 'with decimal percentage' do
+        let(:percentage) { '0.05' }
+
+        it_behaves_like 'gates with percentage'
       end
     end
 
     context 'url-encoded request' do
+      let(:path) { :my_feature }
+
       before do
         flipper[:my_feature].disable
-        post '/features/my_feature/percentage_of_actors', percentage: '10'
+        post "/features/#{path}/percentage_of_actors", percentage: percentage
       end
 
-      it 'enables gate for feature' do
-        expect(flipper[:my_feature].enabled_gate_names).to include(:percentage_of_actors)
+      context 'with integer percentage' do
+        let(:percentage) { '10' }
+
+        it_behaves_like 'gates with percentage'
       end
 
-      it 'returns decorated feature with gate enabled for 10 percent of actors' do
-        gate = json_response['gates'].find { |gate| gate['name'] == 'percentage_of_actors' }
-        expect(gate['value']).to eq('10')
+      context 'with decimal percentage' do
+        let(:percentage) { '0.05' }
+
+        it_behaves_like 'gates with percentage'
       end
     end
 
     context 'json request' do
+      let(:path) { :my_feature }
+
       before do
         flipper[:my_feature].disable
-        post '/features/my_feature/percentage_of_actors',
-             JSON.generate(percentage: '10'),
+        post "/features/#{path}/percentage_of_actors",
+             JSON.generate(percentage: percentage),
              'CONTENT_TYPE' => 'application/json'
       end
 
-      it 'enables gate for feature' do
-        expect(flipper[:my_feature].enabled_gate_names).to include(:percentage_of_actors)
+      context 'with integer percentage' do
+        let(:percentage) { '10' }
+
+        it_behaves_like 'gates with percentage'
       end
 
-      it 'returns decorated feature with gate enabled for 10 percent of actors' do
-        gate = json_response['gates'].find { |gate| gate['name'] == 'percentage_of_actors' }
-        expect(gate['value']).to eq('10')
+      context 'with decimal percentage' do
+        let(:percentage) { '0.05' }
+
+        it_behaves_like 'gates with percentage'
       end
     end
   end
@@ -113,7 +139,7 @@ RSpec.describe Flipper::Api::V1::Actions::PercentageOfActorsGate do
     end
   end
 
-  describe 'percentage parameter not an integer' do
+  describe 'percentage parameter not a number' do
     before do
       flipper[:my_feature].disable
       post '/features/my_feature/percentage_of_actors', percentage: 'foo'

data/spec/flipper/api/v1/actions/percentage_of_time_gate_spec.rb

@@ -4,34 +4,80 @@ RSpec.describe Flipper::Api::V1::Actions::PercentageOfTimeGate do
   let(:app) { build_api(flipper) }
 
   describe 'enable' do
-    before do
-      flipper[:my_feature].disable
-      post '/features/my_feature/percentage_of_time', percentage: '10'
-    end
+    shared_examples 'gates with percentage' do
+      it 'enables gate for feature' do
+        expect(flipper[path].enabled_gate_names).to include(:percentage_of_time)
+      end
 
-    it 'enables gate for feature' do
-      expect(flipper[:my_feature].enabled_gate_names).to include(:percentage_of_time)
+      it 'returns decorated feature with gate enabled for a percent of times' do
+        gate = json_response['gates'].find { |gate| gate['name'] == 'percentage_of_time' }
+        expect(gate['value']).to eq(percentage)
+      end
     end
 
-    it 'returns decorated feature with gate enabled for 5% of time' do
-      gate = json_response['gates'].find { |gate| gate['name'] == 'percentage_of_time' }
-      expect(gate['value']).to eq('10')
-    end
-  end
+    context 'for feature with slash in name' do
+      let(:path) { 'my/feature' }
 
-  describe 'enable for feature with slash in name' do
-    before do
-      flipper["my/feature"].disable
-      post '/features/my/feature/percentage_of_time', percentage: '10'
+      before do
+        flipper[path].disable
+        post "/features/#{path}/percentage_of_time", percentage: percentage
+      end
+
+      context 'with integer percentage' do
+        let(:percentage) { '10' }
+
+        it_behaves_like 'gates with percentage'
+      end
+
+      context 'with decimal percentage' do
+        let(:percentage) { '0.05' }
+
+        it_behaves_like 'gates with percentage'
+      end
     end
 
-    it 'enables gate for feature' do
-      expect(flipper["my/feature"].enabled_gate_names).to include(:percentage_of_time)
+    context 'url-encoded request' do
+      let(:path) { :my_feature }
+
+      before do
+        flipper[:my_feature].disable
+        post "/features/#{path}/percentage_of_time", percentage: percentage
+      end
+
+      context 'with integer percentage' do
+        let(:percentage) { '10' }
+
+        it_behaves_like 'gates with percentage'
+      end
+
+      context 'with decimal percentage' do
+        let(:percentage) { '0.05' }
+
+        it_behaves_like 'gates with percentage'
+      end
     end
 
-    it 'returns decorated feature with gate enabled for 5% of time' do
-      gate = json_response['gates'].find { |gate| gate['name'] == 'percentage_of_time' }
-      expect(gate['value']).to eq('10')
+    context 'json request' do
+      let(:path) { :my_feature }
+
+      before do
+        flipper[:my_feature].disable
+        post "/features/#{path}/percentage_of_time",
+             JSON.generate(percentage: percentage),
+             'CONTENT_TYPE' => 'application/json'
+      end
+
+      context 'with integer percentage' do
+        let(:percentage) { '10' }
+
+        it_behaves_like 'gates with percentage'
+      end
+
+      context 'with decimal percentage' do
+        let(:percentage) { '0.05' }
+
+        it_behaves_like 'gates with percentage'
+      end
     end
   end
 
@@ -93,7 +139,7 @@ RSpec.describe Flipper::Api::V1::Actions::PercentageOfTimeGate do
     end
   end
 
-  describe 'percentage parameter not an integer' do
+  describe 'percentage parameter not an number' do
     before do
       flipper[:my_feature].disable
       post '/features/my_feature/percentage_of_time', percentage: 'foo'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-api
 version: !ruby/object:Gem::Version
-  version: 0.20.0
+  version: 0.21.0.rc1
 platform: ruby
 authors:
 - John Nunemaker
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-12-20 00:00:00.000000000 Z
+date: 2021-05-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -36,14 +36,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.20.0
+        version: 0.21.0.rc1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.20.0
+        version: 0.21.0.rc1
 description: 
 email:
 - nunemaker@gmail.com
@@ -101,9 +101,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.0.3
 signing_key: