Let UI tests customize the UI settings file.
ClosedPublic

Authored by TallFurryMan on May 3 2020, 6:50 PM.

Details

Summary

This differential fixes the regression on the user-customized settings file "kstarsui.rc", which was wrongly replaced with the default UI settings file from the resources.
This was done to display UI icons and menus properly in "kstars_ui_tests", but the regression is that any user-customized settings, stored in a local "kstarsui.rc" are not considered anymore.

Test Plan

Execute "kstars" and check user settings such as keyboard shortcuts are properly enabled.
Execute "kstars_ui_tests" and check icons and menus are properly displayed in the user interface.

Diff Detail

Repository
R321 KStars
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
TallFurryMan created this revision.May 3 2020, 6:50 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptMay 3 2020, 6:50 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
TallFurryMan requested review of this revision.May 3 2020, 6:50 PM

Updated UI tests to properly set kstarsui.rc.

mutlaqja accepted this revision.May 3 2020, 7:24 PM
This revision is now accepted and ready to land.May 3 2020, 7:24 PM
murveit accepted this revision.May 3 2020, 7:56 PM
murveit added inline comments.
kstars/kstarsinit.cpp
135

FWIW, I would have preferred this to be either in the header or at the top of the file, just to make the constant more visible, but I don't know if we have a standard on that. In any event, I'm certainly OK with this this also proceeding as is and being merged into the codebase.

This revision was automatically updated to reflect the committed changes.