fist_of_fury 0.0.1 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: aaac55f7aa5ff55179438b0849987c260000e695
-  data.tar.gz: cb691189d955e8d22fe2b2a25997da15f2321c47
+  metadata.gz: 30fd5d339ee339c1294b986583f7a0abed1cb08a
+  data.tar.gz: 593d60d97908fa37af0f13192fc68487950d8a71
 SHA512:
-  metadata.gz: daec4f0b46b06ae89773d9c68783ce2d59c2c201fde9ea21a5d59f3d17545790c1ecc293edf0227f93f8a39dcf9c5c35a8a9643816af688910d0712e7eb6b812
-  data.tar.gz: 90c799dab5cae395152b2bdecd623cd200a8e93ba0c1f48fae0d021faad3a720a12bf0d010b2c2d60bb095aa1dbcd95ccfb101af4e312ec261dd578345408e75
+  metadata.gz: 01928d25ec21fece7ac05657327f503ac07f287067b0bf979823241bce3e2a324e9124401343e9cff4668cd18187c620e33f96587284e67318d9094a5aec67a0
+  data.tar.gz: f2697fdaf69a3f99e0b14a70f365f10a3cb8588d80b8c4b327d4bb4d1a0775d7ecaaafc98290331737e02611f6ef66005e85356e3067f01115bb5e26ad173534

data/.travis.yml

@@ -0,0 +1,11 @@
+language: ruby
+rvm:
+  - 1.9.3
+  - jruby-19mode
+  - rbx
+  - 2.0.0
+  - 2.1.0
+matrix:
+  allow_failures:
+    - rvm: jruby-19mode
+    - rvm: rbx

data/README.md

@@ -1,12 +1,16 @@
 # Fist of Fury
 
-Recurring jobs for [Sucker Punch](https://github.com/brandonhilkert/sucker_punch).
+[![Build Status](https://travis-ci.org/facto/fist_of_fury.png?branch=master)](https://travis-ci.org/facto/fist_of_fury)
+[![Dependency Status](https://gemnasium.com/facto/fist_of_fury.svg)](https://gemnasium.com/facto/fist_of_fury)
+[![Coverage Status](https://coveralls.io/repos/facto/fist_of_fury/badge.png)](https://coveralls.io/r/facto/fist_of_fury)
+[![Code Climate](https://codeclimate.com/github/facto/fist_of_fury.png)](https://codeclimate.com/github/facto/fist_of_fury)
+[![Gem Version](https://badge.fury.io/rb/fist_of_fury.png)](http://badge.fury.io/rb/fist_of_fury)
 
-**This is alpha software. Use at your own risk!**
+Recurring jobs for [Sucker Punch](https://github.com/brandonhilkert/sucker_punch).
 
 ## Why
 
-Sucker Punch offers the capability to run background jobs within the web server process, allowing for less use of resources and utilization of free hosting on services such as Heroku. However, it does not include the ability to run jobs at scheduled intervals. This is where Fist of Fury comes in.
+Sucker Punch offers the ability to run background jobs *within the web server process*. Yes, this means you can host your app on Heroku without paying for an extra dyno for background jobs. However, Sucker Punch does not implement recurring jobs. Enter Fist of Fury!
 
 ## What
 
@@ -16,7 +20,7 @@ Fist of Fury is heavily inspired by [Sidetiq](https://github.com/tobiassvn/sidet
 
 Add this line to your application's Gemfile:
 
-    gem 'fist_of_fury', '~> 0.1'
+    gem 'fist_of_fury', '~> 0.1.0'
 
 And then execute:
 
@@ -69,10 +73,19 @@ unless defined?(Rails::Console)
 end
 ```
 
+## Configuration Options
+
+In the initializer:
+
+```Ruby
+FistOfFury.configure do |config|
+  config.utc = true # false by default; makes all time within Fist of Fury UTC
+end
+```
 
 ## Contributing
 
-1. Fork it ( http://github.com/<my-github-username>/fist_of_fury/fork )
+1. Fork it ( http://github.com/facto/fist_of_fury/fork )
 2. Create your feature branch (`git checkout -b my-new-feature`)
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)

data/Rakefile

@@ -1 +1,12 @@
 require "bundler/gem_tasks"
+require 'rspec/core/rake_task'
+
+RSpec::Core::RakeTask.new('spec')
+
+# If you want to make this the default task
+task :default => :spec
+task :test => :spec
+
+task :console do
+  exec "irb -r fist_of_fury -I ./lib"
+end

data/fist_of_fury.gemspec

@@ -23,6 +23,11 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'rspec'
   gem.add_development_dependency 'guard'
   gem.add_development_dependency 'guard-rspec'
+  gem.add_development_dependency 'ruby_gntp'
+
+  if RUBY_PLATFORM != "java"
+    gem.add_development_dependency 'coveralls'
+  end
 
   gem.add_dependency 'sucker_punch', '>= 1.0.2'
   gem.add_dependency 'celluloid',    '>= 0.15.2'

data/lib/fist_of_fury/actor/clock.rb

@@ -1,33 +1,15 @@
 module FistOfFury
   module Actor
-    class Clock
-      include Logging
+    class Clock < FistOfFury::Clock
       include FistOfFury::Actor
 
-      attr_reader :schedules
-
       def initialize(*args, &block)
         after(0) do
-          debug 'FistOfFury::Clock looping...'
+          debug 'FistOfFury::Clock starting loop...'
           loop!
         end
       end
 
-      def schedule_for(worker)
-        worker.schedule if worker.respond_to? :schedule
-      end
-
-      def tick(tick = current_time)
-        FistOfFury.workers.each do |worker|
-          FistOfFury.dispatcher.dispatch(worker, tick)
-        end
-      end
-
-      def current_time
-        # TODO: configurable time zone support
-        Time.now
-      end
-
       private
 
       def loop!
@@ -39,12 +21,6 @@ module FistOfFury
         # handle_exception(e, context: 'FistOfFury::Clock#loop!')
         raise e
       end
-
-      def time
-        start = current_time
-        yield
-        1 - (current_time.to_f - start.to_f)
-      end
     end
   end
 end

data/lib/fist_of_fury/actor/dispatcher.rb

@@ -1,32 +1,7 @@
 module FistOfFury
   module Actor
-    class Dispatcher
-      include Logging
+    class Dispatcher < FistOfFury::Dispatcher
       include FistOfFury::Actor
-
-      def dispatch(worker, tick)
-        schedule = worker.schedule
-        return unless schedule.schedule_next?(tick)
-        enqueue(worker, schedule.next_occurrence(tick))
-      rescue StandardError => e
-        # TODO: exception handling support
-        raise e
-      end
-
-      private
-
-      def enqueue(worker, next_run_time)
-        klass = worker.to_s
-        info "Enqueue: #{klass}"
-        last_run_time = FistOfFury.store["#{klass}:next"] || -1
-        next_run_time = next_run_time.to_f
-        FistOfFury.store["#{klass}:last"] = last_run_time
-        FistOfFury.store["#{klass}:next"] = next_run_time
-        worker.new.async.perform
-      rescue StandardError => e
-        # TODO: exception handling support
-        raise e
-      end
     end
   end
 end

data/lib/fist_of_fury/clock.rb

@@ -0,0 +1,29 @@
+module FistOfFury
+  class Clock
+    include Logging
+
+    attr_reader :schedules
+
+    def schedule_for(worker)
+      worker.schedule if worker.respond_to? :schedule
+    end
+
+    def current_time
+      FistOfFury.config.utc ? Time.now.utc : Time.now
+    end
+
+    protected
+
+    def time
+      start = current_time
+      yield
+      1 - (current_time.to_f - start.to_f)
+    end
+
+    def tick(time = current_time)
+      FistOfFury.workers.each do |worker|
+        FistOfFury.dispatcher.dispatch(worker, time)
+      end
+    end
+  end
+end

data/lib/fist_of_fury/config.rb

@@ -0,0 +1,20 @@
+module FistOfFury
+  class << self
+    attr_writer :config
+
+    def configure
+      yield config
+    end
+
+    def configure_with_defaults
+      config.utc = false
+      yield config if block_given?
+    end
+
+    def config
+      @config ||= OpenStruct.new
+    end
+  end
+
+  configure_with_defaults
+end

data/lib/fist_of_fury/dispatcher.rb

@@ -0,0 +1,29 @@
+module FistOfFury
+  class Dispatcher
+    include Logging
+
+    def dispatch(worker, time)
+      worker.schedule_next(time) do
+        enqueue(worker, worker.next_occurrence(time))
+      end
+    rescue StandardError => e
+      # TODO: exception handling support
+      raise e
+    end
+
+    protected
+
+    def enqueue(worker, next_run_time)
+      klass = worker.to_s
+      info "Enqueue: #{klass}"
+      last_run_time = FistOfFury.store["#{klass}:next"] || -1
+      next_run_time = next_run_time.to_f
+      FistOfFury.store["#{klass}:last"] = last_run_time
+      FistOfFury.store["#{klass}:next"] = next_run_time
+      worker.new.async.perform
+    rescue StandardError => e
+      # TODO: exception handling support
+      raise e
+    end
+  end
+end

data/lib/fist_of_fury/recurrent.rb

@@ -24,6 +24,10 @@ module FistOfFury
         schedule.options = options
       end
 
+      def schedule_next(time, &block)
+        schedule.schedule_next(time, &block)
+      end
+
       private
 
       def scheduled_occurrence(key)

data/lib/fist_of_fury/schedule.rb

@@ -1,39 +1,45 @@
 module FistOfFury
   class Schedule
-    attr_accessor :options
-    attr_reader :last_occurrence
-
-    # TODO: configurable timezone support
-    START_TIME = Time.local(2010, 1, 1)
+    attr_accessor :options, :ice_cube_schedule, :last_occurrence
 
     def initialize
-      @schedule = IceCube::Schedule.new(START_TIME)
+      self.ice_cube_schedule = IceCube::Schedule.new(start_time)
     end
 
     def method_missing(meth, *args, &block)
       if IceCube::Rule.respond_to?(meth)
         rule = IceCube::Rule.send(meth, *args, &block)
-        @schedule.add_recurrence_rule(rule)
+        ice_cube_schedule.add_recurrence_rule(rule)
         rule
-      elsif @schedule.respond_to?(meth)
-        @schedule.send(meth, *args, &block)
+      elsif ice_cube_schedule.respond_to?(meth)
+        ice_cube_schedule.send(meth, *args, &block)
       else
         super
       end
     end
 
-    def schedule_next?(time)
-      next_occurrence = @schedule.next_occurrence(time)
-      if @last_scheduled != next_occurrence
-        @last_scheduled = next_occurrence
-        true
-      else
-        false
-      end
+    def schedule_next(time)
+      return unless schedule_next?(time)
+      yield
+      self.last_occurrence = next_occurrence(time)
     end
 
     def to_s
-      @schedule.to_s
+      ice_cube_schedule.to_s
+    end
+
+    def start_time
+      @start_time ||= FistOfFury.config.utc ? Time.utc(2010, 1, 1) : Time.local(2010, 1, 1)
+    end
+
+    private
+
+    def next_occurrence(time)
+      ice_cube_schedule.next_occurrence(time)
+    end
+
+    def schedule_next?(time)
+      last_occurrence != next_occurrence(time)
     end
   end
 end

data/lib/fist_of_fury/supervisor.rb

@@ -21,10 +21,6 @@ module FistOfFury
         info 'FistOfFury::Supervisor start'
         super
       end
-
-      def run
-        raise "FistOfFury::Supervisor should not be run in the foreground."
-      end
     end
   end
 end

data/lib/fist_of_fury/version.rb

@@ -1,3 +1,3 @@
 module FistOfFury
-  VERSION = "0.0.1"
+  VERSION = "0.1.0"
 end

data/lib/fist_of_fury.rb

@@ -10,10 +10,13 @@ require 'sucker_punch'
 require 'celluloid'
 
 # internal
+require 'fist_of_fury/config'
 require 'fist_of_fury/subclass_tracking'
 require 'fist_of_fury/api'
 require 'fist_of_fury/logging'
 require 'fist_of_fury/schedule'
+require 'fist_of_fury/clock'
+require 'fist_of_fury/dispatcher'
 require 'fist_of_fury/recurrent'
 require 'fist_of_fury/version'
 
@@ -49,10 +52,8 @@ module FistOfFury
   end
 
   def attack!
-    return true if @attacking
     yield if block_given?
     FistOfFury::Supervisor.run!
-    @attacking = true
   end
 end
 

data/spec/fist_of_fury/actor_sharedspec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+shared_examples_for 'an actor' do
+  it 'mixes in Celluloid' do
+    expect(subject).to respond_to(:after)
+  end
+end

data/spec/fist_of_fury/actors/clock_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+require 'fist_of_fury/actor_sharedspec'
+require 'fist_of_fury/clock_sharedspec'
+
+describe FistOfFury::Actor::Clock do
+  it_should_behave_like 'an actor'
+  it_should_behave_like 'a clock'
+
+  let(:clock) { subject }
+
+  before :each do
+    expect_any_instance_of(described_class).to receive(:after)
+  end
+
+  describe '#initialize' do
+    it 'makes a deferred call' do
+      clock
+    end
+  end
+end

data/spec/fist_of_fury/actors/dispatcher_spec.rb

@@ -0,0 +1,8 @@
+require 'spec_helper'
+require 'fist_of_fury/actor_sharedspec'
+require 'fist_of_fury/dispatcher_sharedspec'
+
+describe FistOfFury::Actor::Dispatcher do
+  it_should_behave_like 'an actor'
+  it_should_behave_like 'a dispatcher'
+end

data/spec/fist_of_fury/api_sharedspec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+shared_examples_for 'the api' do
+  describe '#workers' do
+    it 'returns the deep subclass list from Recurrent' do
+      expect(FistOfFury::Recurrent).to receive(:subclasses).with(true)
+      subject.workers
+    end
+  end
+
+  describe '#schedules' do
+    it 'returns the worker schedules' do
+      worker_list = double('worker_list')
+      expect(worker_list).to receive(:map)
+      allow_any_instance_of(described_class).to receive(:workers).and_return(worker_list)
+      subject.schedules
+    end
+  end
+end

data/spec/fist_of_fury/clock_sharedspec.rb

@@ -0,0 +1,43 @@
+require 'spec_helper'
+require 'fist_of_fury/logging_sharedspec'
+
+shared_examples_for 'a clock' do
+  it_should_behave_like 'a logger'
+
+  describe '#schedule_for' do
+    it "delegates to the worker's schedule" do
+      worker = double('worker')
+      worker.stub(:schedule)
+      expect(worker).to receive(:schedule)
+      subject.schedule_for(worker)
+    end
+  end
+
+  describe '#current_time' do
+    context 'when UTC is enabled' do
+      before :each do
+        FistOfFury.configure { |config| config.utc = true }
+      end
+
+      it 'returns the current time in UTC' do
+        now = double('now')
+        utc = double('utc')
+        now.stub(:utc).and_return(utc)
+        Time.stub(:now).and_return(now)
+        expect(subject.current_time).to eq utc
+      end
+    end
+
+    context 'when UTC is not enabled' do
+      before :each do
+        FistOfFury.configure { |config| config.utc = false }
+      end
+
+      it 'returns the current time in the default time zone' do
+        now = double('now')
+        Time.stub(:now).and_return(now)
+        expect(subject.current_time).to eq now
+      end
+    end
+  end
+end

data/spec/fist_of_fury/config_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+describe FistOfFury do
+  describe 'config' do
+    describe '#configure' do
+      let(:config) { OpenStruct.new }
+
+      before :each do
+        described_class.config = config
+      end
+
+      it 'calls the block and yields the config ' do
+        block_called = false
+        described_class.configure do |config|
+          expect(config).to eq(config)
+          block_called = true
+        end
+        expect(block_called).to be_true
+      end
+    end
+
+    describe '#configure_with_defaults' do
+      let(:config) { OpenStruct.new }
+
+      before :each do
+        described_class.config = config
+      end
+
+      it 'calls the block and yields the config ' do
+        block_called = false
+        described_class.configure_with_defaults do |config|
+          expect(config).to eq(config)
+          block_called = true
+        end
+        expect(block_called).to be_true
+      end
+    end
+
+    describe '#config' do
+      it 'has some default values' do
+        expect(described_class.config).to eq OpenStruct.new(
+          utc: false
+        )
+      end
+    end
+  end
+end

data/spec/fist_of_fury/dispatcher_sharedspec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+require 'fist_of_fury/logging_sharedspec'
+
+shared_examples_for 'a dispatcher' do
+  it_should_behave_like 'a logger'
+
+  describe '#dispatch' do
+
+    let(:time) { double('time') }
+
+    let(:worker) { double('worker') }
+
+    it 'delegates scheduling to the worker' do
+      expect(worker).to receive(:schedule_next) do |*args, &block|
+        expect(args).to include(time)
+        expect(block).to_not be_nil
+      end
+      subject.dispatch(worker, time)
+    end
+  end
+end

data/spec/fist_of_fury/job_spec.rb

@@ -0,0 +1,6 @@
+require 'spec_helper'
+require 'fist_of_fury/recurrent_sharedspec'
+
+describe AwesomeJob do
+  it_should_behave_like 'a job with recurrence'
+end

data/spec/fist_of_fury/logging_sharedspec.rb

@@ -0,0 +1,9 @@
+require 'spec_helper'
+
+shared_examples_for 'a logger' do
+  [:fatal, :error, :warn, :info, :debug].each do |level|
+    it "responds to :#{level}" do
+      expect(subject).to respond_to(level)
+    end
+  end
+end

data/spec/fist_of_fury/railtie_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+module ::Rails
+  class Railtie
+    def self.initializer(name)
+    end
+  end
+end
+
+require_relative '../../lib/fist_of_fury/railtie'
+
+describe FistOfFury::Railtie do
+  after :all do
+    Object.send(:remove_const, :Rails)
+  end
+
+  it 'derives from ::Rails::Railtie' do
+    expect(described_class.ancestors).to include(::Rails::Railtie)
+  end
+end

data/spec/fist_of_fury/recurrent_sharedspec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+require 'fist_of_fury/subclass_tracking_sharedspec'
+
+shared_examples_for 'a job with recurrence' do
+  it_should_behave_like 'a class that tracks its subclasses'
+
+  after :each do
+    Celluloid::Actor.clear_registry
+  end
+
+  describe '::schedule' do
+    it 'returns a new schedule' do
+      schedule = double('schedule')
+      FistOfFury::Schedule.stub(:new).and_return(schedule)
+      expect(described_class.schedule).to eq schedule
+    end
+  end
+
+  describe '::last_scheduled_occurrence' do
+    it 'calls scheduled_occurrence with the right arguments' do
+      expect(described_class).to receive(:scheduled_occurrence).with('last')
+      described_class.last_scheduled_occurrence
+    end
+  end
+
+  describe '::next_scheduled_occurrence' do
+    it 'calls scheduled_occurrence with the right arguments' do
+      expect(described_class).to receive(:scheduled_occurrence).with('next')
+      described_class.next_scheduled_occurrence
+    end
+  end
+
+  describe '::recurs' do
+    it 'evaluate the block within the context of the schedule and sets the options' do
+      block = lambda {}
+      schedule = double('schedule')
+      options = double('optionss')
+      described_class.schedule = schedule
+      expect(schedule).to receive(:instance_eval)
+      expect(schedule).to receive(:options=).with(options)
+      described_class.recurs(options, &block)
+    end
+  end
+
+  describe '::schedule_next' do
+    let(:block) { lambda {} }
+    let(:time) { double('time') }
+    let(:schedule) { double('schedule') }
+
+    it 'delegates to the schedule' do
+      described_class.schedule = schedule
+      expect(schedule).to receive(:schedule_next).with(time)
+      described_class.schedule_next(time, &block)
+    end
+  end
+end

data/spec/fist_of_fury/schedule_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+
+describe FistOfFury::Schedule do
+  let(:schedule) { subject }
+
+  describe '#initialize' do
+    it 'creates an IceCube schedule' do
+      ice_cube_schedule = double('ice_cube_schedule')
+      start_time = double('start_time')
+      allow_any_instance_of(described_class).to receive(:start_time).and_return(start_time)
+      allow(IceCube::Schedule).to receive(:new).with(start_time).and_return(ice_cube_schedule)
+      expect_any_instance_of(described_class).to receive(:ice_cube_schedule=).with(ice_cube_schedule)
+      schedule
+    end
+  end
+
+  describe '#schedule_next' do
+    let(:time) { double('time') }
+    let(:block) { lambda {} }
+    let(:next_occurrence) { double('next_occurrence') }
+
+    before :each do
+      allow(schedule).to receive(:next_occurrence).and_return(next_occurrence)
+    end
+
+    context 'when it should be scheduled' do
+      before :each do
+        allow(schedule).to receive(:schedule_next?).and_return(true)
+      end
+
+      it 'calls the block' do
+        block_called = false
+        schedule.schedule_next(time) { block_called = true }
+        expect(block_called).to be_true
+      end
+
+      it 'sets the last occurrence' do
+        expect(schedule).to receive(:last_occurrence=).with(next_occurrence)
+        schedule.schedule_next(time, &block)
+      end
+    end
+
+    context 'when it should not be scheduled' do
+      before :each do
+        allow(schedule).to receive(:schedule_next?).and_return(false)
+      end
+
+      it 'does not call the block' do
+        block_called = false
+        schedule.schedule_next(time) { block_called = true }
+        expect(block_called).to be_false
+      end
+
+      it 'does not set the last occurrence' do
+        expect(schedule).not_to receive(:last_occurrence=).with(next_occurrence)
+        schedule.schedule_next(time, &block)
+      end
+    end
+  end
+
+  describe '#to_s' do
+    it 'delegates to ice_cube_schedule' do
+      expect(schedule.ice_cube_schedule).to receive(:to_s)
+      schedule.to_s
+    end
+  end
+
+  describe '#start_time' do
+    context 'when utc is enabled' do
+      before :each do
+        FistOfFury.configure do |config|
+          config.utc = true
+        end
+      end
+
+      it 'returns the utc start time' do
+        expect(Time).to receive(:utc)
+        schedule
+      end
+    end
+
+    context 'when utc is not enabled' do
+      it 'returns the local start time' do
+        expect(Time).to receive(:local)
+        schedule
+      end
+    end
+  end
+end

data/spec/fist_of_fury/subclass_tracking_sharedspec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+shared_examples_for 'a class that tracks its subclasses' do
+  class ClassThatTracksItsSubclasses < described_class
+  end
+
+  class AnotherClassThatTracksItsSubclasses < ClassThatTracksItsSubclasses
+  end
+
+  describe '#subclasses' do
+    context 'when deep is specified' do
+      it 'returns the class' do
+        expect(described_class.subclasses()).to include(ClassThatTracksItsSubclasses)
+        expect(described_class.subclasses(true)).to include(AnotherClassThatTracksItsSubclasses)
+      end
+    end
+
+    context 'when deep is not specified' do
+      it 'returns the class' do
+        expect(described_class.subclasses(false)).to include(ClassThatTracksItsSubclasses)
+        expect(described_class.subclasses(false)).not_to include(AnotherClassThatTracksItsSubclasses)
+      end
+    end
+  end
+end

data/spec/fist_of_fury/version_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe FistOfFury do
+  it 'has a version' do
+    expect(FistOfFury::VERSION).to_not be_nil
+  end
+
+  it 'has a version that matches the version in the README' do
+    readme_path = File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'README.md'))
+    readme = File.read(readme_path)
+    readme_matches = readme.scan(/gem 'fist_of_fury', '~> (\d+).(\d+).+'/)
+    readme_matches
+    readme_major = readme_matches[0][0]
+    readme_minor = readme_matches[0][1]
+
+    version_matches = FistOfFury::VERSION.scan(/(\d+).(\d+).+/)
+    version_major = version_matches[0][0]
+    version_minor = version_matches[0][1]
+
+    expect(readme_major).to eq(version_major)
+    expect(readme_minor).to eq(version_minor)
+  end
+end

data/spec/fist_of_fury_spec.rb

@@ -1,22 +1,54 @@
 require 'spec_helper'
+require 'fist_of_fury/api_sharedspec'
 
 describe FistOfFury do
-  describe 'logger' do
-    it "delegates get to Sucker Punch's logger" do
+  it_should_behave_like 'the api'
+
+  describe '#logger' do
+    before :each do
+      FistOfFury.logger = nil
+    end
+
+    it "delegates to Sucker Punch's logger" do
       expect(FistOfFury.logger).to eq SuckerPunch.logger
     end
+
+    after :each do
+      FistOfFury.logger = Logger.new(nil)
+    end
   end
 
-  describe 'clock' do
-    it "delegates get to FistOfFury::Supervisor's clock" do
+  describe '#clock' do
+    it "delegates to the supervisor's clock" do
       expect(FistOfFury::Supervisor).to receive(:clock)
       FistOfFury.clock
     end
   end
 
-  describe 'store' do
+  describe '#store' do
     it 'starts out as an empty hash' do
       expect(FistOfFury.store).to eq Hash.new
     end
   end
+
+  describe '#dispatcher' do
+    it "delegates to the supervisor's dispatcher" do
+      expect(FistOfFury::Supervisor).to receive(:dispatcher)
+      FistOfFury.dispatcher
+    end
+  end
+
+  describe '#attack!' do
+    it 'calls the block when provided' do
+      allow(FistOfFury::Supervisor).to receive(:run!)
+      block_called = false
+      FistOfFury.attack! { block_called = true }
+      expect(block_called).to be_true
+    end
+
+    it 'calls #run! on the supervisor' do
+      expect(FistOfFury::Supervisor).to receive(:run!)
+      FistOfFury.attack!
+    end
+  end
 end

data/spec/fixtures/awesome_job.rb

@@ -0,0 +1,7 @@
+class AwesomeJob
+  include SuckerPunch::Job
+  include FistOfFury::Recurrent
+
+  def perform
+  end
+end

data/spec/spec_helper.rb

@@ -3,11 +3,36 @@ begin
 rescue LoadError
 end
 
+if RUBY_PLATFORM != "java"
+  require 'coveralls'
+  Coveralls.wear!
+end
+
 require 'fist_of_fury'
 
+Dir[File.join(File.dirname(__FILE__), 'fixtures/**/*.rb')].each do |fixture|
+  require fixture
+end
+
+FistOfFury.logger = Logger.new(nil)
+# Celluloid.logger = Logger.new(nil)
+
 RSpec.configure do |config|
   config.after(:each) do
     # Clean up the master queue list
     SuckerPunch::Queues.instance_variable_set(:@queues, Set.new)
+
+    # Make sure supervisor is fake
+    class ::FistOfFury::Supervisor
+      def self.clock
+        @clock ||= FistOfFury::Clock.new
+      end
+
+      def self.dispatcher
+        @dispatcher ||= FistOfFury::Dispatcher.new
+      end
+    end
+
+    FistOfFury.configure_with_defaults
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fist_of_fury
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.0
 platform: ruby
 authors:
 - Joshua Rieken
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-19 00:00:00.000000000 Z
+date: 2014-03-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
@@ -80,6 +80,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: ruby_gntp
+  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: coveralls
+  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: sucker_punch
   requirement: !ruby/object:Gem::Requirement
@@ -131,6 +159,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".travis.yml"
 - Gemfile
 - Guardfile
 - LICENSE.txt
@@ -142,6 +171,9 @@ files:
 - lib/fist_of_fury/actor/clock.rb
 - lib/fist_of_fury/actor/dispatcher.rb
 - lib/fist_of_fury/api.rb
+- lib/fist_of_fury/clock.rb
+- lib/fist_of_fury/config.rb
+- lib/fist_of_fury/dispatcher.rb
 - lib/fist_of_fury/logging.rb
 - lib/fist_of_fury/railtie.rb
 - lib/fist_of_fury/recurrent.rb
@@ -149,7 +181,22 @@ files:
 - lib/fist_of_fury/subclass_tracking.rb
 - lib/fist_of_fury/supervisor.rb
 - lib/fist_of_fury/version.rb
+- spec/fist_of_fury/actor_sharedspec.rb
+- spec/fist_of_fury/actors/clock_spec.rb
+- spec/fist_of_fury/actors/dispatcher_spec.rb
+- spec/fist_of_fury/api_sharedspec.rb
+- spec/fist_of_fury/clock_sharedspec.rb
+- spec/fist_of_fury/config_spec.rb
+- spec/fist_of_fury/dispatcher_sharedspec.rb
+- spec/fist_of_fury/job_spec.rb
+- spec/fist_of_fury/logging_sharedspec.rb
+- spec/fist_of_fury/railtie_spec.rb
+- spec/fist_of_fury/recurrent_sharedspec.rb
+- spec/fist_of_fury/schedule_spec.rb
+- spec/fist_of_fury/subclass_tracking_sharedspec.rb
+- spec/fist_of_fury/version_spec.rb
 - spec/fist_of_fury_spec.rb
+- spec/fixtures/awesome_job.rb
 - spec/spec_helper.rb
 homepage: https://github.com/facto/fist_of_fury
 licenses:
@@ -176,5 +223,20 @@ signing_key:
 specification_version: 4
 summary: Recurring jobs for Sucker Punch
 test_files:
+- spec/fist_of_fury/actor_sharedspec.rb
+- spec/fist_of_fury/actors/clock_spec.rb
+- spec/fist_of_fury/actors/dispatcher_spec.rb
+- spec/fist_of_fury/api_sharedspec.rb
+- spec/fist_of_fury/clock_sharedspec.rb
+- spec/fist_of_fury/config_spec.rb
+- spec/fist_of_fury/dispatcher_sharedspec.rb
+- spec/fist_of_fury/job_spec.rb
+- spec/fist_of_fury/logging_sharedspec.rb
+- spec/fist_of_fury/railtie_spec.rb
+- spec/fist_of_fury/recurrent_sharedspec.rb
+- spec/fist_of_fury/schedule_spec.rb
+- spec/fist_of_fury/subclass_tracking_sharedspec.rb
+- spec/fist_of_fury/version_spec.rb
 - spec/fist_of_fury_spec.rb
+- spec/fixtures/awesome_job.rb
 - spec/spec_helper.rb