pause 0.0.2 → 0.0.3

data/Gemfile

@@ -1,3 +1,6 @@
+#!/usr/bin/env ruby
+#^syntax detection
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in pause.gemspec

data/Guardfile

@@ -6,8 +6,10 @@
 
 guard 'rspec' do
   watch(%r{^spanx\.gemspec}) { "spec"}
-  watch(%r{^spec/.+_spec\.rb$})
   watch(%r{^lib/(.+)\.rb$}) { "spec" }
+
+  watch(%r{^spec/.+_spec\.rb$})
   watch('spec/spec_helper.rb')  { "spec" }
+  watch(%r{spec/support/.*}) { "spec" }
 end
 

data/lib/pause/action.rb

@@ -56,10 +56,12 @@ module Pause
     end
 
     def ok?
+      return true if self.class.disabled?
       Pause.analyzer.check(self).nil?
     end
 
     def analyze
+      return nil if self.class.disabled?
       Pause.analyzer.check(self)
     end
 
@@ -79,6 +81,33 @@ module Pause
       "#{self.scope}:#{@identifier}"
     end
 
+    # Actions can be globally disabled or re-enabled in a persistent
+    # way.
+    #
+    #   MyAction.disable
+    #   MyAction.enabled? => false
+    #   MyAction.disabled? => true
+    #
+    #   MyAction.enable
+    #   MyAction.enabled? => true
+    #   MyAction.disabled? => false
+    #
+    def self.enable
+      Pause.analyzer.adapter.enable(class_scope)
+    end
+
+    def self.disable
+      Pause.analyzer.adapter.disable(class_scope)
+    end
+
+    def self.enabled?
+      Pause.analyzer.adapter.enabled?(class_scope)
+    end
+
+    def self.disabled?
+      ! enabled?
+    end
+
     private
 
     def self.class_scope

data/lib/pause/redis/adapter.rb

@@ -54,6 +54,22 @@ module Pause
         redis.del (increment_keys + blocked_keys)
       end
 
+      def disable(scope)
+        redis.set("disabled:#{scope}", "1")
+      end
+
+      def enable(scope)
+        redis.del("disabled:#{scope}")
+      end
+
+      def disabled?(scope)
+        ! enabled?(scope)
+      end
+
+      def enabled?(scope)
+        redis.keys("disabled:#{scope}").first.nil?
+      end
+
       private
 
       def redis

data/lib/pause/version.rb

@@ -1,3 +1,3 @@
 module Pause
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/pause/action_spec.rb

@@ -47,14 +47,16 @@ describe Pause::Action do
     end
 
     it "should successfully consider different period checks" do
-      time = Time.now
-      Timecop.freeze time do
+      time = period_marker(resolution, Time.now.to_i + 1)
+
+      Timecop.freeze Time.at(time - 35) do
         4.times do
           action.increment!
           action.ok?.should be_true
         end
       end
-      Timecop.freeze Time.at(time.to_i + 30) do
+
+      Timecop.freeze Time.at(time - 5) do
         2.times do
           action.increment!
           action.ok?.should be_true
@@ -63,6 +65,18 @@ describe Pause::Action do
         action.ok?.should be_false
       end
     end
+
+    context "action is disabled" do
+
+      it "should be true if action is disabled, even if blocked" do
+        10.times { action.increment! }
+        action.ok?.should be_false
+
+        MyNotification.disable
+
+        action.ok?.should be_true
+      end
+    end
   end
 
   describe "#analyze" do
@@ -91,6 +105,17 @@ describe Pause::Action do
         blocked_action.timestamp.should == expected_blocked_action.timestamp
       end
     end
+
+    context "action is disabled" do
+      it "return nil, even if blocked" do
+        10.times { action.increment! }
+        action.should_not be_ok
+
+        MyNotification.disable
+
+        action.analyze.should be_nil
+      end
+    end
   end
 
   describe "#tracked_identifiers" do
@@ -183,3 +208,45 @@ describe Pause::Action, ".scope" do
     DefinedScopeAction.new("1.2.3.4").scope.should == "my:scope"
   end
 end
+
+describe Pause::Action, "enabled/disabled states" do
+  class BlockedAction < Pause::Action
+    scope "blocked"
+    check 10, 0, 10
+  end
+
+  before do
+    Pause.configure do |c|
+      c.resolution = 10
+      c.history = 10
+    end
+  end
+
+  let(:action) { BlockedAction }
+
+  describe "#disable" do
+    before do
+      action.should be_enabled
+      action.should_not be_disabled
+      action.disable
+    end
+
+    it "disables the action" do
+      action.should be_disabled
+      action.should_not be_enabled
+    end
+  end
+
+  describe "#enable" do
+    before do
+      action.disable
+      action.should_not be_enabled
+      action.enable
+    end
+
+    it "enables the action" do
+      action.should be_enabled
+      action.should_not be_disabled
+    end
+  end
+end

data/spec/support/fakeredis.rb

@@ -1 +1,3 @@
+require 'fakeredis'
 require 'fakeredis/rspec'
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pause
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors: