This is an automated email from the ASF dual-hosted git repository.
vavila pushed a commit to branch fix/handle-oauth-updates
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/fix/handle-oauth-updates by
this push:
new e87aa884be Removing comments
e87aa884be is described below
commit e87aa884beda8b387650abcaf7080237eceafc5c
Author: Vitor Avila <[email protected]>
AuthorDate: Fri Jan 10 01:06:22 2025 -0300
Removing comments
---
tests/unit_tests/commands/databases/update_test.py | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/unit_tests/commands/databases/update_test.py
b/tests/unit_tests/commands/databases/update_test.py
index 4fab3bde49..a7a1dd97c5 100644
--- a/tests/unit_tests/commands/databases/update_test.py
+++ b/tests/unit_tests/commands/databases/update_test.py
@@ -399,8 +399,8 @@ def test_remove_oauth_config_purges_tokens(
"find_permission_view_menu",
)
find_permission_view_menu.side_effect = [
- None, # schema1 has no permissions
- "[my_db].[schema2]", # second schema already exists
+ None,
+ "[my_db].[schema2]",
]
add_permission_view_menu = mocker.patch.object(
security_manager,
@@ -435,8 +435,8 @@ def test_update_other_fields_dont_affect_oauth(
"find_permission_view_menu",
)
find_permission_view_menu.side_effect = [
- None, # schema1 has no permissions
- "[my_db].[schema2]", # second schema already exists
+ None,
+ "[my_db].[schema2]",
]
add_permission_view_menu = mocker.patch.object(
security_manager,
