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

Proposed by Alfonso Sanchez-Beato
Status: Merged
Approved by: Simon Fels
Approved revision: 2547205f639b6f5284c0f0082b75cb50a80d971d
Merged at revision: 834892b6539b52afd29cff9ade2b5bd5498c88f4
Proposed branch: ~alfonsosanchezbeato/snappy-hwe-snaps/+git/engineering-tests:wifi-ap-clarify
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/engineering-tests:master
Diff against target: 31 lines (+11/-1)
1 file modified
tests/units/wifi-ap.pxu (+11/-1)
Reviewer Review Type Date Requested Status
Matteo Croce (community) Approve
Simon Fels Approve
Review via email: mp+306238@code.launchpad.net

Commit message

Clarify interface connections for wifi-ap

The interface connections for wifi-ap depend on whether the system is
full-snap based or not.

Description of the change

Clarify interface connections for wifi-ap

The interface connections for wifi-ap depend on whether the system is
full-snap based or not.

To post a comment you must log in.
Revision history for this message
Simon Fels (morphis) wrote :

LGTM

review: Approve
Revision history for this message
Matteo Croce (teknoraver) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/tests/units/wifi-ap.pxu b/tests/units/wifi-ap.pxu
index acd2225..78ee075 100644
--- a/tests/units/wifi-ap.pxu
+++ b/tests/units/wifi-ap.pxu
@@ -18,15 +18,25 @@ _steps:
18 .18 .
19 $ sudo snap connect wifi-ap:firewall-control ubuntu-core:firewall-control19 $ sudo snap connect wifi-ap:firewall-control ubuntu-core:firewall-control
20 $ sudo snap connect wifi-ap:network-control ubuntu-core:network-control20 $ sudo snap connect wifi-ap:network-control ubuntu-core:network-control
21
22 For snaps on Classic:
21 $ sudo snap connect wifi-ap:network-manager ubuntu-core:network-manager23 $ sudo snap connect wifi-ap:network-manager ubuntu-core:network-manager
24
25 For full-snap based system (note that you need to install first
26 network-manager snap in this case):
27 $ sudo snap install network-manager
22 $ sudo snap connect wifi-ap:network-manager network-manager:service28 $ sudo snap connect wifi-ap:network-manager network-manager:service
23 .29 .
24_verification:30_verification:
25 Verify that you see the following when running snap interfaces:31 Verify that you see the following when running snap interfaces:
26 :network-bind wifi-ap32 :network-bind wifi-ap
27 :network-control wifi-ap33 :network-control wifi-ap
34
35 For snaps on Classic:
28 :network-manager wifi-ap36 :network-manager wifi-ap
29 network-manager:servic wifi-ap:network-manager37
38 For full-snap based system:
39 network-manager:service wifi-ap:network-manager
3040
31id: wifi_ap/configuration/get_all41id: wifi_ap/configuration/get_all
32category_id: wifi_ap42category_id: wifi_ap

Subscribers

People subscribed via source and target branches