Skip to content

[roottest] Don't create a TApplication in ForeignCArray test - #23634

Merged
guitargeek merged 1 commit into
root-project:masterfrom
guitargeek:roottest-foreigncarray-no-tapplication
Oct 7, 2026
Merged

guitargeek merged 1 commit into
root-project:masterfrom
guitargeek:roottest-foreigncarray-no-tapplication

Conversation

@guitargeek

Copy link
Copy Markdown
Contributor

The TestForeignCArray executable created a TApplication without "-b". When $DISPLAY is set, ROOT then starts in graphics mode and tries to open the X display. If the X server rejects the connection, Xlib prints "Authorization required, but no authorization protocol specified" to stderr, which ends up in the compared output and fails the test.

The test only exercises I/O of a CArray, so it doesn't need a TApplication at all.

Fixes this test failure on my system:

4/4 Test #1924: roottest-root-io-ForeignCArray-mytest ....................***Failed    0.35 sec
-- TEST COMMAND --
cd /home/rembserj/code/root/root_build/roottest/root/io/ForeignCArray
/nix/store/xjl7p8dvyk2j53kqf7f43kdj4ypbxz7g-coreutils-9.11/bin/timeout -s USR2 270s ./TestForeignCArray
-- BEGIN TEST OUTPUT --
Authorization required, but no authorization protocol specified

CArray[3]
ForeignData 0
ForeignData 0
ForeignData 0
~ForeignData 3
~ForeignData 2
~ForeignData 1
Array
CArray[0]
ForeignData 0
ForeignData 0
ForeignData 0
1
2
3

-- END TEST OUTPUT --
-- BEGIN OUTDIFF OUTPUT --
--- /home/rembserj/code/root/root_src/roottest/root/io/ForeignCArray/mytest.ref	Mon Oct  5 22:38:48 2026
+++ /home/rembserj/code/root/root_build/roottest/root/io/ForeignCArray/mytest.log	Wed Oct  7 09:49:21 2026
@@ -1,3 +1,4 @@
+Authorization required, but no authorization protocol specified
 CArray[3]
 ForeignData 0
 ForeignData 0

-- END OUTDIFF OUTPUT --
CMake Error at /home/rembserj/code/root/root_src/cmake/modules/RootTestDriver.cmake:283 (message):
  compare 'stdout' error: 1

The TestForeignCArray executable created a TApplication without "-b".
When $DISPLAY is set, ROOT then starts in graphics mode and tries to open
the X display. If the X server rejects the connection, Xlib prints
"Authorization required, but no authorization protocol specified" to
stderr, which ends up in the compared output and fails the test.

The test only exercises I/O of a CArray<ForeignData>, so it doesn't need
a TApplication at all.

Fixes this test failure on my system:
```txt
4/4 Test root-project#1924: roottest-root-io-ForeignCArray-mytest ....................***Failed    0.35 sec
-- TEST COMMAND --
cd /home/rembserj/code/root/root_build/roottest/root/io/ForeignCArray
/nix/store/xjl7p8dvyk2j53kqf7f43kdj4ypbxz7g-coreutils-9.11/bin/timeout -s USR2 270s ./TestForeignCArray
-- BEGIN TEST OUTPUT --
Authorization required, but no authorization protocol specified

CArray[3]
ForeignData 0
ForeignData 0
ForeignData 0
~ForeignData 3
~ForeignData 2
~ForeignData 1
Array
CArray[0]
ForeignData 0
ForeignData 0
ForeignData 0
1
2
3

-- END TEST OUTPUT --
-- BEGIN OUTDIFF OUTPUT --
--- /home/rembserj/code/root/root_src/roottest/root/io/ForeignCArray/mytest.ref	Mon Oct  5 22:38:48 2026
+++ /home/rembserj/code/root/root_build/roottest/root/io/ForeignCArray/mytest.log	Wed Oct  7 09:49:21 2026
@@ -1,3 +1,4 @@
+Authorization required, but no authorization protocol specified
 CArray[3]
 ForeignData 0
 ForeignData 0

-- END OUTDIFF OUTPUT --
CMake Error at /home/rembserj/code/root/root_src/cmake/modules/RootTestDriver.cmake:283 (message):
  compare 'stdout' error: 1
```
@guitargeek guitargeek self-assigned this Oct 7, 2026
@guitargeek
guitargeek requested a review from dpiparo as a code owner October 7, 2026 07:57
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 18h 16m 49s ⏱️
 3 884 tests  3 881 ✅ 0 💤 3 ❌
79 333 runs  79 330 ✅ 0 💤 3 ❌

For more details on these failures, see this check.

Results for commit 8903c3a.

@guitargeek
guitargeek requested a review from siliataider October 7, 2026 13:20

@siliataider siliataider left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@guitargeek
guitargeek merged commit 8e5bc6a into root-project:master Oct 7, 2026
30 of 36 checks passed
@guitargeek
guitargeek deleted the roottest-foreigncarray-no-tapplication branch October 7, 2026 14:09
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.

2 participants