gitolite-rugged 1.2.pre.devel

data/spec/repo_spec.rb

@@ -0,0 +1,202 @@
+require 'spec_helper'
+
+describe Gitolite::Config::Repo do
+  before(:each) do
+    @repo = Gitolite::Config::Repo.new("CoolRepo")
+  end
+
+  describe '#new' do
+    it 'should create a repo called "CoolRepo"' do
+      @repo.name.should == "CoolRepo"
+    end
+  end
+
+  describe "deny rules" do
+    it 'should maintain the order of rules for a repository' do
+      @repo.add_permission("RW+", "", "bob", "joe", "sam")
+      @repo.add_permission("R", "", "sue", "jen", "greg")
+      @repo.add_permission("-", "refs/tags/test[0-9]", "@students", "jessica")
+      @repo.add_permission("RW", "refs/tags/test[0-9]", "@teachers", "bill", "todd")
+      @repo.add_permission("R", "refs/tags/test[0-9]", "@profs")
+
+      @repo.permissions.length.should == 2
+      @repo.permissions[0].size.should == 2
+      @repo.permissions[1].size.should == 3
+    end
+  end
+
+  describe '#add_permission' do
+    it 'should allow adding a permission to the permissions list' do
+      @repo.add_permission("RW+")
+      @repo.permissions.length.should == 1
+      @repo.permissions.first.keys.first.should == "RW+"
+    end
+
+    it 'should allow adding a permission while specifying a refex' do
+      @repo.add_permission("RW+", "refs/heads/master")
+      @repo.permissions.length.should == 1
+      @repo.permissions.first.keys.first.should == "RW+"
+      @repo.permissions.first.values.last.first.first.should == "refs/heads/master"
+    end
+
+    it 'should allow specifying users individually' do
+      @repo.add_permission("RW+", "", "bob", "joe", "susan", "sam", "bill")
+      @repo.permissions.first["RW+"][""].should == %w[bob joe susan sam bill]
+    end
+
+    it 'should allow specifying users as an array' do
+      users = %w[bob joe susan sam bill]
+
+      @repo.add_permission("RW+", "", users)
+      @repo.permissions.first["RW+"][""].should == users
+    end
+
+    it 'should not allow adding an invalid permission via an InvalidPermissionError' do
+      expect {@repo.add_permission("BadPerm")}.to raise_error
+    end
+  end
+
+  describe "permissions" do
+    before(:each) do
+      @repo = Gitolite::Config::Repo.new("CoolRepo")
+    end
+
+    it 'should allow adding the permission C' do
+      @repo.add_permission("C", "", "bob")
+    end
+
+    it 'should allow adding the permission -' do
+      @repo.add_permission("-", "", "bob")
+    end
+
+    it 'should allow adding the permission R' do
+      @repo.add_permission("R", "", "bob")
+    end
+
+    it 'should allow adding the permission RM' do
+      @repo.add_permission("RM", "", "bob")
+    end
+
+    it 'should allow adding the permission RW' do
+      @repo.add_permission("RW", "", "bob")
+    end
+
+    it 'should allow adding the permission RWM' do
+      @repo.add_permission("RWM", "", "bob")
+    end
+
+    it 'should allow adding the permission RW+' do
+      @repo.add_permission("RW+", "", "bob")
+    end
+
+    it 'should allow adding the permission RW+M' do
+      @repo.add_permission("RW+M", "", "bob")
+    end
+
+    it 'should allow adding the permission RWC' do
+      @repo.add_permission("RWC", "", "bob")
+    end
+
+    it 'should allow adding the permission RWCM' do
+      @repo.add_permission("RWCM", "", "bob")
+    end
+
+    it 'should allow adding the permission RW+C' do
+      @repo.add_permission("RW+C", "", "bob")
+    end
+
+    it 'should allow adding the permission RW+CM' do
+      @repo.add_permission("RW+CM", "", "bob")
+    end
+
+    it 'should allow adding the permission RWD' do
+      @repo.add_permission("RWD", "", "bob")
+    end
+
+    it 'should allow adding the permission RWDM' do
+      @repo.add_permission("RWDM", "", "bob")
+    end
+
+    it 'should allow adding the permission RW+D' do
+      @repo.add_permission("RW+D", "", "bob")
+    end
+
+    it 'should allow adding the permission RW+DM' do
+      @repo.add_permission("RW+DM", "", "bob")
+    end
+
+    it 'should allow adding the permission RWCD' do
+      @repo.add_permission("RWCD", "", "bob")
+    end
+
+    it 'should allow adding the permission RWCDM' do
+      @repo.add_permission("RWCDM", "", "bob")
+    end
+
+    it 'should allow adding the permission RW+CD' do
+      @repo.add_permission("RW+CD", "", "bob")
+    end
+
+    it 'should allow adding the permission RW+CDM' do
+      @repo.add_permission("RW+CDM", "", "bob")
+    end
+  end
+
+  describe 'git config options' do
+    it 'should allow setting a git configuration option' do
+      email = "bob@zilla.com"
+      @repo.set_git_config("email", email).should == email
+    end
+
+    it 'should allow deletion of an existing git configuration option' do
+      email = "bob@zilla.com"
+      @repo.set_git_config("email", email)
+      @repo.unset_git_config("email").should == email
+    end
+
+  end
+
+  describe 'gitolite options' do
+    it 'should allow setting a gitolite option' do
+      master = "kenobi"
+      slaves = "one"
+      @repo.set_gitolite_option("mirror.master", master).should == master
+      @repo.set_gitolite_option("mirror.slaves", slaves).should == slaves
+      @repo.options.length.should == 2
+    end
+
+    it 'should allow deletion of an existing gitolite option' do
+      master = "kenobi"
+      slaves = "one"
+      @repo.set_gitolite_option("mirror.master", master)
+      @repo.set_gitolite_option("mirror.slaves", slaves)
+      @repo.options.length.should == 2
+      @repo.unset_gitolite_option("mirror.master").should == master
+      @repo.options.length.should == 1
+    end
+  end
+
+  describe 'permission management' do
+    it 'should combine two entries for the same permission and refex' do
+      users = %w[bob joe susan sam bill]
+      more_users = %w[sally peyton andrew erin]
+
+      @repo.add_permission("RW+", "", users)
+      @repo.add_permission("RW+", "", more_users)
+      @repo.permissions.first["RW+"][""].should == users.concat(more_users)
+      @repo.permissions.first["RW+"][""].length.should == 9
+    end
+
+    it 'should not list the same users twice for the same permission level' do
+      users = %w[bob joe susan sam bill]
+      more_users = %w[bob peyton andrew erin]
+      even_more_users = %w[bob jim wayne courtney]
+
+      @repo.add_permission("RW+", "", users)
+      @repo.add_permission("RW+", "", more_users)
+      @repo.add_permission("RW+", "", even_more_users)
+      @repo.permissions.first["RW+"][""].should == users.concat(more_users).concat(even_more_users).uniq!
+      @repo.permissions.first["RW+"][""].length.should == 11
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,21 @@
+require 'rubygems'
+require 'spork'
+require 'forgery'
+
+require 'simplecov'
+require 'simplecov-rcov'
+SimpleCov.formatter = SimpleCov::Formatter::RcovFormatter
+SimpleCov.start
+
+Spork.prefork do
+  # Loading more in this block will cause your tests to run faster. However,
+  # if you change any configuration or code from libraries loaded here, you'll
+  # need to restart spork for it take effect.
+
+end
+
+Spork.each_run do
+  # This code will be run each time you run your specs.
+  require File.expand_path('../../lib/gitolite', __FILE__)
+  include Gitolite
+end

data/spec/ssh_key_spec.rb

@@ -0,0 +1,355 @@
+require 'spec_helper'
+
+describe Gitolite::SSHKey do
+
+  key_dir    = File.join(File.dirname(__FILE__), 'fixtures', 'keys')
+  output_dir = '/tmp'
+  # output_dir = File.join(File.dirname(File.dirname(__FILE__)), 'tmp')
+
+  describe "#from_string" do
+    it 'should construct an SSH key from a string' do
+      key = File.join(key_dir, 'bob.pub')
+      key_string = File.read(key)
+      s = SSHKey.from_string(key_string, "bob")
+
+      s.owner.should == 'bob'
+      s.location.should == ""
+      s.blob.should == key_string.split[1]
+    end
+
+    it 'should raise an ArgumentError when an owner isnt specified' do
+      key_string = "not_a_real_key"
+      lambda { SSHKey.from_string(key_string) }.should raise_error
+    end
+
+    it 'should have a location when one is specified' do
+      key = File.join(key_dir, 'bob.pub')
+      key_string = File.read(key)
+      s = SSHKey.from_string(key_string, "bob", "kansas")
+
+      s.owner.should == 'bob'
+      s.location.should == "kansas"
+      s.blob.should == key_string.split[1]
+    end
+
+    it 'should raise an ArgumentError when owner is nil' do
+      lambda { SSHKey.from_string("bad_string", nil) }.should raise_error
+    end
+
+    it 'should raise an ArgumentError when we get an invalid SSHKey string' do
+      lambda { SSHKey.from_string("bad_string", "bob") }.should raise_error
+    end
+  end
+
+  describe "#from_file" do
+    it 'should load a key from a file' do
+      key = File.join(key_dir, 'bob.pub')
+      s = SSHKey.from_file(key)
+      key_string = File.read(key).split
+
+      s.owner.should == "bob"
+      s.blob.should == key_string[1]
+    end
+
+    it 'should load a key with a location from a file' do
+      key = File.join(key_dir, 'bob@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'bob'
+      s.location.should == 'desktop'
+    end
+
+    it 'should load a key with owner and location from a file' do
+      key = File.join(key_dir, 'joe-bob@god-zilla.com@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'joe-bob@god-zilla.com'
+      s.location.should == 'desktop'
+    end
+  end
+
+  describe '#owner' do
+    it 'owner should be bob for bob.pub' do
+      key = File.join(key_dir, 'bob.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'bob'
+    end
+
+    it 'owner should be bob for bob@desktop.pub' do
+      key = File.join(key_dir, 'bob@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'bob'
+    end
+
+    it 'owner should be bob@zilla.com for bob@zilla.com.pub' do
+      key = File.join(key_dir, 'bob@zilla.com.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'bob@zilla.com'
+    end
+
+    it "owner should be joe-bob@god-zilla.com for joe-bob@god-zilla.com@desktop.pub" do
+      key = File.join(key_dir, 'joe-bob@god-zilla.com@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'joe-bob@god-zilla.com'
+    end
+
+    it "owner should be bob.joe@test.zilla.com for bob.joe@test.zilla.com@desktop.pub" do
+      key = File.join(key_dir, 'bob.joe@test.zilla.com@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'bob.joe@test.zilla.com'
+    end
+
+    it "owner should be bob+joe@test.zilla.com for bob+joe@test.zilla.com@desktop.pub" do
+      key = File.join(key_dir, 'bob+joe@test.zilla.com@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'bob+joe@test.zilla.com'
+    end
+
+    it 'owner should be bob@zilla.com for bob@zilla.com@desktop.pub' do
+      key = File.join(key_dir, 'bob@zilla.com@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'bob@zilla.com'
+    end
+
+    it 'owner should be jakub123 for jakub123.pub' do
+      key = File.join(key_dir, 'jakub123.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'jakub123'
+    end
+
+    it 'owner should be jakub123@foo.net for jakub123@foo.net.pub' do
+      key = File.join(key_dir, 'jakub123@foo.net.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'jakub123@foo.net'
+    end
+
+    it 'owner should be joe@sch.ool.edu for joe@sch.ool.edu' do
+      key = File.join(key_dir, 'joe@sch.ool.edu.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'joe@sch.ool.edu'
+    end
+
+    it 'owner should be joe@sch.ool.edu for joe@sch.ool.edu@desktop.pub' do
+      key = File.join(key_dir, 'joe@sch.ool.edu@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == 'joe@sch.ool.edu'
+    end
+  end
+
+  describe '#location' do
+    it 'location should be "" for bob.pub' do
+      key = File.join(key_dir, 'bob.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == ''
+    end
+
+    it 'location should be "desktop" for bob@desktop.pub' do
+      key = File.join(key_dir, 'bob@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == 'desktop'
+    end
+
+    it 'location should be "" for bob@zilla.com.pub' do
+      key = File.join(key_dir, 'bob@zilla.com.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == ''
+    end
+
+    it 'location should be "desktop" for bob@zilla.com@desktop.pub' do
+      key = File.join(key_dir, 'bob@zilla.com@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == 'desktop'
+    end
+
+    it 'location should be "" for jakub123.pub' do
+      key = File.join(key_dir, 'jakub123.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == ''
+    end
+
+    it 'location should be "" for jakub123@foo.net.pub' do
+      key = File.join(key_dir, 'jakub123@foo.net.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == ''
+    end
+
+    it 'location should be "" for joe@sch.ool.edu' do
+      key = File.join(key_dir, 'joe@sch.ool.edu.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == ''
+    end
+
+    it 'location should be "desktop" for joe@sch.ool.edu@desktop.pub' do
+      key = File.join(key_dir, 'joe@sch.ool.edu@desktop.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == 'desktop'
+    end
+
+    it 'location should be "foo-bar" for bob@foo-bar.pub' do
+      key = File.join(key_dir, 'bob@foo-bar.pub')
+      s = SSHKey.from_file(key)
+      s.location.should == 'foo-bar'
+    end
+  end
+
+  describe '#keys' do
+    it 'should load ssh key properly' do
+      key = File.join(key_dir, 'bob.pub')
+      s = SSHKey.from_file(key)
+      parts = File.read(key).split #should get type, blob, email
+
+      s.type.should == parts[0]
+      s.blob.should == parts[1]
+      s.email.should == parts[2]
+    end
+  end
+
+  describe '#email' do
+    it 'should use owner if email is missing' do
+      key = File.join(key_dir, 'jakub123@foo.net.pub')
+      s = SSHKey.from_file(key)
+      s.owner.should == s.email
+    end
+  end
+
+  describe '#new' do
+    it 'should create a valid ssh key' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+
+      s = SSHKey.new(type, blob, email)
+
+      s.to_s.should == [type, blob, email].join(' ')
+      s.owner.should == email
+    end
+
+    it 'should create a valid ssh key while specifying an owner' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+      owner = Forgery::Name.first_name
+
+      s = SSHKey.new(type, blob, email, owner)
+
+      s.to_s.should == [type, blob, email].join(' ')
+      s.owner.should == owner
+    end
+
+    it 'should create a valid ssh key while specifying an owner and location' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+      owner = Forgery::Name.first_name
+      location = Forgery::Name.location
+
+      s = SSHKey.new(type, blob, email, owner, location)
+
+      s.to_s.should == [type, blob, email].join(' ')
+      s.owner.should == owner
+      s.location.should == location
+    end
+  end
+
+  describe '#hash' do
+    it 'should have two hash equalling one another' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+      owner = Forgery::Name.first_name
+      location = Forgery::Name.location
+
+      hash_test = [owner, location, type, blob, email].hash
+      s = SSHKey.new(type, blob, email, owner, location)
+
+      s.hash.should == hash_test
+    end
+  end
+
+  describe '#filename' do
+    it 'should create a filename that is the <email>.pub' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+
+      s = SSHKey.new(type, blob, email)
+
+      s.filename.should == "#{email}.pub"
+    end
+
+    it 'should create a filename that is the <owner>.pub' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+      owner = Forgery::Name.first_name
+
+      s = SSHKey.new(type, blob, email, owner)
+
+      s.filename.should == "#{owner}.pub"
+    end
+
+    it 'should create a filename that is the <email>@<location>.pub' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+      location = Forgery::Basic.text(:at_least => 8, :at_most => 15)
+
+      s = SSHKey.new(type, blob, email, nil, location)
+
+      s.filename.should == "#{email}@#{location}.pub"
+    end
+
+    it 'should create a filename that is the <owner>@<location>.pub' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+      owner = Forgery::Name.first_name
+      location = Forgery::Basic.text(:at_least => 8, :at_most => 15)
+
+      s = SSHKey.new(type, blob, email, owner, location)
+
+      s.filename.should == "#{owner}@#{location}.pub"
+    end
+  end
+
+  describe '#to_file' do
+    it 'should write a "valid" SSH public key to the file system' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+      owner = Forgery::Name.first_name
+      location = Forgery::Basic.text(:at_least => 8, :at_most => 15)
+
+      s = SSHKey.new(type, blob, email, owner, location)
+
+      ## write file
+      s.to_file(output_dir)
+
+      ## compare raw string with written file
+      s.to_s.should == File.read(File.join(output_dir, s.filename))
+    end
+
+    it 'should return the filename written' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+      owner = Forgery::Name.first_name
+      location = Forgery::Basic.text(:at_least => 8, :at_most => 15)
+
+      s = SSHKey.new(type, blob, email, owner, location)
+
+      s.to_file(output_dir).should == File.join(output_dir, s.filename)
+    end
+  end
+
+  describe '==' do
+    it 'should have two keys equalling one another' do
+      type = "ssh-rsa"
+      blob = Forgery::Basic.text(:at_least => 372, :at_most => 372)
+      email = Forgery::Internet.email_address
+
+      s1 = SSHKey.new(type, blob, email)
+      s2 = SSHKey.new(type, blob, email)
+
+      s1.should == s2
+    end
+  end
+end