Configuration file for Chrome perf and endure tests.

chrome/test/functional/perf.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Basic pyauto performance tests.
 
 For tests that need to be run for multiple iterations (e.g., so that average
 and standard deviation values can be reported), the default number of iterations
 run for each of these tests is specified by |_DEFAULT_NUM_ITERATIONS|.
@@ skipping 579 matching lines @@
                        self._num_iterations, elapsed_time)
       self.assertTrue(self._timeout_count <= self._max_timeout_count,
                       msg='Test exceeded automation timeout threshold.')
       self.assertEqual(1 + num_tabs, self.GetTabCount(),
                        msg='Did not open %d new tab(s).' % num_tabs)
       for _ in range(num_tabs):
         self.CloseTab(tab_index=1)
 
     self._PrintSummaryResults(description, timings, 'milliseconds', graph_name)
 
+  def _GetConfig(self):
+    """Load perf test configuration file.
+
+    Returns:
+      A dictionary is returned that represents the config information.

[dennis_jeffrey, 2012/08/16 18:49:58]

remove 'is returned'

[fdeng1, 2012/08/17 06:39:34]

Done.

+    """
+    config_file = os.path.join(os.path.dirname(__file__), 'perf.cfg')
+    config = {'username': None,
+              'password': None,
+              'google_account_url': 'https://accounts.google.com/',
+              'gmail_url': 'https://www.gmail.com',

[dennis_jeffrey, 2012/08/16 18:49:58]

why is the URL here https whereas the two below are just http?  Can we just use
http for all sites?

[fdeng1, 2012/08/17 06:39:34]

Oh, they are meant to be all https.
I think in the test we want https? 

On 2012/08/16 18:49:58, dennis_jeffrey wrote:

[dennis_jeffrey, 2012/08/17 17:35:44]

Previously the tests navigated to the http site, but I think it doesn't make a
difference either way.  Navigating to http://www.gmail.com seems to redirect to
an https site anyway.

[fdeng1, 2012/08/17 23:19:37]

Ok, I just leave them to be https. 

On 2012/08/17 17:35:44, dennis_jeffrey wrote:

+              'plus_url': 'http://plus.google.com',
+              'docs_url': 'http://docs.google.com'}
+    if os.path.exists(config_file):
+      try:
+        new_config = pyauto.PyUITest.EvalDataFrom(config_file)
+        for key in new_config:
+          if new_config.get(key) is not None:
+            config[key] = new_config.get(key)
+      except SyntaxError, e:
+        logging.info('Could not read %s: %s', config_file, str(e))
+    return config
+
   def _LoginToGoogleAccount(self, account_key='test_google_account'):
     """Logs in to a test Google account.
 
+    Login with user-defined credentials if they exist.
+    Else login with private test credentials if they exist.
+    Else fail.
+
     Args:
-      account_key: The string key associated with the test account login
-          credentials to use.
+      account_key: The string key in private_tests_info.txt which is associated
+                   with the test account login credentials to use. It will only
+                   be used when fail to load user-defined credentials.
+
+    Raises:
+      RuntimeError: if could not get credential information.
     """
-    creds = self.GetPrivateInfo()[account_key]
-    test_utils.GoogleAccountsLogin(self, creds['username'], creds['password'])
+    private_file = os.path.join(pyauto.PyUITest.DataDir(), 'pyauto_private',
+                                'private_tests_info.txt')
+    config_file = os.path.join(os.path.dirname(__file__), 'perf.cfg')
+    config = self._GetConfig()
+    google_account_url = config.get('google_account_url')
+    username = config.get('username')
+    password = config.get('password')
+    if username and password:
+      logging.info(
+          'Using google account credential from %s',
+          os.path.join(os.path.dirname(__file__), 'perf.cfg'))
+    elif os.path.exists(private_file):
+      google_account_url = 'https://accounts.google.com/'

[dennis_jeffrey, 2012/08/16 18:49:58]

this should probably use the login URL from the config info

[fdeng1, 2012/08/17 06:39:34]

I was thinking those accounts in private_tests_info.txt are registered with the
public google server, so I forced it to be the public one if
private_tests_info.txt is used. But on a second thought, I think using the one
from the config info may make more sense as if a user has explicitly changed it,
we probably should use the custom one he has chosen.


On 2012/08/16 18:49:58, dennis_jeffrey wrote:

+      creds = self.GetPrivateInfo()[account_key]
+      username = creds['username']
+      password = creds['password']
+      logging.info(
+          'Google account credential not defined in %s, using %s',

[dennis_jeffrey, 2012/08/16 18:49:58]

probably don't need to say the config file name or the private file name.  Just
say something like this:

'User-defined credentials not found, using private test credentials instead.'

[fdeng1, 2012/08/17 06:39:34]

Done.

+          config_file, private_file)
+    else:
+      message = \
+          'Please sepecify credential information in %s.\n' \

[dennis_jeffrey, 2012/08/16 18:49:58]

sepecify --> specify

[dennis_jeffrey, 2012/08/16 18:49:58]

Right before this, say in the log message that no user-defined or private test
credentials could be found.

[fdeng1, 2012/08/17 06:39:34]

Done.

[fdeng1, 2012/08/17 06:39:34]

Done.

+          'Format:\n\t{\n' \
+          '\t "username": "my_username",\n' \
+          '\t "password": "my_password",\n' \
+          '\t "google_account_url": "https://my-google-account-server",\n' \
+          '\t "gmail_url: "https://my-gmail-server",\n' \
+          '\t "plus_url": "https://my-plus-server",\n' \
+          '\t "docs_url": "https://my-docs-server",\n\t}\n' \
+          % config_file

[dennis_jeffrey, 2012/08/16 18:49:58]

I think we don't need to print the format in this message.  The user can just
look inside the credentials file to figure out the format.

[fdeng1, 2012/08/17 06:39:34]

Done.

+      raise RuntimeError('Could not get credential information.\n' + message)
+    test_utils.GoogleAccountsLogin(
+        self, username, password, url=google_account_url)
     self.NavigateToURL('about:blank')  # Clear the existing tab.
 
   def _GetCPUUsage(self):
     """Returns machine's CPU usage.
 
     This function uses /proc/stat to identify CPU usage, and therefore works
     only on Linux/ChromeOS.
 
     Returns:
       A dictionary with 'user', 'nice', 'system' and 'idle' values.
@@ skipping 1992 matching lines @@
     """Identifies the port number to which the server is currently bound.
 
     Returns:
       The numeric port number to which the server is currently bound.
     """
     return self._server.server_address[1]
 
 
 if __name__ == '__main__':
   pyauto_functional.Main()