diff options
| author | Paul Ganssle <paul@ganssle.io> | 2020-01-02 11:54:27 -0500 |
|---|---|---|
| committer | Paul Ganssle <paul@ganssle.io> | 2020-01-02 14:08:55 -0500 |
| commit | ea7f441b455cabbae2bf98f2d10fd11724001155 (patch) | |
| tree | e773eb33388f57dab41fd697deb517ff67ab47a5 /dateutil/parser | |
| parent | 21fe6e9669191b413e59cd9abd22610191575c1f (diff) | |
| download | dateutil-git-ea7f441b455cabbae2bf98f2d10fd11724001155.tar.gz | |
Fix custom repr for ParserError
This was originally dead code, because an indentation error had `__repr__`
defined after the `return` statement in `__str__`. The definition of the
`__repr__` is also changed to be more in line with the conception of a
repr as "what you would have to evalue to get this object".
Even though this is not guaranteed behavior, this commit also adds a
regression test to avoid simple errors like this in the future.
Diffstat (limited to 'dateutil/parser')
| -rw-r--r-- | dateutil/parser/_parser.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/dateutil/parser/_parser.py b/dateutil/parser/_parser.py index 7fcfa54..8d67584 100644 --- a/dateutil/parser/_parser.py +++ b/dateutil/parser/_parser.py @@ -1600,8 +1600,9 @@ class ParserError(ValueError): except (TypeError, IndexError): return super(ParserError, self).__str__() - def __repr__(self): - return "%s(%s)" % (self.__class__.__name__, str(self)) + def __repr__(self): + args = ", ".join("'%s'" % arg for arg in self.args) + return "%s(%s)" % (self.__class__.__name__, args) class UnknownTimezoneWarning(RuntimeWarning): |