diff options
| author | Patrick Michaud <pmichaud@uw.edu> | 2014-03-22 16:52:05 +0100 |
|---|---|---|
| committer | Claude Paroz <claude@2xlibre.net> | 2014-04-01 20:45:12 +0200 |
| commit | 73474df9541bd5fabbf419debc5d72c09619f625 (patch) | |
| tree | eaaae97565fd01ae03af44d26c0f2c7d20bb2ef4 /tests | |
| parent | 07e2a5681435e493be93dceac611c3870ae6b1c6 (diff) | |
Fixed #22256 -- Replaced bad fallback for missing PATH
Thanks Baptiste Mispelon for the review.
Backport of acee46fc9 from master.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/user_commands/tests.py | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/tests/user_commands/tests.py b/tests/user_commands/tests.py index bf555e66b9..c24a6e7a2d 100644 --- a/tests/user_commands/tests.py +++ b/tests/user_commands/tests.py @@ -1,8 +1,9 @@ +import os import sys from django.core import management from django.core.management import CommandError -from django.core.management.utils import popen_wrapper +from django.core.management.utils import find_command, popen_wrapper from django.test import SimpleTestCase from django.utils import translation from django.utils.six import StringIO @@ -60,6 +61,19 @@ class CommandTests(SimpleTestCase): management.call_command('leave_locale_alone_true', stdout=out) self.assertEqual(out.getvalue(), "pl\n") + def test_find_command_without_PATH(self): + """ + find_command should still work when the PATH environment variable + doesn't exist (#22256). + """ + current_path = os.environ.pop('PATH', None) + + try: + self.assertIsNone(find_command('_missing_')) + finally: + if current_path is not None: + os.environ['PATH'] = current_path + class UtilsTests(SimpleTestCase): |
