dstandish commented on code in PR #22607:
URL: https://github.com/apache/airflow/pull/22607#discussion_r856567749
##########
tests/core/test_providers_manager.py:
##########
@@ -142,6 +145,49 @@ def test_connection_form_widgets(self):
connections_form_widgets =
list(provider_manager.connection_form_widgets.keys())
assert len(connections_form_widgets) > 29
+ @pytest.mark.parametrize(
+ 'scenario',
+ [
+ 'prefix',
+ 'no_prefix',
+ 'both_1',
+ 'both_2',
+ ],
+ )
+ def test_connection_form__add_widgets_prefix_backcompat(self, scenario):
+ """
+ When the field name is prefixed, it should be used as is.
+ When not prefixed, we should add the prefix defined using the conn
type.
Review Comment:
```suggestion
When not prefixed, we should add the prefix
```
--
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]
