spec/mocks_enabled_spec.rb in regrit-0.0.6 vs spec/mocks_enabled_spec.rb in regrit-0.1.0

@@ -5,10 +5,11 @@
   after { Regrit.disable_mock! }
 
   before { @uri = 'git://example.com/does/not/work.git' }
 
   context "(public)" do
+
     # This uri would not work if we weren't mocked
     subject { described_class.new(@uri) }
 
     it { should_not be_private_key_required }
 
@@ -25,28 +26,33 @@
     context "(mocked timeout)" do
       before { Regrit::Provider::Mock.timeout = true }
       it { should_not be_accessible }
 
       it "should raise TimeoutError" do
-        subject { described_class.new(@uri) }
-
         lambda { subject.refs }.should raise_error(Regrit::TimeoutError)
       end
     end
 
   end
 
   context "(private)" do
     # This uri would not work if we weren't mocked
+    before { @uri = '[e-mail]:engineyard/regrit.git' }
 
     context "(mocked accessible)" do
       before { Regrit::Provider::Mock.accessible = true }
 
+      it "would raise if I used it, but can still be requested about auth" do
+        described_class.new(@uri).should be_private_key_required
+      end
+
+      it "still raises on no key" do
+        lambda { described_class.new(@uri).refs }.should raise_error(Regrit::PrivateKeyRequired)
+      end
+
       it "still raises on bad key" do
-        pending("should we care if the key is blank in mock mode?") do
-          lambda { described_class.new(@uri, :private_key => '') }.should raise_error
-        end
+        lambda { described_class.new(@uri, :private_key => '').refs }.should raise_error(Regrit::PrivateKeyRequired)
       end
 
       context "with a key" do
         subject { described_class.new(@uri, :private_key => 'any key') }
         it { subject.should be_accessible }
@@ -54,13 +60,16 @@
     end
 
     context "(mocked inaccessible)" do
       before { Regrit::Provider::Mock.accessible = false }
 
+      it "would raise if I used it, but can still be requested about auth" do
+        described_class.new(@uri).should be_private_key_required
+      end
+
       it "still raises on bad key" do
-        pending("should we care if the key is blank in mock mode?") do
-          lambda { described_class.new(@uri, :private_key => '') }.should raise_error
-        end
+        lambda { described_class.new(@uri).refs }.should raise_error(Regrit::PrivateKeyRequired)
+        lambda { described_class.new(@uri, :private_key => '').refs }.should raise_error(Regrit::PrivateKeyRequired)
       end
 
       context "with a key" do
         subject { described_class.new(@uri, :private_key => private_key) }
         it { subject.should_not be_accessible }