@@ -12,6 +12,10 @@ module Rip if env.strip.empty? return "must give a ripenv to create" end + + if env.strip =~ /^active$/i + return "invalid environment name" + end if File.exists? dir "#{env} exists" lib/rip/env.rb @@ -35,6 +35,10 @@ context "Creating a ripenv" do assert_equal 'must give a ripenv to create', Rip::Env.create("\t") assert_equal 'must give a ripenv to create', Rip::Env.create("\t ") end + + test 'fails if attempt to name env active' do + assert_equal 'invalid environment name', Rip::Env.create('active') + end end context "Using a ripenv" do test/env_test.rb 44e7ecc8971097af1bf2189f3a9c59abcdba53e7 Mark Turner mark@amerine.net http://github.com/Pistos/rip/commit/a7558b4d2717b6c66e0861f42564d1c3e18613f5 a7558b4d2717b6c66e0861f42564d1c3e18613f5 2009-06-13T23:38:00-07:00 2009-06-13T01:25:41-07:00 We can't have people naming their env's 'active'. It creates a self referencing symlink that breaks things. Signed-off-by: Chris Wanstrath <chris@ozmm.org> ad5c79e2871a404347248ebc8b6890cc1c6d41a5 Chris Wanstrath chris@ozmm.org