Eason09053360 commented on PR #72707: URL: https://github.com/apache/airflow/pull/72707#issuecomment-5605564744
Thanks @rjgoyln for the careful trace, applied two of the three: - **`TERM` pin** - applied via the suggestion. Confirmed the gap you described: with the fix reverted, `TERM=dumb` made the test pass, and `TERM` unset / `xterm-256color` made it fail. With the pin it fails under all three. - **`width=200`** - applied. Reproduced it: on an 80-column tty the uploaded report had `system_path` / `python_path` cut at column 80. - **`cleanup_providers_manager`** - left as is. `test_show_info` and `test_show_info_anonymize` in the same class already go through the full `show()` path without that fixture, so the new test is not the first one and I'd rather keep the three consistent. Happy to add it to all three if a maintainer prefers. -- 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]
