check return code from virtualenvwrapper_verify_workon_home everywhere and return an error code if validation fails

Author: dhellmann

test.sh

@@ -45,11 +45,23 @@ deactivate
 
 echo
 echo "LISTING ENVIRONMENTS"
-workon
+envs=`workon | tr '\n' ' '`
+if [ "$envs" = "env1 env2 " ]
+then
+    echo "PASS"
+else
+    echo "FAIL: \""$envs\"""
+fi
 
 echo
 echo "REMOVING ENVIRONMENTS"
 rmvirtualenv "env1"
 rmvirtualenv "env2"
 
 rm -rf $WORKON_HOME
+
+echo
+echo "MISSING WORKON_HOME"
+workon && echo "Failed to detect missing dir" || echo "PASS"
+mkvirtualenv foo && echo "Failed to detect missing dir" || echo "PASS"
+rmvirtualenv foo && echo "Failed to detect missing dir" || echo "PASS"

virtualenvwrapper_bashrc

@@ -82,7 +82,7 @@ function virtualenvwrapper_run_hook () {
 #
 function mkvirtualenv () {
     eval "envname=\$$#"
-    virtualenvwrapper_verify_workon_home
+    virtualenvwrapper_verify_workon_home || return 1
     (cd "$WORKON_HOME"; 
         virtualenv "$@"; 
         virtualenvwrapper_run_hook "./premkvirtualenv" "$envname"
@@ -94,7 +94,7 @@ function mkvirtualenv () {
 # Remove an environment, in the WORKON_HOME.
 function rmvirtualenv () {
     typeset env_name="$1"
-    virtualenvwrapper_verify_workon_home
+    virtualenvwrapper_verify_workon_home || return 1
     if [ "$env_name" = "" ]
     then
         echo "Please specify an enviroment."
@@ -114,7 +114,7 @@ function rmvirtualenv () {
 
 # List the available environments.
 function virtualenvwrapper_show_workon_options () {
-    virtualenvwrapper_verify_workon_home
+    virtualenvwrapper_verify_workon_home || return 1
 	find "$WORKON_HOME" -maxdepth 1 -mindepth 1 -type d -exec basename '{}' \; | sort
 }