build_status_server 0.6 → 0.7

data/Gemfile

@@ -12,3 +12,7 @@ group :development do
   gem "ruby-debug"
   gem "sinatra"
 end
+
+group :development, :test do
+  gem "rake"
+end

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    build_status_server (0.3)
+    build_status_server (0.7)
       json
 
 GEM
@@ -15,6 +15,7 @@ GEM
     rack (1.4.0)
     rack-protection (1.2.0)
       rack
+    rake (0.9.2.2)
     rbx-require-relative (0.0.5)
     rspec (2.9.0)
       rspec-core (~> 2.9.0)
@@ -41,6 +42,7 @@ PLATFORMS
 DEPENDENCIES
   build_status_server!
   json
+  rake
   rspec
   ruby-debug
   sinatra

data/build_status_server.gemspec

@@ -16,11 +16,19 @@ built for my employer ChallengePost (http://challengepost.com). It works in
 conjunction with our Jenkins Continuous Integration server (and its
 Notification Plugin)) and an Arduino powered Traffic Light controller
 (https://github.com/jcmuller/TrafficLightController) with a pseudo-RESTful API.
-
   EOS
   s.description = "A build notifier server for Jenkins CI that controls an XFD over HTTP"
 
-  s.files = Dir["{lib/**/*,spec/**/*}"] + %w(bin/build_status_server config/config-example.yml LICENSE README.md Gemfile Gemfile.lock build_status_server.gemspec)
+  s.files = Dir["{lib/**/*,spec/**/*}"] + %w(
+    Gemfile
+    Gemfile.lock
+    LICENSE
+    README.md
+    bin/build_status_server
+    build_status_server.gemspec
+    config/config-example.yml
+  )
+
   s.require_path = "lib"
   s.bindir = "bin"
   s.executables = %w(build_status_server)
@@ -28,6 +36,7 @@ Notification Plugin)) and an Arduino powered Traffic Light controller
   s.homepage = "http://github.com/jcmuller/build_status_server"
   s.test_files = Dir["spec/**/*_spec.rb"]
 
+  s.add_development_dependency("rake")
   s.add_development_dependency("ruby-debug")
   s.add_development_dependency("sinatra")
 

data/config/config-example.yml

@@ -1,14 +1,14 @@
     udp_server:
-      address: '127.0.0.1'
+      address: 127.0.0.1
       port: 1234
     tcp_client:
-      host: '127.0.0.1'
+      host: 127.0.0.1
       port: 4567
-      pass: "/green"
-      fail: "/red"
+      pass: /green
+      fail: /red
     store:
-      filename: "/tmp/build_result.yml"
+      filename: /tmp/build_result.yml
     mask:
-      regex: ".*master.*"
-      policy: "include"
-    verbose: true
+      regex: !ruby/regexp /.*master.*/
+      policy: include
+    verbose: false

data/lib/build_status_server.rb

@@ -5,5 +5,6 @@ require "timeout"
 require "yaml"
 
 module BuildStatusServer
+  autoload :Config, 'build_status_server/config'
   autoload :Server, 'build_status_server/server'
 end

data/lib/build_status_server.rb~

@@ -1,10 +1,10 @@
 require "rubygems"
 require "json"
-require "sinatra"
 require "socket"
 require "timeout"
 require "yaml"
 
 module BuildStatusServer
+  autoload :Config, 'build_status_server/config'
   autoload :Server, 'build_status_server/server'
 end

data/lib/build_status_server/config.rb

@@ -0,0 +1,91 @@
+module BuildStatusServer
+  class Config
+    attr_reader :config
+
+    def initialize
+      @config = {}
+    end
+
+    # This is responsible of loading the config object
+    def load(options = {})
+      config = load_config_file(options[:config])
+      import_config(config, options)
+    end
+
+    def method_missing(meth, *args, &block)
+      return config[meth.to_s] if config.has_key?(meth.to_s)
+      super
+    end
+
+    def store_file
+      return File.expand_path(".", store["filename"]) if store
+      nil
+    end
+
+    private
+
+    # This will load the passed in config object into the config attribute
+    def import_config(config = {}, options = {})
+      config["verbose"] = options[:verbose] unless options[:verbose].nil?
+      @config = config
+    end
+
+    # This is responsible to return a hash with the contents of a YAML file
+    def load_config_file(config_file = nil)
+      curated_file = nil
+
+      if config_file
+        f = File.expand_path(config_file)
+        if File.exists?(f)
+          curated_file = f
+        else
+          raise "Supplied config file (#{config_file}) doesn't seem to exist"
+        end
+      else
+        locations_to_try.each do |possible_conf_file|
+          f = File.expand_path(possible_conf_file)
+          if File.exists?(f)
+            curated_file = f
+            break
+          end
+        end
+
+        if curated_file.nil?
+          STDERR.puts <<-EOT
+Looks like there isn't an available configuration file for this program.
+We're very diligently going to use some sensible defaults, but you're
+strongly recommended to create one in any of the following locations:
+
+    #{locations_to_try.join("\n    ")}
+
+ Here is a sample of the contents for that file (and the settings we're going
+ to use):
+
+#{get_example_config}
+          EOT
+
+          return YAML.load(get_example_config)
+        end
+      end
+
+      YAML.load_file(curated_file).tap do |config|
+        raise "This is an invalid configuration file!" unless config.class == Hash
+      end
+    end
+
+    def locations_to_try
+        %w(
+          ~/.config/build_status_server/config.yml
+          ./config/config.yml
+          /etc/build_status_server/config.yml
+          /usr/local/etc/build_status_server/config.yml
+        )
+    end
+
+    def get_example_config
+      filename = "#{File.dirname(File.expand_path(__FILE__))}/../../config/config-example.yml"
+      File.open(filename).read
+    end
+  end
+end
+

data/lib/build_status_server/server.rb

@@ -1,28 +1,18 @@
+# TODO
+# move all configuration stuff to Config
+# and just call config[:blaj] instead of if blah
 module BuildStatusServer
   class Server
-    attr_reader   :config, :store_file, :mask_policy, :verbose
-    attr_accessor :store, :mask
+    attr_reader :config, :store
 
     def initialize(options = {})
-      load_config_file(options[:config])
-      @verbose     = options[:verbose] || config["verbose"]
-      @store_file  = File.expand_path(".", config["store"]["filename"])
-      @mask        = Regexp.new(config["mask"]["regex"])
-      @mask_policy = config["mask"]["policy"] || "exclude"
-    end
-
-    def load_store
-      @store = begin
-                 YAML.load_file(store_file)
-               rescue
-                 {}
-               end
-      @store = {} unless store.class == Hash
+      @config = Config.new
+      config.load(options)
     end
 
     def listen
       sock = UDPSocket.new
-      udp_server = config["udp_server"]
+      udp_server = config.udp_server
 
       begin
         sock.bind(udp_server["address"], udp_server["port"])
@@ -35,11 +25,11 @@ is listening at the same port (#{udp_server["address"]}:#{udp_server["port"]}
         exit
       end
 
-      puts "Listening on UDP #{udp_server["address"]}:#{udp_server["port"]}" if verbose
+      puts "Listening on UDP #{udp_server["address"]}:#{udp_server["port"]}" if config.verbose
 
       while true
         data, addr = sock.recvfrom(2048)
-        #require "ruby-debug"; debugger
+
         if process_job(data)
           status = process_all_statuses
           notify(status)
@@ -49,13 +39,25 @@ is listening at the same port (#{udp_server["address"]}:#{udp_server["port"]}
       sock.close
     end
 
+    private
+
+    def load_store
+      @store = begin
+                 YAML.load_file(config.store_file)
+               rescue
+                 {}
+               end
+      @store = {} unless store.class == Hash
+    end
+
+
     def process_job(data = "{}")
       job = JSON.parse(data)
 
       build_name = job["name"]
 
       unless should_process_build(build_name)
-        STDOUT.puts "Ignoring #{build_name} (#{mask}--#{mask_policy})" if verbose
+        STDOUT.puts "Ignoring #{build_name} (#{config.mask["regex"]}--#{config.mask["policy"]})" if config.verbose
         return false
       end
 
@@ -65,77 +67,33 @@ is listening at the same port (#{udp_server["address"]}:#{udp_server["port"]}
         return false
       end
 
-      phase      = job["build"]["phase"]
-      status     = job["build"]["status"]
+      phase  = job["build"]["phase"]
+      status = job["build"]["status"]
 
       if phase == "FINISHED"
-        STDOUT.puts "Got #{status} for #{build_name} on #{Time.now} [#{job.inspect}]" if verbose
+        STDOUT.puts "Got #{status} for #{build_name} on #{Time.now} [#{job.inspect}]" if config.verbose
         case status
         when "SUCCESS", "FAILURE"
           load_store
           store[build_name] = status
-          File.open(store_file, "w") { |file| YAML.dump(store, file) }
+          File.open(config.store_file, "w") { |file| YAML.dump(store, file) }
           return true
         end
       else
-        STDOUT.puts "Started for #{build_name} on #{Time.now} [#{job.inspect}]" if verbose
+        STDOUT.puts "Started for #{build_name} on #{Time.now} [#{job.inspect}]" if config.verbose
       end
 
       return false
     end
 
-    # Ensure config file exists. If not, copy example into it
-    def load_config_file(config_file)
-      curated_file = nil
-
-      if config_file
-        f = File.expand_path(config_file)
-        if File.exists?(f)
-          curated_file = f
-        else
-          puts "Supplied config file (#{config_file}) doesn't seem to exist" if verbose
-          exit
-        end
-      else
-        locations_to_try = %w(
-          ~/.config/build_status_server/config.yml
-          config/config.yml
-          /etc/build_status_server/config.yml
-          /usr/local/etc/build_status_server/config.yml
-        )
-
-        locations_to_try.each do |possible_conf_file|
-          f = File.expand_path(possible_conf_file)
-          if File.exists?(f)
-            puts "Using #{possible_conf_file}!" if verbose if verbose
-            curated_file = f
-            break
-          end
-        end
-
-        puts <<-EOT
-  Looks like there isn't an available configuration file for this program. You
-  can create one in any of the following locations:
-
-   #{locations_to_try.map{|l| File.expand_path(l)}.join("\n   ")}
-
-   Here is a sample of the contents for that file:
-
-#{File.open("#{File.dirname(File.expand_path(__FILE__))}/../../config/config-example.yml").read}
-
-        EOT
-
-        exit
-      end
-
-      puts "Using #{curated_file}!" if verbose
-      @config = YAML.load_file(curated_file)
-    end
-
     def should_process_build(build_name)
       # If mask exists, then ...
-      ! (!!mask && ((mask_policy == "include" && build_name !~ mask) ||
-                    (mask_policy != "include" && build_name =~ mask)))
+      ! (
+        !!config.mask &&
+        !!config.mask["regex"] &&
+        ((config.mask["policy"] == "include" && build_name !~ config.mask["regex"]) ||
+         (config.mask["policy"] != "include" && build_name =~ config.mask["regex"])
+      ))
     end
 
     def process_all_statuses
@@ -149,7 +107,7 @@ is listening at the same port (#{udp_server["address"]}:#{udp_server["port"]}
     end
 
     def notify(status)
-      tcp_client = config["tcp_client"]
+      tcp_client = config.tcp_client
 
       attempts = 0
       light  = status ? tcp_client["pass"] : tcp_client["fail"]
@@ -160,7 +118,7 @@ is listening at the same port (#{udp_server["address"]}:#{udp_server["port"]}
           client = TCPSocket.new(tcp_client["host"], tcp_client["port"])
           client.print "GET #{light} HTTP/1.0\n\n"
           answer = client.gets(nil)
-          STDOUT.puts answer if verbose
+          STDOUT.puts answer if config.verbose
           client.close
         end
       rescue Timeout::Error => ex

data/lib/build_status_server/version.rb

@@ -1,3 +1,3 @@
 module BuildStatusServer
-  VERSION = "0.6"
+  VERSION = "0.7"
 end

data/lib/build_status_server/version.rb~

@@ -1,3 +1,3 @@
 module BuildStatusServer
-  VERSION = "0.5"
+  VERSION = "0.6"
 end

data/spec/lib/build_status_server/config_spec.rb

@@ -0,0 +1,100 @@
+require 'spec_helper'
+require 'build_status_server'
+require 'tempfile'
+
+describe BuildStatusServer::Config do
+  let(:config) { BuildStatusServer::Config.new }
+
+  describe "#load" do
+    it "should call load_config_file with options passed in" do
+      options = {:config => nil}
+      config.should_receive(:load_config_file).with(options[:config])
+      config.load(options)
+    end
+
+    it "should set the config values from yaml file" do
+      config.should_receive(:load_config_file).and_return(YAML.load(config.send(:get_example_config)))
+      config.load
+      config.udp_server.should == {'address' => '127.0.0.1', 'port' => 1234}
+      config.verbose.should == false
+    end
+  end
+
+  describe "#load_config_file" do
+    it "should load the yaml file passed in as a file argument" do
+      file_name = nil
+
+      Tempfile.open('config') do |f|
+        f.puts "---"
+        f.puts "key: value"
+        f.puts "key2: value2"
+        file_name = f.path
+      end
+
+      config.send(:load_config_file, file_name).should == {
+        "key" => "value",
+        "key2" => "value2"
+      }
+    end
+
+    it "should try to load paths from the locations to try" do
+      file_name = nil
+      Tempfile.open("config") do |f|
+        f.puts "---"
+        f.puts "key: value"
+        f.puts "key2: value2"
+        file_name = f.path
+      end
+
+      config.stub!(:locations_to_try).and_return([file_name])
+      config.send(:load_config_file).should == {
+        "key" => "value",
+        "key2" => "value2"
+      }
+    end
+
+    it "should throw an exception if the config file doesn't exist" do
+      file_name = "/tmp/i_dont_exist.yml"
+      expect { config.send(:load_config_file, file_name) }.should raise_error RuntimeError, "Supplied config file (#{file_name}) doesn't seem to exist"
+    end
+
+    it "should throw an exception if the config file isn't a hash" do
+      file_name = nil
+      Tempfile.open(file_name) do |f|
+        f.puts "YADDA YADDA"
+        file_name = f.path
+      end
+      expect { config.send(:load_config_file, file_name) }.should raise_error RuntimeError, "This is an invalid configuration file!"
+    end
+
+    it "should return the default options if no default location is found" do
+      STDERR.should_receive(:puts)
+      config_hash = config.send(:load_config_file)
+      config_hash["udp_server"]["address"].should == '127.0.0.1'
+      config_hash["verbose"].should == false
+    end
+  end
+
+  describe "#store_file" do
+    it "returns the store file configured" do
+      config.stub!(:store).and_return("filename" => "/tmp/build_result.yml")
+      config.store_file.should == "/tmp/build_result.yml"
+    end
+
+    it "returns nil if config doesn't have store option" do
+      config.stub!(:store).and_return(nil)
+      config.store_file.should be_nil
+    end
+  end
+
+  describe "#method_missing" do
+    it "should respond to methods named after elements in the config hash" do
+      config.send(:import_config, "blah" => 1)
+      config.blah.should == 1
+    end
+
+    it "should not respond to methods named after elements that don't exist" do
+      expect { config.blah }.should raise_error NoMethodError
+    end
+  end
+end

data/spec/lib/build_status_server/server_spec.rb

@@ -0,0 +1,121 @@
+require 'spec_helper'
+require 'build_status_server'
+
+describe BuildStatusServer::Server do
+  let(:server) { BuildStatusServer::Server.new }
+
+  before do
+    STDERR.should_receive :puts
+  end
+
+  describe "#listen"
+
+  context "private methods" do
+    describe "#load_store" do
+      before do
+        server.config.stub!(:store_file).and_return("/tmp/build")
+      end
+
+      it "initializes an empty hash if store file doesn't exist" do
+        server.send(:load_store)
+        server.store.should == {}
+      end
+
+      it "initializes an empty hash if store file is empty" do
+        require "tempfile"
+        f = Tempfile.new("server_spec")
+        server.stub!(:store_file).and_return(f.path)
+
+        server.send(:load_store)
+        server.store.should == {}
+      end
+
+      it "initializes a hash with the contents of the store file" do
+        server.config.stub!(:store_file).and_return("spec/support/build_result.yml")
+        server.send(:load_store)
+
+        server.store.should == {"blah" => "SUCCESS", "test" => "SUCCESS"}
+      end
+    end
+
+    describe "#notify"
+    describe "#process_all_statuses"
+    describe "#process_job"
+
+    describe "#should_process_build" do
+      context "when mask exists" do
+        before do
+          server.stub!(:mask).and_return(%r{.*(?:master).*})
+        end
+
+        context "when policy is include" do
+          before do
+            server.stub!(:mask_policy).and_return("include")
+          end
+
+          it "ignores builds if mask doesn't match build name" do
+            server.send(:should_process_build, "blah-development").should be_false
+          end
+
+          it "processes builds if mask matches build name" do
+            server.send(:should_process_build, "blah-master").should be_true
+          end
+        end
+
+        context "when policy is exclude" do
+          before do
+            server.config.stub!(:mask).and_return({"policy" => "exclude", "regex" => /.*(?:master).*/})
+          end
+
+          it "ignores builds if mask matches build name" do
+            server.send(:should_process_build, "blah-master").should be_false
+          end
+
+          it "processes builds if mask doesn't match build name" do
+            server.send(:should_process_build, "blah-development").should be_true
+          end
+        end
+
+        context "when policy is undefined it defaults to ignore" do
+          before do
+            server.config.stub!(:mask).and_return({"policy" => nil, "regex" => /.*(?:master).*/})
+          end
+
+          it "ignores builds if mask matches build name" do
+            server.send(:should_process_build, "blah-master").should be_false
+          end
+
+          it "processes builds if mask doesn't match build name" do
+            server.send(:should_process_build, "blah-development").should be_true
+          end
+        end
+
+        context "when policy is unexpected it defaults to ignore" do
+          before do
+            server.config.stub!(:mask).and_return({"policy" => "trash", "regex" => /.*(?:master).*/})
+          end
+
+          it "ignores builds if mask matches build name" do
+            server.send(:should_process_build, "blah-master").should be_false
+          end
+
+          it "processes builds if mask doesn't match build name" do
+            server.send(:should_process_build, "blah-development").should be_true
+          end
+        end
+      end
+
+      context "when mask doesn't" do
+        before do
+          server.config.stub!(:mask).and_return({"policy" => "include", "regex" => nil})
+        end
+
+        it "should process all jobs" do
+          server.send(:should_process_build, "blah-development").should be_true
+        end
+      end
+    end
+  end
+end
+
+# vim:set foldmethod=syntax foldlevel=1:

metadata

@@ -1,12 +1,12 @@
 --- !ruby/object:Gem::Specification 
 name: build_status_server
 version: !ruby/object:Gem::Version 
-  hash: 7
+  hash: 5
   prerelease: 
   segments: 
   - 0
-  - 6
-  version: "0.6"
+  - 7
+  version: "0.7"
 platform: ruby
 authors: 
 - Juan C. Muller
@@ -14,11 +14,10 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-04-26 00:00:00 -04:00
-default_executable: 
+date: 2012-05-01 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: ruby-debug
+  name: rake
   prerelease: false
   requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
@@ -32,7 +31,7 @@ dependencies:
   type: :development
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
-  name: sinatra
+  name: ruby-debug
   prerelease: false
   requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
@@ -46,7 +45,7 @@ dependencies:
   type: :development
   version_requirements: *id002
 - !ruby/object:Gem::Dependency 
-  name: json
+  name: sinatra
   prerelease: false
   requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
@@ -57,8 +56,22 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  type: :runtime
+  type: :development
   version_requirements: *id003
+- !ruby/object:Gem::Dependency 
+  name: json
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :runtime
+  version_requirements: *id004
 description: A build notifier server for Jenkins CI that controls an XFD over HTTP
 email: jcmuller@gmail.com
 executables: 
@@ -68,27 +81,24 @@ extensions: []
 extra_rdoc_files: []
 
 files: 
-- lib/build_status_server/requirements.rb~
+- lib/build_status_server/config.rb
 - lib/build_status_server/server.rb
-- lib/build_status_server/server.rb~
 - lib/build_status_server/version.rb
 - lib/build_status_server/version.rb~
 - lib/build_status_server.rb
 - lib/build_status_server.rb~
-- spec/lib/build_status_server_spec.rb
-- spec/lib/build_status_server_spec.rb~
+- spec/lib/build_status_server/config_spec.rb
+- spec/lib/build_status_server/server_spec.rb
 - spec/spec_helper.rb
-- spec/spec_helper.rb~
 - spec/support/build_result.yml
 - spec/support/sample.json
-- bin/build_status_server
-- config/config-example.yml
-- LICENSE
-- README.md
 - Gemfile
 - Gemfile.lock
+- LICENSE
+- README.md
+- bin/build_status_server
 - build_status_server.gemspec
-has_rdoc: true
+- config/config-example.yml
 homepage: http://github.com/jcmuller/build_status_server
 licenses: 
 - GPL
@@ -118,9 +128,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 1.8.23
 signing_key: 
 specification_version: 3
 summary: This utility is part of an XFD (eXtreeme Feedback Device) solution designed and built for my employer ChallengePost (http://challengepost.com). It works in conjunction with our Jenkins Continuous Integration server (and its Notification Plugin)) and an Arduino powered Traffic Light controller (https://github.com/jcmuller/TrafficLightController) with a pseudo-RESTful API.
 test_files: 
-- spec/lib/build_status_server_spec.rb
+- spec/lib/build_status_server/config_spec.rb
+- spec/lib/build_status_server/server_spec.rb

data/lib/build_status_server/requirements.rb~

@@ -1,8 +0,0 @@
-require "rubygems"
-require "bundler/setup"
-
-Bundler.require(:default)
-
-require "socket"
-require "timeout"
-require "yaml"

data/lib/build_status_server/server.rb~

@@ -1,189 +0,0 @@
-module BuildStatusServer
-  class Server
-    attr_reader   :config, :store_file, :mask_policy, :verbose
-    attr_accessor :store, :mask
-
-    def initialize(options = {})
-      load_config_file(options[:config])
-      @verbose     = options[:verbose] || config["verbose"]
-      @store_file  = File.expand_path(".", config["store"]["filename"])
-      @mask        = Regexp.new(config["mask"]["regex"])
-      @mask_policy = config["mask"]["policy"] || "exclude"
-    end
-
-    def load_store
-      @store = begin
-                 YAML.load_file(store_file)
-               rescue
-                 {}
-               end
-      @store = {} unless store.class == Hash
-    end
-
-    def listen
-      sock = UDPSocket.new
-      udp_server = config["udp_server"]
-      sock.bind(udp_server["address"], udp_server["port"])
-
-      puts "Listening on UDP #{udp_server["address"]}:#{udp_server["port"]}" if verbose
-
-      while true
-        data, addr = sock.recvfrom(2048)
-        #require "ruby-debug"; debugger
-        if process_job(data)
-          status = process_all_statuses
-          notify(status)
-        end
-      end
-
-      sock.close
-    end
-
-    def process_job(data = "{}")
-      job = JSON.parse(data)
-
-      build_name = job["name"]
-
-      unless should_process_build(build_name)
-        STDOUT.puts "Ignoring #{build_name} (#{mask}--#{mask_policy})" if verbose
-        return false
-      end
-
-      if job.class != Hash or
-        job["build"].class != Hash
-        STDERR.puts "Pinged with an invalid payload"
-        return false
-      end
-
-      phase      = job["build"]["phase"]
-      status     = job["build"]["status"]
-
-      if phase == "FINISHED"
-        STDOUT.puts "Got #{status} for #{build_name} on #{Time.now} [#{job.inspect}]" if verbose
-        case status
-        when "SUCCESS", "FAILURE"
-          load_store
-          store[build_name] = status
-          File.open(store_file, "w") { |file| YAML.dump(store, file) }
-          return true
-        end
-      else
-        STDOUT.puts "Started for #{build_name} on #{Time.now} [#{job.inspect}]" if verbose
-      end
-
-      return false
-    end
-
-    # Ensure config file exists. If not, copy example into it
-    def load_config_file(config_file)
-      curated_file = nil
-
-      if config_file
-        f = File.expand_path(config_file)
-        if File.exists?(f)
-          curated_file = f
-        else
-          puts "Supplied config file (#{config_file}) doesn't seem to exist" if verbose
-          exit
-        end
-      else
-        locations_to_try = %w(
-          ~/.config/build_status_server/config.yml
-          config/config.yml
-          /etc/build_status_server/config.yml
-          /usr/local/etc/build_status_server/config.yml
-        )
-
-        locations_to_try.each do |possible_conf_file|
-          f = File.expand_path(possible_conf_file)
-          if File.exists?(f)
-            puts "Using #{possible_conf_file}!" if verbose if verbose
-            curated_file = f
-            break
-          end
-        end
-
-        puts <<-EOT
-  Looks like there isn't an available configuration file for this program. You
-  can create one in any of the following locations:
-
-   #{locations_to_try.map{|l| File.expand_path(l)}.join("\n   ")}
-
-   Here is a sample of the contents for that file:
-
-#{File.open("#{File.dirname(File.expand_path(__FILE__))}/../../config/config-example.yml").read}
-
-        EOT
-
-        exit
-      end
-
-      puts "Using #{curated_file}!" if verbose
-      @config = YAML.load_file(curated_file)
-    end
-
-    def should_process_build(build_name)
-      # If mask exists, then ...
-      ! (!!mask && ((mask_policy == "include" && build_name !~ mask) ||
-                    (mask_policy != "include" && build_name =~ mask)))
-    end
-
-    def process_all_statuses
-      pass = true
-
-      @store.values.each do |val|
-        pass &&= (val == "pass" || val == "SUCCESS")
-      end
-
-      pass
-    end
-
-    def notify(status)
-      tcp_client = config["tcp_client"]
-
-      attempts = 0
-      light  = status ? tcp_client["pass"] : tcp_client["fail"]
-
-      begin
-        timeout(5) do
-          attempts += 1
-          client = TCPSocket.new(tcp_client["host"], tcp_client["port"])
-          client.print "GET #{light} HTTP/1.0\n\n"
-          answer = client.gets(nil)
-          STDOUT.puts answer if verbose
-          client.close
-        end
-      rescue Timeout::Error => ex
-        STDERR.puts "Error: #{ex} while trying to send #{light}"
-        retry unless attempts > 2
-      rescue Errno::ECONNREFUSED, Errno::EHOSTUNREACH => ex
-        STDERR.puts "Error: #{ex} while trying to send #{light}"
-        STDERR.puts "Will wait for 2 seconds and try again..."
-        sleep 2
-        retry unless attempts > 2
-      end
-    end
-  end
-end
-
-__END__
-
-Example payload:
-{
-  "name":"test",
-  "url":"job/test/",
-  "build":{
-    "full_url":"http://cronus.local:3001/job/test/20/",
-    "number":20,
-    "phase":"FINISHED",
-    "status":"SUCCESS",
-    "url":"job/test/20/"
-  }
-}
-
-We're getting this error once in a while:
-/usr/local/lib/ruby/1.8/timeout.rb:64:in `notify': execution expired (Timeout::Error)
-        from /home/jcmuller/build_notifier/lib/server.rb:102:in `notify'
-        from /home/jcmuller/build_notifier/lib/server.rb:33:in `listen'
-        from bin/server:5
-

data/spec/lib/build_status_server_spec.rb

@@ -1,124 +0,0 @@
-$:.push File.expand_path("lib", __FILE__)
-
-require 'spec_helper'
-require 'build_status_server'
-
-describe BuildStatusServer::Server, :pending => true do
-  let!(:server) { BuildStatusServer::Server.new(:config => '/dev/null') }
-
-  describe "#load_config_file" do
-    it "should use the supplied argument" do
-      config_file = '/dev/null'
-      File.should_receive(:exists?).and_return(true)
-      server.load_config_file(config_file)
-    end
-  end
-
-  describe "#load_store" do
-
-    before do
-      server.stub!(:store_file).and_return("/tmp/build")
-    end
-
-    it "initializes an empty hash if store file doesn't exist" do
-      server.load_store
-      server.store.should == {}
-    end
-
-    it "initializes an empty hash if store file is empty" do
-      require "tempfile"
-      f = Tempfile.new("server_spec")
-      server.stub!(:store_file).and_return(f.path)
-
-      server.load_store
-      server.store.should == {}
-    end
-
-    it "initializes a hash with the contents of the store file" do
-      server.stub!(:store_file).and_return("spec/support/build_result.yml")
-      server.load_store
-
-      server.store.should == {"blah" => "SUCCESS", "test" => "SUCCESS"}
-    end
-  end
-
-  describe "#notify"
-  describe "#process_all_statuses"
-  describe "#process_job"
-
-  describe "#should_process_build" do
-    context "when mask exists" do
-      before do
-        server.stub!(:mask).and_return(/.*master.*/)
-      end
-
-      context "when policy is include" do
-        before do
-          server.stub!(:mask_policy).and_return("include")
-        end
-
-        it "ignores builds if mask doesn't match build name" do
-          server.should_process_build("blah-development").should be_false
-        end
-
-        it "processes builds if mask matches build name" do
-          server.should_process_build("blah-master").should be_true
-        end
-      end
-
-      context "when policy is exclude" do
-        before do
-          server.stub!(:mask_policy).and_return("exclude")
-        end
-
-        it "ignores builds if mask matches build name" do
-          server.should_process_build("blah-master").should be_false
-        end
-
-        it "processes builds if mask doesn't match build name" do
-          server.should_process_build("blah-development").should be_true
-        end
-      end
-
-      context "when policy is undefined" do
-        before do
-          server.stub!(:mask_policy).and_return(nil)
-        end
-
-        it "ignores builds if mask matches build name" do
-          server.should_process_build("blah-master").should be_false
-        end
-
-        it "processes builds if mask doesn't match build name" do
-          server.should_process_build("blah-development").should be_true
-        end
-      end
-
-      context "when policy is unexpected" do
-        before do
-          server.stub!(:mask_policy).and_return("trash")
-        end
-
-        it "ignores builds if mask matches build name" do
-          server.should_process_build("blah-master").should be_false
-        end
-
-        it "processes builds if mask doesn't match build name" do
-          server.should_process_build("blah-development").should be_true
-        end
-      end
-    end
-
-    context "when mask doesn't" do
-      before do
-        server.stub!(:mask).and_return(nil)
-      end
-
-      it "should process all jobs" do
-        server.should_process_build("blah-development").should be_true
-      end
-    end
-  end
-end
-
-# vim:set foldmethod=syntax foldlevel=1:

data/spec/lib/build_status_server_spec.rb~

@@ -1,124 +0,0 @@
-$:.push File.expand_path("lib", __FILE__)
-
-require 'spec_helper'
-require 'build_status_server'
-
-describe BuildStatusServer::Server do
-  let!(:server) { BuildStatusServer::Server.new(:config => '/dev/null') }
-
-  describe "#load_config_file" do
-    it "should use the supplied argument" do
-      config_file = '/dev/null'
-      File.should_receive(:exists?).and_return(true)
-      server.load_config_file(config_file)
-    end
-  end
-
-  describe "#load_store" do
-
-    before do
-      server.stub!(:store_file).and_return("/tmp/build")
-    end
-
-    it "initializes an empty hash if store file doesn't exist" do
-      server.load_store
-      server.store.should == {}
-    end
-
-    it "initializes an empty hash if store file is empty" do
-      require "tempfile"
-      f = Tempfile.new("server_spec")
-      server.stub!(:store_file).and_return(f.path)
-
-      server.load_store
-      server.store.should == {}
-    end
-
-    it "initializes a hash with the contents of the store file" do
-      server.stub!(:store_file).and_return("spec/support/build_result.yml")
-      server.load_store
-
-      server.store.should == {"blah" => "SUCCESS", "test" => "SUCCESS"}
-    end
-  end
-
-  describe "#notify"
-  describe "#process_all_statuses"
-  describe "#process_job"
-
-  describe "#should_process_build" do
-    context "when mask exists" do
-      before do
-        server.stub!(:mask).and_return(/.*master.*/)
-      end
-
-      context "when policy is include" do
-        before do
-          server.stub!(:mask_policy).and_return("include")
-        end
-
-        it "ignores builds if mask doesn't match build name" do
-          server.should_process_build("blah-development").should be_false
-        end
-
-        it "processes builds if mask matches build name" do
-          server.should_process_build("blah-master").should be_true
-        end
-      end
-
-      context "when policy is exclude" do
-        before do
-          server.stub!(:mask_policy).and_return("exclude")
-        end
-
-        it "ignores builds if mask matches build name" do
-          server.should_process_build("blah-master").should be_false
-        end
-
-        it "processes builds if mask doesn't match build name" do
-          server.should_process_build("blah-development").should be_true
-        end
-      end
-
-      context "when policy is undefined" do
-        before do
-          server.stub!(:mask_policy).and_return(nil)
-        end
-
-        it "ignores builds if mask matches build name" do
-          server.should_process_build("blah-master").should be_false
-        end
-
-        it "processes builds if mask doesn't match build name" do
-          server.should_process_build("blah-development").should be_true
-        end
-      end
-
-      context "when policy is unexpected" do
-        before do
-          server.stub!(:mask_policy).and_return("trash")
-        end
-
-        it "ignores builds if mask matches build name" do
-          server.should_process_build("blah-master").should be_false
-        end
-
-        it "processes builds if mask doesn't match build name" do
-          server.should_process_build("blah-development").should be_true
-        end
-      end
-    end
-
-    context "when mask doesn't" do
-      before do
-        server.stub!(:mask).and_return(nil)
-      end
-
-      it "should process all jobs" do
-        server.should_process_build("blah-development").should be_true
-      end
-    end
-  end
-end
-
-# vim:set foldmethod=syntax foldlevel=1:

data/spec/spec_helper.rb~

@@ -1,15 +0,0 @@
-# This file was generated by the `rspec --init` command. Conventionally, all
-# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
-# Require this file using `require "spec_helper.rb"` to ensure that it is only
-# loaded once.
-#
-# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
-
-#require "rspec/core"
-#require "rspec/mocks"
-#
-#RSpec.configure do |config|
-#  config.treat_symbols_as_metadata_keys_with_true_values = true
-#  config.run_all_when_everything_filtered = true
-#  config.filter_run :focus
-#end