Fix #22

AlexandreDecan · Mar 9, 2020 · fb473a4 · fb473a4
1 parent 2513632
commit fb473a4
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 13 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,12 @@
 # Changelog
 
+## Development
+
+### Fixed
+
+ - Fix invalid representations of non-atomic intervals composed of a singleton ([#22](https://github.com/AlexandreDecan/portion/issues/22)).
+
+
 ## 2.0.0 (2020-03-06)
 
 ### Added

diff --git a/portion/interval.py b/portion/interval.py
@@ -570,17 +570,20 @@ def __hash__(self):
         return hash(tuple([self.lower, self.upper]))
 
     def __repr__(self):
-        if self.empty:
-            return '()'
-        elif self.lower == self.upper:
-            return '[{}]'.format(repr(self.lower))
-        else:
-            return ' | '.join(
-                '{}{},{}{}'.format(
-                    '[' if i.left == Bound.CLOSED else '(',
-                    repr(i.lower),
-                    repr(i.upper),
-                    ']' if i.right == Bound.CLOSED else ')',
+        intervals = []
+
+        for interval in self:
+            if interval.empty:
+                intervals.append('()')
+            elif interval.lower == interval.upper:
+                intervals.append('[{}]'.format(repr(interval.lower)))
+            else:
+                intervals.append(
+                    '{}{},{}{}'.format(
+                        '[' if interval.left == Bound.CLOSED else '(',
+                        repr(interval.lower),
+                        repr(interval.upper),
+                        ']' if interval.right == Bound.CLOSED else ')',
+                    )
                 )
-                for i in self._intervals
-            )
+        return ' | '.join(intervals)