Merge ~alfonsosanchezbeato/snappy-hwe-snaps/+git/network-manager:add-no-wowlan-test into ~snappy-hwe-team/snappy-hwe-snaps/+git/network-manager:master

Proposed by Alfonso Sanchez-Beato
Status: Merged
Approved by: Simon Fels
Approved revision: e8c313204f44c1475e84f4c8614179198265228a
Merged at revision: 7874081a294fbda5596fa37074933a76cd7c5e6b
Proposed branch: ~alfonsosanchezbeato/snappy-hwe-snaps/+git/network-manager:add-no-wowlan-test
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/network-manager:master
Diff against target: 49 lines (+43/-0)
1 file modified
tests/main/wifi-no-wowlan-iface-removed/task.yaml (+43/-0)
Reviewer Review Type Date Requested Status
Simon Fels Approve
Konrad Zapałowicz (community) lgtm Approve
System Enablement Bot continuous-integration Approve
Review via email: mp+323107@code.launchpad.net

Commit message

Add test that checks interfaces on NM stop

Test that verifies that wifi interfaces are unconfigured when NM is
stopped in WoWLAN is not supported

Description of the change

Add test that checks interfaces on NM stop

Test that verifies that wifi interfaces are unconfigured when NM is
stopped in WoWLAN is not supported.

To post a comment you must log in.
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Konrad Zapałowicz (kzapalowicz) wrote :

ack

review: Approve (lgtm)
Revision history for this message
Simon Fels (morphis) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tests/main/wifi-no-wowlan-iface-removed/task.yaml b/tests/main/wifi-no-wowlan-iface-removed/task.yaml
2new file mode 100644
3index 0000000..1e8f759
4--- /dev/null
5+++ b/tests/main/wifi-no-wowlan-iface-removed/task.yaml
6@@ -0,0 +1,43 @@
7+summary: |
8+ Verify that WiFi interface is removed after NM stops if WoWLAN is
9+ not supported, even if enabled by NM.
10+
11+execute: |
12+ # Enable WoWLAN globally for all new WiFi connections
13+ snap set network-manager wifi.wake-on-wlan=any
14+
15+ # Replace core wpa-supplicant with one from snap
16+ sudo systemctl stop wpa_supplicant.service
17+ # TODO Install from stable when wowlan change lands
18+ snap install --edge wpa-supplicant
19+ snap connect network-manager:wpa wpa-supplicant:service
20+ sudo systemctl restart snap.network-manager.networkmanager.service
21+
22+ # mac80211_hwsim does not support wowlan, check that
23+ ! iw phy phy0 wowlan show
24+
25+ # Get us an AP via the wifi-ap we can use to create a connection to
26+ snap install wifi-ap
27+
28+ # AP needs a bit to appear and be visible for NetworkManager
29+ /snap/bin/network-manager.nmcli d wifi rescan
30+ while ! /snap/bin/network-manager.nmcli d wifi | grep Ubuntu ; do
31+ /snap/bin/network-manager.nmcli d wifi rescan
32+ sleep 1
33+ done
34+
35+ # Connect to the AP and ensure the connection was established
36+ /snap/bin/network-manager.nmcli d wifi | grep Ubuntu
37+ /snap/bin/network-manager.nmcli d wifi connect Ubuntu
38+ /snap/bin/network-manager.nmcli d | grep 'wlan1.*connected'
39+ ip address show wlan1 | grep -q inet
40+
41+ # Now that we're connected a new connection exists which we can look at
42+ # to verify that the WoWLAN key is correctly set.
43+ /snap/bin/network-manager.nmcli c show Ubuntu | grep '802-11-wireless.wake-on-wlan:.*default (1)'
44+
45+ # Stop NM, then check that the interface is not configured anymore
46+ sudo systemctl stop snap.network-manager.networkmanager.service
47+
48+ ip address show wlan1 | grep -vq inet
49+ ip address show wlan1 | grep -q "state DOWN"

Subscribers

People subscribed via source and target branches