Skip to content

Fix misaligned headers in the failed tests report table - #6753

Merged
albertvillanova merged 1 commit into
mainfrom
ci-fix-report-table-headers
Aug 17, 2026
Merged

Fix misaligned headers in the failed tests report table#6753
albertvillanova merged 1 commit into
mainfrom
ci-fix-report-table-headers

Conversation

@albertvillanova

@albertvillanova albertvillanova commented Aug 14, 2026

Copy link
Copy Markdown
Member

This PR fixes the headers of the failed tests table reported by the slow tests workflow.

Motivation

Each row of the table holds three fields, the test file, the test class and the truncated test name, but only two headers are passed to tabulate. As a result the header row is padded on the left and every column ends up with the wrong label, the file being reported under Test Location and the class under Test Name.

Before:

+-----------------+-----------------+----------------------------------+
|                 | Test Location   | Test Name                        |
+=================+=================+==================================+
| tests/test_b.py | TestBaz         | test_qux_with_a_very_long_name.. |
+-----------------+-----------------+----------------------------------+

After:

+-----------------+---------+-----------------------------+
| File            | Class   | Test Name                   |
+=================+=========+=============================+
| tests/test_b.py | TestBaz | test_qux_with_a_long_name.. |
+-----------------+---------+-----------------------------+

Changes

  • Name the three columns of the failed tests table

Note

Low Risk
Reporting-only change in CI log formatting with no runtime or security impact.

Overview
Fixes mislabeled columns in the failed tests grid that log_reports.py prints for CI/Slack when pytest failures are reported.

Each row already splits nodeid into file, class, and a truncated test name, but tabulate was only given two headers (Test Location, Test Name), so labels shifted across columns. The change passes three headers—File, Class, Test Name—so the grid matches the data.

Reviewed by Cursor Bugbot for commit df5473d. Bugbot is set up for automated code reviews on this repo. Configure here.

@bot-ci-comment

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@qgallouedec
qgallouedec force-pushed the ci-fix-report-table-headers branch from c1304e5 to df5473d Compare August 14, 2026 18:41
@albertvillanova
albertvillanova merged commit ec630be into main Aug 17, 2026
31 of 33 checks passed
@albertvillanova
albertvillanova deleted the ci-fix-report-table-headers branch August 17, 2026 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants