buildbox 0.0.4 → 0.1

data/spec/integration/running_a_build_spec.rb

@@ -1,9 +1,28 @@
+# encoding: UTF-8
+
 require 'spec_helper'
 
 describe 'running a build' do
   let(:commit)    { "3e0c65433b241ff2c59220f80bcdcd2ebb7e4b96" }
   let(:command)   { "rspec test_spec.rb" }
-  let(:build)     { Buildbox::Build.new(:project_id => 1, :build_id => 1, :repository => FIXTURES_PATH.join("repo.git").to_s, :commit => commit, :command => command) }
+  let(:env)       { { } }
+  let(:build)     { Buildbox::Build.new(:project => { :name => "test", :team => { :name => "test" } }, :number => 1, :script => script, :env => env) }
+  let(:runner)    { Buildbox::Runner.new(build) }
+  let(:script) do
+<<-SCRIPT
+#/bin/bash
+set -e
+echo `pwd`
+if [ ! -d ".git" ]; then
+  git clone "#{FIXTURES_PATH.join("repo.git").to_s}" ./ -q
+fi
+git clean -fd
+git fetch -q
+git checkout -qf #{commit}
+bundle install --local
+#{command}
+SCRIPT
+  end
 
   before do
     Buildbox.stub(:root_path).and_return(TEMP_PATH)
@@ -15,10 +34,21 @@ describe 'running a build' do
 
   context 'running a working build' do
     it "returns a successfull result" do
-      result = build.start
+      runner.start
+
+      build.should be_success
+      build.output.should =~ /1 example, 0 failures/
+    end
+  end
+
+  context 'running a working build with a semi colon in the command' do
+    let(:command) { "rspec test_spec.rb;" }
+
+    it "returns a successfull result" do
+      runner.start
 
-      result.should be_success
-      result.output.should =~ /1 example, 0 failures/
+      build.should be_success
+      build.output.should =~ /1 example, 0 failures/
     end
   end
 
@@ -26,10 +56,98 @@ describe 'running a build' do
     let(:commit) { "2d762cdfd781dc4077c9f27a18969efbd186363c" }
 
     it "returns a unsuccessfull result" do
-      result = build.start
+      runner.start
+
+      build.should_not be_success
+      build.output.should =~ /1 example, 1 failure/
+    end
+  end
+
+  context 'running a failing build that has commands after the one that failed' do
+    let(:commit)  { "2d762cdfd781dc4077c9f27a18969efbd186363c" }
+    let(:command) { "rspec test_spec.rb; echo 'oh no you didnt!'" }
+
+    it "returns a unsuccessfull result" do
+      runner.start
+
+      build.should_not be_success
+      build.output.should =~ /1 example, 1 failure/
+    end
+  end
+
+  context 'running a ruby script' do
+    let(:script) do
+      <<-SCRIPT
+#!/usr/bin/env ruby
+puts 'hello'
+exit 123
+SCRIPT
+    end
+
+    it "runs and returns the correct output" do
+      runner.start
+
+      build.output.should == 'hello'
+      build.exit_status.should == 123
+    end
+  end
+
+  context 'accessing ENV variables with a ruby script' do
+    let(:env) { { "FOO" => "great" } }
+    let(:script) do
+      <<-SCRIPT
+#!/usr/bin/env ruby
+puts ENV["FOO"]
+SCRIPT
+    end
+
+    it "runs and returns the correct output" do
+      runner.start
+
+      build.output.should == 'great'
+    end
+  end
+
+  context 'accessing ENV variables with a bash script' do
+    let(:env) { { "BAR" => "great" } }
+    let(:script) do
+      <<-SCRIPT
+echo $BAR
+SCRIPT
+    end
+
+    it "runs and returns the correct output" do
+      runner.start
+
+      build.output.should == 'great'
+    end
+  end
+
+  context 'running a failing build that returns a non standard exit status' do
+    let(:command) { "exit 123" }
+
+    it "returns a unsuccessfull result" do
+      runner.start
 
-      result.should_not be_success
-      result.output.should =~ /1 example, 1 failure/
+      build.should_not be_success
+      build.exit_status.should == 123
+    end
+  end
+
+  context 'a build that has a command with a syntax error' do
+    let(:command) { 'if ( echo yay' }
+
+    it "returns a unsuccessfull result" do
+      runner.start
+
+      build.should_not be_success
+
+      # bash 3.2.48 prints "syntax error" in lowercase.
+      # freebsd 9.1 /bin/sh prints "Syntax error" with capital S.
+      # zsh 5.0.2 prints "parse error" which we do not handle.
+      # localized systems will print the message in not English which
+      # we do not handle either.
+      build.output.should =~ /(syntax|parse) error/i
     end
   end
 
@@ -37,25 +155,24 @@ describe 'running a build' do
     let(:command) { 'foobar' }
 
     it "returns a unsuccessfull result" do
-      result = build.start
+      runner.start
 
-      result.should_not be_success
+      build.should_not be_success
       # ubuntu: sh: 1: foobar: not found
       # osx: sh: foobar: command not found
-      result.output.should =~ /foobar.+not found/
+      build.output.should =~ /foobar.+not found/
     end
   end
 
   context 'running multiple builds in a row' do
     it "returns a successfull result when the build passes" do
-      first_result  = build.start
-      second_result = build.start
-
-      first_result.should be_success
-      first_result.output.should =~ /1 example, 0 failures/
+      runner.start
+      build.should be_success
+      build.output.should =~ /1 example, 0 failures/
 
-      second_result.should be_success
-      second_result.output.should =~ /1 example, 0 failures/
+      runner.start
+      build.should be_success
+      build.output.should =~ /1 example, 0 failures/
     end
   end
 
@@ -63,12 +180,12 @@ describe 'running a build' do
     it "returns a successfull result" do
       result = nil
       thread = Thread.new do
-        result = build.start
+        result = runner.start
       end
       thread.join
 
-      result.should be_success
-      result.output.should =~ /1 example, 0 failures/
+      build.should be_success
+      build.output.should =~ /1 example, 0 failures/
     end
   end
 
@@ -78,12 +195,12 @@ describe 'running a build' do
     it "returns a successfull result" do
       result = nil
       thread = Thread.new do
-        result = build.start
+        result = runner.start
       end
       thread.join
 
-      result.should_not be_success
-      result.output.should =~ /1 example, 1 failure/
+      build.should_not be_success
+      build.output.should =~ /1 example, 1 failure/
     end
   end
 end

data/spec/support/silence_logger.rb

@@ -1,7 +1,12 @@
+# encoding: UTF-8
+
+require 'logger'
+require 'celluloid'
+
 RSpec.configure do |config|
   config.before(:each) do
     logger = Logger.new(StringIO.new)
 
-    Buildbox.stub(:logger => logger)
+    Celluloid.logger = logger
   end
 end

data/spec/support/webmock.rb

@@ -0,0 +1 @@
+require 'webmock/rspec'

metadata

@@ -1,71 +1,85 @@
 --- !ruby/object:Gem::Specification
 name: buildbox
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: '0.1'
 platform: ruby
 authors:
 - Keith Pitt
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-15 00:00:00.000000000 Z
+date: 2013-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.3'
-  type: :development
+        version: '0.8'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0.8'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: faraday_middleware
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.9'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.9'
+- !ruby/object:Gem::Dependency
+  name: hashie
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: '2.0'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: multi_json
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: '1.7'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
-  name: dotenv
+  name: celluloid
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: '0.14'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.14'
 description: Ruby client for buildbox
 email:
 - me@keithpitt.com
@@ -74,10 +88,11 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .buildbox.toml
 - .gitignore
 - .rspec
+- .ruby-version
 - Gemfile
+- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -85,22 +100,17 @@ files:
 - buildbox-ruby.gemspec
 - lib/buildbox.rb
 - lib/buildbox/api.rb
-- lib/buildbox/auth.rb
 - lib/buildbox/build.rb
-- lib/buildbox/client.rb
+- lib/buildbox/cli.rb
 - lib/buildbox/command.rb
 - lib/buildbox/configuration.rb
-- lib/buildbox/observer.rb
-- lib/buildbox/pid_file.rb
-- lib/buildbox/response.rb
-- lib/buildbox/result.rb
+- lib/buildbox/environment.rb
+- lib/buildbox/monitor.rb
+- lib/buildbox/runner.rb
 - lib/buildbox/utf8.rb
 - lib/buildbox/version.rb
 - lib/buildbox/worker.rb
-- spec/buildbox/buildbox/build_spec.rb
 - spec/buildbox/buildbox/command_spec.rb
-- spec/buildbox/buildbox/configuration_spec.rb
-- spec/buildbox/buildbox_spec.rb
 - spec/fixtures/repo.git/HEAD
 - spec/fixtures/repo.git/config
 - spec/fixtures/repo.git/description
@@ -124,8 +134,8 @@ files:
 - spec/fixtures/rspec/test_spec.rb
 - spec/integration/running_a_build_spec.rb
 - spec/spec_helper.rb
-- spec/support/dotenv.rb
 - spec/support/silence_logger.rb
+- spec/support/webmock.rb
 homepage: ''
 licenses:
 - MIT
@@ -146,15 +156,12 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.0
+rubygems_version: 2.0.3
 signing_key: 
 specification_version: 4
 summary: Ruby client for buildbox
 test_files:
-- spec/buildbox/buildbox/build_spec.rb
 - spec/buildbox/buildbox/command_spec.rb
-- spec/buildbox/buildbox/configuration_spec.rb
-- spec/buildbox/buildbox_spec.rb
 - spec/fixtures/repo.git/HEAD
 - spec/fixtures/repo.git/config
 - spec/fixtures/repo.git/description
@@ -178,5 +185,5 @@ test_files:
 - spec/fixtures/rspec/test_spec.rb
 - spec/integration/running_a_build_spec.rb
 - spec/spec_helper.rb
-- spec/support/dotenv.rb
 - spec/support/silence_logger.rb
+- spec/support/webmock.rb

data/.buildbox.toml

@@ -1,2 +0,0 @@
-[build]
-commands = [ "bundle", "rspec" ]

data/lib/buildbox/auth.rb

@@ -1,37 +0,0 @@
-module Buildbox
-  class Auth
-    def login(options)
-      if Buildbox.configuration.api_key
-        error "You have already authentication. To unauthenticate, run `buildbox auth:logout`"
-      end
-
-      key = options[:api_key]
-      api = Buildbox::API.new(:api_key => key)
-
-      if api.login.success?
-        Buildbox.configuration.update :api_key, key
-        info "Authentication successful"
-      end
-    end
-
-    def logout
-      if Buildbox.configuration.api_key.nil?
-        error "You are currently not logged in. To authenticate, run: `buildbox auth:login`"
-      end
-
-      Buildbox.configuration.update :api_key, nil
-      info "You have successfuly logged out"
-    end
-
-    private
-
-    def info(message)
-      Buildbox.logger.info message
-    end
-
-    def error(message)
-      Buildbox.logger.error message
-      exit 1
-    end
-  end
-end