This is an automated email from the ASF dual-hosted git repository.
weilee pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 7362055f5b Fix typo in fixture set_auth_role_public (#42488)
7362055f5b is described below
commit 7362055f5b6eb0d58966fb1e5f13c1f3a861da06
Author: Daniel Standish <[email protected]>
AuthorDate: Thu Sep 26 01:33:05 2024 -0700
Fix typo in fixture set_auth_role_public (#42488)
auto -> auth
---
.../test_role_and_permission_endpoint.py | 36 +++++++++++-----------
tests/providers/fab/auth_manager/conftest.py | 2 +-
2 files changed, 19 insertions(+), 19 deletions(-)
diff --git
a/tests/providers/fab/auth_manager/api_endpoints/test_role_and_permission_endpoint.py
b/tests/providers/fab/auth_manager/api_endpoints/test_role_and_permission_endpoint.py
index 77e3107a0b..30cfaeb227 100644
---
a/tests/providers/fab/auth_manager/api_endpoints/test_role_and_permission_endpoint.py
+++
b/tests/providers/fab/auth_manager/api_endpoints/test_role_and_permission_endpoint.py
@@ -108,11 +108,11 @@ class TestGetRoleEndpoint(TestRoleEndpoint):
assert response.status_code == 403
@pytest.mark.parametrize(
- "set_auto_role_public, expected_status_code",
+ "set_auth_role_public, expected_status_code",
(("Public", 403), ("Admin", 200)),
- indirect=["set_auto_role_public"],
+ indirect=["set_auth_role_public"],
)
- def test_with_auth_role_public_set(self, set_auto_role_public,
expected_status_code):
+ def test_with_auth_role_public_set(self, set_auth_role_public,
expected_status_code):
response = self.client.get("/auth/fab/v1/roles/Admin")
assert response.status_code == expected_status_code, response.json
@@ -146,11 +146,11 @@ class TestGetRolesEndpoint(TestRoleEndpoint):
assert response.status_code == 403
@pytest.mark.parametrize(
- "set_auto_role_public, expected_status_code",
+ "set_auth_role_public, expected_status_code",
(("Public", 403), ("Admin", 200)),
- indirect=["set_auto_role_public"],
+ indirect=["set_auth_role_public"],
)
- def test_with_auth_role_public_set(self, set_auto_role_public,
expected_status_code):
+ def test_with_auth_role_public_set(self, set_auth_role_public,
expected_status_code):
response = self.client.get("/auth/fab/v1/roles")
assert response.status_code == expected_status_code, response.json
@@ -208,11 +208,11 @@ class TestGetPermissionsEndpoint(TestRoleEndpoint):
assert response.status_code == 403
@pytest.mark.parametrize(
- "set_auto_role_public, expected_status_code",
+ "set_auth_role_public, expected_status_code",
(("Public", 403), ("Admin", 200)),
- indirect=["set_auto_role_public"],
+ indirect=["set_auth_role_public"],
)
- def test_with_auth_role_public_set(self, set_auto_role_public,
expected_status_code):
+ def test_with_auth_role_public_set(self, set_auth_role_public,
expected_status_code):
response = self.client.get("/auth/fab/v1/permissions")
assert response.status_code == expected_status_code, response.json
@@ -346,11 +346,11 @@ class TestPostRole(TestRoleEndpoint):
assert response.status_code == 403
@pytest.mark.parametrize(
- "set_auto_role_public, expected_status_code",
+ "set_auth_role_public, expected_status_code",
(("Public", 403), ("Admin", 200)),
- indirect=["set_auto_role_public"],
+ indirect=["set_auth_role_public"],
)
- def test_with_auth_role_public_set(self, set_auto_role_public,
expected_status_code):
+ def test_with_auth_role_public_set(self, set_auth_role_public,
expected_status_code):
payload = {
"name": "Test2",
"actions": [{"resource": {"name": "Connections"}, "action":
{"name": "can_create"}}],
@@ -393,11 +393,11 @@ class TestDeleteRole(TestRoleEndpoint):
assert response.status_code == 403
@pytest.mark.parametrize(
- "set_auto_role_public, expected_status_code",
+ "set_auth_role_public, expected_status_code",
(("Public", 403), ("Admin", 204)),
- indirect=["set_auto_role_public"],
+ indirect=["set_auth_role_public"],
)
- def test_with_auth_role_public_set(self, set_auto_role_public,
expected_status_code):
+ def test_with_auth_role_public_set(self, set_auth_role_public,
expected_status_code):
role = create_role(self.app, "mytestrole")
response = self.client.delete(f"/auth/fab/v1/roles/{role.name}")
assert response.status_code == expected_status_code, response.location
@@ -579,11 +579,11 @@ class TestPatchRole(TestRoleEndpoint):
assert response.status_code == 403
@pytest.mark.parametrize(
- "set_auto_role_public, expected_status_code",
+ "set_auth_role_public, expected_status_code",
(("Public", 403), ("Admin", 200)),
- indirect=["set_auto_role_public"],
+ indirect=["set_auth_role_public"],
)
- def test_with_auth_role_public_set(self, set_auto_role_public,
expected_status_code):
+ def test_with_auth_role_public_set(self, set_auth_role_public,
expected_status_code):
role = create_role(self.app, "mytestrole")
response = self.client.patch(
f"/auth/fab/v1/roles/{role.name}",
diff --git a/tests/providers/fab/auth_manager/conftest.py
b/tests/providers/fab/auth_manager/conftest.py
index da18f9d6c0..22c29dd229 100644
--- a/tests/providers/fab/auth_manager/conftest.py
+++ b/tests/providers/fab/auth_manager/conftest.py
@@ -50,7 +50,7 @@ def minimal_app_for_auth_api():
@pytest.fixture
-def set_auto_role_public(request):
+def set_auth_role_public(request):
app = request.getfixturevalue("minimal_app_for_auth_api")
auto_role_public = app.config["AUTH_ROLE_PUBLIC"]
app.config["AUTH_ROLE_PUBLIC"] = request.param
