jbox-gitolite 1.1.2

data/spec/dirty_proxy_spec.rb

@@ -0,0 +1,63 @@
+require 'gitolite/dirty_proxy'
+require 'spec_helper'
+
+describe Gitolite::DirtyProxy do
+
+  it "should create a new instance given valid attributes" do
+    Gitolite::DirtyProxy.new([]).should_not be_nil
+  end
+
+  let(:target) { ['foo', 'bar'] }
+  let(:proxy) { Gitolite::DirtyProxy.new(target) }
+
+  describe 'delegating to the target object' do
+    it 'should act as instance of the target' do
+      proxy.should be_instance_of target.class
+    end
+
+    it 'should respond to all methods of the target' do
+      proxy.should respond_to(*target.methods)
+    end
+
+    it 'should equal the target' do
+      proxy.should equal target
+    end
+  end
+
+  describe 'dirty checking methods' do
+    it 'should respond to clean_up!' do
+      proxy.respond_to?(:clean_up!).should be_true
+    end
+
+    it 'should respond to dirty?' do
+      proxy.respond_to?(:dirty?).should be_true
+    end
+
+    context 'when just initialized' do
+      it 'should be clean' do
+        proxy.dirty?.should be_false
+      end
+    end
+
+    shared_examples 'dirty? clean_up!' do
+      it 'should be dirty' do
+        proxy.dirty?.should be_true
+      end
+
+      it 'should be clean again after clean_up!' do
+        proxy.clean_up!
+        proxy.dirty?.should be_false
+      end
+    end
+
+    context 'when target object has changed directly' do
+      before(:each) { proxy << 'baz' }
+      include_examples 'dirty? clean_up!'
+    end
+
+    context 'when target object has changed in depth' do
+      before(:each) { proxy[0] << 'ooo' }
+      include_examples 'dirty? clean_up!'
+    end
+  end
+end

data/spec/gitolite_admin_spec.rb

@@ -0,0 +1,4 @@
+require 'gitolite/gitolite_admin'
+
+describe Gitolite::GitoliteAdmin do
+end

data/spec/group_spec.rb

@@ -0,0 +1,126 @@
+require 'gitolite/config/group'
+require 'spec_helper'
+
+describe Gitolite::Config::Group do
+  describe "#new" do
+    it "should create a new group with an empty list of users" do
+      group = Gitolite::Config::Group.new("testgroup")
+      group.users.empty?.should be true
+      group.name.should == "testgroup"
+    end
+
+    it "should create a new group with a name containing #{Gitolite::Config::Group::PREPEND_CHAR}" do
+      name = "#{Gitolite::Config::Group::PREPEND_CHAR}testgroup"
+      group = Gitolite::Config::Group.new(name)
+      group.name.should == "testgroup"
+    end
+  end
+
+  describe "users" do
+    before :each do
+      @group = Gitolite::Config::Group.new('testgroup')
+    end
+
+    describe "#add_user" do
+      it "should allow adding one user with a string" do
+        @group.add_user("bob")
+        @group.size.should == 1
+        @group.users.first.should == "bob"
+      end
+
+      it "should allow adding one user with a symbol" do
+        @group.add_user(:bob)
+        @group.size.should == 1
+        @group.users.first.should == "bob"
+      end
+
+      it "should not add the same user twice" do
+        @group.add_user("bob")
+        @group.size.should == 1
+        @group.add_user(:bob)
+        @group.size.should == 1
+        @group.users.first.should == "bob"
+      end
+
+      it "should maintain users in sorted order" do
+        @group.add_user("susan")
+        @group.add_user("peyton")
+        @group.add_user("bob")
+        @group.users.first.should == "bob"
+        @group.users.last.should == "susan"
+      end
+    end
+
+    describe "#add_users" do
+      it "should allow adding multiple users at once" do
+        @group.add_users("bob", "joe", "sue", "sam", "dan")
+        @group.size.should == 5
+      end
+
+      it "should allow adding multiple users in nested arrays" do
+        @group.add_users(["bob", "joe", ["sam", "sue", "dan"]], "bill")
+        @group.size.should == 6
+      end
+
+      it "should allow adding users of symbols and strings" do
+        @group.add_users("bob", :joe, :sue, "sam")
+        @group.size.should == 4
+      end
+
+      it "should not add the same user twice" do
+        @group.add_users("bob", :bob, "bob", "sam")
+        @group.size.should == 2
+      end
+    end
+
+    describe "#rm_user" do
+      before :each do
+        @group.add_users("bob", "joe", "susan", "sam", "alex")
+      end
+
+      it "should support removing a user via a String" do
+        @group.rm_user("bob")
+        @group.size.should == 4
+      end
+
+      it "should support removing a user via a Symbol" do
+        @group.rm_user(:bob)
+        @group.size.should == 4
+      end
+    end
+
+    describe "#empty!" do
+      it "should clear all users from the group" do
+        @group.add_users("bob", "joe", "sue", "jim")
+        @group.size.should == 4
+        @group.empty!
+        @group.size.should == 0
+      end
+    end
+
+    describe "#size" do
+      it "should reflect how many users are in the group" do
+        @group.add_users("bob", "joe", "sue", "jim")
+        @group.users.length.should == @group.size
+      end
+    end
+
+    describe "#has_user?" do
+      it "should search for a user via a String" do
+        @group.add_user("bob")
+        @group.has_user?("bob").should be true
+      end
+
+      it "should search for a user via a Symbol" do
+        @group.add_user(:bob)
+        @group.has_user?(:bob).should be true
+      end
+    end
+  end
+
+  describe "#to_s" do
+    group = Gitolite::Config::Group.new("testgroup")
+    group.add_users("bob", "joe", "sam", "sue")
+    group.to_s.should == "@testgroup          = bob joe sam sue\n" #10 spaces after @testgroup
+  end
+end

data/spec/keys/bob-ins@zilla-site.com@desktop.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAsRofYfjAUdw0McxGpLTxBbZyKN05HGY19ZIjEQmBPUe2Skt1i/SLwIYzTzKv6vEgAdT9SwmCsO/jpDY/ZM+MhWy4okBisn04yIHCW10q4+YNo2WatzttEa1W+hC58nKtoq/0HkvPVdoeOZxcNtpjvgvkrXH2zdmX7xnA1AAWtqRNkRYWPKjiojkg92aqmJLISSIDeZs2wvXbFO3BJhkvyr3W60cinKhGBBscvdCYUi5pb9dXIFhEEqRf1JkT5CEmF3p4GqSt4/L79nR0LV45grS4NbcN+fCnjWZ8PQhmJ+WPLKkydgR1YvobeY7zDHdHJXWLhsPa+SjwI3WfzrB+GQ== bob@zilla.com

data/spec/keys/bob.joe@test.zilla.com@desktop.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAsRofYfjAUdw0McxGpLTxBbZyKN05HGY19ZIjEQmBPUe2Skt1i/SLwIYzTzKv6vEgAdT9SwmCsO/jpDY/ZM+MhWy4okBisn04yIHCW10q4+YNo2WatzttEa1W+hC58nKtoq/0HkvPVdoeOZxcNtpjvgvkrXH2zdmX7xnA1AAWtqRNkRYWPKjiojkg92aqmJLISSIDeZs2wvXbFO3BJhkvyr3W60cinKhGBBscvdCYUi5pb9dXIFhEEqRf1JkT5CEmF3p4GqSt4/L79nR0LV45grS4NbcN+fCnjWZ8PQhmJ+WPLKkydgR1YvobeY7zDHdHJXWLhsPa+SjwI3WfzrB+GQ== bob@zilla.com

data/spec/keys/bob.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEA6EFlh48tzCnepmggd09sUEM4m1zH3Fs/X6XWm1MAkEnMsD5hFGjkcNabDM8vq9zIRZ05YC6Gxo2plstAf+X4Y636+hyFvbDONB9mRP7DxJhFRaBScSFH60jeTz4ue2ExH3xA1JkaHMcV5vooUqG4BW8Vy/sz8wt/s0aIg9xqkrPOnfvqwunZ/zFUNyL8tC1HY3zGUkRzEVd2yRKaI+DGyRsh8HuYIb2X3NQ0YsU3uGGud7ObmxDbM7WGniyxRVK3lYCvgnTjvdPGi7Xx9QNQz53zLFbklGPZSfpFFHS84qR0Rd/+MnpT50FODhTmXHZtZF1eik09z63GW3YVt4PGoQ== bob@zilla.com

data/spec/keys/bob@desktop.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAwZc2kw4Bc8Kksp0XBiDR/rDrUAmv2THB9OVbRwJ7TEelU8OFJqCx1syjdjAgS+LE3K5d5xtDMYRlxE2hkl8V6EySY9Q35qWqaqhmS8erZap0qiCWM5vlzxWFKOpnhQdNbNjA7a7FSTDT6ThOC9sTCJfMvNdBiiUUGGDE48AvtwtF3er24h8w7J5623AICfOdhfPAFavTO6/QpeHRe8QOkbBJ2oaJDer4rDetgtBUgHXh3jANgD2ICRFd9l5eCozkDCISFJ6Xg2Eq8gMJ2DKj3BSzBpRPiAJ1YVsr/cBzGxRVRPS6Go485E2r7l7zAsE1t/t6eRUU4W5ZeopxD4GUXQ== bob@zilla.com

data/spec/keys/bob@foo-bar.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQDajoou61M6Fjebh2WEIgdCDf76YCVXTepYogBgE8/rd0LaPB0zdzwWjt+nDTJTx2ZMI+riCB8M7vzrXwG6zUTF49KrzQ4WEsV66yfEk+kyQ5WCx9L05seeFIhbYrk3TbEWUUWwCeXU3yj9cHiNwAxzWK3OwDcPbRu5zRk5JjPE491wrywmhn3klxy6BQcFS42qbci9DsNq9tTcfqJn+3m3s7JBVc2f58Blf45PAMlYUe8TH8up/vtUeg1NPysozKRu9+sUw8ChVUZOlA3tL9Zs76GQG2Q+fgXEV4eRAjF/jrmMT5vE7lHMkG0V5WEO7SPsFXoibmRLRgyRA2wQuS49 bob@zilla.com

data/spec/keys/bob@zilla.com.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAu1cY+0C5QV/0mcdobqS673BVlT7weW1lZKp83v8kTYHJJ8NLKixnPf4VePUoE5IMdmWMHNtAtSCG+X/Gdo/2VBHWbZ/O2mC2WSYH0u/nk+Bw0l30/vbV0wP3rcapY42BiMl/H0CPuxql99SBcVJoAaKkxJ4uvyR/hQ+mEmSIYz7mD11opCC6owmWm7R5Sdf2UGXzYXHwu5G9DdIE+QcmYbLkltCBHwa17syoM5QtetbHc/S/wbudrLclJXPHJ0xEtJMhGEHzNJndwYpOA4R61O+phIXIwdtdBeCAxAQyb9hf+qrOBaMfXyFn/WF0ov8hAnKDWWKg74e4ZHYJ2qPSRw== bob@zilla.com

data/spec/keys/bob@zilla.com@desktop.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAsRofYfjAUdw0McxGpLTxBbZyKN05HGY19ZIjEQmBPUe2Skt1i/SLwIYzTzKv6vEgAdT9SwmCsO/jpDY/ZM+MhWy4okBisn04yIHCW10q4+YNo2WatzttEa1W+hC58nKtoq/0HkvPVdoeOZxcNtpjvgvkrXH2zdmX7xnA1AAWtqRNkRYWPKjiojkg92aqmJLISSIDeZs2wvXbFO3BJhkvyr3W60cinKhGBBscvdCYUi5pb9dXIFhEEqRf1JkT5CEmF3p4GqSt4/L79nR0LV45grS4NbcN+fCnjWZ8PQhmJ+WPLKkydgR1YvobeY7zDHdHJXWLhsPa+SjwI3WfzrB+GQ== bob@zilla.com

data/spec/keys/jakub123.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAvtAkUG0e/dgqzb9Vtj6mspYCueII5U/gx3fnpIb2j8ng/ZJPPLYIb2UCwHdFBj/VZerKiJn6epJRi8qj4ddC4Z/LPNgjn06gbjAAubyb2Lki/XSvizpP6j4OtRnt/Vf17n1hyUKN0OAQmSaf1DoXsvWHpb64U7WxbY3Z3XLdJlqtIXW0vr3X13RKx6JZDGM2re/1ymZ5SJZT/KTruqtC42Yu4C2ktSa+JRh82+jALGy/5A/o1lsCziCSE/+iycVNM9hSfbwGTC1AVT1WO2BiQlduUL3tk4XyMEjlOXF2K4FobcBhLTrNtLc24vnlx5HkcBmuubq7x6g1pw4znwItgQ== jakub123@foo.net

data/spec/keys/jakub123@foo.net.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEArDPbrc0rP+Pux38Y14rMkgbPHo7iY89/VG80BH8dXVB1nisFHgY6zJdSPJ3zrEm1j81Hc7tZvd3FGjOKP8Ae02/J8O4WaVdB5pMo81YxQWu3eV6nD7x7AIeZEBauK7zV3eqyxTZ/V0oBd7e9VKMukxoiv+RXlXdItpH0md3bpQ3IreFjfaBrFnX0QLdGx0CjimKtMNOU7yKRlvuGcrvMoRtXSUzssAUtjmy8cRabBqZyJVOFrk6MhE1VpcAEE47Sb2ovFgfx5w4kA7mRAz6jO+MAM5go0lGoK26WrR9p/ZXuhmmdjQMPdSCssGQxni657vLp3F7PGV2AG35M58IwuQ==

data/spec/keys/joe-bob@god-zilla.com@desktop.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAsRofYfjAUdw0McxGpLTxBbZyKN05HGY19ZIjEQmBPUe2Skt1i/SLwIYzTzKv6vEgAdT9SwmCsO/jpDY/ZM+MhWy4okBisn04yIHCW10q4+YNo2WatzttEa1W+hC58nKtoq/0HkvPVdoeOZxcNtpjvgvkrXH2zdmX7xnA1AAWtqRNkRYWPKjiojkg92aqmJLISSIDeZs2wvXbFO3BJhkvyr3W60cinKhGBBscvdCYUi5pb9dXIFhEEqRf1JkT5CEmF3p4GqSt4/L79nR0LV45grS4NbcN+fCnjWZ8PQhmJ+WPLKkydgR1YvobeY7zDHdHJXWLhsPa+SjwI3WfzrB+GQ== bob@zilla.com

data/spec/keys/joe@sch.ool.edu.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAu9Y0aeYzlF/e/JZDYr4s/BmCnU5WATugdxnEDboTFgSt0rIZ3P+93m+wchGgP/GzspHa/gSIBBjDBwmzPlhdNieJdiNyJ0n0BvUWOBlI3wNQLU+wdqa1UwxQyPZNSplxyzhcsIgD6kPqYR9JE6Yga7QUnJdP/vAB+XtMoZlc+CVKjxLrjijSHX57WJ0bSXwKtYVHtgCGtJ1mmAZV/q64SR3mqEwB7WG0BTNwT0ruHRk4Zg/EpuutStYGne23/1FBSrfMirQCEneTiLOO/vAibWLzTRWSVyRT9YJYeH0Cp5dXnu3MqPL6+PhaP+iDtx6Vo/h2jeCJZpJUJb7dX9nl/Q== joe@sch.ool.edu

data/spec/keys/joe@sch.ool.edu@desktop.pub

@@ -0,0 +1 @@
+ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAt6+DWErgYKwCLQ+1DiD05iZc1FFgU2n+QXSqAOIjU5O2FKVgGU5JLOGydidOUK/vWRrPmS045Iv4BxTPIUPMLjn9lBjUkY7ZxfJb5aSW+9zwE/5/073DswRwHVZuxsxfScgqkwenTjN6bbM+qnASn+b6yxfRLmp86IzIXxKp2EiJ4Anw/9oUxre/K25WemTo8eUfKLfTS+z0KLSs1C9TJpO/MkDkHiv1S2wcnduA+nVcbSo7MuxrVDdQIQcDf0gKwwLwxtvKZXtUWC3Ji9kn3F86aRVBuMcl068t+UbbKvLc0BKo7RFMQ+nHXHosu1GJBwal+wWygbBksTk+92hRGQ== joe@sch.ool.edu

data/spec/repo_spec.rb

@@ -0,0 +1,204 @@
+require 'hashery'
+require 'gitolite/config/repo'
+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,2 @@
+require 'forgery'
+require 'tmpdir'

data/spec/ssh_key_spec.rb

@@ -0,0 +1,335 @@
+require 'spec_helper'
+require 'gitolite/ssh_key'
+include Gitolite
+
+describe Gitolite::SSHKey do
+  key_dir = File.join(File.dirname(__FILE__),'keys')
+
+  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@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 '#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)
+
+      tmpdir = Dir.tmpdir
+      s.to_file(tmpdir)
+
+      s.to_s.should == File.read(File.join(tmpdir, 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)
+
+      tmpdir = Dir.tmpdir
+
+
+      s.to_file(tmpdir).should == File.join(tmpdir, 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