Tests for GSM compliance.

functional/chromeos_gsm_compliance.py

+#!/usr/bin/python
+# Copyright (c) 2011 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import subprocess as sub
+
+import pyauto_functional  # Must be imported before pyauto
+import pyauto
+
+
+class ChromeosGSMCompliance(pyauto.PyUITest):
+  """Tests for ChromeOS GSM compliance.
+
+  Fakes connection to a network and verifies that the network name is correct.
+  """
+
+  _ethernet = None
+
+  def _GetEthernet(self):
+    """Get the ethernet to which the device is connected."""
+    result = sub.Popen('ifconfig | cut -d\' \' -f 1 | grep eth',
+                        stdout=sub.PIPE, shell=True).communicate()
+    self.assertNotEqual(result[0], '', msg='Not connected to Ethernet.')
+    self._ethernet = result[0].rstrip()
+
+  def setUp(self):
+    pyauto.PyUITest.setUp(self)
+    sub.call(['sudo', 'stop', 'cromo'])
+    self._GetEthernet()
+    sub.call(['sudo', '/bin/sh', '/usr/local/lib/flimflam/test/backchannel',
+              'setup', self._ethernet, 'pseudo-modem0'])
+
+  def tearDown(self):
+    if self._ethernet:
+      sub.call(['sudo', '/bin/sh', '/usr/local/lib/flimflam/test/backchannel',
+                'teardown', self._ethernet, 'pseudo-modem0'])
+    pyauto.PyUITest.tearDown(self)
+
+  def _IsFakeGSMRunning(self):
+    """Check if fake-gsm-modem is running.
+
+    Returns:
+       True if fake-gsm-modem process is running and False if not.  

[dennis_jeffrey, 2011/10/21 01:02:16]

nit: indent this line by 1 fewer space (should only be indented 2 spaces
underneath the previous line).

[deepakg, 2011/10/21 01:06:43]

Done.

+    """
+    call = sub.Popen('ps -ef | grep fake-gsm-modem | grep -v grep',

[dennis_jeffrey, 2011/10/21 01:02:16]

sorry, my previous comment where I said "change to call?" was a temporary
comment for myself that I meant to delete, but forgot to delete it.  I think the
variable name "ps" that you originally had here is ok.

[deepakg, 2011/10/21 01:06:43]

Done.

+                     shell=True, stdout=sub.PIPE).communicate()
+    return len(call[0].split('fake-gsm-modem')) > 0
+
+  def _VerifyBasicCarrierCompliance(self, carrier_name, cellular_name):
+    """Faking the specified carrier and checking the name.
+
+    Args:
+       carrier_name: The name of the carrier.
+       cellular_name: The name in the icon of the cellular network.

[dennis_jeffrey, 2011/10/21 01:02:16]

nit: indent the above 2 lines by 1 fewer space each

[deepakg, 2011/10/21 01:06:43]

Done.

+    """
+    fake_gsm = sub.Popen(['sudo', '/usr/bin/python', 
+                          '/usr/local/lib/flimflam/test/fake-gsm-modem',
+                          '-c', carrier_name])
+    # Wait for the fake GSM to connect.
+    cellular = 'cellular_networks'
+    self.assertTrue(self.WaitUntil(lambda: cellular in
+                    self.NetworkScan().keys(), timeout=10, retry_sleep=1),
+                    msg='Did not connect to any cellular network')

[stanleyw, 2011/10/21 01:21:16]

"No cellular networks appeared on scan.".  No connection is happening here.  

[deepakg, 2011/10/21 22:40:42]

Done.

+    network_info = self.NetworkScan()[cellular]
+    result = any([network_info[key]['name'] == cellular_name
+                  for key in network_info.keys()])
+    self.assertTrue(self._IsFakeGSMRunning(), msg='Fake GSM is not running.')

[stanleyw, 2011/10/21 01:21:16]

This assertion should probably happen before we perform the network scan but
after we've set up the fake-gsm.  No purpose in scanning if fake-gsm isn't even
running.  

Also, referring to the line below.  We probably want to terminate on every
assertion.  If one of the above assertion fails, this could lead to several
instances of fake-gsm-modem being run.  It might be safe to use popen.poll to
make sure the process has actually terminated so other tests are not affected by
a failure.

[deepakg, 2011/10/21 22:40:42]

Done.

+    fake_gsm.terminate()
+    self.assertTrue(result, msg='Did not connect to cellular network %s.'

[stanleyw, 2011/10/21 01:21:16]

We're testing if name displayed in the menu matches the desired display name of
the network, not if the network connects.  This assertion/message should
probably reflect that.

[deepakg, 2011/10/21 22:40:42]

Done.

+                    % cellular_name)
+
+  def testConnectThree(self):
+    """Testing connection to Three."""
+    self._VerifyBasicCarrierCompliance('three', '3')
+
+  def testConnectSFR(self):
+    """Testing connection to SFR."""
+    self._VerifyBasicCarrierCompliance('SFR', 'SFR')
+
+  def testConnectKPN(self):
+    """Testing connection to KPN."""
+    self._VerifyBasicCarrierCompliance('KPN', 'KPN NL')
+
+  def testConnectSimyo(self):
+    """Testing connection to Simyo."""
+    self._VerifyBasicCarrierCompliance('Simyo', 'E-Plus')
+
+  def testConnectMovistar(self):
+    """Testing connection to Movistar."""
+    self._VerifyBasicCarrierCompliance('Movistar', 'Movistar')
+
+  def testConnectThreeita(self):
+    """Testing connection to threeita."""
+    self._VerifyBasicCarrierCompliance('threeita', '3ITA')
+
+  def testConnectAtt(self):
+    """Testing connection to att."""
+    self._VerifyBasicCarrierCompliance('att', 'AT&T')
+
+  def testConnectTmobile(self):
+    """Testing connection to Tmobile."""
+    self._VerifyBasicCarrierCompliance('Tmobile', 'T-Mobile')
+
+
+if __name__ == '__main__':
+  pyauto_functional.Main()