Merge lp:~jonas-drange/ubuntu-system-settings/wifi-hide-other-fix-1377076 into lp:ubuntu-system-settings

Proposed by Jonas G. Drange
Status: Merged
Approved by: Ken VanDine
Approved revision: 1145
Merged at revision: 1169
Proposed branch: lp:~jonas-drange/ubuntu-system-settings/wifi-hide-other-fix-1377076
Merge into: lp:ubuntu-system-settings
Diff against target: 156 lines (+75/-3)
3 files modified
plugins/wifi/PageComponent.qml (+2/-1)
tests/autopilot/ubuntu_system_settings/__init__.py (+34/-0)
tests/autopilot/ubuntu_system_settings/tests/test_wifi.py (+39/-2)
To merge this branch: bzr merge lp:~jonas-drange/ubuntu-system-settings/wifi-hide-other-fix-1377076
Reviewer Review Type Date Requested Status
Ken VanDine Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+238922@code.launchpad.net

Commit message

[wifi] use actiongroup to hide "connect to other networks"

Description of the change

[wifi] use actiongroup to hide "connect to other networks"

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Looks good, but the tests need to be fixed as well.

review: Needs Fixing
Revision history for this message
Jonas G. Drange (jonas-drange) wrote :

How do we fix this?

On Mon, Oct 20, 2014 at 8:36 PM, Ken VanDine <email address hidden>
wrote:

> Review: Needs Fixing
>
> Looks good, but the tests need to be fixed as well.
> --
>
> https://code.launchpad.net/~jonas-drange/ubuntu-system-settings/wifi-hide-other-fix-1377076/+merge/238922
> You are the owner of
> lp:~jonas-drange/ubuntu-system-settings/wifi-hide-other-fix-1377076.
>

1143. By Jonas G. Drange

merge trunk

1144. By Jonas G. Drange

making test work for unity7 and unity8.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1145. By Jonas G. Drange

translate wifi

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Great, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/wifi/PageComponent.qml'
--- plugins/wifi/PageComponent.qml 2014-09-26 12:52:31 +0000
+++ plugins/wifi/PageComponent.qml 2014-10-22 12:52:36 +0000
@@ -138,6 +138,8 @@
138 objectName: "connectToHiddenNetwork"138 objectName: "connectToHiddenNetwork"
139 text: i18n.tr("Connect to hidden network…")139 text: i18n.tr("Connect to hidden network…")
140 progression: true140 progression: true
141 visible : (actionGroup.actionObject.valid ?
142 actionGroup.actionObject.state : false)
141 onClicked: {143 onClicked: {
142 otherNetworLoader.source = "OtherNetwork.qml";144 otherNetworLoader.source = "OtherNetwork.qml";
143 PopupUtils.open(otherNetworLoader.item);145 PopupUtils.open(otherNetworLoader.item);
@@ -155,5 +157,4 @@
155 Flickable.DragAndOvershootBounds :157 Flickable.DragAndOvershootBounds :
156 Flickable.StopAtBounds158 Flickable.StopAtBounds
157 }159 }
158
159}160}
160161
=== modified file 'tests/autopilot/ubuntu_system_settings/__init__.py'
--- tests/autopilot/ubuntu_system_settings/__init__.py 2014-09-26 12:52:31 +0000
+++ tests/autopilot/ubuntu_system_settings/__init__.py 2014-10-22 12:52:36 +0000
@@ -678,6 +678,40 @@
678 return True678 return True
679 return False679 return False
680680
681 @autopilot.logging.log_action(logger.debug)
682 def enable_wireless(self):
683 self._set_wireless(True)
684
685 @autopilot.logging.log_action(logger.debug)
686 def disable_wireless(self):
687 self._set_wireless(False)
688
689 """
690 :returns: Whether or not WiFi can be used
691 """
692 @autopilot.logging.log_action(logger.debug)
693 def have_wireless(self):
694 try:
695 self.wait_select_single('SwitchMenuItem', text=_('Wi-Fi'))
696 except:
697 return False
698 return True
699
700 """Returns the current WiFi state
701
702 :returns: Whether or not WiFi is enabled
703 """
704 @autopilot.logging.log_action(logger.debug)
705 def get_wireless(self):
706 return self.wait_select_single(
707 'SwitchMenuItem', text=_('Wi-Fi')).checked
708
709 @autopilot.logging.log_action(logger.debug)
710 def _set_wireless(self, state):
711 obj = self.wait_select_single('SwitchMenuItem', text=_('Wi-Fi'))
712 if obj.checked != state:
713 self.pointing_device.click_object(obj)
714
681 """Connects to hidden network715 """Connects to hidden network
682716
683 :param name: Network name string (SSID)717 :param name: Network name string (SSID)
684718
=== modified file 'tests/autopilot/ubuntu_system_settings/tests/test_wifi.py'
--- tests/autopilot/ubuntu_system_settings/tests/test_wifi.py 2014-09-26 12:52:31 +0000
+++ tests/autopilot/ubuntu_system_settings/tests/test_wifi.py 2014-10-22 12:52:36 +0000
@@ -25,6 +25,11 @@
25 Equals(_('Wi-Fi')))25 Equals(_('Wi-Fi')))
2626
27 def test_connect_to_hidden_network(self):27 def test_connect_to_hidden_network(self):
28 if not self.wifi_page.have_wireless():
29 self.skipTest('Cannot test wireless since it cannot be enabled')
30 self.addCleanup(
31 self.wifi_page._set_wireless, self.wifi_page.get_wireless())
32 self.wifi_page.enable_wireless()
28 dialog = self.wifi_page.connect_to_hidden_network(33 dialog = self.wifi_page.connect_to_hidden_network(
29 'yeah',34 'yeah',
30 scroll_to_and_click=self.system_settings.main_view35 scroll_to_and_click=self.system_settings.main_view
@@ -47,7 +52,11 @@
47 dialog.wait_until_destroyed()52 dialog.wait_until_destroyed()
4853
49 def test_connect_to_nonexistant_hidden_network(self):54 def test_connect_to_nonexistant_hidden_network(self):
5055 if not self.wifi_page.have_wireless():
56 self.skipTest('Cannot test wireless since it cannot be enabled')
57 self.addCleanup(
58 self.wifi_page._set_wireless, self.wifi_page.get_wireless())
59 self.wifi_page.enable_wireless()
51 dialog = self.wifi_page.connect_to_hidden_network(60 dialog = self.wifi_page.connect_to_hidden_network(
52 'yeah',61 'yeah',
53 scroll_to_and_click=self.system_settings.main_view62 scroll_to_and_click=self.system_settings.main_view
@@ -72,6 +81,11 @@
7281
73 @skip('skipped due to bug 1337556')82 @skip('skipped due to bug 1337556')
74 def test_connect_to_hidden_network_using_secrets(self):83 def test_connect_to_hidden_network_using_secrets(self):
84 if not self.wifi_page.have_wireless():
85 self.skipTest('Cannot test wireless since it cannot be enabled')
86 self.addCleanup(
87 self.wifi_page._set_wireless, self.wifi_page.get_wireless())
88 self.wifi_page.enable_wireless()
75 dialog = self.wifi_page.connect_to_hidden_network(89 dialog = self.wifi_page.connect_to_hidden_network(
76 'yeah', security='wpa', password='abcdefgh',90 'yeah', security='wpa', password='abcdefgh',
77 scroll_to_and_click=self.system_settings.main_view91 scroll_to_and_click=self.system_settings.main_view
@@ -95,6 +109,11 @@
95109
96 @skip('skipped due to bug 1337556')110 @skip('skipped due to bug 1337556')
97 def test_connect_to_hidden_network_using_incorrect_secrets(self):111 def test_connect_to_hidden_network_using_incorrect_secrets(self):
112 if not self.wifi_page.have_wireless():
113 self.skipTest('Cannot test wireless since it cannot be enabled')
114 self.addCleanup(
115 self.wifi_page._set_wireless, self.wifi_page.get_wireless())
116 self.wifi_page.enable_wireless()
98 dialog = self.wifi_page.connect_to_hidden_network(117 dialog = self.wifi_page.connect_to_hidden_network(
99 'yeah', security='wpa', password='abcdefgh',118 'yeah', security='wpa', password='abcdefgh',
100 scroll_to_and_click=self.system_settings.main_view119 scroll_to_and_click=self.system_settings.main_view
@@ -117,7 +136,11 @@
117 _('Your authentication details were incorrect'))))136 _('Your authentication details were incorrect'))))
118137
119 def test_connect_to_hidden_network_then_cancel(self):138 def test_connect_to_hidden_network_then_cancel(self):
120139 if not self.wifi_page.have_wireless():
140 self.skipTest('Cannot test wireless since it cannot be enabled')
141 self.addCleanup(
142 self.wifi_page._set_wireless, self.wifi_page.get_wireless())
143 self.wifi_page.enable_wireless()
121 dialog = self.wifi_page.connect_to_hidden_network(144 dialog = self.wifi_page.connect_to_hidden_network(
122 'foo',145 'foo',
123 scroll_to_and_click=self.system_settings.main_view146 scroll_to_and_click=self.system_settings.main_view
@@ -134,6 +157,18 @@
134 len(self.device_mock.GetMethodCalls('Disconnect')),157 len(self.device_mock.GetMethodCalls('Disconnect')),
135 Eventually(Equals(1)))158 Eventually(Equals(1)))
136159
160 def test_connect_to_hidden_network_dialog_visibility(self):
161 if not self.wifi_page.have_wireless():
162 self.skipTest('Cannot test wireless since it cannot be enabled')
163 self.addCleanup(
164 self.wifi_page._set_wireless, self.wifi_page.get_wireless())
165 self.wifi_page.disable_wireless()
166 self.assertThat(
167 lambda: bool(self.wifi_page.select_single(
168 '*',
169 objectName='connectToHiddenNetwork').visible),
170 Eventually(Equals(False)), 'other net dialog not hidden')
171
137 """Note: this test does not actually remove previous networks from the UI.172 """Note: this test does not actually remove previous networks from the UI.
138 The NetworkManager dbusmock template does not currently support deletion173 The NetworkManager dbusmock template does not currently support deletion
139 of connections."""174 of connections."""
@@ -157,5 +192,7 @@
157192
158 self.system_settings.main_view.go_back()193 self.system_settings.main_view.go_back()
159194
195 # wait for ui to update
196 sleep(2)
160 self.wifi_page.remove_previous_network(197 self.wifi_page.remove_previous_network(
161 previous_networks[2]['ssid'], scroll_to_and_click=click_method)198 previous_networks[2]['ssid'], scroll_to_and_click=click_method)

Subscribers

People subscribed via source and target branches