ampel_extase 0.1.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: aa656566f184d93b15926cec998ebdafbfda9818
-  data.tar.gz: 85e86d33c060fa95f5160c0fefcd195df190bedf
+  metadata.gz: 185c3c458af6b31e6b267f17af22fbceefef5447
+  data.tar.gz: 279c80d9fa98fb8878d9a0299789903fd607f810
 SHA512:
-  metadata.gz: e63cb5d8603d5dc1fbf0da83d7f2fe6011971976f893ff0826f14ae4d724ad37798989783aecc534bf6d038814e188b25392246d5a3ca130ae1aaee0ea57fba0
-  data.tar.gz: f4258d55ae89298bea4db1dfdd42fc22284af742de0c3cdf0573ca7bfd39c06c03f55d5b952dcb72f24ece1631a64ccf9b6c6d7670e28076ea767c14b4119f70
+  metadata.gz: 8ac46db41479a33a543bf87735d633a9efa49952cceaf6a8bcf9b5c6b0fd33b32485b4be2615ea422d56406d5f0fc5884882ceec8daa4c3350d5beb6c9c5331c
+  data.tar.gz: efe0863f59b6af56b2e4d4eb792fa4d34af22cf1328af1dc77bd543f1781de8f5723e96bd3fc30474dab704990dd9a4e00adb5f6a1f2ce2a17815931d7fa1897

data/.gitignore

@@ -5,5 +5,6 @@
 .rvmrc
 Gemfile.lock
 coverage
+errors.lst
 pkg
 tags

data/.utilsrc

@@ -0,0 +1,21 @@
+# vim: set ft=ruby:
+
+search do
+  prune_dirs /\A(\.svn|\.git|CVS|tmp|tags|coverage)\z/
+  skip_files /(\A\.|\.sw[pon]\z|\.(log|fnm|jpg|jpeg|png|pdf|svg)\z|tags|brakeman.ignore|errors\.lst|~\z)/i
+end
+
+discover do
+  prune_dirs /\A(\.svn|\.git|CVS|tmp|tags|coverage)\z/
+  skip_files /(\A\.|\.sw[pon]\z|\.log\z|~\z)/
+  binary false
+end
+
+strip_spaces do
+  prune_dirs /\A(\..*|CVS)\z/
+  skip_files /(\A\.|\.sw[pon]\z|\.log\z|~\z)/
+end
+
+probe do
+  test_framework :'rspec'
+end

data/Rakefile

@@ -12,15 +12,16 @@ GemHadar do
   licenses    << 'GPL-2'
   test_dir    'spec'
   ignore      '.*.sw[pon]', 'pkg', 'Gemfile.lock', '.rvmrc', 'coverage',
-    'tags', '.bundle', '.DS_Store', '.build'
+    'tags', '.bundle', '.DS_Store', '.build', 'errors.lst'
 
   readme      'README.md'
   executables.merge Dir['bin/*'].map { |x| File.basename(x) }
 
-  dependency             'tins',        '~>1.0'
+  dependency             'tins',           '~>1.0'
+  dependency             'term-ansicolor', '~>1.0'
   dependency             'socket_switcher'
-  development_dependency 'simplecov',   '~>0.9'
-  development_dependency 'rspec',       '~>3.0'
+  development_dependency 'simplecov',      '~>0.9'
+  development_dependency 'rspec',          '~>3.0'
   development_dependency 'byebug'
 
   default_task_dependencies :spec

data/VERSION

@@ -1 +1 @@
-0.1.0
+0.3.0

data/ampel_extase.gemspec

@@ -1,50 +1,53 @@
 # -*- encoding: utf-8 -*-
-# stub: ampel_extase 0.1.0 ruby lib
+# stub: ampel_extase 0.3.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "ampel_extase"
-  s.version = "0.1.0"
+  s.version = "0.3.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Florian Frank"]
-  s.date = "2015-01-02"
+  s.date = "2015-08-25"
   s.description = "Library to control the build traffic light. Yes, we can\u{2026}"
   s.email = "flori@ping.de"
   s.executables = ["ampel_control"]
-  s.extra_rdoc_files = ["README.md", "lib/ampel_extase.rb", "lib/ampel_extase/build_state.rb", "lib/ampel_extase/controller.rb", "lib/ampel_extase/jenkins_client.rb", "lib/ampel_extase/light_switcher.rb", "lib/ampel_extase/version.rb"]
-  s.files = [".gitignore", "Gemfile", "README.md", "Rakefile", "VERSION", "ampel_extase.gemspec", "bin/ampel_control", "lib/ampel_extase.rb", "lib/ampel_extase/build_state.rb", "lib/ampel_extase/controller.rb", "lib/ampel_extase/jenkins_client.rb", "lib/ampel_extase/light_switcher.rb", "lib/ampel_extase/version.rb", "spec/ampel_extase/jenkins_client_spec.rb", "spec/ampel_extase/light_switcher_spec.rb", "spec/spec_helper.rb"]
+  s.extra_rdoc_files = ["README.md", "lib/ampel_extase.rb", "lib/ampel_extase/build_state.rb", "lib/ampel_extase/controller.rb", "lib/ampel_extase/jenkins_client.rb", "lib/ampel_extase/jenkins_state_observer.rb", "lib/ampel_extase/light_switcher.rb", "lib/ampel_extase/version.rb"]
+  s.files = [".gitignore", ".utilsrc", "Gemfile", "README.md", "Rakefile", "VERSION", "ampel_extase.gemspec", "bin/ampel_control", "lib/ampel_extase.rb", "lib/ampel_extase/build_state.rb", "lib/ampel_extase/controller.rb", "lib/ampel_extase/jenkins_client.rb", "lib/ampel_extase/jenkins_state_observer.rb", "lib/ampel_extase/light_switcher.rb", "lib/ampel_extase/version.rb", "spec/ampel_extase/build_state_spec.rb", "spec/ampel_extase/controller_spec.rb", "spec/ampel_extase/jekins_state_observer_spec.rb", "spec/ampel_extase/jenkins_client_spec.rb", "spec/ampel_extase/light_switcher_spec.rb", "spec/spec_helper.rb"]
   s.homepage = "http://github.com/flori/ampel_extase"
   s.licenses = ["GPL-2"]
   s.rdoc_options = ["--title", "AmpelExtase - Library to control the build traffic light", "--main", "README.md"]
-  s.rubygems_version = "2.2.2"
+  s.rubygems_version = "2.4.5.1"
   s.summary = "Library to control the build traffic light"
-  s.test_files = ["spec/ampel_extase/jenkins_client_spec.rb", "spec/ampel_extase/light_switcher_spec.rb", "spec/spec_helper.rb"]
+  s.test_files = ["spec/ampel_extase/build_state_spec.rb", "spec/ampel_extase/controller_spec.rb", "spec/ampel_extase/jekins_state_observer_spec.rb", "spec/ampel_extase/jenkins_client_spec.rb", "spec/ampel_extase/light_switcher_spec.rb", "spec/spec_helper.rb"]
 
   if s.respond_to? :specification_version then
     s.specification_version = 4
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_development_dependency(%q<gem_hadar>, ["~> 1.0.0"])
+      s.add_development_dependency(%q<gem_hadar>, ["~> 1.3.1"])
       s.add_development_dependency(%q<simplecov>, ["~> 0.9"])
       s.add_development_dependency(%q<rspec>, ["~> 3.0"])
       s.add_development_dependency(%q<byebug>, [">= 0"])
       s.add_runtime_dependency(%q<tins>, ["~> 1.0"])
+      s.add_runtime_dependency(%q<term-ansicolor>, ["~> 1.0"])
       s.add_runtime_dependency(%q<socket_switcher>, [">= 0"])
     else
-      s.add_dependency(%q<gem_hadar>, ["~> 1.0.0"])
+      s.add_dependency(%q<gem_hadar>, ["~> 1.3.1"])
       s.add_dependency(%q<simplecov>, ["~> 0.9"])
       s.add_dependency(%q<rspec>, ["~> 3.0"])
       s.add_dependency(%q<byebug>, [">= 0"])
       s.add_dependency(%q<tins>, ["~> 1.0"])
+      s.add_dependency(%q<term-ansicolor>, ["~> 1.0"])
       s.add_dependency(%q<socket_switcher>, [">= 0"])
     end
   else
-    s.add_dependency(%q<gem_hadar>, ["~> 1.0.0"])
+    s.add_dependency(%q<gem_hadar>, ["~> 1.3.1"])
     s.add_dependency(%q<simplecov>, ["~> 0.9"])
     s.add_dependency(%q<rspec>, ["~> 3.0"])
     s.add_dependency(%q<byebug>, [">= 0"])
     s.add_dependency(%q<tins>, ["~> 1.0"])
+    s.add_dependency(%q<term-ansicolor>, ["~> 1.0"])
     s.add_dependency(%q<socket_switcher>, [">= 0"])
   end
 end

data/bin/ampel_control

@@ -2,15 +2,16 @@
 
 require 'ampel_extase'
 
-serial      = ENV['SERIAL'] or
+serial      = ENV['SERIAL'].full? or
   fail 'need env var SERIAL for path to serial port'
-jenkins_url = ENV['JENKINS_URL'] or
+jenkins_url = ENV['JENKINS_URL'].full? or
   fail 'need env var JENKINS_URL of the form http://test2.local:8080/job/foo'
+warning_jenkins_url = ENV['WARNING_JENKINS_URL'].full?
 sleep_duration = (ENV['SLEEP'] || 10).to_i
 
-AmpelExtase::Controller.new(
-  serial,
-  jenkins_url,
-  debug: true,
-  sleep: sleep_duration,
+AmpelExtase::Controller.for(
+  serial: serial,
+  jenkins_url: jenkins_url,
+  warning_jenkins_url: warning_jenkins_url,
+  sleep: sleep_duration
 ).start

data/lib/ampel_extase/controller.rb

@@ -1,17 +1,33 @@
 #!/usr/bin/env ruby
 
-require 'ampel_extase/jenkins_client'
+require 'term/ansicolor'
 require 'ampel_extase/light_switcher'
-require 'ampel_extase/build_state'
+require 'ampel_extase/jenkins_state_observer'
 
 class AmpelExtase::Controller
-  def initialize(serial, jenkins_url, debug: false, sleep: 10)
-    @jenkins = AmpelExtase::JenkinsClient.new(jenkins_url, debug: debug)
-    check_jenkins
-    @lights   = AmpelExtase::LightSwitcher.new(serial, debug: debug)
+  include Term::ANSIColor
+
+  def self.for(
+    serial:,
+    jenkins_url:,
+    warning_jenkins_url: nil,
+    sleep: 10
+  )
+    ampel_jenkins   = AmpelExtase::JenkinsStateObserver.for_url(jenkins_url)
+    warning_jenkins = AmpelExtase::JenkinsStateObserver.for_url(warning_jenkins_url)
+    lights = AmpelExtase::LightSwitcher.for(serial: serial)
+    new(ampel_jenkins, warning_jenkins, lights, sleep: sleep)
+  end
+
+  def initialize(
+    ampel_jenkins,
+    warning_jenkins,
+    lights,
+    sleep: 10
+  )
+    @ampel_jenkins, @warning_jenkins, @lights, @sleep =
+      ampel_jenkins, warning_jenkins, lights, sleep
     check_lights
-    @sleep = sleep
-    @build_state = AmpelExtase::BuildState.for
   end
 
   def start
@@ -41,50 +57,65 @@ class AmpelExtase::Controller
 
   def perform
     @crashed = false
-    on_state_change do |state|
-      perform_switch state
+    @ampel_jenkins.on_state_change do |state|
+      perform_lights_switch state
+    end
+    @warning_jenkins.on_state_change do |state|
+      perform_warning state
     end
   end
 
-  def perform_switch(state)
+  def perform_lights_switch(state)
     case state.last_result
     when 'SUCCESS'
       @lights.green.on
       @lights.red.off
+      puts success('LIGHTS SUCCESS')
     when 'FAILURE', 'ABORTED'
       @lights.red.on
       if state.building?
         @lights.green.on
+        puts failure_building('LIGHTS FAILURE BUILDING')
       else
         @lights.green.off
+        puts failure('LIGHTS FAILURE')
       end
     end
   end
 
-  def on_state_change
-    new_state = AmpelExtase::BuildState.for [ last_result, building? ]
-    if new_state == @build_state
-      puts "state did not change, is still #@build_state => do nothing"
-    else
-      puts "state changed from #@build_state to #{new_state} => taking action"
-      yield new_state
+  def perform_warning(state)
+    case state.last_result
+    when 'SUCCESS'
+      @lights.aux.off
+      puts success('WARNING SUCCESS')
+    when 'FAILURE', 'ABORTED'
+      if state.building?
+        puts failure_building('WARNING FAILURE BUILDING')
+      else
+        @lights.aux.on
+        puts failure('WARNING FAILURE')
+      end
     end
-  ensure
-    @build_state = new_state
   end
 
-  def last_result
-    @jenkins.fetch_build(:last_completed_build)['result']
+  def success(message)
+    green message
   end
 
-  def building?
-    @jenkins.fetch_build(:last_build)['building']
+  def failure(message)
+    red message
+  end
+
+  def failure_building(message)
+    green on_red message
   end
 
   def switch_all_lights_off
-    for light in @lights
-      light.off
-    end
+    @lights.each(&:off)
+  end
+
+  def switch_all_lights_on
+    @lights.each(&:on)
   end
 
   def handle_crash(exception)
@@ -98,19 +129,10 @@ class AmpelExtase::Controller
 
   def check_lights
     puts "checking lights configuration"
-    for light in @lights
-      light.on
-    end
+    switch_all_lights_on
     sleep 1
-    for light in @lights
-      light.off
-    end
+    switch_all_lights_off
     sleep 1
     puts "OK"
   end
-
-  def check_jenkins
-    puts "checking jenkins configuration"
-    @jenkins.fetch and puts "OK"
-  end
 end

data/lib/ampel_extase/jenkins_client.rb

@@ -5,21 +5,19 @@ require 'uri'
 
 module AmpelExtase
   class JenkinsClient
-    def initialize(url, debug: false)
+    def initialize(url)
       url = URI.parse(url.to_s) unless URI::HTTP === url
-      @url, @debug = url, debug
+      @url = url
     end
 
+    attr_reader :url
+
     def api_url(*path)
       [ @url, *path, 'api', 'json' ].compact * '/'
     end
 
-    def console_url(number)
-      [ @url, number, 'consoleText' ] * '/'
-    end
-
     def fetch(url = api_url)
-      @debug and STDERR.puts "Fetching #{url.to_s.inspect}."
+      puts "Fetching #{url.to_s.inspect}."
       JSON open(url).read
     rescue => e
       e.message << " for #{url.inspect}"
@@ -36,17 +34,5 @@ module AmpelExtase
             end
       fetch url
     end
-
-    def fetch_build_revision(type)
-      data = fetch_build(type)
-      data['actions'].find { |x| x.keys.include?('lastBuiltRevision') }['lastBuiltRevision']['SHA1']
-    rescue
-      nil
-    end
-
-    def fetch_console_text(type)
-      build = fetch_build(type)
-      open(console_url(build['number'])).read
-    end
   end
 end

data/lib/ampel_extase/jenkins_state_observer.rb

@@ -0,0 +1,48 @@
+require 'ampel_extase/build_state'
+require 'ampel_extase/jenkins_client'
+
+class AmpelExtase::JenkinsStateObserver
+  def self.for_url(jenkins_url)
+    if jenkins_url
+      jenkins = AmpelExtase::JenkinsClient.new(jenkins_url)
+      new(jenkins)
+    else
+      Tins::NULL
+    end
+  end
+
+  def initialize(jenkins)
+    @jenkins = jenkins
+    @build_state = AmpelExtase::BuildState.for
+    check
+  end
+
+  def check
+    puts "checking jenkins configuration for #{@jenkins.url.to_s.inspect}"
+    @jenkins.fetch and puts "OK"
+  end
+
+  def last_result
+    @jenkins.fetch_build(:last_completed_build)['result']
+  end
+
+  def building?
+    @jenkins.fetch_build(:last_build)['building']
+  end
+
+  def state_changed?(new_state)
+    new_state != @build_state
+  end
+
+  def on_state_change
+    new_state = AmpelExtase::BuildState.for [ last_result, building? ]
+    if state_changed?(new_state)
+      puts "state changed from #@build_state to #{new_state} => taking action"
+      yield new_state
+    else
+      puts "state did not change, is still #@build_state => do nothing"
+    end
+  ensure
+    @build_state = new_state
+  end
+end

data/lib/ampel_extase/light_switcher.rb

@@ -2,8 +2,16 @@ require 'socket_switcher'
 
 module AmpelExtase
   class LightSwitcher
-    def initialize(serial, debug: false)
-      @port = SocketSwitcher::Port.new(serial, debug: debug)
+    def self.for(serial:)
+      if serial
+        new SocketSwitcher::Port.new(serial)
+      else
+        Tins::NULL
+      end
+    end
+
+    def initialize(port)
+      @port = port
     end
 
     attr_reader :port
@@ -24,9 +32,8 @@ module AmpelExtase
       [
         :green,
         :red,
-        # not in use for now :aux,
+        :aux,
       ].map { |color| __send__(color) }.each(&block)
     end
   end
 end
-

data/lib/ampel_extase/version.rb

@@ -1,6 +1,6 @@
 module AmpelExtase
   # AmpelExtase version
-  VERSION         = '0.1.0'
+  VERSION         = '0.3.0'
   VERSION_ARRAY   = VERSION.split('.').map(&:to_i) # :nodoc:
   VERSION_MAJOR   = VERSION_ARRAY[0] # :nodoc:
   VERSION_MINOR   = VERSION_ARRAY[1] # :nodoc:

data/spec/ampel_extase/build_state_spec.rb

@@ -0,0 +1,80 @@
+require 'spec_helper'
+
+describe AmpelExtase::BuildState do
+  describe 'initial state' do
+    let :bs do
+      described_class.for
+    end
+
+    it 'has a to_a' do
+      expect(bs.to_a).to eq [ 'N/A', nil ]
+    end
+
+    it 'has N/A as the last result' do
+      expect(bs.last_result).to eq 'N/A'
+    end
+
+    it 'does not build atm' do
+      expect(bs.building?).to eq false
+    end
+
+    it 'reads N/A as a string' do
+      expect(bs.to_s).to eq 'N/A'
+    end
+  end
+
+  describe 'an intermediate state' do
+    let :bs do
+      described_class.for [ 'SUCCESS', true ]
+    end
+
+
+    it 'has SUCCESS as the last result' do
+      expect(bs.last_result).to eq 'SUCCESS'
+    end
+
+    describe 'building' do
+      it 'has a to_a' do
+        expect(bs.to_a).to eq [ 'SUCCESS', true ]
+      end
+
+      it 'can be building' do
+        expect(bs.building?).to eq true
+      end
+
+      it 'reads SUCCESS (building) as a string' do
+        expect(bs.to_s).to eq 'SUCCESS (building)'
+      end
+    end
+
+    describe 'not building' do
+      let :bs do
+        described_class.for [ 'SUCCESS', false ]
+      end
+
+      it 'has a to_a' do
+        expect(bs.to_a).to eq [ 'SUCCESS', false ]
+      end
+
+      it 'can not be building' do
+        expect(bs.building?).to eq false
+      end
+
+      it 'reads SUCCESS (building) as a string' do
+        expect(bs.to_s).to eq 'SUCCESS'
+      end
+    end
+  end
+
+  describe 'equality' do
+    it 'can be equal or unequal' do
+      foo_true = described_class.for([ 'FOO', true ])
+      foo_false = described_class.for([ 'FOO', false ])
+      bar_true = described_class.for([ 'BAR', true ])
+      expect(foo_true).to eq described_class.for([ 'FOO', true ])
+      expect(foo_true).not_to eq foo_false
+      expect(foo_true).not_to eq bar_true
+    end
+
+  end
+end

data/spec/ampel_extase/controller_spec.rb

@@ -0,0 +1,226 @@
+require 'spec_helper'
+
+describe AmpelExtase::Controller do
+  before do
+    allow_any_instance_of(AmpelExtase::JenkinsClient).to receive(:puts)
+    allow_any_instance_of(AmpelExtase::Controller).to receive(:puts)
+    allow_any_instance_of(AmpelExtase::JenkinsStateObserver).to receive(:puts)
+    allow_any_instance_of(described_class).to receive(:sleep)
+  end
+
+  let :ampel_client do
+    double('AmpelExtase::JenkinsClient', url: 'http://foo/bar')
+  end
+
+  let :warning_client do
+    double('AmpelExtase::JenkinsClient', url: 'http://foo/bar')
+  end
+
+  let :ampel_jenkins do
+    AmpelExtase::JenkinsStateObserver.new ampel_client
+  end
+
+  let :warning_jenkins do
+    AmpelExtase::JenkinsStateObserver.new warning_client
+  end
+
+  let :lights do
+    double('AmpelExtase::LightSwitcher')
+  end
+
+  let :controller do
+    described_class.new(
+      ampel_jenkins,
+      warning_jenkins,
+      lights
+    )
+  end
+
+  before do
+    for client in [ ampel_client, warning_client ]
+      allow(client).to receive(:fetch).and_return true
+      allow(client).to receive(:fetch_build).and_return('result' => 'N/A')
+    end
+  end
+
+  before do
+    allow(lights).to receive(:each)
+  end
+
+  describe '.for' do
+    it 'can create a demo object that does not really do anything' do
+      demo_object = described_class.for(
+        serial: nil,
+        jenkins_url: nil
+      )
+      expect(demo_object.instance_eval { @ampel_jenkins }).to be_nil
+      expect(demo_object.instance_eval { @warning_jenkins }).to be_nil
+      expect(demo_object.instance_eval { @lights }).to be_nil
+    end
+  end
+
+  describe '#start' do
+    it 'performs and then sleeps' do
+      allow(controller).to receive(:sleep).and_raise(
+        sleep_error = StandardError.new
+      )
+      allow(controller).to receive(:at_exit)
+      expect(controller).to receive(:perform)
+      expect { controller.start }.to raise_error sleep_error
+    end
+
+    it 'can handle crashes' do
+      allow(controller).to receive(:sleep).and_raise(
+        sleep_error = StandardError.new
+      )
+      allow(controller).to receive(:at_exit)
+      expect(controller).to receive(:perform).and_raise(e = StandardError.new)
+      expect(controller).to receive(:handle_crash).with(e)
+      expect { controller.start }.to raise_error sleep_error
+    end
+
+  end
+
+  describe '#stop' do
+    before do
+      controller
+    end
+
+    it 'iterates over all lights' do
+      expect(lights).to receive(:each).once
+      controller.stop
+    end
+  end
+
+  describe '#perform' do
+    let :perform do
+      controller.instance_eval { perform }
+    end
+
+    it 'reacts to state changes' do
+      state = AmpelExtase::BuildState.for
+      allow(ampel_jenkins).to receive(:state_changed?).and_return true
+      expect { |b| ampel_jenkins.on_state_change(&b) }.to yield_with_args(state)
+      allow(warning_jenkins).to receive(:state_changed?).and_return true
+      expect { |b| warning_jenkins.on_state_change(&b) }.to yield_with_args(state)
+      perform
+    end
+  end
+
+
+  describe '#handle_crash' do
+    let :exception do
+      raise StandardError rescue $!
+    end
+
+    let :handle_crash do
+      my_exception = exception
+      controller.instance_eval { handle_crash(my_exception) }
+    end
+
+    before do
+      controller
+      allow(controller).to receive(:warn)
+    end
+
+    it 'warns on crash' do
+      expect(controller).to receive(:warn).with(/^Caught/)
+      handle_crash
+    end
+
+    it 'switches the lights of once' do
+      expect(controller).to receive(:switch_all_lights_off)
+      handle_crash
+      expect(controller).not_to receive(:switch_all_lights_off)
+      handle_crash
+    end
+  end
+
+  describe '#perform_lights_switch' do
+    before do
+      controller.instance_variable_set :@lights, Tins::NULL
+    end
+
+    let :perform_lights_switch do
+      my_state = state
+      controller.instance_eval { perform_lights_switch(my_state) }
+    end
+
+    context 'success' do
+      let :state do
+        AmpelExtase::BuildState.for [ 'SUCCESS', false ]
+      end
+
+      it 'goes green on success' do
+        expect(controller).to receive(:success).and_call_original
+        perform_lights_switch
+      end
+    end
+
+    context 'failure' do
+      let :state do
+        AmpelExtase::BuildState.for [ 'FAILURE', false ]
+      end
+
+      it 'goes red on failure' do
+        expect(controller).to receive(:failure).and_call_original
+        perform_lights_switch
+      end
+    end
+
+    context 'failure building' do
+      let :state do
+        AmpelExtase::BuildState.for [ 'FAILURE', true ]
+      end
+
+      it 'goes green on red on failure and building' do
+        expect(controller).to receive(:failure_building).and_call_original
+        perform_lights_switch
+      end
+    end
+  end
+
+  describe '#perform_warning' do
+    before do
+      controller.instance_variable_set :@lights, Tins::NULL
+    end
+
+    let :perform_warning do
+      my_state = state
+      controller.instance_eval { perform_warning(my_state) }
+    end
+
+    context 'success' do
+      let :state do
+        AmpelExtase::BuildState.for [ 'SUCCESS', false ]
+      end
+
+      it 'goes green on success' do
+        expect(controller).to receive(:success).and_call_original
+        perform_warning
+      end
+    end
+
+    context 'failure' do
+      let :state do
+        AmpelExtase::BuildState.for [ 'FAILURE', false ]
+      end
+
+      it 'goes red on failure' do
+        expect(controller).to receive(:failure).and_call_original
+        perform_warning
+      end
+    end
+
+    context 'failure building' do
+      let :state do
+        AmpelExtase::BuildState.for [ 'FAILURE', true ]
+      end
+
+      it 'goes green on red on failure and building' do
+        expect(controller).to receive(:failure_building).and_call_original
+        perform_warning
+      end
+    end
+  end
+end

data/spec/ampel_extase/jekins_state_observer_spec.rb

@@ -0,0 +1,115 @@
+require 'spec_helper'
+
+describe AmpelExtase::JenkinsStateObserver do
+  let :client do
+    double('AmpelExtase::JenkinsClient', url: 'http://foo/bar')
+  end
+
+  before do
+    allow(client).to receive(:fetch).and_return true
+  end
+
+  let :jso do
+    described_class.new client
+  end
+
+  describe '.for_url' do
+    it 'creates a state observer for an URL' do
+      url = 'http://foo/bar'
+      expect(AmpelExtase::JenkinsClient).to\
+        receive(:new).with(url).and_return client
+      expect(described_class).to\
+        receive(:new).with(client)
+      described_class.for_url url
+    end
+
+    it 'returns a null object if url was nil' do
+      expect(described_class.for_url(nil)).to be_nil
+    end
+  end
+
+  describe '#check' do
+    it 'outputs "OK" if it can fetch data from CI' do
+      allow(jso).to receive(:puts)
+      expect(jso).to receive(:puts).with('OK')
+      jso.check
+    end
+
+    it 'lets an exception pass if raised by the client' do
+      allow(client).to receive(:fetch).and_raise e = StandardError.new
+      expect { jso.check }.to raise_error e
+    end
+  end
+
+  describe '#last_result' do
+    it 'uses client to fetch last_completed_build' do
+      allow(client).to receive(:fetch_build).with(:last_completed_build).
+        and_return('result' => :foo)
+      expect(jso.last_result).to eq :foo
+    end
+  end
+
+  describe '#building?' do
+    it 'uses client to fetch last_build' do
+      allow(client).to receive(:fetch_build).with(:last_build).
+        and_return('building' => :foo)
+      expect(jso.building?).to eq :foo
+    end
+  end
+
+  describe '#on_state_change' do
+    let :bs_initial do
+      AmpelExtase::BuildState.for [ 'N/A', nil ]
+    end
+
+    let :bs_success do
+      AmpelExtase::BuildState.for [ 'SUCCESS', true ]
+    end
+
+    let :bs_failure do
+      AmpelExtase::BuildState.for [ 'FAILURE', true ]
+    end
+
+    it 'tracks changes in build state' do
+      expect(jso.instance_variable_get(:@build_state)).to eq bs_initial
+      allow(client).to receive(:fetch_build).with(:last_completed_build).
+        and_return('result' => 'SUCCESS')
+      allow(client).to receive(:fetch_build).with(:last_build).
+        and_return('building' => true)
+      expect(jso).to receive(:puts).with(
+        "state changed from N/A to SUCCESS (building) => "\
+        "taking action"
+      )
+      expect { |b| jso.on_state_change(&b) }.to yield_with_args(bs_success)
+      expect(jso.instance_variable_get(:@build_state)).to eq bs_success
+      allow(client).to receive(:fetch_build).with(:last_completed_build).
+        and_return('result' => 'FAILURE')
+      allow(client).to receive(:fetch_build).with(:last_build).
+        and_return('building' => true)
+      expect(jso).to receive(:puts).with(
+        "state changed from SUCCESS (building) to FAILURE (building) => "\
+        "taking action"
+      )
+      expect { |b| jso.on_state_change(&b) }.to yield_with_args(bs_failure)
+      expect(jso.instance_variable_get(:@build_state)).to eq bs_failure
+    end
+
+    it 'notices if nothing has changed' do
+      allow(client).to receive(:fetch_build).twice.with(:last_completed_build).
+        and_return('result' => 'SUCCESS')
+      allow(client).to receive(:fetch_build).twice.with(:last_build).
+        and_return('building' => true)
+      expect(jso).to receive(:puts).with(
+        "state changed from N/A to SUCCESS (building) => "\
+        "taking action"
+      )
+      expect { |b| jso.on_state_change(&b) }.to yield_with_args(bs_success)
+      expect(jso.instance_variable_get(:@build_state)).to eq bs_success
+      expect(jso).to receive(:puts).with(
+        "state did not change, is still SUCCESS (building) => do nothing"
+      )
+      expect(jso.instance_variable_get(:@build_state)).to eq bs_success
+      expect { |b| jso.on_state_change(&b) }.not_to yield_with_args(bs_success)
+    end
+  end
+end

data/spec/ampel_extase/jenkins_client_spec.rb

@@ -1,5 +1,47 @@
 require 'spec_helper'
 
 describe AmpelExtase::JenkinsClient do
-  pending
+  let :client do
+    AmpelExtase::JenkinsClient.new 'http://foo/bar'
+  end
+
+  let :opened_uri do
+    double('URI', read: '{"hello":"world"}')
+  end
+
+  describe '#fetch' do
+    let :api_url do
+      'http://foo/bar/api/json'
+    end
+
+    it 'can fetch JSON data from CI' do
+      expect(client).to receive(:open).with(api_url).
+        and_return opened_uri
+      expect(client.fetch).to eq("hello" => "world")
+    end
+
+    it 'modifies exception messages and reraises' do
+      expect(client).to receive(:open).with(api_url).
+        and_raise e = StandardError.new('foo')
+      expect { client.fetch }.to raise_error(
+        StandardError, /for #{api_url.inspect}/
+      )
+    end
+  end
+
+  describe '#fetch_build' do
+    let :api_url do
+      'http://foo/bar/buildType/api/json'
+    end
+
+    it 'can can fetch a build type responding to #to_s' do
+      expect(client).to receive(:open).with(api_url).and_return opened_uri
+      client.fetch_build double(to_s: 'buildType')
+    end
+
+    it 'can interpret a build type symbol correctly' do
+      expect(client).to receive(:open).with(api_url).and_return opened_uri
+      client.fetch_build :build_type
+    end
+  end
 end

data/spec/ampel_extase/light_switcher_spec.rb

@@ -1,5 +1,40 @@
 require 'spec_helper'
 
 describe AmpelExtase::LightSwitcher do
-  pending
+  let :port do
+    double('SocketSwitcher::Port')
+  end
+
+  let :switcher do
+    allow(SocketSwitcher::Port).to receive(:new).and_return port
+    described_class.for serial: 'foo/bar'
+  end
+
+  describe '#green' do
+    it 'returns device 0' do
+      expect(port).to receive(:device).with(0)
+      switcher.green
+    end
+  end
+
+  describe '#red' do
+    it 'returns device 1' do
+      expect(port).to receive(:device).with(1)
+      switcher.red
+    end
+  end
+
+  describe '#aux' do
+    it 'returns device 3' do
+      expect(port).to receive(:device).with(2)
+      switcher.aux
+    end
+  end
+
+  describe '#each' do
+    it 'iterates over all port devices' do
+      expect(port).to receive(:device).thrice.and_return :ok
+      expect(switcher.each.to_a).to eq %i[ ok ] * 3
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ampel_extase
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Florian Frank
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-02 00:00:00.000000000 Z
+date: 2015-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gem_hadar
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.3.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.3.1
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: term-ansicolor
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: socket_switcher
   requirement: !ruby/object:Gem::Requirement
@@ -105,10 +119,12 @@ extra_rdoc_files:
 - lib/ampel_extase/build_state.rb
 - lib/ampel_extase/controller.rb
 - lib/ampel_extase/jenkins_client.rb
+- lib/ampel_extase/jenkins_state_observer.rb
 - lib/ampel_extase/light_switcher.rb
 - lib/ampel_extase/version.rb
 files:
 - ".gitignore"
+- ".utilsrc"
 - Gemfile
 - README.md
 - Rakefile
@@ -119,8 +135,12 @@ files:
 - lib/ampel_extase/build_state.rb
 - lib/ampel_extase/controller.rb
 - lib/ampel_extase/jenkins_client.rb
+- lib/ampel_extase/jenkins_state_observer.rb
 - lib/ampel_extase/light_switcher.rb
 - lib/ampel_extase/version.rb
+- spec/ampel_extase/build_state_spec.rb
+- spec/ampel_extase/controller_spec.rb
+- spec/ampel_extase/jekins_state_observer_spec.rb
 - spec/ampel_extase/jenkins_client_spec.rb
 - spec/ampel_extase/light_switcher_spec.rb
 - spec/spec_helper.rb
@@ -148,11 +168,14 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: Library to control the build traffic light
 test_files:
+- spec/ampel_extase/build_state_spec.rb
+- spec/ampel_extase/controller_spec.rb
+- spec/ampel_extase/jekins_state_observer_spec.rb
 - spec/ampel_extase/jenkins_client_spec.rb
 - spec/ampel_extase/light_switcher_spec.rb
 - spec/spec_helper.rb