Merge lp:~aacid/qtmir/load_generic_sensors_test_5_4_1 into lp:qtmir

Proposed by Albert Astals Cid
Status: Merged
Approved by: Gerry Boland
Approved revision: 325
Merged at revision: 324
Proposed branch: lp:~aacid/qtmir/load_generic_sensors_test_5_4_1
Merge into: lp:qtmir
Diff against target: 14 lines (+5/-0)
1 file modified
tests/mirserver/Screen/screen_test.cpp (+5/-0)
To merge this branch: bzr merge lp:~aacid/qtmir/load_generic_sensors_test_5_4_1
Reviewer Review Type Date Requested Status
Gerry Boland (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Michał Sawicz code Approve
Review via email: mp+251970@code.launchpad.net

Commit message

Make the test pass with Qt >= 5.4.1

To post a comment you must log in.
325. By Albert Astals Cid

Set envvar only if not set

Revision history for this message
Michał Sawicz (saviq) wrote :

Yeaaah, ok :P

review: Approve (code)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Gerry Boland (gerboland) :
review: Approve
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/mirserver/Screen/screen_test.cpp'
2--- tests/mirserver/Screen/screen_test.cpp 2014-10-16 18:27:04 +0000
3+++ tests/mirserver/Screen/screen_test.cpp 2015-03-05 16:24:00 +0000
4@@ -51,6 +51,11 @@
5
6 TEST(ScreenTest, OrientationSensor)
7 {
8+ if (!qEnvironmentVariableIsSet("QT_ACCEL_FILEPATH")) {
9+ // Trick Qt >= 5.4.1 to load the generic sensors
10+ qputenv("QT_ACCEL_FILEPATH", "dummy");
11+ }
12+
13 Screen::skipDBusRegistration = true;
14 Screen *screen = new Screen(fake_output);
15

Subscribers

People subscribed via source and target branches