Signed-off-by: Soumik Dutta <shalearkane@gmail.com>
@@ -29,9 +29,7 @@ def ping_monitor(hook_config, config_filename, state, monitoring_log_level, dry_
'''
if state not in MONITOR_STATE_TO_CRONHUB:
logger.debug(
- '{}: Ignoring unsupported monitoring {} in Cronhub hook'.format(
- config_filename, state.name.lower()
- )
+ f'{config_filename}: Ignoring unsupported monitoring {state.name.lower()} in Cronhub hook'
)
return
if state not in MONITOR_STATE_TO_CRONITOR:
- '{}: Ignoring unsupported monitoring {} in Cronitor hook'.format(
+ f'{config_filename}: Ignoring unsupported monitoring {state.name.lower()} in Cronitor hook'
@@ -106,9 +106,7 @@ def test_ping_monitor_with_other_error_logs_warning():
def test_ping_monitor_with_unsupported_monitoring_state():
hook_config = {'ping_url': 'https://example.com'}
- flexmock(module.logger).should_receive("debug").once().with_args(
- '{}: Ignoring unsupported monitoring {} in Cronhub hook'.format("config.yaml", "log")
+ flexmock(module.requests).should_receive('get').never()
module.ping_monitor(
hook_config, 'config.yaml', module.monitor.State.LOG, monitoring_log_level=1, dry_run=False,
@@ -91,9 +91,7 @@ def test_ping_monitor_with_other_error_logs_warning():
- '{}: Ignoring unsupported monitoring {} in Cronitor hook'.format("config.yaml", "log")