kalluripradeep commented on PR #64908: URL: https://github.com/apache/airflow/pull/64908#issuecomment-4229219983
> Could we add a regression test for the `flask_app is None` path? Right now the route tests patch `_get_flask_app()` directly, so they do not exercise the new lazy init branch. A small unit test that leaves `auth_manager.flask_app` unset and checks that `_get_flask_app()` creates and stores the app would make this safer. Thanks for the feedback! I've added a specific regression test in test_login_lazy_init.py that exercises the flask_app is None path directly to ensure the lazy initialization works as expected and is only called once. Verified that it also handles initialization errors gracefully. -- 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]
