foca-integrity 0.1.9.2 → 0.1.9.3

data/lib/integrity/notifier/base.rb

@@ -10,7 +10,7 @@ module Integrity
       end
 
       def self.to_haml
-        raise NoMethodError, "you need to implement this method in your notifier"
+        raise NotImplementedError, "you need to implement this method in your notifier"
       end
 
       attr_reader :commit
@@ -26,7 +26,7 @@ module Integrity
       end
 
       def deliver!
-        raise NoMethodError, "you need to implement this method in your notifier"
+        raise NotImplementedError, "you need to implement this method in your notifier"
       end
 
       def short_message

data/lib/integrity/project.rb

@@ -1,7 +1,13 @@
+require "integrity/project/notifiers"
+require "integrity/project/push"
+
 module Integrity
   class Project
     include DataMapper::Resource
 
+    include Helpers::Notifiers
+    include Helpers::Push
+
     property :id,         Integer,  :serial => true
     property :name,       String,   :nullable => false
     property :permalink,  String
@@ -35,41 +41,14 @@ module Integrity
       Build.queue(commit)
     end
 
-    def push(payload)
-      payload = parse_payload(payload)
-      raise ArgumentError unless valid_payload?(payload)
-
-      commits =
-        if Integrity.config[:build_all_commits]
-          payload["commits"]
-        else
-          [ payload["commits"].first ]
-        end
-
-      commits.each do |commit_data|
-        create_commit_from(commit_data)
-        build(commit_data["id"])
-      end
-    end
-
     def last_commit
       commits.first(:project_id => id, :order => [:committed_at.desc])
     end
 
-    def last_build
-      warn "Project#last_build is deprecated, use Project#last_commit (#{caller[0]})"
-      last_commit
-    end
-
     def previous_commits
       commits.all(:project_id => id, :order => [:committed_at.desc]).tap {|commits| commits.shift }
     end
 
-    def previous_builds
-      warn "Project#previous_builds is deprecated, use Project#previous_commits (#{caller[0]})"
-      previous_commits
-    end
-
     def status
       last_commit && last_commit.status
     end
@@ -85,19 +64,6 @@ module Integrity
       end)
     end
 
-    def config_for(notifier)
-      notifier = notifiers.first(:name => notifier.to_s.split(/::/).last, :project_id => id)
-      notifier.blank? ? {} : notifier.config
-    end
-
-    def notifies?(notifier)
-      !notifiers.first(:name => notifier.to_s.split(/::/).last, :project_id => id).blank?
-    end
-
-    def enable_notifiers(*args)
-      Notifier.enable_notifiers(id, *args)
-    end
-
     private
       def find_or_create_commit_with_identifier(commit_identifier)
         # We abuse +committed_at+ here setting it to Time.now because we use it
@@ -116,13 +82,6 @@ module Integrity
         SCM.new(uri, branch).head
       end
 
-      def create_commit_from(data)
-        commits.create(:identifier   => data["id"],
-                       :author       => "#{data["author"]["name"]} <#{data["author"]["email"]}>",
-                       :message      => data["message"],
-                       :committed_at => data["timestamp"])
-      end
-
       def set_permalink
         self.permalink = (name || "").downcase.
           gsub(/'s/, "s").
@@ -137,17 +96,5 @@ module Integrity
       rescue SCM::SCMUnknownError => error
         Integrity.log "Problem while trying to deleting code: #{error}"
       end
-
-      def valid_payload?(payload)
-        payload && payload["ref"].to_s.include?(branch) &&
-                               !payload["commits"].nil? &&
-                               !payload["commits"].to_a.empty?
-      end
-
-      def parse_payload(payload)
-        JSON.parse(payload.to_s)
-      rescue JSON::ParserError
-        false
-      end
   end
 end

data/lib/integrity/project/notifiers.rb

@@ -0,0 +1,33 @@
+module Integrity
+  class Project
+    module Helpers
+      module Notifiers
+        def notifies?(notifier)
+          return false unless notifier = notifiers.first(:name => notifier)
+
+          notifier.enabled?
+        end
+
+        def enabled_notifiers
+          notifiers.all(:enabled => true)
+        end
+
+        def config_for(notifier)
+          notifier = notifiers.first(:name => notifier)
+          notifier ? notifier.config : {}
+        end
+
+        def update_notifiers(to_enable, config)
+          config.each_pair { |name, config|
+            notifier = notifiers.first(:name => name)
+            notifier ||= notifiers.new(:name => name)
+
+            notifier.enabled = to_enable.include?(name)
+            notifier.config  = config
+            notifier.save
+          }
+        end
+      end
+    end
+  end
+end

data/lib/integrity/project/push.rb

@@ -0,0 +1,44 @@
+module Integrity
+  class Project
+    module Helpers
+      module Push
+        def push(payload)
+          payload = parse_payload(payload)
+          raise ArgumentError unless valid_payload?(payload)
+
+          commits =
+            if Integrity.config[:build_all_commits]
+              payload["commits"]
+            else
+              [ payload["commits"].first ]
+            end
+
+          commits.each do |commit_data|
+            create_commit_from(commit_data)
+            build(commit_data["id"])
+          end
+        end
+
+        private
+          def create_commit_from(data)
+            commits.create(:identifier => data["id"],
+              :author  => "#{data["author"]["name"]} <#{data["author"]["email"]}>",
+              :message => data["message"],
+              :committed_at => data["timestamp"])
+          end
+
+          def valid_payload?(payload)
+            payload && payload["ref"].to_s.include?(branch) &&
+                                 !payload["commits"].nil? &&
+                                 !payload["commits"].to_a.empty?
+          end
+
+          def parse_payload(payload)
+            JSON.parse(payload.to_s)
+          rescue JSON::ParserError
+          false
+          end
+      end
+    end
+  end
+end

data/test/acceptance/browse_project_test.rb

@@ -48,6 +48,10 @@ class BrowsePublicProjectsTest < Test::Unit::AcceptanceTestCase
     click_link "My Test Project"
 
     assert_have_tag("h1", :content => "My Test Project")
+
+    # He can then go back to the project listing
+    click_link "projects"
+    assert_have_tag("a", :content => "My Test Project")
   end
 
   scenario "a user gets a 404 when browsing to an unexisting project" do

data/test/acceptance/build_notifications_test.rb

@@ -1,7 +1,9 @@
 require File.dirname(__FILE__) + "/../helpers/acceptance"
-require "helpers/acceptance/textfile_notifier"
+require "helpers/acceptance/notifier_helper"
 
 class BuildNotificationsTest < Test::Unit::AcceptanceTestCase
+  include NotifierHelper
+
   story <<-EOS
     As an administrator,
     I want to setup notifiers on my projects
@@ -12,11 +14,14 @@ class BuildNotificationsTest < Test::Unit::AcceptanceTestCase
     # This is needed before any available notifier is unset
     # in the global #before
     load "helpers/acceptance/textfile_notifier.rb"
+    load "helpers/acceptance/email_notifier.rb"
+    Notifier.register(Integrity::Notifier::Textfile)
+    Notifier.register(Integrity::Notifier::Email)
   end
 
-  scenario "an admin sets up a notifier for a project that didn't have any" do
+  scenario "an admin sets up a notifier and issue a manual build" do
     git_repo(:my_test_project).add_successful_commit
-    Project.gen(:my_test_project, :notifiers => [], :uri => git_repo(:my_test_project).path)
+    Project.gen(:my_test_project, :uri => git_repo(:my_test_project).path)
     rm_f "/tmp/textfile_notifications.txt"
 
     login_as "admin", "test"
@@ -33,10 +38,58 @@ class BuildNotificationsTest < Test::Unit::AcceptanceTestCase
     notification = File.read("/tmp/textfile_notifications.txt")
     notification.should =~ /=== Built #{git_repo(:my_test_project).short_head} successfully ===/
     notification.should =~ /Build #{git_repo(:my_test_project).head} was successful/
-    notification.should =~ %r(http://www.example.com/my-test-project/commits/#{git_repo(:my_test_project).head})
+    notification.should =~
+      %r(http://www.example.com/my-test-project/commits/#{git_repo(:my_test_project).head})
     notification.should =~ /Commit Author: John Doe/
     notification.should =~ /Commit Date: (.+)/
     notification.should =~ /Commit Message: This commit will work/
     notification.should =~ /Build Output:\n\nRunning tests...\n/
   end
+
+  scenario "an admin can setup a notifier without enabling it" do
+    Project.gen(:integrity)
+
+    login_as "admin", "test"
+
+    visit "/integrity"
+    click_link "Edit Project"
+    fill_in_email_notifier
+    click_button "Update Project"
+
+    visit "/integrity/edit"
+    assert_have_email_notifier
+  end
+
+  scenario "an admin configures various notifiers accros multiple 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/installer_test.rb

@@ -24,7 +24,10 @@ class InstallerTest < Test::Unit::AcceptanceTestCase
   end
 
   scenario "Installing integrity into a given directory" do
-    assert install.include?("Awesome")
+    post_install_message = install
+
+    assert post_install_message.include?("Awesome")
+    assert post_install_message.include?("integrity migrate_db #{root.join("config.yml")}")
 
     assert root.join("builds").directory?
     assert root.join("log").directory?

data/test/helpers.rb

@@ -1,27 +1,24 @@
 $:.unshift File.dirname(__FILE__) + "/../lib", File.dirname(__FILE__)
 
-%w(test/unit
-context
-pending
-matchy
-storyteller
-webrat/sinatra
-rr
-mocha
-dm-sweatshop).each { |dependency|
-  begin
-    require dependency
-  rescue LoadError => e
-    puts "You're missing some gems required to run the tests."
-    puts "Please run `rake test:setup`"
-    puts "NOTE: You'll probably need to run that command as root or with sudo."
-
-    puts "Thanks :)"
-    puts
-
-    raise
-  end
-}
+require "rubygems"
+
+require "test/unit"
+require "rr"
+require "mocha"
+require "dm-sweatshop"
+require "webrat/sinatra"
+
+gem "jeremymcanally-context"
+gem "jeremymcanally-matchy"
+gem "jeremymcanally-pending"
+require "context"
+require "matchy"
+require "pending"
+
+require "integrity"
+require "integrity/notifier/test/fixtures"
+
+require "helpers/expectations"
 
 begin
   require "ruby-debug"
@@ -29,10 +26,6 @@ begin
 rescue LoadError
 end
 
-require "integrity"
-require "helpers/expectations"
-require "integrity/notifier/test/fixtures"
-
 module TestHelper
   def ignore_logs!
     Integrity.config[:log] = "/tmp/integrity.test.log"
@@ -65,29 +58,25 @@ class Test::Unit::TestCase
 
   before(:all) do
     DataMapper.setup(:default, "sqlite3::memory:")
+
+    require "integrity/migrations"
   end
 
   before(:each) do
+    [Project, Build, Commit, Notifier].each(&:auto_migrate_down!)
+    capture_stdout { Integrity.migrate_db }
+
     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
+    Notifier.available.clear
+    Integrity.instance_variable_set(:@config, nil)
   end
 
   after(:each) do
-    repository(:default) do
-      while repository.adapter.current_transaction
-        repository.adapter.current_transaction.rollback
-        repository.adapter.pop_transaction
-      end
-    end
+    capture_stdout { Integrity::Migrations.migrate_down! }
   end
 end

data/test/helpers/acceptance.rb

@@ -1,4 +1,8 @@
 require File.dirname(__FILE__) + "/../helpers"
+
+gem "foca-storyteller"
+require "storyteller"
+
 require "helpers/acceptance/git_helper"
 
 module AcceptanceHelper

data/test/helpers/acceptance/notifier_helper.rb

@@ -0,0 +1,47 @@
+module NotifierHelper
+  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
+end