2011-10-08 21:36:38 +00:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe Project do
|
|
|
|
describe "Associations" do
|
|
|
|
it { should have_many(:users) }
|
|
|
|
it { should have_many(:users_projects) }
|
2011-10-21 18:40:36 +00:00
|
|
|
it { should have_many(:issues) }
|
|
|
|
it { should have_many(:notes) }
|
|
|
|
it { should have_many(:snippets) }
|
2011-12-14 16:38:52 +00:00
|
|
|
it { should have_many(:web_hooks).dependent(:destroy) }
|
2011-10-08 21:36:38 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
describe "Validation" do
|
|
|
|
it { should validate_presence_of(:name) }
|
|
|
|
it { should validate_presence_of(:path) }
|
2011-10-21 18:40:36 +00:00
|
|
|
it { should validate_presence_of(:code) }
|
2011-10-08 21:36:38 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
describe "Respond to" do
|
|
|
|
it { should respond_to(:readers) }
|
|
|
|
it { should respond_to(:writers) }
|
2011-12-05 07:43:53 +00:00
|
|
|
it { should respond_to(:repository_writers) }
|
2011-10-08 21:36:38 +00:00
|
|
|
it { should respond_to(:admins) }
|
|
|
|
it { should respond_to(:add_access) }
|
|
|
|
it { should respond_to(:reset_access) }
|
2011-12-05 07:43:53 +00:00
|
|
|
it { should respond_to(:update_repository) }
|
|
|
|
it { should respond_to(:destroy_repository) }
|
2011-10-08 21:36:38 +00:00
|
|
|
it { should respond_to(:public?) }
|
|
|
|
it { should respond_to(:private?) }
|
|
|
|
it { should respond_to(:url_to_repo) }
|
|
|
|
it { should respond_to(:path_to_repo) }
|
|
|
|
it { should respond_to(:valid_repo?) }
|
|
|
|
it { should respond_to(:repo_exists?) }
|
|
|
|
it { should respond_to(:repo) }
|
|
|
|
it { should respond_to(:tags) }
|
|
|
|
it { should respond_to(:commit) }
|
2011-12-14 16:38:52 +00:00
|
|
|
it { should respond_to(:commits_between) }
|
2011-10-08 21:36:38 +00:00
|
|
|
end
|
|
|
|
|
2011-12-05 07:43:53 +00:00
|
|
|
it "should not allow 'gitolite-admin' as repo name" do
|
2011-10-21 18:40:36 +00:00
|
|
|
should allow_value("blah").for(:path)
|
2011-12-05 07:43:53 +00:00
|
|
|
should_not allow_value("gitolite-admin").for(:path)
|
2011-10-21 18:40:36 +00:00
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
it "should return valid url to repo" do
|
2011-10-08 21:36:38 +00:00
|
|
|
project = Project.new(:path => "somewhere")
|
|
|
|
project.url_to_repo.should == "git@localhost:somewhere.git"
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
it "should return path to repo" do
|
2011-10-08 21:36:38 +00:00
|
|
|
project = Project.new(:path => "somewhere")
|
2011-10-09 19:55:06 +00:00
|
|
|
project.path_to_repo.should == File.join(Rails.root, "tmp", "tests", "somewhere")
|
2011-10-08 21:36:38 +00:00
|
|
|
end
|
|
|
|
|
2011-12-14 16:38:52 +00:00
|
|
|
it "returns the full web URL for this repo" do
|
|
|
|
project = Project.new(:code => "somewhere")
|
|
|
|
project.web_url.should == "#{GIT_HOST['host']}/somewhere"
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
describe :valid_repo? do
|
|
|
|
it "should be valid repo" do
|
2011-10-08 21:36:38 +00:00
|
|
|
project = Factory :project
|
2011-10-26 13:46:25 +00:00
|
|
|
project.valid_repo?.should be_true
|
2011-10-08 21:36:38 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it "should be invalid repo" do
|
|
|
|
project = Project.new(:name => "ok_name", :path => "/INVALID_PATH/", :code => "NEOK")
|
|
|
|
project.valid_repo?.should be_false
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2011-12-14 16:38:52 +00:00
|
|
|
describe "web hooks" do
|
|
|
|
let(:project) { Factory :project }
|
|
|
|
|
|
|
|
context "with no web hooks" do
|
|
|
|
it "raises no errors" do
|
|
|
|
lambda {
|
|
|
|
project.execute_web_hooks('oldrev', 'newrev', 'ref')
|
|
|
|
}.should_not raise_error
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "with web hooks" do
|
|
|
|
before do
|
|
|
|
@webhook = Factory(:web_hook)
|
|
|
|
@webhook_2 = Factory(:web_hook)
|
|
|
|
project.web_hooks << [@webhook, @webhook_2]
|
|
|
|
end
|
|
|
|
|
|
|
|
it "executes multiple web hook" do
|
|
|
|
@webhook.should_receive(:execute).once
|
|
|
|
@webhook_2.should_receive(:execute).once
|
|
|
|
|
2011-12-15 09:33:20 +00:00
|
|
|
project.execute_web_hooks('oldrev', 'newrev', 'refs/heads/master')
|
2011-12-14 16:38:52 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2011-12-15 09:33:20 +00:00
|
|
|
context "does not execute web hooks" do
|
|
|
|
before do
|
|
|
|
@webhook = Factory(:web_hook)
|
|
|
|
project.web_hooks << [@webhook]
|
|
|
|
end
|
|
|
|
|
|
|
|
it "when pushing a branch for the first time" do
|
|
|
|
@webhook.should_not_receive(:execute)
|
2011-12-26 09:10:21 +00:00
|
|
|
project.execute_web_hooks('00000000000000000000000000000000', 'newrev', 'refs/heads/master')
|
2011-12-15 09:33:20 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it "when pushing tags" do
|
|
|
|
@webhook.should_not_receive(:execute)
|
|
|
|
project.execute_web_hooks('oldrev', 'newrev', 'refs/tags/v1.0.0')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "when pushing new branches" do
|
|
|
|
|
|
|
|
end
|
|
|
|
|
2011-12-14 16:38:52 +00:00
|
|
|
context "when gathering commit data" do
|
|
|
|
before do
|
|
|
|
@oldrev, @newrev, @ref = project.fresh_commits(2).last.sha, project.fresh_commits(2).first.sha, 'refs/heads/master'
|
|
|
|
@commit = project.fresh_commits(2).first
|
|
|
|
|
|
|
|
# Fill nil/empty attributes
|
|
|
|
project.description = "This is a description"
|
|
|
|
|
|
|
|
@data = project.web_hook_data(@oldrev, @newrev, @ref)
|
|
|
|
end
|
|
|
|
|
|
|
|
subject { @data }
|
|
|
|
|
|
|
|
it { should include(before: @oldrev) }
|
|
|
|
it { should include(after: @newrev) }
|
|
|
|
it { should include(ref: @ref) }
|
|
|
|
|
|
|
|
context "with repository data" do
|
|
|
|
subject { @data[:repository] }
|
|
|
|
|
|
|
|
it { should include(name: project.name) }
|
|
|
|
it { should include(url: project.web_url) }
|
|
|
|
it { should include(description: project.description) }
|
|
|
|
it { should include(homepage: project.web_url) }
|
|
|
|
it { should include(private: project.private?) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context "with commits" do
|
|
|
|
subject { @data[:commits] }
|
|
|
|
|
|
|
|
it { should be_an(Array) }
|
|
|
|
it { should have(1).element }
|
|
|
|
|
|
|
|
context "the commit" do
|
|
|
|
subject { @data[:commits].first }
|
|
|
|
|
|
|
|
it { should include(id: @commit.id) }
|
|
|
|
it { should include(message: @commit.safe_message) }
|
|
|
|
it { should include(timestamp: @commit.date.xmlschema) }
|
|
|
|
it { should include(url: "http://localhost/#{project.code}/commits/#{@commit.id}") }
|
|
|
|
|
|
|
|
context "with a author" do
|
|
|
|
subject { @data[:commits].first[:author] }
|
|
|
|
|
|
|
|
it { should include(name: @commit.author_name) }
|
|
|
|
it { should include(email: @commit.author_email) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2011-11-15 08:34:30 +00:00
|
|
|
describe "updates" do
|
2011-11-02 20:14:03 +00:00
|
|
|
let(:project) { Factory :project }
|
|
|
|
|
2011-11-15 08:34:30 +00:00
|
|
|
before do
|
2011-11-02 20:14:03 +00:00
|
|
|
@issue = Factory :issue,
|
|
|
|
:project => project,
|
|
|
|
:author => Factory(:user),
|
|
|
|
:assignee => Factory(:user)
|
|
|
|
|
|
|
|
@note = Factory :note,
|
|
|
|
:project => project,
|
|
|
|
:author => Factory(:user)
|
|
|
|
|
|
|
|
@commit = project.fresh_commits(1).first
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "return commit, note & issue" do
|
|
|
|
it { project.updates(3).count.should == 3 }
|
|
|
|
it { project.updates(3).last.id.should == @commit.id }
|
|
|
|
it { project.updates(3).include?(@issue).should be_true }
|
|
|
|
it { project.updates(3).include?(@note).should be_true }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2011-11-15 08:34:30 +00:00
|
|
|
describe "last_activity" do
|
2011-11-02 20:14:03 +00:00
|
|
|
let(:project) { Factory :project }
|
|
|
|
|
2011-11-15 08:34:30 +00:00
|
|
|
before do
|
2011-11-02 20:14:03 +00:00
|
|
|
@note = Factory :note,
|
|
|
|
:project => project,
|
|
|
|
:author => Factory(:user)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { project.last_activity.should == @note }
|
|
|
|
it { project.last_activity_date.to_s.should == @note.created_at.to_s }
|
|
|
|
end
|
|
|
|
|
2011-11-15 08:34:30 +00:00
|
|
|
describe "fresh commits" do
|
2011-11-02 20:14:03 +00:00
|
|
|
let(:project) { Factory :project }
|
|
|
|
|
|
|
|
it { project.fresh_commits(3).count.should == 3 }
|
|
|
|
it { project.fresh_commits.first.id.should == "2fb376f61875b58bceee0492e270e9c805294b1a" }
|
|
|
|
it { project.fresh_commits.last.id.should == "0dac878dbfe0b9c6104a87d65fe999149a8d862c" }
|
|
|
|
end
|
|
|
|
|
2011-12-14 16:38:52 +00:00
|
|
|
describe "commits_between" do
|
|
|
|
let(:project) { Factory :project }
|
|
|
|
|
|
|
|
subject do
|
|
|
|
commits = project.commits_between("a6d1d4aca0c85816ddfd27d93773f43a31395033",
|
|
|
|
"2fb376f61875b58bceee0492e270e9c805294b1a")
|
|
|
|
commits.map { |c| c.id }
|
|
|
|
end
|
|
|
|
|
|
|
|
it { should have(2).elements }
|
|
|
|
it { should include("2fb376f61875b58bceee0492e270e9c805294b1a") }
|
|
|
|
it { should include("4571e226fbcd7be1af16e9fa1e13b7ac003bebdf") }
|
|
|
|
it { should_not include("a6d1d4aca0c85816ddfd27d93773f43a31395033") }
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
describe "Git methods" do
|
2011-10-08 21:36:38 +00:00
|
|
|
let(:project) { Factory :project }
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
describe :repo do
|
|
|
|
it "should return valid repo" do
|
2011-10-08 21:36:38 +00:00
|
|
|
project.repo.should be_kind_of(Grit::Repo)
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
it "should return nil" do
|
2011-10-08 21:36:38 +00:00
|
|
|
lambda { Project.new(:path => "invalid").repo }.should raise_error(Grit::NoSuchPathError)
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
it "should return nil" do
|
2011-10-08 21:36:38 +00:00
|
|
|
lambda { Project.new.repo }.should raise_error(TypeError)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
describe :commit do
|
|
|
|
it "should return first head commit if without params" do
|
2011-10-08 21:36:38 +00:00
|
|
|
project.commit.id.should == project.repo.commits.first.id
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
it "should return valid commit" do
|
2011-10-08 21:36:38 +00:00
|
|
|
project.commit(ValidCommit::ID).should be_valid_commit
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
it "should return nil" do
|
2011-10-08 21:36:38 +00:00
|
|
|
project.commit("+123_4532530XYZ").should be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
describe :tree do
|
|
|
|
before do
|
2011-10-08 21:36:38 +00:00
|
|
|
@commit = project.commit(ValidCommit::ID)
|
|
|
|
end
|
|
|
|
|
2011-10-26 13:46:25 +00:00
|
|
|
it "should raise error w/o arguments" do
|
2011-10-08 21:36:38 +00:00
|
|
|
lambda { project.tree }.should raise_error
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should return root tree for commit" do
|
|
|
|
tree = project.tree(@commit)
|
|
|
|
tree.contents.size.should == ValidCommit::FILES_COUNT
|
|
|
|
tree.contents.map(&:name).should == ValidCommit::FILES
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should return root tree for commit with correct path" do
|
|
|
|
tree = project.tree(@commit, ValidCommit::C_FILE_PATH)
|
|
|
|
tree.contents.map(&:name).should == ValidCommit::C_FILES
|
|
|
|
end
|
|
|
|
|
|
|
|
it "should return root tree for commit with incorrect path" do
|
|
|
|
project.tree(@commit, "invalid_path").should be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
# == Schema Information
|
|
|
|
#
|
|
|
|
# Table name: projects
|
|
|
|
#
|
2011-12-18 14:09:16 +00:00
|
|
|
# id :integer not null, primary key
|
|
|
|
# name :string(255)
|
|
|
|
# path :string(255)
|
|
|
|
# description :text
|
|
|
|
# created_at :datetime
|
|
|
|
# updated_at :datetime
|
|
|
|
# private_flag :boolean default(TRUE), not null
|
|
|
|
# code :string(255)
|
|
|
|
# owner_id :integer
|
|
|
|
# default_branch :string(255) default("master"), not null
|
2011-10-08 21:36:38 +00:00
|
|
|
#
|
|
|
|
|