Differential D17573

Fix the X11TimestampUpdateTest
ClosedPublic
Actions

Authored by graesslin on Dec 14 2018, 6:10 AM.

Details

Reviewers
zzag
Group Reviewers
KWin
Commits
R108:c3e712e967bd: Fix the X11TimestampUpdateTest
Summary

It also searches for the platform plugin, so we need to ensure it finds
it in the build directory.

This was a regression caused by building all platform plugins in the
correct location.

Test Plan

strace on the failing test, verified correct plugin is loaded

Diff Detail

Repository
R108 KWin
Branch
fix-x11-timestamp-update-test
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 6008
Build 6026: arc lint + arc unit
graesslin created this revision.Dec 14 2018, 6:10 AM
Restricted Application added a project: KWin. · View Herald TranscriptDec 14 2018, 6:10 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
graesslin requested review of this revision.Dec 14 2018, 6:10 AM
zzag accepted this revision.Dec 14 2018, 8:31 AM
zzag added a subscriber: zzag.
zzag added inline comments.
autotests/test_x11_timestamp_update.cpp
51

Please add an explanatory comment.

This revision is now accepted and ready to land.Dec 14 2018, 8:31 AM
Closed by commit R108:c3e712e967bd: Fix the X11TimestampUpdateTest (authored by graesslin). · Explain WhyDec 14 2018, 4:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 47547

View Options

autotests/test_x11_timestamp_update.cpp

Loading...