Merge ~alfonsosanchezbeato/snappy-hwe-snaps/+git/engineering-tests:wifi-ap-def-ssid into ~snappy-hwe-team/snappy-hwe-snaps/+git/engineering-tests:master

Proposed by Alfonso Sanchez-Beato
Status: Merged
Approved by: Roberto Mier Escandon
Approved revision: 06863a52a56df598e0c31e92aa87973d22c18ff3
Merged at revision: b3ea64e799c808f0595a1cf83210ee18488fa7e7
Proposed branch: ~alfonsosanchezbeato/snappy-hwe-snaps/+git/engineering-tests:wifi-ap-def-ssid
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/engineering-tests:master
Diff against target: 48 lines (+33/-0)
1 file modified
wifi-ap/units/wifi-ap.pxu (+33/-0)
Reviewer Review Type Date Requested Status
Simon Fels Approve
Roberto Mier Escandon (community) Approve
System Enablement Bot continuous-integration Approve
Konrad Zapałowicz (community) lgtm Approve
Review via email: mp+325809@code.launchpad.net

Description of the change

wifi-ap: add test for default SSID in gadget snap

To post a comment you must log in.
Revision history for this message
Konrad Zapałowicz (kzapalowicz) wrote :

ack

review: Approve (lgtm)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Roberto Mier Escandon (rmescandon) wrote :

lgtm

review: Approve
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/wifi-ap/units/wifi-ap.pxu b/wifi-ap/units/wifi-ap.pxu
2index 503197a..229cd87 100644
3--- a/wifi-ap/units/wifi-ap.pxu
4+++ b/wifi-ap/units/wifi-ap.pxu
5@@ -4,6 +4,7 @@
6 # Written by:
7 # Jim Hodapp <jim.hodapp@canonical.com>
8 # Simon Fels <simon.fels@canonical.com>
9+# Alfonso Sanchez-Beato <alfonso.sanchez-beato@canonical.com>
10
11 unit: category
12 id: wifi_ap
13@@ -156,3 +157,35 @@ _verification:
14 "status-code": 200,
15 "type": "sync"
16 }
17+
18+id: wifi_ap/configuration/default_ssid
19+category_id: wifi_ap
20+plugin: manual
21+_summary: WiFi AP default SSID is set from gadget snap
22+_purpose:
23+ Check if the configuration of the WiFi AP SSID is set to the value
24+ coming from the gadget snap, when present.
25+_steps:
26+ 1. Install a gadget snap which contains in gadget.yaml:
27+ .
28+ defaults:
29+ # Below snap ID matches the one assigned for wifi-ap in the Ubuntu Store
30+ 2rGgvyaY0CCzlWuKAPwFtCWrgwkM8lqS:
31+ default.wifi.ssid: GadgetSnap
32+ .
33+ 2. Reboot
34+ 3. Remove wifi-ap if present:
35+ .
36+ $ snap remove wifi-ap
37+ .
38+ 4. Install wifi-ap (this must be done from the store, otherwise the gadget
39+ will not set the defaults for the snap). Assuming we are testing candidate:
40+ .
41+ $ snap install --candidate wifi-ap
42+ .
43+ 5. Get the SSID value:
44+ .
45+ $ /snap/bin/wifi-ap.config get wifi.ssid
46+ .
47+_verification:
48+ Check that wifi.ssid=GadgetSnap

Subscribers

People subscribed via source and target branches