Skip to content

Commit 04664ca

Browse files
committed
Added tests to verify that cpvirtualenv, lsvirtualenv, and mkproject work with spaces in env names.
1 parent f4f2370 commit 04664ca

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

tests/test_cp.sh

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,10 @@ test_new_env_activated () {
3333
assertTrue virtualenvwrapper_verify_active_environment
3434
}
3535

36-
test_virtual_env_variable () {
37-
mkvirtualenv "source" >/dev/null 2>&1
38-
cpvirtualenv "source" "destination" >/dev/null 2>&1
39-
assertSame "Wrong virtualenv name" "destination" $(basename "$VIRTUAL_ENV")
36+
test_virtual_env_variable_space_in_name () {
37+
mkvirtualenv " space source" >/dev/null 2>&1
38+
cpvirtualenv " space source" " space destination" >/dev/null 2>&1
39+
assertSame "Wrong virtualenv name" " space destination" "$(basename "$VIRTUAL_ENV")"
4040
assertTrue "$WORKON_HOME not in $VIRTUAL_ENV" "echo $VIRTUAL_ENV | grep -q $WORKON_HOME"
4141
}
4242

tests/test_ls.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ test_get_site_packages_dir () {
2525
deactivate
2626
}
2727

28-
test_lssitepackages () {
29-
mkvirtualenv "lssitepackagestest" >/dev/null 2>&1
28+
test_lssitepackages_space_in_name () {
29+
mkvirtualenv " space lssitepackagestest" >/dev/null 2>&1
3030
contents="$(lssitepackages)"
3131
assertTrue "did not find easy_install in site-packages" "echo $contents | grep -q easy_install"
3232
deactivate

virtualenvwrapper.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -945,9 +945,9 @@ function cpvirtualenv {
945945
echo "Please provide a valid virtualenv to copy."
946946
return 1
947947
else
948-
# see if its already in workon
948+
# see if it's already in workon
949949
if [ ! -e "$WORKON_HOME/$src_name" ]; then
950-
# so its a virtualenv we are importing
950+
# so it's a virtualenv we are importing
951951
# make sure we have a full path
952952
# and get the name
953953
src="$(virtualenvwrapper_expandpath "$src_name")"

0 commit comments

Comments
 (0)