Merge lp:~bladernr/checkbox/912038 into lp:checkbox

[Jeff Lane ]

Added a test called networking/info_automated that is an automated version of networking/info

with one difference... this one also dumps the config of wireless devices as well as ethernet.

Tested on a VM with a single ethernet device and a laptop with one ethernet and one wi-fi (not running):

root@klaatu:/usr/share/checkbox# ./scripts/udev_resource | ./scripts/filter_templates -w "category=WIRELESS" -w "category=NETWORK"| awk "/path: / { print \$2 }"| xargs -n 1 sh -c "for i in \`ls /sys\$0/net 2>/dev/null\`; do .\/scripts\/network_info \$i; done"
Interface: eth0
Connected: Yes
IP: 192.168.1.11
MAC: 00:26:b9:63:e4:52

Interface: wlan0
Connected: No
IP: n/a
MAC: 00:21:6a:a8:03:22

So this should provide decent data regardless of whether the wireless or ethernet device is configured or in use...

[Brendan Donegan (brendan-donegan)]

Looks like there are some conflicts in the changelog. Can you fix them and resubmit?

review: Needs Fixing

[lp:~bladernr/checkbox/912038]

1162. By Jeff Lane  on 2012-01-09

merged in current trunk

[Jeff Lane  (bladernr)]

You coulda fixed that while merging... it's because daniel has merged other things into trunk since I made this branch... in any case, I've merged the current trunk into my branch, fixed the changelog and pushed the update here.

review: Needs Resubmitting

[Brendan Donegan (brendan-donegan)]

Ok, merging it now.

review: Approve

=== modified file 'debian/changelog'
--- debian/changelog	2012-01-09 11:45:22 +0000
+++ debian/changelog	2012-01-09 22:32:23 +0000
@@ -39,10 +39,12 @@
   * Fixed spelling mistakes in user_apps job file. (LP: #904209)
 
   [Jeff Lane]
+  * Created automated network info test to get some config info during automated 
+    runs. (LP: #912038)
   * Added requires to suspend wireless jobs so they won't run if wireless isn't
     present (LP: #907150)
 
- -- Daniel Manrique <[e-mail]>  Fri, 06 Jan 2012 17:14:39 -0500
+ -- Jeff Lane <[e-mail]>  Mon, 09 Jan 2012 17:28:10 -0500
 
 checkbox (0.13) precise; urgency=low
 

=== modified file 'jobs/networking.txt.in'
--- jobs/networking.txt.in	2011-10-07 18:59:33 +0000
+++ jobs/networking.txt.in	2012-01-09 22:32:23 +0000
@@ -30,6 +30,15 @@
   VERIFICATION:
       Is this correct?
 
+plugin: shell 
+name: networking/info_automated
+requires: 
+ package.name == 'network-manager'
+ device.category == 'NETWORK'
+command: udev_resource | filter_templates -w "category=WIRELESS" -w "category=NETWORK"| awk "/path: / { print \$2 }"| xargs -n 1 sh -c "for i in \`ls /sys\$0/net 2>/dev/null\`; do network_info \$i; done"
+_description:
+ This is an automated test to gather some info on the current state of your network devices. If no devices are found, the test will exit with an error.
+
 plugin: manual
 name: networking/wired_connection
 command: network_check