integrity 0.1.8 → 0.1.9.0

data/test/acceptance/manual_build_project_test.rb

@@ -0,0 +1,82 @@
+require File.dirname(__FILE__) + "/helpers"
+
+class ManualBuildProjectTest < Test::Unit::AcceptanceTestCase
+  story <<-EOS
+    As an administrator,
+    I want to manually build my project
+    So that I know if it builds properly
+  EOS
+
+  scenario "clicking on 'Manual Build' triggers a successful build" do
+    git_repo(:my_test_project).add_successful_commit
+    Project.gen(:my_test_project, :uri => git_repo(:my_test_project).path)
+    login_as "admin", "test"
+
+    visit "/my-test-project"
+    click_button "manual build"
+
+    assert_have_tag("h1", :content =>
+      "Built #{git_repo(:my_test_project).short_head} successfully")
+    assert_have_tag("blockquote p", :content => "This commit will work")
+    assert_have_tag("span.who",     :content => "by: John Doe")
+    assert_have_tag("span.when",    :content => "today")
+    assert_have_tag("pre.output",   :content => "Running tests...")
+  end
+
+  scenario "clicking on 'Manual Build' triggers a failed build" do
+    git_repo(:my_test_project).add_failing_commit
+    Project.gen(:my_test_project, :uri => git_repo(:my_test_project).path)
+    login_as "admin", "test"
+
+    visit "/my-test-project"
+    click_button "manual build"
+
+    assert_have_tag("h1",
+      :content => "Built #{git_repo(:my_test_project).short_head} and failed")
+    assert_have_tag("blockquote p", :content => "This commit will fail")
+  end
+
+  scenario "fixing the build command and then rebuilding result in a successful build" do
+    git_repo(:my_test_project).add_successful_commit
+    Project.gen(:my_test_project,
+                :uri => git_repo(:my_test_project).path,
+                :command => "ruby not-found.rb")
+
+    login_as "admin", "test"
+
+    visit "/my-test-project"
+    click_button "manual build"
+    assert_have_tag("h1", :content => "failed")
+
+    visit "/my-test-project/edit"
+    fill_in "Build script", :with => "./test"
+    click_button "Update Project"
+
+    visit "/my-test-project"
+    click_button "Build the last commit"
+
+    assert_have_tag("h1", :content => "success")
+  end
+
+  scenario "Successful builds should not display the 'Rebuild' button" do
+    git_repo(:my_test_project).add_successful_commit
+    Project.gen(:my_test_project, :uri => git_repo(:my_test_project).path)
+    login_as "admin", "test"
+
+    visit "/my-test-project"
+    click_button "manual build"
+
+    assert_have_no_tag("button", :content => "Rebuild")
+  end
+
+  scenario "Failed builds should display the 'Rebuild' button" do
+    git_repo(:my_test_project).add_failing_commit
+    Project.gen(:my_test_project, :uri => git_repo(:my_test_project).path)
+    login_as "admin", "test"
+
+    visit "/my-test-project"
+    click_button "manual build"
+
+    assert_have_tag("button", :content => "Rebuild")
+  end
+end

data/test/acceptance/notifier_test.rb

@@ -0,0 +1,109 @@
+require File.dirname(__FILE__) + "/helpers"
+
+class NotifierConfigIssues < Test::Unit::AcceptanceTestCase
+  story <<-EOS
+    As an administrator,
+    I want to add multiple projects to Integrity,
+    So that I can be certain notifiers remain functional (cf #43)
+  EOS
+
+  before(:each) do
+    # This is needed before any available notifier is unset
+    # in the global #before
+    load File.dirname(__FILE__) + "/../helpers/acceptance/email_notifier.rb"
+  end
+
+  def fill_in_email_notifier
+    fill_in "notifiers[Email][to]",     :with => "quentin@example.com"
+    fill_in "notifiers[Email][from]",   :with => "ci@example.com"
+    fill_in "notifiers[Email][user]",   :with => "inspector"
+    fill_in "notifiers[Email][pass]",   :with => "gadget"
+    fill_in "notifiers[Email][auth]",   :with => "simple"
+    fill_in "notifiers[Email][domain]", :with => "example.com"
+  end
+
+  def fill_in_project_info(name, repo)
+    fill_in "Name",            :with => name
+    fill_in "Git repository",  :with => repo
+    fill_in "Branch to track", :with => "master"
+    fill_in "Build script",    :with => "rake"
+    check   "Public project"
+
+    fill_in_email_notifier
+  end
+
+  def assert_have_email_notifier
+    assert_have_tag "input#email_notifier_to[@value='quentin@example.com']"
+    assert_have_tag "input#email_notifier_from[@value='ci@example.com']"
+    assert_have_tag "input#email_notifier_user[@value='inspector']"
+    assert_have_tag "input#email_notifier_pass[@value='gadget']"
+    assert_have_tag "input#email_notifier_auth[@value='simple']"
+    assert_have_tag "input#email_notifier_domain[@value='example.com']"
+  end
+
+  def add_project(name, repo)
+    visit "/new"
+    fill_in_project_info(name, repo)
+    click_button "Create Project"
+
+    assert_have_tag("h1", :content => name)
+    click_link 'Edit Project'
+    assert_have_email_notifier
+  end
+
+  def edit_project(name)
+    visit "/#{name}"
+    click_link "Edit Project"
+    assert_have_email_notifier
+    fill_in :branch, :with => "testing"
+    click_button "Update Project"
+  end
+
+  scenario "an admin can create a public project and retain mailer info" do
+    Project.first(:permalink => "integrity").should be_nil
+
+    login_as "admin", "test"
+
+    visit "/"
+    add_project  "Integrity", "git://github.com/foca/integrity.git"
+    edit_project "integrity"
+
+    visit "/integrity"
+    click_link "Edit Project"
+
+    assert_have_email_notifier
+  end
+
+  scenario "an admin can create multiple public projects" do
+    Project.first(:permalink => "integrity").should be_nil
+
+    login_as "admin", "test"
+
+    visit "/"
+
+    add_project "Integrity", "git://github.com/foca/integrity.git"
+    click_link  "projects"
+
+    add_project "Webrat", "git://github.com/brynary/webrat.git"
+    click_link  "projects"
+
+    add_project "Rails", "git://github.com/rails/rails.git"
+    click_link  "projects"
+
+    edit_project "integrity"
+    edit_project "webrat"
+    edit_project "rails"
+
+    visit "/integrity"
+    click_link "Edit Project"
+    assert_have_email_notifier
+
+    visit "/webrat"
+    click_link "Edit Project"
+    assert_have_email_notifier
+
+    visit "/rails"
+    click_link "Edit Project"
+    assert_have_email_notifier
+  end
+end

data/test/acceptance/project_syndication_test.rb

@@ -0,0 +1,30 @@
+require File.dirname(__FILE__) + "/helpers"
+
+class ProjectSyndicationTest < Test::Unit::AcceptanceTestCase
+  story <<-EOS
+    As a user,
+    I want to subscribe to a public project's Atom feed
+    So I can know the status of my favorite projects while having my morning coffee
+  EOS
+
+  scenario "a public project's page includes an autodiscovery link tag for the feed" do
+    Project.gen(:integrity, :public => true)
+    visit "/integrity"
+
+    assert_have_tag("link[@href='/integrity.atom']")
+  end
+
+  scenario "a public project's feed should include the latest builds" do
+    commits = 10.of { Commit.gen(:successful) } + 1.of { Commit.gen(:failed) }
+    Project.gen(:integrity, :public => true, :commits => commits)
+
+    visit "/integrity.atom"
+
+    # TODO: check for content-type
+
+    assert_have_tag("feed title", :content => "Build history for Integrity")
+    assert_have_tag("feed entry", :count => 11)
+    assert_have_tag("feed entry:first title", :content => "success")
+    assert_have_tag("feed entry:last title",  :content => "failed")
+  end
+end

data/test/acceptance/stylesheet_test.rb

@@ -0,0 +1,18 @@
+require File.dirname(__FILE__) + "/helpers"
+
+class IntegrityStylesheetTest < Test::Unit::AcceptanceTestCase
+  story <<-EOS
+    As a user,
+    I want the stylesheet to work (even with Sinatra 0.9.1)
+    So that Integrity isn't a PITA to use
+  EOS
+
+  scenario "browsing on some Integrity install" do
+    visit "/integrity.css"
+
+    assert_contain("body {")
+    # TODO: better test
+    assert_equal %Q{"2465c472aacf302259dde5146a841e45"},
+      webrat_session.send(:response).headers["ETag"]
+  end
+end

data/test/helpers.rb

@@ -1,37 +1,43 @@
-require File.dirname(__FILE__) + "/../lib/integrity"
+$:.unshift File.dirname(__FILE__) + "/../lib", File.dirname(__FILE__),
+  File.dirname(__FILE__) + "/../vendor/webrat/lib"
+
+%w(test/unit
+context
+pending
+matchy
+storyteller
+webrat/sinatra
+rr
+mocha
+test/zentest_assertions
+dm-sweatshop).each { |dependency|
+  begin
+    require dependency
+  rescue LoadError
+    puts "You're missing some gems required to run the tests."
+    puts "Please run `rake test:install_dependencies`"
+    puts "You'll probably need to run that command as root or with sudo."
+
+    puts "Thanks :)"
+    puts
+
+    exit 1
+  end
+}
 
 begin
-  require "test/unit"
-  require "redgreen"
-  require "context"
-  require "storyteller"
-  require "pending"
-  require "matchy"
-  require "rr"
-  require "mocha"
   require "ruby-debug"
+  require "redgreen"
 rescue LoadError
-  puts "You're missing some gems required to run the tests."
-  puts "Please run `rake test:install_dependencies`"
-  puts "You'll probably need to run that command as root or with sudo."
-  puts
-  puts "Thanks :)"
-  puts
-
-  exit 1
 end
 
-require File.dirname(__FILE__) / "helpers" / "expectations"
-require File.dirname(__FILE__) / "helpers" / "fixtures"
+require "integrity"
+require "helpers/expectations"
+require "helpers/fixtures"
 
 module TestHelper
-  def setup_and_reset_database!
-    DataMapper.setup(:default, "sqlite3::memory:")
-    DataMapper.auto_migrate!
-  end
-
   def ignore_logs!
-    stub(Integrity).log { nil }
+    Integrity.config[:log] = "/tmp/integrity.test.log"
   end
 end
 
@@ -43,5 +49,32 @@ class Test::Unit::TestCase
   include RR::Adapters::TestUnit
   include Integrity
   include TestHelper
-end
 
+  before(:all) do
+    DataMapper.setup(:default, "sqlite3::memory:")
+  end
+
+  before(:each) do
+    RR.reset
+    DataMapper.auto_migrate!
+    Integrity.instance_variable_set(:@config, nil)
+    Notifier.available.each { |n|
+      Notifier.send(:remove_const, n.to_s.split(":").last.to_sym)
+    }
+
+    repository(:default) do
+      transaction = DataMapper::Transaction.new(repository)
+      transaction.begin
+      repository.adapter.push_transaction(transaction)
+    end
+  end
+
+  after(:each) do
+    repository(:default) do
+      while repository.adapter.current_transaction
+        repository.adapter.current_transaction.rollback
+        repository.adapter.pop_transaction
+      end
+    end
+  end
+end

data/test/helpers/acceptance.rb

@@ -1,67 +1,10 @@
-require 'webrat/rack'
-require 'sinatra'
-require 'sinatra/test'
-
-set :environment, :test
-disable :run
-disable :reload
-
-Webrat.configuration.instance_variable_set("@mode", :sinatra)
-
-module Webrat
-  class SinatraSession < Session
-    DEFAULT_DOMAIN = "integrity.example.org"
-
-    def initialize(context = nil)
-      super(context)
-      @sinatra_test = Sinatra::TestHarness.new
-    end
-
-    %w(get head post put delete).each do |verb|
-      class_eval <<-METHOD
-        def #{verb}(path, data, headers = {})
-          params = data.inject({}) do |data, (key,value)|
-            data[key] = Rack::Utils.unescape(value)
-            data
-          end
-          headers['HTTP_HOST'] = DEFAULT_DOMAIN
-          @sinatra_test.#{verb}(path, params, headers)
-        end
-      METHOD
-    end
-
-    def response_body
-      @sinatra_test.body
-    end
-
-    def response_code
-      @sinatra_test.status
-    end
-
-    private
-
-    def response
-      @sinatra_test.response
-    end
-
-    def current_host
-      URI.parse(current_url).host || DEFAULT_DOMAIN
-    end
-
-    def response_location_host
-      URI.parse(response_location).host || DEFAULT_DOMAIN
-    end
-  end
-end
-
-require Integrity.root / "app"
-require File.dirname(__FILE__) / "acceptance/git_helper"
+require "helpers/acceptance/git_helper"
 
 module AcceptanceHelper
   include FileUtils
 
   def export_directory
-    Integrity.root / "exports"
+    File.dirname(__FILE__) + "/../../exports"
   end
 
   def enable_auth!
@@ -75,7 +18,7 @@ module AcceptanceHelper
     def AcceptanceHelper.logged_in; true; end
     basic_auth user, password
     visit "/login"
-    Sinatra::Application.before { login_required if AcceptanceHelper.logged_in }
+    Integrity::App.before { login_required if AcceptanceHelper.logged_in }
   end
 
   def log_out
@@ -94,9 +37,9 @@ module AcceptanceHelper
   end
 
   def setup_log!
-    pathname = Integrity.root / "integrity.log"
-    FileUtils.rm pathname if File.exists?(pathname)
-    Integrity.config[:log] = pathname
+    log_file = Pathname(File.dirname(__FILE__) + "/../../integrity.log")
+    log_file.delete if log_file.exist?
+    Integrity.config[:log] = log_file
   end
 end
 
@@ -105,15 +48,25 @@ class Test::Unit::AcceptanceTestCase < Test::Unit::TestCase
   include Test::Storyteller
   include GitHelper
   include Webrat::Methods
+  include Webrat::Matchers
+  include Webrat::HaveTagMatcher
+
+  # TODO: does this belongs in Webrat::SinatraSession?
   Webrat::Methods.delegate_to_session :response_code
 
+  def app
+    Integrity::App.tap { |app|
+      app.set     :environment, :test
+      app.disable :raise_errors, :run, :reload
+    }
+  end
+
   before(:all) do
-    Integrity.config[:base_uri] = "http://#{Webrat::SinatraSession::DEFAULT_DOMAIN}"
+    Integrity.config[:base_uri] = "http://www.example.com"
   end
 
   before(:each) do
     # ensure each scenario is run in a clean sandbox
-    setup_and_reset_database!
     enable_auth!
     setup_log!
     set_and_create_export_directory!
@@ -124,4 +77,5 @@ class Test::Unit::AcceptanceTestCase < Test::Unit::TestCase
     destroy_all_git_repos
     rm_r export_directory if File.directory?(export_directory)
   end
+
 end