Adding #remove_remote to Git::Base (adding some tests too)

robertodecurnex · robertodecurnex · commit 35683a16e012 · 2013-04-15T12:07:56.000-03:00
diff --git a/lib/git/base.rb b/lib/git/base.rb
@@ -346,6 +346,13 @@ def add_remote(name, url, opts = {})
       Git::Remote.new(self, name)
     end
 
+    # removes a remote from this repository
+    #
+    # @git.remove_remote('scott_git')
+    def remove_remote(name)
+      self.lib.remote_remove(name)
+    end
+
     # returns an array of all Git::Tag objects for this repository
     def tags
       self.lib.tags.map { |r| tag(r) }
diff --git a/tests/units/test_remotes.rb b/tests/units/test_remotes.rb
@@ -15,16 +15,34 @@ def test_add_remote
       local.add_remote('testremote', remote)
 
       assert(!local.branches.map{|b| b.full}.include?('testremote/master'))
+      assert(local.remotes.map{|b| b.name}.include?('testremote'))
 
       local.add_remote('testremote2', remote, :fetch => true)
 
       assert(local.branches.map{|b| b.full}.include?('remotes/testremote2/master'))
+      assert(local.remotes.map{|b| b.name}.include?('testremote2'))
 
       local.add_remote('testremote3', remote, :track => 'master')
       
       assert(local.branches.map{|b| b.full}.include?('master')) #We actually a new branch ('test_track') on the remote and track that one intead. 
+      assert(local.remotes.map{|b| b.name}.include?('testremote3'))
     end 
   end
+
+  def test_remove_remote_remove
+    in_temp_dir do |path|
+      local = Git.clone(@wbare, 'local')
+      remote = Git.clone(@wbare, 'remote')
+      
+      local.add_remote('testremote', remote)
+
+      assert(local.remotes.map{|b| b.name}.include?('testremote'))
+ 
+      local.remove_remote('testremote')
+      
+      assert(!local.remotes.map{|b| b.name}.include?('testremote'))
+    end
+  end
   
   def test_remote_fun
     in_temp_dir do |path|
