[autotests] Fix failing of testDontCrashReinitializeCompositor
ClosedPublic
Actions

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

Details

Reviewers

davidedmundson

Group Reviewers

KWin

Commits

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.

Repository

R108 KWin

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

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

This revision is now accepted and ready to land.Dec 6 2018, 5:41 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			autotests/integration/dont_crash_reinitialize_compositor.cpp (23 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	46976	576c667		Dec 6 2018, 5:30 PM	★	★
Diff 2	46979	576c667	R108:3956d6ab4a4e0100a0ec979024c75b9c3945ba06	Dec 6 2018, 5:42 PM	★	★

Commit	Tree	Parents	Author	Summary	Date
7411d73cbe07	b6bd0b5720c0	576c6671245c	Vlad Zagorodniy	[autotests] Fix failing of testDontCrashReinitializeCompositor (Show More…)	Dec 6 2018, 10:05 AM