pause 0.2.1 → 0.4.0

data/spec/pause/redis/adapter_spec.rb

@@ -31,16 +31,22 @@ describe Pause::Redis::Adapter do
       expect(set[0].size).to eql(2)
     end
 
-    it 'should remove old key from a redis set' do
-      time = Time.now
-      expect(redis_conn).to receive(:zrem).with(tracked_key, [adapter.period_marker(resolution, time)])
-
-      adapter.time_blocks_to_keep = 1
-      Timecop.freeze time do
-        adapter.increment(scope, identifier, Time.now.to_i)
+    context 'removing two elements' do
+      let(:to_delete) { 2 }
+      let(:time) { Time.now }
+      before do
+        adapter
+        to_delete.times do |t|
+          expect(redis_conn).to receive(:zrem).with(tracked_key, [adapter.period_marker(resolution, time + t)]).once
+        end
+        adapter.time_blocks_to_keep = 1
       end
-      Timecop.freeze time + (adapter.resolution + 1) do
-        adapter.increment(scope, identifier, Time.now.to_i)
+      it 'should remove old elements' do
+        adapter.increment(scope, identifier, time.to_i)
+        to_delete.times do |t|
+          next_time = time + (adapter.resolution + t + 1)
+          adapter.increment(scope, identifier, next_time.to_i)
+        end
       end
     end
 

data/spec/pause/redis/sharded_adapter_spec.rb

@@ -21,4 +21,18 @@ describe Pause::Redis::ShardedAdapter do
       expect { adapter.all_keys('cake') }.to raise_error(Pause::Redis::OperationNotSupported)
     end
   end
+
+  describe '#with_multi' do
+    let(:redis) { adapter.send(:redis) }
+    it 'should not call redis.multi' do
+      expect(redis).to_not receive(:multi)
+      expect { adapter.increment(:scope, 123, Time.now) }.to_not raise_error
+    end
+  end
+
+  describe '#redis' do
+    it 'should not use redis db when connecting' do
+      expect(adapter.send(:redis_connection_opts)).to_not include(:db)
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -5,20 +5,29 @@
 #
 # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 
-ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', __FILE__)
-require 'rubygems'
-require 'bundler/setup' if File.exists?(ENV['BUNDLE_GEMFILE'])
 require 'pause'
-require 'pry'
-require 'support/fakeredis'
+require 'fileutils'
+
+if ENV['PAUSE_REAL_REDIS']
+  require 'pause/redis/adapter'
+  puts ; puts "NOTE: Using real Redis-server at #{Pause::Redis::Adapter.redis.inspect}\n\n"
+else
+  require 'fakeredis/rspec'
+end
 
 RSpec.configure do |config|
   config.run_all_when_everything_filtered = true
   config.filter_run :focus
 
-  # Run specs in random order to surface order dependencies. If you find an
-  # order dependency and want to debug it, you can fix the order by providing
-  # the seed, which is printed after each run.
-  #     --seed 1234
+  rspec_dir = './.spec'.freeze
+  FileUtils.mkdir_p(rspec_dir)
+  config.example_status_persistence_file_path = "#{rspec_dir}/results.txt"
+
   config.order = 'random'
+
+  if ENV['PAUSE_REAL_REDIS']
+    config.before(:example) do
+      Pause::Redis::Adapter.redis.flushdb
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pause
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.4.0
 platform: ruby
 authors:
 - Atasay Gokkaya
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-07 00:00:00.000000000 Z
+date: 2018-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
@@ -27,13 +27,127 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: Real time rate limiting for multi-process ruby environments based on
-  Redis
+- !ruby/object:Gem::Dependency
+  name: hiredis
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: fakeredis
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: timecop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: This gem provides highly flexible and easy to use interface to define
+  rate limit checks, register events as they come, and verify if the rate limit is
+  reached. Multiple checks for the same metric are easily supported. This gem is used
+  at very high scale on several popular web sites.
 email:
 - atasay@wanelo.com
 - paul@wanelo.com
-- sax@wanelo.com
-- kig@wanelo.com
+- sax@ericsaxby.com
+- kigster@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -65,8 +179,7 @@ files:
 - spec/pause/redis/adapter_spec.rb
 - spec/pause/redis/sharded_adapter_spec.rb
 - spec/spec_helper.rb
-- spec/support/fakeredis.rb
-homepage: https://github.com/wanelo/pause
+homepage: https://github.com/kigster/pause
 licenses: []
 metadata: {}
 post_install_message: 
@@ -85,10 +198,11 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
-summary: RReal time rate limiting for multi-process ruby environments based on Redis
+summary: Fast, scalable, and flexible real time rate limiting library for distributed
+  Ruby environments backed by Redis.
 test_files:
 - spec/pause/action_spec.rb
 - spec/pause/analyzer_spec.rb
@@ -97,4 +211,3 @@ test_files:
 - spec/pause/redis/adapter_spec.rb
 - spec/pause/redis/sharded_adapter_spec.rb
 - spec/spec_helper.rb
-- spec/support/fakeredis.rb

data/spec/support/fakeredis.rb

@@ -1,2 +0,0 @@
-require 'fakeredis/rspec'
-