Skip to content
This repository was archived by the owner on May 5, 2025. It is now read-only.

Commit e41a4a1

Browse files
chore(logs): change level of unactioned message (#455)
* chore(logs): change level of unactioned message * chore(test): set level to debug
1 parent 0363e7f commit e41a4a1

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

shared/validation/install.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,7 @@ def validate_install_configuration(inputted_dict):
403403
validator = CodecovYamlValidator(show_secret=True)
404404
is_valid = validator.validate(inputted_dict, config_schema)
405405
if not is_valid:
406-
log.warning(
406+
log.debug(
407407
"Configuration considered invalid, using dict as it is",
408408
extra=dict(errors=validator.errors),
409409
)

tests/unit/validation/test_install_validation.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,12 @@ def test_validate_install_configuration_simple(mocker):
1818

1919

2020
def test_validate_install_configuration_invalid(mocker):
21-
mock_warning = mocker.patch.object(install_log, "warning")
21+
install_log.setLevel("DEBUG")
22+
mock_debug = mocker.patch.object(install_log, "debug")
2223
assert validate_install_configuration(
2324
{"setup": {"codecov_url": "http://codecov.company.com"}, "gitlab": 1}
2425
) == {"setup": {"codecov_url": "http://codecov.company.com"}, "gitlab": 1}
25-
assert mock_warning.call_count == 1
26+
assert mock_debug.call_count == 1
2627

2728

2829
def test_validate_install_configuration_with_user_yaml(mocker):
@@ -448,7 +449,8 @@ def test_admins(mocker):
448449

449450

450451
def test_validate_install_configuration_raise_warning(mocker):
451-
mock_warning = mocker.patch.object(install_log, "warning")
452+
install_log.setLevel("DEBUG")
453+
mock_debug = mocker.patch.object(install_log, "debug")
452454
input = {
453455
"setup": {
454456
"tasks": {
@@ -464,7 +466,7 @@ def test_validate_install_configuration_raise_warning(mocker):
464466
}
465467
}
466468
validate_install_configuration(input)
467-
mock_warning.assert_called_with(
469+
mock_debug.assert_called_with(
468470
"Configuration considered invalid, using dict as it is",
469471
extra={
470472
"errors": {

0 commit comments

Comments
 (0)