integrity 0.1.8 → 0.1.9.0

data/test/unit/project_test.rb

@@ -0,0 +1,282 @@
+require File.dirname(__FILE__) + "/../helpers"
+
+class ProjectTest < Test::Unit::TestCase
+  before(:each) do
+    RR.reset
+    ignore_logs!
+  end
+
+  specify "default fixture is valid and can be saved" do
+    lambda do
+      Project.generate.tap do |project|
+        project.should be_valid
+        project.save
+      end
+    end.should change(Project, :count).by(1)
+  end
+
+  specify "integrity fixture is valid and can be saved" do
+    lambda do
+      Project.generate(:integrity).tap do |project|
+        project.should be_valid
+        project.save
+      end
+    end.should change(Project, :count).by(1)
+  end
+
+  describe "Properties" do
+    before(:each) do
+      @project = Project.generate(:integrity)
+    end
+
+    it "has a name" do
+      @project.name.should == "Integrity"
+    end
+
+    it "has a permalink" do
+      @project.permalink.should == "integrity"
+
+      @project.tap do |project|
+        project.name = "foo's bar/baz and BACON?!"
+        project.save
+      end.permalink.should == "foos-bar-baz-and-bacon"
+    end
+
+    it "has an URI" do
+      @project.uri.should == Addressable::URI.parse("git://github.com/foca/integrity.git")
+    end
+
+    it "has a branch" do
+      @project.branch.should == "master"
+    end
+
+    specify "branch defaults to master" do
+      Project.new.branch.should == "master"
+    end
+
+    it "has a command" do
+      # TODO: rename to build_command
+      @project.command.should == "rake"
+    end
+
+    specify "command defaults to 'rake'" do
+      Project.new.command.should == "rake"
+    end
+
+    it "has a building flag" do
+      @project.should_not be_building
+    end
+
+    specify "building flag default to false" do
+      Project.new.should_not be_building
+    end
+
+    it "knows it's visibility" do
+      # TODO: rename Project#public property to visibility
+      # TODO: and have utility method to query its state instead
+
+      Project.new.should be_public
+
+      @project.should be_public
+      @project.tap { |p| p.public = "1" }.should be_public
+      @project.tap { |p| p.public = "0" }.should_not be_public
+
+      Project.gen(:public => "false").should be_public
+      Project.gen(:public => "true").should be_public
+      Project.gen(:public => false).should_not be_public
+      Project.gen(:public => nil).should_not be_public
+    end
+
+    it "has a created_at" do
+      @project.created_at.should be_a(DateTime)
+    end
+
+    it "has an updated_at" do
+      @project.updated_at.should be_a(DateTime)
+    end
+
+    it "knows it's status" do
+      Project.gen(:commits => 1.of{ Commit.gen(:successful) }).status.should == :success
+      Project.gen(:commits => 2.of{ Commit.gen(:successful) }).status.should == :success
+      Project.gen(:commits => 2.of{ Commit.gen(:failed) }).status.should == :failed
+      Project.gen(:commits => 1.of{ Commit.gen(:pending) }).status.should == :pending
+      Project.gen(:commits => []).status.should be_nil
+    end
+
+    it "knows it's last build" do
+      Project.gen(:commits => []).last_commit.should be_nil
+
+      commits = 5.of { Commit.gen(:successful) }
+      project = Project.gen(:commits => commits)
+      project.last_commit.should == commits.sort_by {|c| c.committed_at }.last
+    end
+  end
+
+  describe "Validation" do
+    it "requires a name" do
+      lambda do
+        Project.gen(:name => nil).should_not be_valid
+      end.should_not change(Project, :count)
+    end
+
+    it "requires an URI" do
+      lambda do
+        Project.gen(:uri => nil).should_not be_valid
+      end.should_not change(Project, :count)
+    end
+
+    it "requires a branch" do
+      lambda do
+        Project.gen(:branch => nil).should_not be_valid
+      end.should_not change(Project, :count)
+    end
+
+    it "requires a command" do
+      lambda do
+        Project.gen(:command => nil).should_not be_valid
+      end.should_not change(Project, :count)
+    end
+
+    it "ensures its name is unique" do
+      Project.gen(:name => "Integrity")
+      lambda do
+        Project.gen(:name => "Integrity").should_not be_valid
+      end.should_not change(Project, :count)
+    end
+  end
+
+  describe "Finding public or private projects" do
+    before(:each) do
+      @public_project = Project.gen(:public => true)
+      @private_project = Project.gen(:public => false)
+    end
+
+    it "finds only public projects if the condition passed is false" do
+      projects = Project.only_public_unless(false)
+      projects.should_not include(@private_project)
+      projects.should include(@public_project)
+    end
+
+    it "finds both private and public projects if the condition passed is true" do
+      projects = Project.only_public_unless(true)
+      projects.should include(@private_project)
+      projects.should include(@public_project)
+    end
+  end
+
+  describe "When finding its previous builds" do
+    before(:each) do
+      @project = Project.generate(:commits => 5.of { Commit.gen })
+      @commits = @project.commits.sort_by {|c| c.committed_at }.reverse
+    end
+
+    it "has 4 previous builds" do
+      @project.should have(4).previous_commits
+    end
+
+    it "returns the builds ordered chronogicaly (desc) by creation date" do
+      @project.previous_commits.should == @commits[1..-1]
+    end
+
+    it "excludes the last build" do
+      @project.previous_commits.should_not include(@project.last_commit)
+    end
+
+    it "returns an empty array if it has only one build" do
+      project = Project.gen(:commits => 1.of { Integrity::Commit.gen })
+      project.should have(:no).previous_commits
+    end
+
+    it "returns an empty array if there are no builds" do
+      project = Project.gen(:commits => [])
+      project.should have(:no).previous_commits
+    end
+  end
+
+  describe "When getting destroyed" do
+    before(:each) do
+      @commits  = 7.of { Commit.gen }
+      @project = Project.generate(:commits => @commits)
+    end
+
+    it "destroys itself and tell ProjectBuilder to delete the code from disk" do
+      lambda do
+        stub.instance_of(ProjectBuilder).delete_code
+        @project.destroy
+      end.should change(Project, :count).by(-1)
+    end
+
+    it "destroys its builds" do
+      lambda do
+        @project.destroy
+      end.should change(Commit, :count).by(-7)
+    end
+  end
+
+  describe "When retrieving state about its notifier" do
+    before(:each) do
+      @project = Project.generate
+      @irc     = Notifier.generate(:irc)
+    end
+
+    specify "#config_for returns given notifier's configuration" do
+      @project.update_attributes(:notifiers => [@irc])
+      @project.config_for("IRC").should == {:uri => "irc://irc.freenode.net/integrity"}
+    end
+
+    specify "#config_for returns an empty hash if no such notifier" do
+      @project.config_for("IRC").should == {}
+    end
+
+    specify "#notifies? is true if it uses the given notifier" do
+      @project.update_attributes(:notifiers => [@irc])
+      @project.notifies?("IRC").should == true
+    end
+
+    specify "#notifies? is false if it doesnt use the given notifier" do
+      @project.update_attributes(:notifiers => [])
+
+      @project.notifies?("IRC").should == false
+      @project.notifies?("UndefinedNotifier").should == false
+    end
+  end
+
+  describe "When building a commit" do
+    before(:each) do
+      @commits = 2.of { Commit.gen }
+      @project = Project.gen(:integrity, :commits => @commits)
+      stub.instance_of(ProjectBuilder).build { nil }
+    end
+
+    it "gets the specified commit and creates a pending build for it" do
+      commit = @commits.last
+
+      lambda {
+        @project.build(commit.identifier)
+      }.should change(Build, :count).by(1)
+
+      build = Build.all.last
+      build.commit.should be(commit)
+      build.should be_pending
+
+      commit.should be_pending
+    end
+
+    it "creates an empty commit with the head of the project if passed 'HEAD' (the default)" do
+      mock(@project).head_of_remote_repo { "FOOBAR" }
+
+      lambda {
+        @project.build("HEAD")
+      }.should change(Commit, :count).by(1)
+
+      build = Build.all.last
+      build.commit.should be(@project.last_commit)
+
+      @project.last_commit.should be_pending
+      @project.last_commit.identifier.should be("FOOBAR")
+
+      @project.last_commit.author.name.should == "<Commit author not loaded>"
+      @project.last_commit.message.should == "<Commit message not loaded>"
+    end
+  end
+end

data/test/unit/scm_test.rb

@@ -0,0 +1,54 @@
+require File.dirname(__FILE__) + "/../helpers"
+
+class SCMTest < Test::Unit::TestCase
+  def scm(uri)
+    SCM.new(Addressable::URI.parse(uri), "master", "foo")
+  end
+
+  it "recognizes git URIs" do
+    scm("git://example.org/repo").should be_an(SCM::Git)
+    scm("git@example.org/repo.git").should be_an(SCM::Git)
+    scm("git://example.org/repo.git").should be_an(SCM::Git)
+  end
+
+  it "raises SCMUnknownError if it can't figure the SCM from the URI" do
+    lambda { scm("scm://example.org") }.should raise_error(SCM::SCMUnknownError)
+  end
+
+  it "doesn't need the working tree path for all operations, so it's not required on the constructor" do
+    lambda {
+      SCM.new(Addressable::URI.parse("git://github.com/foca/integrity.git"), "master")
+    }.should_not raise_error
+  end
+
+  describe "SCM::Git::URI" do
+    uris = [
+      "rsync://host.xz/path/to/repo.git/",
+      "rsync://host.xz/path/to/repo.git",
+      "rsync://host.xz/path/to/repo.gi",
+      "http://host.xz/path/to/repo.git/",
+      "https://host.xz/path/to/repo.git/",
+      "git://host.xz/path/to/repo.git/",
+      "git://host.xz/~user/path/to/repo.git/",
+      "ssh://[user@]host.xz[:port]/path/to/repo.git/",
+      "ssh://[user@]host.xz/path/to/repo.git/",
+      "ssh://[user@]host.xz/~user/path/to/repo.git/",
+      "ssh://[user@]host.xz/~/path/to/repo.git",
+      "host.xz:/path/to/repo.git/",
+      "host.xz:~user/path/to/repo.git/",
+      "host.xz:path/to/repo.git",
+      "user@host.xz:/path/to/repo.git/",
+      "user@host.xz:~user/path/to/repo.git/",
+      "user@host.xz:path/to/repo.git",
+      "user@host.xz:path/to/repo",
+      "user@host.xz:path/to/repo.a_git"
+    ]
+
+    uris.each do |uri|
+      it "parses the uri #{uri}" do
+        git_url = SCM::Git::URI.new(uri)
+        git_url.working_tree_path.should == "path-to-repo"
+      end
+    end
+  end
+end

data/views/_commit_info.haml

@@ -0,0 +1,24 @@
+%h1&= commit.human_readable_status
+
+- if commit.failed?
+  %form{ :action => commit_path(commit, :builds), :method => :post }
+    %p.submit
+      %button{ :type => :submit, :title => "Rebuild this commit" }<
+        Rebuild
+
+%blockquote
+  %p&= commit.message
+  %p.meta<
+    %span.who<
+      &== by: #{commit.author.name}
+    |
+    %span.when{ :title => commit.commited_at }<
+      &= pretty_date commit.committed_at
+    |
+    %span.what<
+      &== commit: #{commit.identifier}
+
+%h2 Build Output:
+%pre.output
+  :preserve
+    #{bash_color_codes h(commit.output)}

data/views/build.haml

@@ -1,2 +1,2 @@
-#build{ :class => @build.status }
-  = partial(:build_info, :build => @build)
+#build{ :class => @commit.status }
+  = partial(:commit_info, :commit => @commit)

data/views/error.haml

@@ -13,9 +13,10 @@
 
     %dt What can I do?
     %dd
-      Is your
-      %a{ :href => project_url(@project, :edit) } config
-      ok?
+      - if @project
+        Is your
+        %a{ :href => project_url(@project, :edit) } config
+        ok?
       Need
       %a{ :href => "http://integrityapp.com/configure" } help?
       Remember to restart Integrity.

data/views/home.haml

@@ -8,16 +8,14 @@
 - else
   %ul#projects
     - @projects.each do |project|
-      %li{ :class => cycle("even", "odd") + (project.building? ? ' building' : '') + (project.last_build ? (project.last_build.successful? ? ' success' : ' failed') : '') }
+      %li{ :class => cycle("even", "odd") + ' ' + project.status.to_s }
         %a{ :href => project_path(project) }&= project.name
         .meta
           - if project.building?
             Building!
-          - elsif project.last_build.nil?
+          - elsif project.last_commit.nil?
             Never built yet
           - else
-            == Built #{project.last_build.short_commit_identifier}
-            = pretty_date(project.last_build.created_at)
-            = project.last_build.successful? ? "successfully" : "and failed"
+            = project.human_readable_status
   %p#new
     %a{ :href => "/new" } Add a new project

data/views/integrity.sass

@@ -108,6 +108,7 @@ a
       :width 30em
       :padding .2em .4em
       :color = !title_color
+    input.text
       :height 1.4em
     label
       :float left
@@ -177,11 +178,22 @@ a
         :content "."
         :text-indent -9999em
         :text-align left
-      &.destroy
-        :margin-top 0
+      &.destroy, &.manual-build
         button
           :float none
           :display inline
+      &.manual-build
+        button
+          :margin-right 0
+
+  #build form, #last_build form
+    :font-size .75em
+    p.submit
+      :margin 0
+      :padding 0
+      :position absolute
+      :right .5em
+      :top 1.25em
 
   .blank_slate, .error
     p
@@ -213,7 +225,7 @@ a
         :family = !nice_fonts
     dd
       :line-height 1.4
-    
+
     .backtrace
       :margin 1em 0
       :overflow scroll
@@ -254,8 +266,8 @@ a
         :color = !success_color
       &.failed .meta
         :color = !failed_color
-      
-  
+
+
   #previous_builds
     li
       a
@@ -288,9 +300,10 @@ a
           :color = !failed_bg - #444
         &:hover
           :background-color = !failed_bg + #222
-  
+
 
   #build, #last_build
+    :position relative
     h1, blockquote
       :border
         :width 0 1px