ensure the old ripenv is removed during a clone

lib/rip/env.rb

@@ -13,7 +13,7 @@ def create(env)
       if env.strip.empty?
         return "must give a ripenv to create"
       end
-      
+
       if env.strip =~ PRIVATE_ENV
         return "invalid environment name"
       end
@@ -108,6 +108,7 @@ def copy(new)
 
       if File.exists? ripfile = File.join(dest, "#{env}.ripenv")
         FileUtils.cp ripfile, File.join(dest, "#{new}.ripenv")
+        FileUtils.rm ripfile
       end
 
       use new

test/env_test.rb

@@ -36,7 +36,7 @@
     assert_equal 'must give a ripenv to create', Rip::Env.create("\t ")
     assert_equal "must give a ripenv to create", Rip::Env.call(:create)
   end
-  
+
   test 'fails if attempt to name env active' do
     assert_equal 'invalid environment name', Rip::Env.create('active')
   end
@@ -168,6 +168,13 @@
     assert_equal @name, Rip::Env.active
   end
 
+  test "properly renames the ripenv file" do
+    Rip::Env.copy(@name)
+
+    assert File.exists?(File.join(@ripenv, "#{@name}.ripenv"))
+    assert !File.exists?(File.join(@ripenv, "base.ripenv"))
+  end
+
   test "makes the new env a copy of the active env"
 
   test "fails if the new env exists" do
@@ -185,4 +192,4 @@
     assert_equal 'must give a ripenv to copy to', Rip::Env.copy("\t ")
     assert_equal "must give a ripenv to copy to", Rip::Env.call(:copy)
   end
-end
+end

test/test_helper.rb

@@ -47,6 +47,8 @@ def self.setup_with_fs(&block)
       Rip::Env.create('other')
       Rip::Setup.setup_ripenv(Rip.dir)
       Rip::Env.create('base')
+      FileUtils.touch(File.join(Rip.dir, 'other', 'other.ripenv'))
+      FileUtils.touch(File.join(Rip.dir, 'base', 'base.ripenv'))
       setup_block
     end