traffic_light_controller 0.0.3 → 0.0.4

data/.travis.yml

@@ -0,0 +1,6 @@
+---
+language: ruby
+bundler_args: --without development
+rvm:
+  - 1.9.2
+  - 1.9.3

data/Gemfile

@@ -2,9 +2,20 @@ source :rubygems
 
 gemspec
 
+group :development do
+  gem "guard"
+  gem "guard-bundler"
+  gem "guard-rspec"
+  gem "rb-fchange", :require => false
+  gem "rb-fsevent", :require => false
+  gem "rb-inotify", :require => false
+  gem "terminal-notifier-guard"
+end
+
 group :test do
-  gem "rspec"
   gem "ci_reporter"
+  gem "rake"
+  gem "rspec"
   gem "simplecov", :require => false
   gem "simplecov-rcov"
 end

data/Gemfile.lock

@@ -12,11 +12,11 @@ GEM
   specs:
     arduino (0.4)
       serialport (>= 1.0.4)
-    binding_of_caller (0.6.7)
-    builder (3.1.2)
-    ci_reporter (1.7.1)
+    binding_of_caller (0.6.8)
+    builder (3.1.4)
+    ci_reporter (1.7.3)
       builder (>= 2.1.2)
-    coderay (1.0.7)
+    coderay (1.0.8)
     columnize (0.3.6)
     debugger (1.1.4)
       columnize (>= 0.3.1)
@@ -24,12 +24,26 @@ GEM
       debugger-ruby_core_source (~> 1.1.3)
     debugger-linecache (1.1.2)
       debugger-ruby_core_source (>= 1.1.1)
-    debugger-ruby_core_source (1.1.3)
+    debugger-ruby_core_source (1.1.4)
     diff-lcs (1.1.3)
+    ffi (1.1.5)
+    guard (1.5.4)
+      listen (>= 0.4.2)
+      lumberjack (>= 1.0.2)
+      pry (>= 0.9.10)
+      thor (>= 0.14.6)
+    guard-bundler (1.0.0)
+      bundler (~> 1.0)
+      guard (~> 1.1)
+    guard-rspec (2.1.1)
+      guard (>= 1.1)
+      rspec (~> 2.11)
     hashie (1.2.0)
     json (1.7.5)
-    method_source (0.8)
-    multi_json (1.3.6)
+    listen (0.5.3)
+    lumberjack (1.0.2)
+    method_source (0.8.1)
+    multi_json (1.3.7)
     pry (0.9.10)
       coderay (~> 1.0.5)
       method_source (~> 0.8)
@@ -37,12 +51,17 @@ GEM
     pry-debugger (0.2.0)
       debugger (~> 1.1.3)
       pry (~> 0.9.9)
-    pry-stack_explorer (0.4.5)
-      binding_of_caller (~> 0.6.2)
+    pry-stack_explorer (0.4.7)
+      binding_of_caller (~> 0.6.8)
     rack (1.4.1)
     rack-protection (1.2.0)
       rack
-    rake (0.9.2.2)
+    rake (10.0.0)
+    rb-fchange (0.0.6)
+      ffi
+    rb-fsevent (0.9.2)
+    rb-inotify (0.8.8)
+      ffi (>= 0.5.0)
     rspec (2.11.0)
       rspec-core (~> 2.11.0)
       rspec-expectations (~> 2.11.0)
@@ -50,12 +69,12 @@ GEM
     rspec-core (2.11.1)
     rspec-expectations (2.11.3)
       diff-lcs (~> 1.1.3)
-    rspec-mocks (2.11.2)
+    rspec-mocks (2.11.3)
     serialport (1.1.0)
-    simplecov (0.6.4)
+    simplecov (0.7.1)
       multi_json (~> 1.0)
-      simplecov-html (~> 0.5.3)
-    simplecov-html (0.5.3)
+      simplecov-html (~> 0.7.1)
+    simplecov-html (0.7.1)
     simplecov-rcov (0.2.3)
       simplecov (>= 0.4.1)
     sinatra (1.3.3)
@@ -63,6 +82,8 @@ GEM
       rack-protection (~> 1.2)
       tilt (~> 1.3, >= 1.3.3)
     slop (3.3.3)
+    terminal-notifier-guard (1.5.3)
+    thor (0.16.0)
     tilt (1.3.3)
 
 PLATFORMS
@@ -70,11 +91,18 @@ PLATFORMS
 
 DEPENDENCIES
   ci_reporter
+  guard
+  guard-bundler
+  guard-rspec
   pry
   pry-debugger
   pry-stack_explorer
   rake
+  rb-fchange
+  rb-fsevent
+  rb-inotify
   rspec
   simplecov
   simplecov-rcov
+  terminal-notifier-guard
   traffic_light_controller!

data/Guardfile

@@ -0,0 +1,14 @@
+# A sample Guardfile
+# More info at https://github.com/guard/guard#readme
+
+guard 'bundler' do
+  watch('Gemfile')
+  watch(/^.+\.gemspec/)
+end
+
+guard 'rspec' do
+  watch(%r{^spec/.+_spec\.rb$})
+  watch(%r{^lib/(.+)\.rb$})     { |m| "spec/lib/#{m[1]}_spec.rb" }
+  watch('spec/spec_helper.rb')  { "spec" }
+end
+

data/README.md

@@ -0,0 +1,6 @@
+# Traffic Light Controller (ruby)
+[![Build Status](https://secure.travis-ci.org/jcmuller/traffic_light_controller.png?branch=master)](http://travis-ci.org/jcmuller/traffic_light_controller)
+[![Dependency Status](https://gemnasium.com/jcmuller/traffic_light_controller.png "Dependency Status")](https://gemnasium.com/jcmuller/traffic_light_controller)
+[![Code Climate](https://codeclimate.com/badge.png)](https://codeclimate.com/github/jcmuller/traffic_light_controller)
+
+This piece of work goes along with [Build Status Server](http://github.com/jcmuller/build_status_server)

data/lib/traffic_light_controller/server.rb

@@ -6,53 +6,97 @@ module TrafficLightController
       @config = Config.new
       @server = TCPServer.new(config.server.address, config.server.port)
     rescue Errno::EADDRINUSE
-      puts "Address in use"
+      address_in_use_error(config.server)
     rescue Errno::EADDRNOTAVAIL, SocketError
-      puts "Address not available"
-    end
-
-    def work(run_forever = true)
-      while(run_forever)
-        Thread.start(server.accept) do |client|
-          begin
-            path = nil
-
-            while(line = client.readline) do
-              break if line == "\r\n"
-              if match = line.match(%r{GET /+(?<path>\w*) HTTP/1\.})
-                path = match[:path]
-              end
-            end
-
-            if config.arduino.lights.has_key?(path)
-              process(path)
-              client.print "HTTP/1.1 200 OK\r\nContent-type:text/plain\r\n\r\n"
-              client.puts path
-            else
-              client.print "HTTP/1.1 404 Not Found\r\nContent-type:text/plain\r\n\r\n"
-              client.puts "The requested path doesn't exist"
-            end
-          rescue => e
-            puts "Got #{e}!"
-          ensure
-            client.close
-          end
+      address_not_available_error(config.server.address)
+    end
+
+    def work
+      while(true)
+        start_thread_and_do_work
+      end
+    end
+
+    private
+
+    attr_reader :server, :config, :board
+
+    def start_thread_and_do_work
+      Thread.start(server.accept) do |client|
+        work_with_client(client)
+      end
+    end
+
+    def work_with_client(client)
+      puts "work_with_client"
+      process_path(process_request(client))
+    rescue => e
+      puts "Got #{e.class}: #{e}!"
+    ensure
+      client.close
+    end
+
+    def process_request(client)
+      puts "process_request"
+      path = nil
+
+      while(line = client.readline) do
+        break if line == "\r\n"
+        if match = line.match(%r{GET /+(?<path>\w*) HTTP/1\.})
+          path = match[:path]
         end
       end
+
+      path
+    end
+
+    def process_path(path)
+      puts "process_path"
+      if config.arduino.lights.has_key?(path)
+        change_pins(path)
+        client.print "HTTP/1.1 200 OK\r\nContent-type:text/plain\r\n\r\n"
+        client.puts path
+      else
+        client.print "HTTP/1.1 404 Not Found\r\nContent-type:text/plain\r\n\r\n"
+        client.puts "The requested path doesn't exist"
+      end
     end
 
-    def process(path)
+    def change_pins(path)
+      puts "change_pins"
+      init_board
       board.turnOff
       board.setHigh(config.arduino.lights[path]) unless path == "off"
       board.close
     end
 
-    private
+    def init_board
+      puts "board"
+      @board = Arduino.new(config.arduino.port)
+    rescue Errno::ENOENT
+      STDERR.puts "The port #{config.arduino.port} doesn't exist"
+      exit
+    rescue ArgumentError
+      STDERR.puts "#{config.arduino.port} is not a serial port"
+      exit
+    end
 
-    attr_reader :server, :config
+    # ERRORS
+    def address_in_use_error(server)
+      STDERR.puts <<-EOT
+There appears that another instance is running, or another process
+is listening on the same port (#{server.address}:#{server.port})
 
-    def board
-      @board = Arduino.new(config.arduino.port)
+      EOT
+      exit
+    end
+
+    def address_not_available_error(address)
+      STDERR.puts <<-EOT
+The address configured is not available (#{address})
+
+      EOT
+      exit
     end
   end
 end

data/lib/traffic_light_controller/version.rb

@@ -1,3 +1,3 @@
 module TrafficLightController
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/spec/lib/traffic_light_controller/server_spec.rb

@@ -1,4 +1,96 @@
 require 'spec_helper'
 
 describe TrafficLightController::Server do
+  let(:board) { mock }
+  let(:config) { mock }
+  let(:server) { mock(accept: client) }
+  let(:client) { mock }
+
+  before do
+    config.stub_chain(:server, :address).and_return("127.0.0.1")
+    config.stub_chain(:server, :port).and_return(10000 + rand(1000))
+    config.stub_chain(:arduino, :port).and_return("port")
+
+    TrafficLightController::Config.stub(:new).and_return(config)
+    TCPServer.stub(:new).and_return(server)
+    STDOUT.stub(:puts)
+  end
+
+  context "public methods" do
+    before do
+      subject.stub(:board).and_return(board)
+    end
+
+    describe "#initialize" do
+      it "should call address_in_use_error if address is in use" do
+        TCPServer.should_receive(:new).and_raise(Errno::EADDRINUSE)
+        described_class.any_instance.should_receive(:address_in_use_error)
+        described_class.new
+      end
+
+      it "should call address_in_use_error if address is not available" do
+        TCPServer.should_receive(:new).and_raise(Errno::EADDRNOTAVAIL)
+        described_class.any_instance.should_receive(:address_not_available_error)
+        described_class.new
+      end
+    end
+
+    describe "#change_pins" do
+      before { subject.stub(:init_board) }
+
+      it "set high the pin pointed to by path" do
+        config.stub_chain(:arduino, :lights).and_return({"blah" => 4})
+        board.should_receive(:turnOff)
+        board.should_receive(:setHigh).with(4)
+        board.should_receive(:close)
+
+        subject.send(:change_pins, "blah")
+      end
+
+      it "should not set high any pins" do
+        board.should_receive(:turnOff)
+        board.should_not_receive(:setHigh)
+        board.should_receive(:close)
+
+        subject.send(:change_pins, "off")
+      end
+    end
+  end
+
+  context "private methods" do
+    describe "#board" do
+      it "should instantiate a new arduino object" do
+        Arduino.should_receive(:new).with("port")
+        subject.send(:init_board)
+      end
+    end
+  end
+
+  describe "#start_thread_and_do_work" do
+    it "should start a thread" do
+      Thread.should_receive(:start).and_yield(client)
+      subject.should_receive(:work_with_client).with(client)
+      subject.send(:start_thread_and_do_work)
+    end
+  end
+
+  describe "#work_with_client" do
+    it "should call process path" do
+      subject.should_receive(:process_request).with(client).and_return(:processed)
+      subject.should_receive(:process_path).with(:processed)
+      client.should_receive(:close)
+
+      subject.send(:work_with_client, client)
+    end
+
+    it "should rescue from all exceptions and log them to STDOUT" do
+      subject.should_receive(:process_request).and_raise RuntimeError
+
+      STDOUT.should_receive(:puts).with("Got RuntimeError: RuntimeError!")
+
+      client.should_receive(:close)
+
+      subject.send(:work_with_client, client)
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -5,6 +5,14 @@
 #
 # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 
+require "simplecov"
+require 'simplecov-rcov'
+SimpleCov.formatter = SimpleCov::Formatter::RcovFormatter
+SimpleCov.start do
+  add_filter "vendor/bundler_gems" # Ignore gems
+  add_filter "spec"
+end
+
 require "rspec/core"
 require "rspec/mocks"
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: traffic_light_controller
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-24 00:00:00.000000000 Z
+date: 2012-11-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -147,9 +147,12 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .travis.yml
 - Gemfile
 - Gemfile.lock
+- Guardfile
 - LICENSE
+- README.md
 - Rakefile
 - bin/traffic_light_controller
 - config/.gitignore