Differential D17386

[autotests] Fix failing of testDontCrashReinitializeCompositor
ClosedPublic
Actions

Authored by zzag on Dec 6 2018, 5:30 PM.

Details

Reviewers
davidedmundson
Group Reviewers
KWin
Commits
R108:3956d6ab4a4e: [autotests] Fix failing of testDontCrashReinitializeCompositor
Summary

Currently, testDontCrashReinitializeCompositor is unstable because it
doesn't take into account that effects can be loaded asynchronously.

In general, we don't need all default effects in this test, so let's
disable them.

Test Plan

This should fix ASan issues.

Diff Detail

Repository
R108 KWin
Branch
fix-dont-crash-reinitialize-compositor
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 5755
Build 5773: arc lint + arc unit
zzag created this revision.Dec 6 2018, 5:30 PM
Restricted Application added a project: KWin. · View Herald TranscriptDec 6 2018, 5:30 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Dec 6 2018, 5:30 PM
davidedmundson accepted this revision.Dec 6 2018, 5:41 PM
This revision is now accepted and ready to land.Dec 6 2018, 5:41 PM
Closed by commit R108:3956d6ab4a4e: [autotests] Fix failing of testDontCrashReinitializeCompositor (authored by zzag). · Explain WhyDec 6 2018, 5:42 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 46976

View Options

autotests/integration/dont_crash_reinitialize_compositor.cpp

Loading...