potiuk commented on code in PR #46891:
URL: https://github.com/apache/airflow/pull/46891#discussion_r2203939742
##########
airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_plugins.py:
##########
@@ -33,29 +33,20 @@ class TestGetPlugins:
# Filters
(
{},
- 13,
+ 4,
[
"MetadataCollectionPlugin",
- "OpenLineageProviderPlugin",
- "databricks_workflow",
"decreasing_priority_weight_strategy_plugin",
- "edge_executor",
- "hive",
- "plugin-a",
- "plugin-b",
- "plugin-c",
- "postload",
"priority_weight_strategy_plugin",
- "test_plugin",
"workday_timetable_plugin",
],
),
(
{"limit": 3, "offset": 2},
- 13,
- ["databricks_workflow",
"decreasing_priority_weight_strategy_plugin", "edge_executor"],
+ 4,
+ ["priority_weight_strategy_plugin",
"workday_timetable_plugin"],
Review Comment:
Yeah. This experiment failed. It's far too complex to separate out those
fab-specific tests - I will go back to simply dynamically excluding them (which
is a bit of a hack but i think for now it's the best approach - until we
support (as legacy and deprecated) the old Flask plugin mechanism - it's better
to keep the tests in the core.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
