Solution to detect classpath issues in serialize method of Async Trigger about astronomer-providers HOT 3 CLOSED

This is now implemented as part of the unit test for the serialization method for each of the async trigger.
Refer to a sample implementationn below for the snowflake trigger.

def test_snowflake_trigger_serialization():
"""
Asserts that the SnowflakeTrigger correctly serializes its arguments
and classpath.
"""
trigger = SnowflakeTrigger(
task_id=TASK_ID,
polling_period_seconds=POLLING_PERIOD_SECONDS,
query_ids=[],
snowflake_conn_id="test_conn",
)
classpath, kwargs = trigger.serialize()
assert classpath == "astronomer_operators.snowflake.triggers.snowflake_trigger.SnowflakeTrigger"
assert kwargs == {
"task_id": TASK_ID,
"polling_period_seconds": 1.0,
"query_ids": [],
"snowflake_conn_id": "test_conn",
}

from astronomer-providers.

Traceback (most recent call last):

File "/usr/local/lib/python3.9/site-packages/***/utils/module_loading.py", line 35, in import_string
return getattr(module, class_name)

AttributeError: module 'astronomer.providers.microsoft.azure.triggers.data_factory' has no attribute 'WasbToSynpaseTrigger'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

File "/usr/local/lib/python3.9/site-packages/***/jobs/triggerer_job.py", line 406, in update_triggers
trigger_class = self.get_trigger_by_classpath(new_triggers[new_id].classpath)

File "/usr/local/lib/python3.9/site-packages/***/jobs/triggerer_job.py", line 423, in get_trigger_by_classpath
self.trigger_cache[classpath] = import_string(classpath)

File "/usr/local/lib/python3.9/site-packages/***/utils/module_loading.py", line 37, in import_string
raise ImportError(f'Module "{module_path}" does not define a "{class_name}" attribute/class')

ImportError: Module "astronomer.providers.microsoft.azure.triggers.data_factory" does not define a "WasbToSynpaseTrigger" attribute/class

from astronomer-providers.

in airflow trigger now sends the error message with import error mentioned in the above comment,

from astronomer-providers.