diff --git a/.changes/unreleased/Fixes-20230331-095428.yaml b/.changes/unreleased/Fixes-20230331-095428.yaml new file mode 100644 index 00000000000..772ce077b7d --- /dev/null +++ b/.changes/unreleased/Fixes-20230331-095428.yaml @@ -0,0 +1,6 @@ +kind: Fixes +body: Improved failed event serialization handling and associated tests +time: 2023-03-31T09:54:28.701963-07:00 +custom: + Author: QMalcolm + Issue: 7113 7108 6568 diff --git a/core/dbt/events/functions.py b/core/dbt/events/functions.py index 8a20dd2be80..c353e7740ec 100644 --- a/core/dbt/events/functions.py +++ b/core/dbt/events/functions.py @@ -3,7 +3,7 @@ from dbt.events.base_types import BaseEvent, Cache, EventLevel, NoFile, NoStdOut, EventMsg from dbt.events.eventmgr import EventManager, LoggerConfig, LineFormat, NoFilter from dbt.events.helpers import env_secrets, scrub_secrets -from dbt.events.types import EmptyLine +from dbt.events.types import EmptyLine, Note import dbt.flags as flags from dbt.logger import GLOBAL_LOGGER, make_log_dir_if_missing from functools import partial @@ -156,7 +156,9 @@ def msg_to_dict(msg: EventMsg) -> dict: msg_dict = msg.to_dict(casing=betterproto.Casing.SNAKE, include_default_values=True) # type: ignore except AttributeError as exc: event_type = type(msg).__name__ - raise Exception(f"type {event_type} is not serializable. {str(exc)}") + fire_event( + Note(msg=f"type {event_type} is not serializable. {str(exc)}"), level=EventLevel.WARN + ) # We don't want an empty NodeInfo in output if ( "data" in msg_dict diff --git a/tests/unit/test_base_context.py b/tests/unit/test_base_context.py new file mode 100644 index 00000000000..0dc2d93ddca --- /dev/null +++ b/tests/unit/test_base_context.py @@ -0,0 +1,22 @@ +import os + +from dbt.context.base import BaseContext +from jinja2.runtime import Undefined + + +class TestBaseContext: + def test_log_jinja_undefined(self): + # regression test for CT-2259 + try: + os.environ["DBT_ENV_SECRET_LOG_TEST"] = "cats_are_cool" + BaseContext.log(msg=Undefined(), info=True) + except Exception as e: + assert False, f"Logging an jinja2.Undefined object raises an exception: {e}" + + def test_log_with_dbt_env_secret(self): + # regression test for CT-1783 + try: + os.environ["DBT_ENV_SECRET_LOG_TEST"] = "cats_are_cool" + BaseContext.log({"fact1": "I like cats"}, info=True) + except Exception as e: + assert False, f"Logging while a `DBT_ENV_SECRET` was set raised an exception: {e}" diff --git a/tests/unit/test_events.py b/tests/unit/test_events.py index 1e58da879a2..eb20e9ed1b3 100644 --- a/tests/unit/test_events.py +++ b/tests/unit/test_events.py @@ -1,7 +1,10 @@ # flake8: noqa +import pytest +import os +import datetime from dbt.events.test_types import UnitTestInfo from dbt.events import AdapterLogger -from dbt.events.functions import msg_to_json, LOG_VERSION, msg_to_dict +from dbt.events.functions import msg_to_json, LOG_VERSION, msg_to_dict, fire_event from dbt.events.base_types import msg_from_base_event from dbt.events.types import * from dbt.events.test_types import * diff --git a/tests/unit/test_functions.py b/tests/unit/test_functions.py index 4f7cb6845ac..f79648164fe 100644 --- a/tests/unit/test_functions.py +++ b/tests/unit/test_functions.py @@ -2,8 +2,8 @@ import pytest import dbt.flags as flags -from dbt.events.functions import warn_or_error -from dbt.events.types import NoNodesForSelectionCriteria +from dbt.events.functions import msg_to_dict, warn_or_error +from dbt.events.types import InfoLevel, NoNodesForSelectionCriteria from dbt.exceptions import EventCompilationError @@ -47,3 +47,19 @@ def test_warn_or_error_warn_error(warn_error, expect_compilation_exception): warn_or_error(NoNodesForSelectionCriteria()) else: warn_or_error(NoNodesForSelectionCriteria()) + + +def test_msg_to_dict_handles_exceptions_gracefully(): + class BadEvent(InfoLevel): + """A spoof Note event which breaks dictification""" + + def __init__(self): + self.__class__.__name__ = "Note" + + event = BadEvent() + try: + msg_to_dict(event) + except Exception as exc: + assert ( + False + ), f"We expect `msg_to_dict` to gracefully handle exceptions, but it raised {exc}"