aminghadersohi commented on code in PR #37973:
URL: https://github.com/apache/superset/pull/37973#discussion_r2911776071
##########
tests/unit_tests/security/api_test.py:
##########
@@ -39,3 +40,21 @@ def test_csrf_not_exempt(app_context: None) -> None:
"PermissionApi",
"ViewMenuApi",
}
+
+
[email protected](
+ "app",
+ [
Review Comment:
Fixed — added `app` to the function signature so the indirect parametrize
correctly passes the config dict to the `app` fixture. Previously the
parametrized config may not have been applied since the test function only
declared `app_context`.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
