Added tests to verify that cpvirtualenv, lsvirtualenv, and mkproject work with spaces in env names.

Author: jessamynsmith

tests/test_cp.sh

@@ -33,10 +33,10 @@ test_new_env_activated () {
     assertTrue virtualenvwrapper_verify_active_environment
 }
 
-test_virtual_env_variable () {
-    mkvirtualenv "source" >/dev/null 2>&1
-    cpvirtualenv "source" "destination" >/dev/null 2>&1
-    assertSame "Wrong virtualenv name" "destination" $(basename "$VIRTUAL_ENV")
+test_virtual_env_variable_space_in_name () {
+    mkvirtualenv " space source" >/dev/null 2>&1
+    cpvirtualenv " space source" " space destination" >/dev/null 2>&1
+    assertSame "Wrong virtualenv name" " space destination" "$(basename "$VIRTUAL_ENV")"
     assertTrue "$WORKON_HOME not in $VIRTUAL_ENV" "echo $VIRTUAL_ENV | grep -q $WORKON_HOME"
 }
 

tests/test_ls.sh

@@ -25,8 +25,8 @@ test_get_site_packages_dir () {
     deactivate
 }
 
-test_lssitepackages () {
-    mkvirtualenv "lssitepackagestest" >/dev/null 2>&1
+test_lssitepackages_space_in_name () {
+    mkvirtualenv " space lssitepackagestest" >/dev/null 2>&1
     contents="$(lssitepackages)"    
     assertTrue "did not find easy_install in site-packages" "echo $contents | grep -q easy_install"
     deactivate

virtualenvwrapper.sh

@@ -945,9 +945,9 @@ function cpvirtualenv {
         echo "Please provide a valid virtualenv to copy."
         return 1
     else
-        # see if its already in workon
+        # see if it's already in workon
         if [ ! -e "$WORKON_HOME/$src_name" ]; then
-            # so its a virtualenv we are importing
+            # so it's a virtualenv we are importing
             # make sure we have a full path
             # and get the name
             src="$(virtualenvwrapper_expandpath "$src_name")"