localjob 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0917b722ab080a63864457aa154296a191d095ec
-  data.tar.gz: f15b71a3239ae2fd38cba2861c02d61b33b4c019
+  metadata.gz: 3f40a0417920e5c4e5ef5bc6cdb5586f09f67e2a
+  data.tar.gz: 3851bec9273ffdbb4c30ea47010588e51af98b98
 SHA512:
-  metadata.gz: 094f2204527babf51a1da02a0482826305c1510dac6c14e2ca75b7ad7417c8b00c60acb0242783edc24e866a4165ce3425ae909a6127623666aebfc71504ac95
-  data.tar.gz: fb2f04ea2bd0c6510d9e3ce9d3f3cb6feb8b62cb99d132dff6976ff18871adf07ca67c9ac41e5e5b132281dc50e189867a72b800ffe9544b7efe89569563557e
+  metadata.gz: d80362eb2ee2f24eef60f7500cf22d692222dd6ef7bccdda64f44c6b0a853bc84b4fb84542aadd2ba3ceaaf7320f197c5c31ec4cf2b6719c055fd8344e0682f9
+  data.tar.gz: 27a5dec0c5281dd20fd698eefd99487203ff764fa13ca777a0046fc744639e62b1ff0c5c0ec7eedfee2e4ee418e0dc60c60b598778574c28aaefaefe906465c0

data/Gemfile

@@ -2,3 +2,6 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in localjob.gemspec
 gemspec
+
+gem 'coveralls', require: false
+gem 'pry'

data/README.md

@@ -1,4 +1,4 @@
-# Localjob [![Build Status](https://travis-ci.org/Sirupsen/localjob.png?branch=master)](https://travis-ci.org/Sirupsen/localjob) [![Code Climate](https://codeclimate.com/github/Sirupsen/localjob.png)](https://codeclimate.com/github/Sirupsen/localjob)
+# Localjob [![Build Status](https://travis-ci.org/Sirupsen/localjob.png?branch=master)](https://travis-ci.org/Sirupsen/localjob) [![Coverage Status](https://coveralls.io/repos/Sirupsen/localjob/badge.png?branch=master)](https://coveralls.io/r/Sirupsen/localjob?branch=master)
 
 Localjob is a simple, self-contained background queue built on top of [System V
 message queues][sysv] (SysV Message Queue => SysV MQ for short). Workers and the
@@ -84,8 +84,8 @@ Create your instance of the queue as normal in your setup:
 
 ```ruby
 def setup
-  @queue  = Localjob.new("test-queue")
-  @worker = Localjob::Worker.new("test-queue")
+  @queue  = Localjob.new
+  @worker = Localjob::Worker.new(@queue)
 end
 ```
 

data/lib/localjob/version.rb

@@ -1,3 +1,3 @@
 class Localjob
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

data/lib/localjob/worker.rb

@@ -28,7 +28,6 @@ class Localjob
       return work_thread if thread
 
       create_pid_file(@options[:pid_file])
-      deamonize if @options[:deamon]
 
       loop { break unless shift_and_process }
     end
@@ -84,10 +83,6 @@ class Localjob
       Signal.trap("INT") { shutdown }
     end
 
-    def deamonize
-      Process.daemon(true, true)
-    end
-
     def create_pid_file(path)
       File.open(path, 'w') { |f| f << self.pid } if path
     end

data/lib/localjob.rb

@@ -12,7 +12,7 @@ class Localjob
   attr_reader :name
   attr_accessor :queue
 
-  def_delegators :queue, :to_io, :destroy, :size
+  def_delegators :queue, :stats, :destroy, :size
 
   # LOCALJOB in 1337speak
   def initialize(name = 0x10CA110B)

data/test/sysv_adapter_test.rb

@@ -20,4 +20,10 @@ class SysvAdapterTest < LocaljobTestCase
     @localjob << "Hello World"
     assert_equal 1, @localjob.size
   end
+
+  def test_stats
+    @localjob << "Hello World"
+    assert_equal 20, @localjob.stats[:size]
+    assert_equal 1,  @localjob.stats[:count]
+  end
 end

data/test/test_helper.rb

@@ -1,3 +1,6 @@
+require 'coveralls'
+Coveralls.wear!
+
 require 'minitest/unit'
 require 'minitest/autorun'
 $:<< File.dirname(__FILE__) + "/../lib"

data/test/worker_test.rb

@@ -6,6 +6,10 @@ class WorkerTest < LocaljobTestCase
     @worker   = worker(@localjob)
   end
 
+  def teardown
+    @localjob.destroy
+  end
+
   def test_pop_and_send_to_worker
     WalrusJob.any_instance.expects(:perform)
 
@@ -58,17 +62,21 @@ class WorkerTest < LocaljobTestCase
     sleep 0.1
 
     Process.kill("QUIT", pid)
+    sleep 0.1
 
     assert_equal 0, @localjob.size
   end
 
-  def test_sigquit_terminates_the_worker
+  def test_sigint_terminates_the_worker
     @localjob << WalrusJob.new("move")
 
     pid = fork { @worker.work }
     sleep 0.1
 
     Process.kill("INT", pid)
+    sleep 0.1
+
+    assert_equal 0, @localjob.size
   end
 
   def test_thread_worker
@@ -82,4 +90,24 @@ class WorkerTest < LocaljobTestCase
 
     assert_equal 0, @localjob.size
   end
+
+  def test_kill_terminates_forked_worker
+    @localjob << WalrusJob.new("move")
+
+    assert_equal 1, @localjob.size
+
+    fork { @worker.work }
+    # Can do this immediately after, since it pushes a termination message to
+    # the queue.
+    @worker.kill
+    sleep 0.1
+
+    assert_equal 0, @localjob.size
+  end
+
+  def test_sending_termination_message_calls_shutdown!
+    @localjob << Localjob::Worker::TERMINATION_MESSAGE
+    @worker.expects(:exit!)
+    @worker.work
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: localjob
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Simon Eskildsen