snappy-hwe-snaps

Merge ~teknoraver/snappy-hwe-snaps/+git/engineering-tests:master into ~snappy-hwe-team/snappy-hwe-snaps/+git/engineering-tests:master

  1. Git
  2. lp:~teknoraver/snappy-hwe-snaps/+git/engineering-tests
  3. master
  4. Merge into master
Proposed by Matteo Croce
Status: Merged
Approved by: Simon Fels
Approved revision: b2148b39ddccfa7a0197122865355dabc8a51ef4
Merged at revision: 5450c617689d1d96f8d06809a0c975e7872c2df2
Proposed branch: ~teknoraver/snappy-hwe-snaps/+git/engineering-tests:master
Merge into: ~snappy-hwe-team/snappy-hwe-snaps/+git/engineering-tests:master
Diff against target: 44 lines (+36/-0)
1 file modified
wifi-ap/units/wifi-ap.pxu (+36/-0)
Reviewer Review Type Date Requested Status
Simon Fels continuous-integration Approve
System Enablement Bot continuous-integration Needs Fixing
Review via email: mp+314375@code.launchpad.net

Description of the change

tests for wifi-ap:content

To post a comment you must log in.
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:3fea580f7a7448bf9ca27b43b8dd013c3fe9c1c2
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/598/
Executed test runs:
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/506/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/598/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Simon Fels (morphis) :
review: Needs Fixing
Revision history for this message
Simon Fels (morphis) :
review: Needs Fixing
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:ed1f005b83b35fc6bbd46bee3f61bf7917a54267
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/602/
Executed test runs:
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/510/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/602/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Simon Fels (morphis) :
review: Needs Fixing
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:4c4094eab8b2baf654564d70c2ded145059f7dd1
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/603/
Executed test runs:
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/511/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/603/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
System Enablement Bot (system-enablement-ci-bot) wrote :

FAILED: Continuous integration, rev:b2148b39ddccfa7a0197122865355dabc8a51ef4
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/874/
Executed test runs:
    None: https://jenkins.canonical.com/system-enablement/job/generic-update-snap-mp/782/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/generic-build-snap/874/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Simon Fels (morphis) wrote :

LGTM

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

Approving for CI which doesn't work for this project.

review: Approve (continuous-integration)

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 affa766..f367cfe 100644
3--- a/wifi-ap/units/wifi-ap.pxu
4+++ b/wifi-ap/units/wifi-ap.pxu
5@@ -108,3 +108,39 @@ _verification:
6 Verify that the new value for the disabled is set to 0. Also verify that a
7 different machine such as a phone or a laptop can connect to the AP using
8 SSID of "Ubuntu" and no security type.
9+
10+id: wifi_ap/configuration/control_socket
11+category_id: wifi_ap
12+plugin: manual
13+depends: wifi_ap/setup/ensure_interface_connection_setup
14+_summary: Check content interface
15+_purpose:
16+ Check if the content interface is accessible and
17+ it exports a socket for a consumer snap.
18+_steps:
19+ 1. Install the wifiap-consumer snap
20+ .
21+ $ sudo snap install wifiap-consumer
22+ .
23+ 2. Connect the wifi-ap and wifiap-consumer content interface
24+ .
25+ $ sudo snap connect wifiap-consumer:control wifi-ap:control
26+ .
27+ 3. Get the configuration with the unix HTTP client
28+ .
29+ $ sudo wifiap-consumer.unixhttpc /v1/configuration
30+ .
31+_verification:
32+ Verify that the configuration is a valid JSON with reasonable data, like:
33+ {
34+ "result": {
35+ "debug": false,
36+ "disabled": false,
37+ "wifi.interface": "wlan0",
38+ "wifi.security": "open",
39+ "wifi.ssid": "Ubuntu"
40+ },
41+ "status": "OK",
42+ "status-code": 200,
43+ "type": "sync"
44+ }

Subscribers

People subscribed via source and target branches

to all changes: