disallow normalize=True with Tick classes

doc/source/whatsnew/v0.24.0.txt

@@ -112,7 +112,7 @@ Timezones
 Offsets
 ^^^^^^^
 
--
+- :class:`Tick` subclasses can no longer be created with the argument `normalize=True` (:issue:`21427`)
 -
 -
 

pandas/tests/tseries/offsets/test_offsets.py

@@ -28,7 +28,7 @@
                                     YearEnd, Day,
                                     QuarterEnd, BusinessMonthEnd, FY5253,
                                     Nano, Easter, FY5253Quarter,
-                                    LastWeekOfMonth)
+                                    LastWeekOfMonth, Tick)
 from pandas.core.tools.datetimes import format, ole2datetime
 import pandas.tseries.offsets as offsets
 from pandas.io.pickle import read_pickle
@@ -225,6 +225,11 @@ def test_offset_freqstr(self, offset_types):
 
     def _check_offsetfunc_works(self, offset, funcname, dt, expected,
                                 normalize=False):
+
+        if normalize and issubclass(offset, Tick):
+            # normalize=True disallowed for Tick subclasses GH#21427
+            return
+
         offset_s = self._get_offset(offset, normalize=normalize)
         func = getattr(offset_s, funcname)
 
@@ -413,6 +418,9 @@ def test_onOffset(self, offset_types):
         assert offset_s.onOffset(dt)
 
         # when normalize=True, onOffset checks time is 00:00:00
+        if issubclass(offset_types, Tick):
+            # normalize=True disallowed for Tick subclasses GH#21427
+            return
         offset_n = self._get_offset(offset_types, normalize=True)
         assert not offset_n.onOffset(dt)
 
@@ -440,7 +448,9 @@ def test_add(self, offset_types, tz):
         assert isinstance(result, Timestamp)
         assert result == expected_localize
 
-        # normalize=True
+        # normalize=True, disallowed for Tick subclasses GH#21427
+        if issubclass(offset_types, Tick):
+            return
         offset_s = self._get_offset(offset_types, normalize=True)
         expected = Timestamp(expected.date())
 
@@ -3140,6 +3150,14 @@ def test_require_integers(offset_types):
         cls(n=1.5)
 
 
+def test_tick_normalize_raises(tick_classes):
+    # check that trying to create a Tick object with normalize=True raises
+    # GH#21427
+    cls = tick_classes
+    with pytest.raises(ValueError):
+        cls(n=3, normalize=True)
+
+
 def test_weeks_onoffset():
     # GH#18510 Week with weekday = None, normalize = False should always
     # be onOffset

pandas/tseries/offsets.py

@@ -2219,6 +2219,9 @@ class Tick(SingleConstructorOffset):
     def __init__(self, n=1, normalize=False):
         # TODO: do Tick classes with normalize=True make sense?
         self.n = self._validate_n(n)
+        if normalize:
+            raise ValueError("Tick offset with `normalize=True` are not "
+                             "allowed.")  # GH#21427
         self.normalize = normalize
 
     __gt__ = _tick_comp(operator.gt)