Merge branch 'jacebrowning-fix-nonetype-transition'

Author: Sébastien Eustace

pendulum/tz/timezone.py

@@ -107,25 +107,26 @@ def _normalize(self, dt, dst_rule=None):  # type: (_D, Optional[str]) -> _D
             else:
                 transition = transition.previous
 
-        if transition.is_ambiguous(sec):
-            # Ambiguous time
-            if dst_rule == TRANSITION_ERROR:
-                raise AmbiguousTime(dt)
-
-            # We set the fold attribute for later
-            if dst_rule == POST_TRANSITION:
-                fold = 1
-        elif transition.is_missing(sec):
-            # Skipped time
-            if dst_rule == TRANSITION_ERROR:
-                raise NonExistingTime(dt)
-
-            # We adjust accordingly
-            if dst_rule == POST_TRANSITION:
-                sec += transition.fix
-                fold = 1
-            else:
-                sec -= transition.fix
+        if transition:
+            if transition.is_ambiguous(sec):
+                # Ambiguous time
+                if dst_rule == TRANSITION_ERROR:
+                    raise AmbiguousTime(dt)
+
+                # We set the fold attribute for later
+                if dst_rule == POST_TRANSITION:
+                    fold = 1
+            elif transition.is_missing(sec):
+                # Skipped time
+                if dst_rule == TRANSITION_ERROR:
+                    raise NonExistingTime(dt)
+
+                # We adjust accordingly
+                if dst_rule == POST_TRANSITION:
+                    sec += transition.fix
+                    fold = 1
+                else:
+                    sec -= transition.fix
 
         kwargs = {"tzinfo": self}
         if _HAS_FOLD or isinstance(dt, pendulum.DateTime):

tests/date/test_sub.py

@@ -2,7 +2,7 @@
 
 import pendulum
 
-from datetime import timedelta
+from datetime import datetime, timedelta
 
 from ..conftest import assert_date
 
@@ -43,6 +43,11 @@ def test_subtract_days_negative():
     assert pendulum.Date(1975, 5, 30).subtract(days=-1).day == 31
 
 
+def test_subtract_days_max():
+    delta = pendulum.now() - pendulum.instance(datetime.min)
+    assert pendulum.now().subtract(days=delta.days - 1).year == 1
+
+
 def test_subtract_weeks_positive():
     assert pendulum.Date(1975, 5, 28).subtract(weeks=1).day == 21