fix tests

Author: lobsterkatie

tests/integrations/celery/test_celery_beat_crons.py

@@ -1,3 +1,4 @@
+from unittest.mock import patch
 import pytest
 
 pytest.importorskip("celery")
@@ -213,7 +214,7 @@ def test_get_monitor_config_crontab():
     app.conf.timezone = "Europe/Vienna"
 
     celery_schedule = crontab(day_of_month="3", hour="12", minute="*/10")
-    monitor_config = _get_monitor_config(celery_schedule, app)
+    monitor_config = _get_monitor_config(celery_schedule, app, "foo")
     assert monitor_config == {
         "schedule": {
             "type": "crontab",
@@ -224,13 +225,19 @@ def test_get_monitor_config_crontab():
     assert "unit" not in monitor_config["schedule"]
 
 
-def test_get_monitor_config_seconds():
+@patch("sentry_sdk.integrations.celery.logger.warning")
+def test_get_monitor_config_seconds(mock_logger_warning):
     app = MagicMock()
     app.conf = MagicMock()
     app.conf.timezone = "Europe/Vienna"
 
     celery_schedule = schedule(run_every=3)  # seconds
-    monitor_config = _get_monitor_config(celery_schedule, app)
+    monitor_config = _get_monitor_config(celery_schedule, app, "foo")
+    mock_logger_warning.assert_called_with(
+        "Intervals shorter than one minute are not supported by Sentry Crons. Monitor '%s' has an interval of %s seconds. Use the `exclude_beat_tasks` option in the celery integration to exclude it.",
+        "foo",
+        3,
+    )
     assert monitor_config == {}
 
 
@@ -240,7 +247,7 @@ def test_get_monitor_config_minutes():
     app.conf.timezone = "Europe/Vienna"
 
     celery_schedule = schedule(run_every=60)  # seconds
-    monitor_config = _get_monitor_config(celery_schedule, app)
+    monitor_config = _get_monitor_config(celery_schedule, app, "foo")
     assert monitor_config == {
         "schedule": {
             "type": "interval",
@@ -257,7 +264,7 @@ def test_get_monitor_config_unknown():
     app.conf.timezone = "Europe/Vienna"
 
     unknown_celery_schedule = MagicMock()
-    monitor_config = _get_monitor_config(unknown_celery_schedule, app)
+    monitor_config = _get_monitor_config(unknown_celery_schedule, app, "foo")
     assert monitor_config == {}
 
 
@@ -268,7 +275,7 @@ def test_get_monitor_config_default_timezone():
 
     celery_schedule = crontab(day_of_month="3", hour="12", minute="*/10")
 
-    monitor_config = _get_monitor_config(celery_schedule, app)
+    monitor_config = _get_monitor_config(celery_schedule, app, "foo")
 
     assert monitor_config["timezone"] == "UTC"