fourchette 0.0.7 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e3d2dcc715d8a2c65fe34a0248a6089bcfb5762a
-  data.tar.gz: 74456ce01a2cff7e4939f04efc7b0c49891597a1
+  metadata.gz: 6dd6591b8703b74bde51dd854677cb614f5f3aa1
+  data.tar.gz: 345b066b668937adb7a2c5efbf81eab03e80f3a1
 SHA512:
-  metadata.gz: b8055c8a81b2dc7253139b50b42e5fbd7188c137246ff73a44a4e0ce9a15ea9539a12dacab07f7893f56d389a7bac072aa18688f7f51357617aac2971118e4e4
-  data.tar.gz: 69ba6c5cc5d1e26972a865ef438168e154f879cc9e419405d760c0c881a0e58897f98adf5a8d026cf1281776c9c4b78a54162b3f87c01330f545cac63234acb9
+  metadata.gz: 7221c0baa009a6b26853b653c194b2ee51b90414dd2e28ed06d9c113f82323adef5f330f215594b9abc6174efcb64409f8c73fa1526dcd4865a1d6fe09570ba8
+  data.tar.gz: 06b66d34ba4241156741e090ee34c3c89c2446b7ba674958b5844803151aef4cdd41db987c9f89a042872be5cbcb25652a659aab9cac54b4e1f068414ca16052

data/.travis.yml

@@ -1,6 +1,6 @@
 language: ruby
 rvm:
-  - 1.9.3
   - 2.0.0-p353
   - 2.1.0
   - 2.1.1
+  - 2.1.2

data/Gemfile.lock.orig

@@ -0,0 +1,163 @@
+PATH
+  remote: .
+  specs:
+    fourchette (0.0.8)
+      git
+      heroku
+      octokit
+      platform-api (~> 0.2.0)
+      rake
+      sinatra
+      sinatra-contrib
+      sucker_punch
+      thor
+
+GEM
+  remote: http://rubygems.org/
+  specs:
+    addressable (2.3.6)
+    backports (3.6.0)
+    byebug (2.7.0)
+      columnize (~> 0.3)
+      debugger-linecache (~> 1.2)
+    celluloid (0.15.2)
+      timers (~> 1.1.0)
+    celluloid-io (0.15.0)
+      celluloid (>= 0.15.0)
+      nio4r (>= 0.5.0)
+    coderay (1.1.0)
+    columnize (0.8.9)
+    coveralls (0.7.0)
+      multi_json (~> 1.3)
+      rest-client
+      simplecov (>= 0.7)
+      term-ansicolor
+      thor
+    debugger-linecache (1.2.0)
+    diff-lcs (1.2.5)
+    docile (1.1.3)
+    dotenv (0.7.0)
+    erubis (2.7.0)
+    excon (0.34.0)
+    faraday (0.9.0)
+      multipart-post (>= 1.2, < 3)
+    ffi (1.9.3)
+    foreman (0.64.0)
+      dotenv (~> 0.7.0)
+      thor (>= 0.13.6)
+    formatador (0.2.4)
+    git (1.2.6)
+    guard (2.5.1)
+      formatador (>= 0.2.4)
+      listen (~> 2.6)
+      lumberjack (~> 1.0)
+      pry (>= 0.9.12)
+      thor (>= 0.18.1)
+    guard-rspec (4.2.7)
+      guard (~> 2.1)
+      rspec (>= 2.14, < 4.0)
+    heroics (0.0.10)
+      erubis (~> 2.7.0)
+      excon
+      moneta
+      multi_json (>= 1.9.2)
+      netrc
+    heroku (3.8.2)
+      heroku-api (~> 0.3.17)
+      launchy (>= 0.3.2)
+      netrc (~> 0.7.7)
+      rest-client (~> 1.6.1)
+      rubyzip
+    heroku-api (0.3.18)
+      excon (~> 0.27)
+      multi_json (~> 1.8)
+    launchy (2.4.2)
+      addressable (~> 2.3)
+    listen (2.6.2)
+      celluloid (>= 0.15.2)
+      celluloid-io (>= 0.15.0)
+      rb-fsevent (>= 0.9.3)
+      rb-inotify (>= 0.9)
+    lumberjack (1.0.4)
+    method_source (0.8.2)
+    mime-types (2.2)
+    moneta (0.7.20)
+    multi_json (1.9.2)
+    multipart-post (2.0.0)
+    netrc (0.7.7)
+    nio4r (1.0.0)
+    octokit (3.1.0)
+      sawyer (~> 0.5.3)
+    platform-api (0.2.0)
+      heroics (~> 0.0.10)
+    pry (0.9.12.6)
+      coderay (~> 1.0)
+      method_source (~> 0.8)
+      slop (~> 3.4)
+    pry-byebug (1.3.2)
+      byebug (~> 2.7)
+      pry (~> 0.9.12)
+    puma (2.8.2)
+      rack (>= 1.1, < 2.0)
+    rack (1.5.2)
+    rack-protection (1.5.3)
+      rack
+    rack-test (0.6.2)
+      rack (>= 1.0)
+    rake (10.3.2)
+    rb-fsevent (0.9.4)
+    rb-inotify (0.9.3)
+      ffi (>= 0.5.0)
+    rest-client (1.6.7)
+      mime-types (>= 1.16)
+    rspec (2.14.1)
+      rspec-core (~> 2.14.0)
+      rspec-expectations (~> 2.14.0)
+      rspec-mocks (~> 2.14.0)
+    rspec-core (2.14.8)
+    rspec-expectations (2.14.5)
+      diff-lcs (>= 1.1.3, < 2.0)
+    rspec-mocks (2.14.6)
+    rubyzip (1.1.4)
+    sawyer (0.5.4)
+      addressable (~> 2.3.5)
+      faraday (~> 0.8, < 0.10)
+    simplecov (0.8.2)
+      docile (~> 1.1.0)
+      multi_json
+      simplecov-html (~> 0.8.0)
+    simplecov-html (0.8.0)
+    sinatra (1.4.5)
+      rack (~> 1.4)
+      rack-protection (~> 1.4)
+      tilt (~> 1.3, >= 1.3.4)
+    sinatra-contrib (1.4.2)
+      backports (>= 2.0)
+      multi_json
+      rack-protection
+      rack-test
+      sinatra (~> 1.4.0)
+      tilt (~> 1.3)
+    slop (3.5.0)
+    sucker_punch (1.0.5)
+      celluloid (~> 0.15.2)
+    term-ansicolor (1.3.0)
+      tins (~> 1.0)
+    terminal-notifier-guard (1.5.3)
+    thor (0.19.1)
+    tilt (1.4.1)
+    timers (1.1.0)
+    tins (1.1.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  coveralls
+  foreman
+  fourchette!
+  guard-rspec
+  pry-byebug
+  puma
+  rspec (~> 2.14.1)
+  terminal-notifier-guard

data/README.md

@@ -6,7 +6,7 @@
   <b>Your new best friend for isolated testing environments on Heroku</b>
   <br />
   <a href="https://codeclimate.com/github/jipiboily/fourchette"><img src="https://codeclimate.com/github/jipiboily/fourchette.png" /></a>
-  <a href="https://github.com/jipiboily/fourchette"><img src="https://travis-ci.org/jipiboily/fourchette.png?branch=master" /></a>
+  <a href="https://travis-ci.org/jipiboily/fourchette"><img src="https://travis-ci.org/jipiboily/fourchette.png?branch=master" /></a>
   <a href='https://coveralls.io/r/jipiboily/fourchette'><img src='https://coveralls.io/repos/jipiboily/fourchette/badge.png' alt='Coverage Status' /></a>
   <a href="http://badge.fury.io/rb/fourchette"><img src="https://badge.fury.io/rb/fourchette.svg" alt="Gem Version" height="18"></a>
 </p>
@@ -16,18 +16,17 @@ Fourchette is your new best friend for having isolated testing environements. It
 **IMPORTANT: Please note that forking your Heroku app means it will copy the same addon plans and that you will pay for multiple apps and their addons. Watch out!**
 
 ## Table of content
-1. How does that work exactly?
-- Features
-- Installation
-  * Configuration
-  * Enable your Fourchette instance
-  * Enable, disable, update or delete the hook
-  * Before & after steps, aka, callbacks
-- Rake tasks
-- Async processing note
-- Contribute
-  - Logging
-- Contributors
+1. [How does that work exactly?](#how-does-that-work-exactly)
+- [Installation](#installation)
+  * [Configuration](#configuration)
+  * [Enable your Fourchette instance](#enable-your-fourchette-instance)
+  * [Enable, disable, update or delete the hook](#enable-disable-update-or-delete-the-hook)
+  * [Before & after steps, aka, callbacks](#before--after-steps-aka-callbacks)
+- [Rake tasks](#rake-tasks)
+- [Async processing note](#async-processing-note)
+- [Contribute](#contribute)
+  - [Logging](#logging)
+- [Contributors](#contributors)
 
 ## How does that work exactly?
 
@@ -37,6 +36,8 @@ Fourchette is your new best friend for having isolated testing environements. It
   - any new commit against that PR will update the code
 - closing the PR will delete the forked app
 - re-opening the PR will re-create a fork
+ 
+We use it a lot at [Rainforest QA](https://www.rainforestqa.com/). If you want to see a sample Fourchette app, here is one for you to look at: https://github.com/rainforestapp/fourchette.
 
 ## Installation
 

data/fourchette.gemspec

@@ -23,15 +23,15 @@ Gem::Specification.new do |spec|
   spec.add_dependency "sinatra-contrib"
   spec.add_dependency "octokit"
   spec.add_dependency "git"
-  spec.add_dependency "heroku" # Deprecated, but best/easiest solution for the pgbackups...
+  spec.add_dependency "heroku", "~> 3.9" # Deprecated, but best/easiest solution for the pgbackups...
   spec.add_dependency 'platform-api', '~> 0.2.0'
   spec.add_dependency "sucker_punch"
   spec.add_dependency "thor"
 
   spec.add_development_dependency 'foreman'
-  spec.add_development_dependency 'pry-debugger'
+  spec.add_development_dependency 'pry-byebug'
   spec.add_development_dependency 'rspec', '~> 2.14.1'
   spec.add_development_dependency 'guard-rspec'
   spec.add_development_dependency 'terminal-notifier-guard'
   spec.add_development_dependency 'coveralls'
-end
+end

data/lib/fourchette/fork.rb

@@ -24,11 +24,11 @@ class Fourchette::Fork
   def monitor_build build
     logger.info "Start of the build process on Heroku..."
     build_info = @heroku.client.build.info(fork_name, build['id'])
-    # Let's just leave some time to Heroku to download the tarball and start 
+    # Let's just leave some time to Heroku to download the tarball and start
     # the process. This is some random timing that seems to make sense at first.
     sleep 30
     if build_info['status'] == 'failed'
-      @github.comment_pr(pr_number, "The build failed on Herok. See the activity tab on Heroku.")
+      @github.comment_pr(pr_number, "The build failed on Heroku. See the activity tab on Heroku.")
       fail Fourchette::DeployException
     end
   end
@@ -41,11 +41,9 @@ class Fourchette::Fork
 
   def delete
     @heroku.delete(fork_name)
-
-    # Update PR with URL
     @github.comment_pr(pr_number, "Test app deleted!")
   end
-  
+
   def fork_name
     "#{ENV['FOURCHETTE_HEROKU_APP_PREFIX']}-PR-#{pr_number}".downcase # It needs to be lowercase only.
   end
@@ -58,15 +56,22 @@ class Fourchette::Fork
     @params['pull_request']['number']
   end
 
-  private
   def create_unless_exists
     unless @heroku.app_exists?(fork_name)
       @heroku.fork(ENV['FOURCHETTE_HEROKU_APP_TO_FORK'] ,fork_name)
-      # Update PR with URL
-      @github.comment_pr(pr_number, "Test URL: #{@heroku.client.app.info(fork_name)['web_url']}")
+      post_fork_url
     end
   end
 
+  private
+
+  # Update PR with URL. This is a method so that we can override it and just not
+  # have that, if we don't want. Use case: we have custom domains, so we post
+  # the URLs later on.
+  def post_fork_url
+    @github.comment_pr(pr_number, "Test URL: #{@heroku.client.app.info(fork_name)['web_url']}")
+  end
+
   def git_branch_name
     "remotes/origin/#{branch_name}"
   end

data/lib/fourchette/heroku.rb

@@ -10,6 +10,7 @@ class Fourchette::Heroku
     copy_config(from, to)
     copy_add_ons(from, to)
     copy_pg(from, to)
+    copy_RACK_AND_RAILS_ENV_again(from, to)
   end
 
   def delete app_name
@@ -33,7 +34,6 @@ class Fourchette::Heroku
     client.app.info(app_name)['git_url']
   end
 
-  private
   def create_app name
     logger.info "Creating #{name}"
     client.app.create({ name: name })
@@ -67,4 +67,21 @@ class Fourchette::Heroku
     backup = Fourchette::Pgbackups.new
     backup.copy(from, to)
   end
+
+  def copy_RACK_AND_RAILS_ENV_again(from, to)
+    env_to_update = get_original_env(from)
+    unless env_to_update.empty?
+      client.config_var.update(to, env_to_update)
+    end
+  end
+
+  def get_original_env(from)
+    environments = {}
+    ['RACK_ENV', 'RAILS_ENV'].each do |var|
+      if client.config_var.info(from)[var]
+        environments[var] = client.config_var.info(from)[var]
+      end
+    end
+    environments
+  end
 end

data/lib/fourchette/tarball.rb

@@ -27,7 +27,7 @@ class Fourchette::Tarball
   def tar(path)
     logger.info "Preparing tarball..."
     filepath = "#{path}/#{expiration_timestamp}.tar.gz"
-    system("tar -zcvf #{filepath} -C #{path} .")
+    system("tar -zcf #{filepath} -C #{path} .")
     filepath
   end
 

data/lib/fourchette/version.rb

@@ -1,3 +1,3 @@
 module Fourchette
-  VERSION = "0.0.7"
+  VERSION = "0.0.8"
 end

data/spec/lib/fourchette/fork_spec.rb

@@ -0,0 +1,120 @@
+require 'spec_helper'
+
+describe Fourchette::Fork do
+  subject { described_class.new(params) }
+
+  let(:params) {
+    {
+      'pull_request' => {
+        'number' => 1,
+        'head' => {
+          'ref' => '123456'
+        }
+      }
+    }
+  }
+  let(:fork_name) { 'my-fork-pr-1' }
+
+  before do
+    stub_const('ENV', {
+        'FOURCHETTE_HEROKU_APP_PREFIX' => 'my-fork',
+        'FOURCHETTE_HEROKU_APP_TO_FORK' => 'my-heroku-app-name'
+        })
+  end
+
+  describe '#update' do
+    before do
+      subject.stub(:create_unless_exists)
+      subject.stub(:monitor_build)
+      Fourchette::Heroku.any_instance.stub_chain(:client, :build, :create)
+    end
+
+    after do
+      # subject.update
+    end
+
+    it 'calls #create_unless_exists' do
+      subject.should_receive(:create_unless_exists)
+
+    end
+
+    it 'starts the build process with the generated tarball URL' do
+      heroku_client_stub = double
+      heroku_client_stub.stub(:build)
+      heroku_client_stub.build.should_receive(:create)
+      Fourchette::Heroku.any_instance.stub(:client).and_return(heroku_client_stub)
+      subject.update
+    end
+
+    it 'starts monitoring the build' do
+    end
+  end
+
+  describe '#create' do
+    it 'calls #update and #create_unless_exists' do
+      subject.should_receive(:create_unless_exists)
+      subject.should_receive(:update)
+      subject.create
+    end
+  end
+
+  describe '#create_unless_exists' do
+    after do
+      subject.create_unless_exists
+    end
+
+    context 'app does NOT exists' do
+      before do
+        Fourchette::Heroku.any_instance.stub(:app_exists?).and_return(false)
+      end
+
+      it 'calls the fork creation' do
+        subject.stub(:post_fork_url)
+        Fourchette::Heroku.any_instance.should_receive(:fork).with('my-heroku-app-name', fork_name)
+      end
+
+      it 'post the URL to the fork on the GitHub PR' do
+        Fourchette::Heroku.any_instance.stub(:fork)
+        Fourchette::Heroku.any_instance.stub_chain(:client, :app, :info).and_return({'web_url' => 'rainforestqa.com'})
+        Fourchette::GitHub.any_instance.should_receive(:comment_pr).with(1, 'Test URL: rainforestqa.com')
+      end
+    end
+
+    context 'app DOES exists' do
+      before do
+        Fourchette::Heroku.any_instance.stub(:app_exists?).and_return(true)
+      end
+
+      it 'does nothing' do
+        Fourchette::GitHub.any_instance.should_not_receive(:comment_pr)
+        Fourchette::Heroku.any_instance.should_not_receive(:fork)
+      end
+    end
+  end
+
+  describe '#delete' do
+    it 'calls deletes the fork' do
+      Fourchette::GitHub.any_instance.stub(:comment_pr)
+      Fourchette::Heroku.any_instance.should_receive(:delete).with(fork_name)
+      subject.delete
+    end
+
+    it 'comments on the GitHub PR' do
+      Fourchette::Heroku.any_instance.stub(:delete)
+      Fourchette::GitHub.any_instance.should_receive(:comment_pr).with(1, 'Test app deleted!')
+      subject.delete
+    end
+  end
+
+  describe '#fork_name' do
+    it { expect(subject.fork_name).to eq fork_name }
+  end
+
+  describe '#branch_name' do
+    it { expect(subject.branch_name).to eq '123456' }
+  end
+
+  describe '#pr_number' do
+    it { expect(subject.pr_number).to eq 1 }
+  end
+end

data/spec/lib/fourchette/heroku_spec.rb

@@ -4,14 +4,18 @@ describe Fourchette::Heroku do
   let(:heroku) { Fourchette::Heroku.new }
   let(:from_app_name) { 'awesome app' }
   let(:to_app_name) { 'awesomer app!' }
+  let(:app_list) do
+    [
+      { 'name' => 'fourchette-pr-7' },
+      { 'name' => 'fourchette-pr-8' }
+    ]
+  end
 
   before do
     client = double('client')
     client_app = double('client')
-    app_list = [ { 'name' => 'fourchette-pr-7' }, { 'name' => 'fourchette-pr-8' } ]
     client_app.stub(:list).and_return(app_list)
     client.stub(:app).and_return(client_app)
-
     config_var = double('config_var')
     client.stub(:config_var).and_return(config_var)
 
@@ -32,9 +36,10 @@ describe Fourchette::Heroku do
       heroku.stub(:copy_config)
       heroku.stub(:copy_add_ons)
       heroku.stub(:copy_pg)
+      heroku.stub(:copy_RACK_AND_RAILS_ENV_again)
     end
 
-    ['create_app', 'copy_config', 'copy_add_ons', 'copy_pg'].each do |method_name|
+    ['create_app', 'copy_config', 'copy_add_ons', 'copy_pg', 'copy_RACK_AND_RAILS_ENV_again'].each do |method_name|
       it "calls `#{method_name}'" do
         heroku.should_receive(method_name)
         heroku.fork(from_app_name, to_app_name)
@@ -60,56 +65,90 @@ describe Fourchette::Heroku do
     end
   end
 
-  describe 'private functions' do
-    describe '#create_app' do
-      it 'calls app.create on the Heroku client' do
-        heroku.client.app.should_receive(:create).with({ name: to_app_name })
-        heroku.send(:create_app, to_app_name)
-      end
+  describe '#create_app' do
+    it 'calls app.create on the Heroku client' do
+      heroku.client.app.should_receive(:create).with({ name: to_app_name })
+      heroku.create_app(to_app_name)
     end
+  end
 
-    describe '#copy_config' do
-      let(:vars) { { 'WHATEVER' => 'ok', 'HEROKU_POSTGRESQL_SOMETHING_URL' => 'FAIL@POSTGRES/DB' } }
-      let(:cleaned_vars) { { 'WHATEVER' => 'ok'} }
+  describe '#copy_config' do
+    let(:vars) { { 'WHATEVER' => 'ok', 'HEROKU_POSTGRESQL_SOMETHING_URL' => 'FAIL@POSTGRES/DB' } }
+    let(:cleaned_vars) { { 'WHATEVER' => 'ok'} }
 
-      it 'calls #config_vars' do
-        heroku.client.config_var.stub(:update)
-        heroku.should_receive(:config_vars).with(from_app_name).and_return(vars)
-        heroku.send(:copy_config, from_app_name, to_app_name)
-      end
+    it 'calls #config_vars' do
+      heroku.client.config_var.stub(:update)
+      heroku.should_receive(:config_vars).with(from_app_name).and_return(vars)
+      heroku.copy_config(from_app_name, to_app_name)
+    end
 
-      it 'updates config vars without postgres URLs' do
-        heroku.client.config_var.should_receive(:update).with(to_app_name, cleaned_vars )
-        heroku.stub(:config_vars).and_return(vars)
-        heroku.send(:copy_config, 'from', to_app_name)
-      end
+    it 'updates config vars without postgres URLs' do
+      heroku.client.config_var.should_receive(:update).with(to_app_name, cleaned_vars )
+      heroku.stub(:config_vars).and_return(vars)
+      heroku.copy_config('from', to_app_name)
+    end
+  end
+
+  describe '#copy_add_ons' do
+    let(:addon_list) { [ { 'plan' => { 'name' => 'redistogo' } } ] }
+
+    before do
+      heroku.client.stub(:addon).and_return( double('addon') )
+      heroku.client.addon.stub(:create)
+      heroku.client.addon.stub(:list).and_return(addon_list)
+    end
+
+    it 'gets the addon list' do
+      heroku.client.addon.should_receive(:list).with(from_app_name).and_return(addon_list)
+      heroku.copy_add_ons(from_app_name, to_app_name)
     end
 
-    describe '#copy_add_ons' do
-      let(:addon_list) { [ { 'plan' => { 'name' => 'redistogo' } } ] }
+    it 'creates addons' do
+      heroku.client.addon.should_receive(:create).with(to_app_name, { plan: 'redistogo' })
+      heroku.copy_add_ons(from_app_name, to_app_name)
+    end
+  end
 
+  describe '#copy_pg' do
+    it 'calls Fourchette::Pgbackups#copy' do
+      Fourchette::Pgbackups.any_instance.should_receive(:copy).with(from_app_name, to_app_name)
+      heroku.copy_pg(from_app_name, to_app_name)
+    end
+  end
+
+  describe '#copy_RACK_AND_RAILS_ENV_again' do
+    context 'with RACK_ENV or RAILS_ENV setup' do
       before do
-        heroku.client.stub(:addon).and_return( double('addon') )
-        heroku.client.addon.stub(:create)
-        heroku.client.addon.stub(:list).and_return(addon_list)
+        heroku.stub(:get_original_env).and_return({ 'RACK_ENV' => 'qa' })
+      end
+
+      it 'updates the config vars' do
+        heroku.client.config_var.should_receive(:update).with(to_app_name,  { 'RACK_ENV' => 'qa' })
+        heroku.copy_RACK_AND_RAILS_ENV_again(from_app_name, to_app_name)
       end
+    end
 
-      it 'gets the addon list' do
-        heroku.client.addon.should_receive(:list).with(from_app_name).and_return(addon_list)
-        heroku.send(:copy_add_ons, from_app_name, to_app_name)
+    context 'with NO env setup' do
+      before do
+        heroku.stub(:get_original_env).and_return({})
       end
 
-      it 'creates addons' do
-        heroku.client.addon.should_receive(:create).with(to_app_name, { plan: 'redistogo' })
-        heroku.send(:copy_add_ons, from_app_name, to_app_name)
+      it 'does not update config vars' do
+        heroku.client.config_var.should_not_receive(:update)
+        heroku.copy_RACK_AND_RAILS_ENV_again(from_app_name, to_app_name)
       end
     end
+  end
 
-    describe '#copy_pg' do
-      it 'calls Fourchette::Pgbackups#copy' do
-        Fourchette::Pgbackups.any_instance.should_receive(:copy).with(from_app_name, to_app_name)
-        heroku.send(:copy_pg, from_app_name, to_app_name)
-      end
+  describe '#get_original_env' do
+    before do
+      stub_cong_var = { 'RACK_ENV' => 'qa', 'RAILS_ENV' => 'staging', 'DATABASE_URL' => 'postgres://....' }
+      heroku.stub_chain(:client, :config_var, :info).and_return(stub_cong_var)
+    end
+
+    it 'returns the set env vars' do
+      return_value = heroku.get_original_env(from_app_name)
+      expect(return_value).to eq({'RACK_ENV' => 'qa', 'RAILS_ENV' => 'staging'})
     end
   end
-end
+end

data/spec/lib/fourchette/logger_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe Fourchette::Logger do
+  class FakeClassToTest
+    include Fourchette::Logger
+  end
+
+  subject { FakeClassToTest.new }
+
+  it { expect(subject.logger.level).to be Logger::INFO }
+
+  context 'first time called' do
+    it { expect(subject.logger).to be_a(Logger) }
+  end
+
+  context 'each time after' do
+    it 'returns the cached version' do
+      logger = subject.logger
+      expect(subject.logger).to be(logger)
+    end
+  end
+end

data/spec/lib/fourchette/tarball_spec.rb

@@ -33,7 +33,7 @@ describe Fourchette::Tarball do
 
     it 'creates the tarball' do
       subject.unstub(:tar)
-      subject.should_receive(:system).with 'tar -zcvf tmp/1234567/123.tar.gz -C tmp/1234567 .'
+      subject.should_receive(:system).with 'tar -zcf tmp/1234567/123.tar.gz -C tmp/1234567 .'
       subject.url(git_repo_url, branch_name, github_repo)
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fourchette
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.8
 platform: ruby
 authors:
 - Jean-Philippe Boily
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-06 00:00:00.000000000 Z
+date: 2014-07-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -84,16 +84,16 @@ dependencies:
   name: heroku
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.9'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.9'
 - !ruby/object:Gem::Dependency
   name: platform-api
   requirement: !ruby/object:Gem::Requirement
@@ -151,7 +151,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry-debugger
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -235,6 +235,7 @@ files:
 - ".gitignore"
 - ".travis.yml"
 - Gemfile
+- Gemfile.lock.orig
 - Guardfile
 - LICENSE.txt
 - Procfile
@@ -258,7 +259,10 @@ files:
 - lib/fourchette/web/hooks.rb
 - lib/fourchette/web/tarball.rb
 - spec/factories/fake_file
+- spec/lib/fourchette/fork_spec.rb
+- spec/lib/fourchette/github_spec.rb
 - spec/lib/fourchette/heroku_spec.rb
+- spec/lib/fourchette/logger_spec.rb
 - spec/lib/fourchette/pull_request_spec.rb
 - spec/lib/fourchette/tarball_spec.rb
 - spec/lib/web/tarball_spec.rb
@@ -296,7 +300,10 @@ specification_version: 4
 summary: Your new best friend for isolated testing environments on Heroku.
 test_files:
 - spec/factories/fake_file
+- spec/lib/fourchette/fork_spec.rb
+- spec/lib/fourchette/github_spec.rb
 - spec/lib/fourchette/heroku_spec.rb
+- spec/lib/fourchette/logger_spec.rb
 - spec/lib/fourchette/pull_request_spec.rb
 - spec/lib/fourchette/tarball_spec.rb
 - spec/lib/web/tarball_spec.rb