IJHack / QtPass / 41635856782
93%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 127
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 92.671%. Remained the same
41635856782

push

github

web-flow 
tst_mainwindow: restyle the field frames synchronously in the palette test (#2121)

* tst_mainwindow: restyle the field frames synchronously in the palette test

fieldFrameBorderFollowsRuntimePaletteChange set the application
palette and waited up to 3 s for the window's PaletteChange. On Windows
CI it failed now and then (#1866, #1869, #2119) with the frame it had
captured gone: something cleared the panel during the wait, and the
test then read a deleted QFrame. Fifteen runs of the suite on a Windows
runner did not reproduce it.

The palette is now set on the window: QWidget::setPalette() sends its
PaletteChange before returning on every platform, and the check runs
at once. That is still MainWindow::changeEvent() and refreshPalette(),
the code under test; the delivery from QApplication to the window is
Qt's. The captured frame is a QPointer, and a failure says whether it
was deleted and what the field frames carry.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VJygpMUf461yZgfAqwMJ6d
Signed-off-by: Anne Jan Brouwer <brouwer@annejan.com>

* tst_mainwindow: cover the application palette path deterministically

CodeRabbit: the window-level palette no longer covered a desktop theme
switch, which arrives as an application palette. Qt applies that to
child widgets at once but posts the top-level window's
ApplicationPaletteChange (QApplication::event); that posted delivery is
what the old test waited for on the event loop. The test now sets the
application palette and delivers just those posted events with
QCoreApplication::sendPostedEvents(), so the window gets its
PaletteChange with nothing else run meanwhile, then checks the
window-level path as well. The application palette is restored after.

Without the sendPostedEvents() call the first phase fails: the window
is reached only through the posted event.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/... (continued)

10242 of 11052 relevant lines covered (92.67%)

397.33 hits per line

Coverage Regressions

Lines Coverage ∆ File
8
0.0
0.0% src/qrc_qmake_qmake_qm_files.cpp
Jobs
ID Job ID Ran Files Coverage
1 41635856782.1 127
92.67
GitHub Action Run
Source Files on build 41635856782