Fix cleanup of kconfigtest kdeglobals
ClosedPublic
Actions

Authored by davidedmundson on May 22 2018, 9:13 AM.

Details

Reviewers

broulik

Commits

R237:499edcea2576: Fix cleanup of kconfigtest kdeglobals

Summary

We expand the path of kdeglobals to
~/.qttest/kconfigtests/../kdeglobals

which works during the test, but this cleanup is run both on startup and
after removing ~/.qttest/kconfigtests at which point the expansion
fails.

Test Plan

Ran test, looked at qttest dir. It had cleaned up

Repository

R237 KConfig

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 subscriber: kde-frameworks-devel. · View Herald Transcript

davidedmundson requested review of this revision.May 22 2018, 9:13 AM

This revision is now accepted and ready to land.Aug 8 2018, 10:53 AM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			autotests/kconfigtest.cpp (2 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	34627	c1fa17e		May 22 2018, 9:13 AM	★	★
Diff 2	39320	63037ce	R237:499edcea257637b498a64ffb80df35b4e04f6f4b	Aug 8 2018, 5:36 PM	★	★

Commit	Tree	Parents	Author	Summary	Date
31d918b639c3	d3943da6b08a	c1fa17e99ad9	David Edmundson	Fix cleanup of kconfigtest kdeglobals (Show More…)	May 14 2018, 5:26 AM

Status	Author	Revision
Closed	davidedmundson	D13034 Add mechanism to notify other clients of config changes over DBus
Closed	davidedmundson	D13033 Expose getter method for KConfig::addConfigSources
Closed	davidedmundson	D13032 Fix cleanup of kconfigtest kdeglobals