c612a9f · unity8 · git.iko.soy

git.iko.soy/unity8

Don't allow swiping greeter when in tablet mode; fix lockscreen autopilot tests to use tablet prompt when appropriate

c612a9fMichael Terry13 years ago

Greeter/Greeter.qml+2−2

compare with xenial

unifiedsplit

@@ -34,7 +34,7 @@ Showable {

35 property alias dragHandleWidth: dragHandle.width

36 property alias model: greeterContentLoader.model

37− property bool locked: shown && multiUser && !greeterContentLoader.promptless

37+ property bool locked: shown && !greeterContentLoader.promptless

39 readonly property bool narrowMode: !multiUser && width <= units.gu(60)

40 readonly property bool multiUser: LightDM.Users.count > 1

@@ -113,7 +113,7 @@ Showable {

113 // UI element)

114 distanceThreshold: 0

115

116− enabled: !greeter.locked

116+ enabled: greeter.narrowMode

117

118 direction: Direction.Leftwards

119 }

Shell.qml+1−1

@@ -552,7 +552,7 @@ FocusScope {

552 anchors.bottom: parent.bottom

553 width: parent.width

554 dragAreaWidth: shell.edgeSize

555− available: !greeter.locked

555+ available: greeter.narrowMode

556 onDashItemSelected: {

557 greeter.hide()

558 // Animate if moving between application and dash

tests/autopilot/unity8/shell/emulators/greeter.py+4−1

@@ -25,7 +25,7 @@ class Greeter(UnityEmulatorBase):

26 """An emulator that understands the greeter screen."""

28− def unlock(self):

28+ def swipe(self):

29 """Swipe the greeter screen away."""

30 self.created.wait_for(True)

31 touch = Touch.create()

@@ -38,3 +38,6 @@ class Greeter(UnityEmulatorBase):

38 touch.drag(start_x, start_y, stop_x, stop_y)

40 self.created.wait_for(False)

41+

42+ def get_prompt(self):

43+ return self.select_single("TextField", objectName="passwordInput")

tests/autopilot/unity8/shell/tests/test_lock_screen.py+59−26

@@ -40,52 +40,68 @@ class TestLockscreen(UnityTestCase):

40 """Must be able to unlock the PIN entry lock screen."""

41 self.launch_unity()

42 greeter = self.main_window.get_greeter()

43− greeter.unlock()

44+ if greeter.narrowMode:

45+ greeter.swipe()

45− lockscreen = self._wait_for_lockscreen()

46+ lockscreen = self._wait_for_lockscreen()

46− self._enter_pincode("1234")

47+ self._enter_pincode("1234")

47−

48+ self.assertThat(lockscreen.shown, Eventually(Equals(False)))

49+ else:

50+ self._enter_prompt_passphrase("1234")

48− self.assertThat(lockscreen.shown, Eventually(Equals(False)))

51+ self.assertThat(greeter.shown, Eventually(Equals(False)))

50 @with_lightdm_mock("single-passphrase")

53 @with_lightdm_mock("single-passphrase")

51 def test_can_unlock_passphrase_screen(self):

54 def test_can_unlock_passphrase_screen(self):

52 """Must be able to unlock the passphrase entry screen."""

55 """Must be able to unlock the passphrase entry screen."""

53 self.launch_unity()

56 self.launch_unity()

54 greeter = self.main_window.get_greeter()

57 greeter = self.main_window.get_greeter()

55− greeter.unlock()

56−

57− lockscreen = self._wait_for_lockscreen()

58− self._enter_passphrase("password")

59+ if greeter.narrowMode:

60+ greeter.swipe()

61+ lockscreen = self._wait_for_lockscreen()

62+ self._enter_pin_passphrase("password")

60− self.assertThat(lockscreen.shown, Eventually(Equals(False)))

63+ self.assertThat(lockscreen.shown, Eventually(Equals(False)))

64+ else:

65+ self._enter_prompt_passphrase("password")

66+ self.assertThat(greeter.shown, Eventually(Equals(False)))

62 @with_lightdm_mock("single-pin")

68 @with_lightdm_mock("single-pin")

63 def test_pin_screen_wrong_code(self):

69 def test_pin_screen_wrong_code(self):

64 """Entering the wrong pin code must not dismiss the lock screen."""

70 """Entering the wrong pin code must not dismiss the lock screen."""

65 self.launch_unity()

71 self.launch_unity()

66 greeter = self.main_window.get_greeter()

72 greeter = self.main_window.get_greeter()

67− greeter.unlock()

68−

69− lockscreen = self._wait_for_lockscreen()

70− self._enter_pincode("4321")

74+ if greeter.narrowMode:

75+ greeter.swipe()

76+ lockscreen = self._wait_for_lockscreen()

77+ self._enter_pincode("4321")

72− pinentryField = self.main_window.get_pinentryField()

78+ pinentryField = self.main_window.get_pinentryField()

73− self.assertThat(pinentryField.text, Eventually(Equals("")))

79+ self.assertThat(pinentryField.text, Eventually(Equals("")))

80+ self.assertThat(lockscreen.shown, Eventually(Equals(True)))

81+ else:

82+ self._enter_prompt_passphrase("4231")

83+ prompt = self.main_window.get_greeter().get_prompt()

84+ self.assertThat(prompt.text, Eventually(Equals("")))

74− self.assertThat(lockscreen.shown, Eventually(Equals(True)))

85+ self.assertThat(greeter.shown, Eventually(Equals(True)))

76 @with_lightdm_mock("single-passphrase")

87 @with_lightdm_mock("single-passphrase")

77 def test_passphrase_screen_wrong_password(self):

88 def test_passphrase_screen_wrong_password(self):

78 """Entering the wrong password must not dismiss the lock screen."""

89 """Entering the wrong password must not dismiss the lock screen."""

79 self.launch_unity()

90 self.launch_unity()

80 greeter = self.main_window.get_greeter()

91 greeter = self.main_window.get_greeter()

81− greeter.unlock()

93+ if greeter.narrowMode:

94+ greeter.swipe()

83− lockscreen = self._wait_for_lockscreen()

95+ lockscreen = self._wait_for_lockscreen()

84− self._enter_passphrase("foobar")

96+ self._enter_pin_passphrase("foobar")

85−

86− pinentryField = self.main_window.get_pinentryField()

97+ pinentryField = self.main_window.get_pinentryField()

87− self.assertThat(pinentryField.text, Eventually(Equals("")))

98+ self.assertThat(pinentryField.text, Eventually(Equals("")))

99+ self.assertThat(lockscreen.shown, Eventually(Equals(True)))

100+ else:

101+ self._enter_prompt_passphrase("foobar")

102+ prompt = self.main_window.get_greeter().get_prompt()

103+ self.assertThat(prompt.text, Eventually(Equals("")))

88− self.assertThat(lockscreen.shown, Eventually(Equals(True)))

104+ self.assertThat(greeter.shown, Eventually(Equals(True)))

105

90 def _wait_for_lockscreen(self):

106 def _wait_for_lockscreen(self):

91 """Wait for the lock screen to load, and return it."""

107 """Wait for the lock screen to load, and return it."""

@@ -114,9 +130,9 @@ class TestLockscreen(UnityTestCase):

114 )

130 )

115 self.touch.tap_object(self.main_window.get_pinPadButton(int(num)))

131 self.touch.tap_object(self.main_window.get_pinPadButton(int(num)))

116

132

117− def _enter_passphrase(self, passphrase):

133+ def _enter_pin_passphrase(self, passphrase):

118 """Enter the password specified in 'passphrase' into the password entry

134 """Enter the password specified in 'passphrase' into the password entry

119− field.

135+ field of the pin lock screen.

120

136

121 :param passphrase: The string you want to enter.

137 :param passphrase: The string you want to enter.

122 :raises: TypeError if passphrase is not a string.

138 :raises: TypeError if passphrase is not a string.

@@ -129,3 +145,20 @@ class TestLockscreen(UnityTestCase):

129 self.touch.tap_object(pinentryField)

145 self.touch.tap_object(pinentryField)

130 self.keyboard.type(passphrase)

146 self.keyboard.type(passphrase)

131 self.keyboard.type("\n")

147 self.keyboard.type("\n")

148+

149+ def _enter_prompt_passphrase(self, passphrase):

150+ """Enter the password specified in 'passphrase' into the password entry

151+ field of the main user list's prompt.

152+

153+ :param passphrase: The string you want to enter.

154+ :raises: TypeError if passphrase is not a string.

155+

156+ """

157+ if not isinstance(passphrase, basestring):

158+ raise TypeError("'passphrase' parameter must be a string.")

159+

160+ prompt = self.main_window.get_greeter().get_prompt()

161+ self.touch.tap_object(prompt)

162+ self.keyboard.type(passphrase)

163+ self.keyboard.type("\n")

164+