Skip to content

Commit

Permalink
Allow to use_given_order for languages too (#997)
Browse files Browse the repository at this point in the history
  • Loading branch information
whalebot-helmsman committed Oct 27, 2021
1 parent 8af4f29 commit 5ec1a6e
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
4 changes: 2 additions & 2 deletions dateparser/date.py
Expand Up @@ -358,8 +358,8 @@ def __init__(self, languages=None, locales=None, region=None, try_previous_local
raise TypeError("use_given_order argument must be a boolean (%r given)"
% type(use_given_order))

if not locales and use_given_order:
raise ValueError("locales must be given if use_given_order is True")
if not locales and not languages and use_given_order:
raise ValueError("locales or languages must be given if use_given_order is True")

check_settings(settings)

Expand Down
7 changes: 5 additions & 2 deletions tests/test_date.py
Expand Up @@ -652,10 +652,13 @@ def test_error_raised_for_invalid_use_given_order_argument(self, use_given_order
TypeError, ["use_given_order argument must be a boolean (%r given)"
% type(use_given_order)])

def test_error_is_raised_when_use_given_order_is_True_and_locales_is_None(self):
def test_error_is_raised_when_use_given_order_is_True_and_locales_and_languages_is_None(self):
self.when_parser_is_initialized(use_given_order=True)
self.then_error_was_raised(
ValueError, ["locales must be given if use_given_order is True"])
ValueError, ["locales or languages must be given if use_given_order is True"])

def test_no_error_for_order_with_languages_without_locales(self):
self.when_parser_is_initialized(languages=['en', 'fr'], use_given_order=True)

def when_parser_is_initialized(self, languages=None, locales=None, region=None,
try_previous_locales=True, use_given_order=False):
Expand Down

0 comments on commit 5ec1a6e

Please sign in to comment.