on xenial help sign in
Don't change active state when transitioning from locked greeter app back to greeter
commit8553a44c5f0a3ac240d2cec019ce7e4b13cf1ad1 authorMichael Terry <michael.terry@canonical.com> dateMon 14 Dec 2015, 13:07 (11 years ago) parente9f921b
2 files changed+26−4
qml/Greeter/Greeter.qml+1−1 tests/qmltests/Greeter/tst_Greeter.qml+25−3
qml/Greeter/Greeter.qml+1−1
@@ -33,7 +33,7 @@ Showable {
3333 // How far to offset the top greeter layer during a launcher left-drag
3434 property real launcherOffset
3535
36− readonly property bool active: shown || hasLockedApp
36+ readonly property bool active: required || hasLockedApp
3737 readonly property bool fullyShown: loader.item ? loader.item.fullyShown : false
3838
3939 // True when the greeter is waiting for PAM or other setup process
tests/qmltests/Greeter/tst_Greeter.qml+25−3
@@ -71,9 +71,9 @@ Item {
7171 }
7272
7373 SignalSpy {
74− id: emergencyCallSpy
74+ id: activeChangedSpy
7575 target: loader.item
76− signalName: "emergencyCall"
76+ signalName: "activeChanged"
7777 }
7878
7979 GSettings {
@@ -142,7 +142,7 @@ Item {
142142 resetLoader();
143143 teaseSpy.clear();
144144 sessionStartedSpy.clear();
145− emergencyCallSpy.clear();
145+ activeChangedSpy.clear();
146146 viewShowMessageSpy.clear();
147147 viewShowPromptSpy.clear();
148148 viewShowLastChanceSpy.clear();
@@ -471,6 +471,28 @@ Item {
471471 tryCompare(view, "delayMinutes", 0);
472472 }
473473
474+ function test_activeIsConstantDuringLockedApp() {
475+ // Regression test for bug 1525981: if we flicker active state even
476+ // briefly, the mpt-server will allow access to the device's drive.
477+
478+ selectUser("has-password");
479+ verify(greeter.active);
480+
481+ // Test opening a locked app
482+ greeter.lockedApp = "test-app";
483+ greeter.notifyAppFocused("test-app");
484+ verify(greeter.hasLockedApp);
485+ verify(!greeter.shown);
486+
487+ // Test going back to greeter from that locked app
488+ LightDM.Greeter.showGreeter();
489+ verify(!greeter.hasLockedApp);
490+ verify(greeter.shown);
491+
492+ // Active state should never have changed
493+ compare(activeChangedSpy.count, 0);
494+ }
495+
474496 function test_dbusRequestAuthenticationUser() {
475497 selectUser("no-password");
476498 LightDM.Greeter.requestAuthenticationUser("has-password");