uranusjr commented on code in PR #70190:
URL: https://github.com/apache/airflow/pull/70190#discussion_r3689800503
##########
dev/registry/tests/test_extract_parameters.py:
##########
@@ -1086,3 +1157,31 @@ def test_extra_whitespace_is_tolerated(self):
def test_duplicate_providers_collapsed(self):
assert _parse_requested_providers("amazon amazon google") ==
{"amazon", "google"}
+
+
+class TestResolveDottedPath:
+ @pytest.mark.parametrize(
+ "class_path",
+ (
+ "no_dot_here",
+ "this.module.does.not.exist.at.all.Foo",
+ ),
+ )
+ def test__resolve_dotted_path_returns_none(self, class_path: str):
+ assert _resolve_dotted_path(class_path) is None
+
+ @pytest.mark.parametrize(
+ ("class_path", "expected"),
+ (
+ (
+ "extract_parameters.ThisAttrDoesNotExist",
+ ("extract_parameters", "ThisAttrDoesNotExist", None),
+ ),
+ (
+ "extract_parameters.get_category",
+ ("extract_parameters", "get_category", get_category),
+ ),
+ ),
+ )
+ def test__resolve_dottted_path(self, class_path: str, expected: tuple[str,
str, object]):
Review Comment:
```suggestion
def test__resolve_dotted_path(self, class_path: str, expected:
tuple[str, str, object]):
```
--
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]
