Added spec tests for env vars with whitespaces

openshift · Aug 21, 2013 · 4588853 · 4588853
1 parent bfe6019
commit 4588853
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/spec/rhc/commands/env_spec.rb b/spec/rhc/commands/env_spec.rb
@@ -115,6 +115,7 @@ def succeed_without_message(message="done")
      ['env', 'set', '_FOO=1', '--app', 'mock_app_0', '--noprompt', '--confirm'],
      ['env', 'set', 'FOO=BAR=BAZ', '--app', 'mock_app_0', '--noprompt', '--confirm'],
      ['env', 'set', 'FOO==', '--app', 'mock_app_0', '--noprompt', '--confirm'],
+     ['env', 'set', 'FOO=Test 1 2 3', '--app', 'mock_app_0', '--noprompt', '--confirm'],
      #['env', 'set', '--env', 'TEST_ENV_VAR="1"', '--app', 'mock_app_0', '--noprompt', '--confirm' ],
      #['env', 'set', '--env', "TEST_ENV_VAR='1'", '--app', 'mock_app_0', '--noprompt', '--confirm' ]
     ].each_with_index do |args, i|

diff --git a/spec/rhc/helpers_spec.rb b/spec/rhc/helpers_spec.rb
@@ -456,8 +456,15 @@ def options
     it { subject.collect_env_vars('FOO==').first.to_hash.should == { :name => 'FOO', :value => '=' } }
     it { subject.collect_env_vars('FOO=BAR=ZEE').first.to_hash.should == { :name => 'FOO', :value => 'BAR=ZEE' } }
     it { subject.collect_env_vars('foo25_=BAR=\][#%*').first.to_hash.should == { :name => 'foo25_', :value => 'BAR=\][#%*' } }
+    it { subject.collect_env_vars('FOO=Test 1 2 3').first.to_hash.should == { :name => 'FOO', :value => 'Test 1 2 3' } }
     it { subject.collect_env_vars('2FOO=BAR').empty?.should be_true }
     it { subject.collect_env_vars('FOO.2=BAR').empty?.should be_true }
+    it { subject.collect_env_vars('FOO BAR=ZEE').empty?.should be_true }
+    it { subject.collect_env_vars('FOO*BAR=ZEE').empty?.should be_true }
+    it { subject.collect_env_vars('FOO&BAR=ZEE').empty?.should be_true }
+    it { subject.collect_env_vars('FOO:BAR=ZEE').empty?.should be_true }
+    it { subject.collect_env_vars('FOO@BAR=ZEE').empty?.should be_true }
+    it { subject.collect_env_vars('FOO!BAR=ZEE').empty?.should be_true }
   end
 end