Skip to content

[fitpanel test] switch to testing with xvfb, disable TestTree1D only in batch#22024

Merged
linev merged 2 commits intoroot-project:masterfrom
linev:fitpanel_test
Apr 23, 2026
Merged

[fitpanel test] switch to testing with xvfb, disable TestTree1D only in batch#22024
linev merged 2 commits intoroot-project:masterfrom
linev:fitpanel_test

Conversation

@linev
Copy link
Copy Markdown
Member

@linev linev commented Apr 23, 2026

Provide full include path to let run macro directly

TestTree1D works in interactive mode so let run it in interactive mode

Switch testing into xvfb-run - if it found

Pure batch is unstable therefore disable it for the time been again.

Provide full include path to let run macro directly
@linev linev requested a review from ferdymercury April 23, 2026 09:00
@linev linev self-assigned this Apr 23, 2026
@linev linev requested a review from bellenot as a code owner April 23, 2026 09:00
Comment thread gui/fitpanel/test/UnitTesting.cxx
Comment thread gui/fitpanel/test/UnitTesting.cxx
Comment thread gui/fitpanel/test/UnitTesting.cxx
Copy link
Copy Markdown
Collaborator

@ferdymercury ferdymercury left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok for me if tests pass

We can look into the other two tests in subsequent PRs

It is utility which emulates X11 on display-less environment
@linev linev changed the title [fitpanel] disable TestTree1D only in batch [fitpanel test] switch to testing with xvfb, disable TestTree1D only in batch Apr 23, 2026
@github-actions
Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 9h 52m 25s ⏱️
 3 848 tests  3 847 ✅  1 💤 0 ❌
75 966 runs  75 948 ✅ 18 💤 0 ❌

Results for commit be50179.

@linev linev merged commit caf200c into root-project:master Apr 23, 2026
30 checks passed
@linev linev deleted the fitpanel_test branch April 23, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants