Amin Hassani | 3587fb3 | 2021-04-28 10:10:01 -0700 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
Luis Hector Chavez | a151805 | 2018-06-14 08:19:34 -0700 | [diff] [blame] | 2 | # -*- coding: utf-8 -*- |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 3 | # Copyright (c) 2013 The Chromium OS Authors. All rights reserved. |
| 4 | # Use of this source code is governed by a BSD-style license that can be |
| 5 | # found in the LICENSE file. |
| 6 | |
| 7 | """Integration tests for the devserver. |
| 8 | |
| 9 | This module is responsible for testing the actual devserver APIs and should be |
| 10 | run whenever changes are made to the devserver. |
| 11 | |
xixuan | 52c2fba | 2016-05-20 17:02:48 -0700 | [diff] [blame] | 12 | To run the integration test for devserver: |
| 13 | python ./devserver_integration_test.py |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 14 | """ |
| 15 | |
Gabe Black | 3b56720 | 2015-09-23 14:07:59 -0700 | [diff] [blame] | 16 | from __future__ import print_function |
| 17 | |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 18 | import os |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 19 | import shutil |
Gilad Arnold | 7de05f7 | 2014-02-14 13:14:20 -0800 | [diff] [blame] | 20 | import socket |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 21 | import subprocess |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 22 | import sys |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 23 | import tempfile |
| 24 | import time |
| 25 | import unittest |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 26 | |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 27 | from string import Template |
| 28 | |
Amin Hassani | 2aa3428 | 2020-11-18 01:18:19 +0000 | [diff] [blame] | 29 | from xml.dom import minidom |
| 30 | |
Amin Hassani | 6eec879 | 2020-01-09 14:06:48 -0800 | [diff] [blame] | 31 | import requests |
| 32 | |
Amin Hassani | 2aa3428 | 2020-11-18 01:18:19 +0000 | [diff] [blame] | 33 | from six.moves import urllib |
| 34 | |
Amin Hassani | 469f570 | 2019-10-21 15:35:06 -0700 | [diff] [blame] | 35 | import psutil # pylint: disable=import-error |
| 36 | |
Amin Hassani | 469f570 | 2019-10-21 15:35:06 -0700 | [diff] [blame] | 37 | import setup_chromite # pylint: disable=unused-import |
Luis Hector Chavez | a151805 | 2018-06-14 08:19:34 -0700 | [diff] [blame] | 38 | from chromite.lib import cros_logging as logging |
Achuith Bhandarkar | 662fb72 | 2019-10-31 16:12:49 -0700 | [diff] [blame] | 39 | from chromite.lib.xbuddy import devserver_constants |
Luis Hector Chavez | a151805 | 2018-06-14 08:19:34 -0700 | [diff] [blame] | 40 | |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 41 | |
| 42 | # Paths are relative to this script's base directory. |
| 43 | LABEL = 'devserver' |
| 44 | TEST_IMAGE_PATH = 'testdata/devserver' |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 45 | TEST_UPDATE_PAYLOAD_NAME = 'update.gz' |
| 46 | TEST_UPDATE_PAYLOAD_METADATA_NAME = 'update.gz.json' |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 47 | |
| 48 | # Update request based on Omaha v3 protocol format. |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 49 | UPDATE_REQUEST = Template("""<?xml version="1.0" encoding="UTF-8"?> |
Amin Hassani | 495f1de | 2019-02-26 11:13:39 -0800 | [diff] [blame] | 50 | <request protocol="3.0" updater="ChromeOSUpdateEngine" updaterversion="0.1.0.0" ismachine="1"> |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 51 | <os version="Indy" platform="Chrome OS" sp="0.11.254.2011_03_09_1814_i686"></os> |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 52 | <app appid="$appid" version="11.254.2011_03_09_1814" lang="en-US" track="developer-build" board="x86-generic" hardware_class="BETA DVT" delta_okay="true"> |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 53 | <updatecheck></updatecheck> |
| 54 | </app> |
| 55 | </request> |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 56 | """) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 57 | |
| 58 | # RPC constants. |
| 59 | STAGE = 'stage' |
| 60 | IS_STAGED = 'is_staged' |
| 61 | STATIC = 'static' |
| 62 | UPDATE = 'update' |
| 63 | CHECK_HEALTH = 'check_health' |
| 64 | CONTROL_FILES = 'controlfiles' |
| 65 | XBUDDY = 'xbuddy' |
Prashanth B | a06d2d2 | 2014-03-07 15:35:19 -0800 | [diff] [blame] | 66 | LIST_IMAGE_DIR = 'list_image_dir' |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 67 | |
| 68 | # API rpcs and constants. |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 69 | API_SET_UPDATE_REQUEST = 'new_update-test/the-new-update' |
| 70 | API_TEST_IP_ADDR = '127.0.0.1' |
| 71 | |
| 72 | DEVSERVER_START_TIMEOUT = 15 |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 73 | DEVSERVER_START_SLEEP = 1 |
Gilad Arnold | 7de05f7 | 2014-02-14 13:14:20 -0800 | [diff] [blame] | 74 | MAX_START_ATTEMPTS = 5 |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 75 | |
| 76 | |
| 77 | class DevserverFailedToStart(Exception): |
| 78 | """Raised if we could not start the devserver.""" |
| 79 | |
| 80 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 81 | class DevserverTestBase(unittest.TestCase): |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 82 | """Class containing common logic between devserver test classes.""" |
| 83 | |
| 84 | def setUp(self): |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 85 | """Creates and populates a test directory, temporary files.""" |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 86 | self.test_data_path = tempfile.mkdtemp() |
| 87 | self.src_dir = os.path.dirname(__file__) |
| 88 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 89 | # Copy the payload to the location of the update label. |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 90 | self._CreateLabelAndCopyUpdatePayloadFiles(LABEL) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 91 | |
| 92 | # Copy the payload to the location of forced label. |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 93 | self._CreateLabelAndCopyUpdatePayloadFiles(API_SET_UPDATE_REQUEST) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 94 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 95 | # Allocate temporary files for various devserver outputs. |
| 96 | self.pidfile = self._MakeTempFile('pid') |
| 97 | self.portfile = self._MakeTempFile('port') |
| 98 | self.logfile = self._MakeTempFile('log') |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 99 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 100 | # Initialize various runtime values. |
| 101 | self.devserver_url = self.port = self.pid = None |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 102 | self.devserver = None |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 103 | |
| 104 | def tearDown(self): |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 105 | """Kill the server, remove the test directory and temporary files.""" |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 106 | |
| 107 | self._StopServer() |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 108 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 109 | self._RemoveFile(self.pidfile) |
| 110 | self._RemoveFile(self.portfile) |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 111 | # If the unittest did not succeed, print out the devserver log. |
| 112 | if sys.exc_info() != (None, None, None): |
| 113 | with open(self.logfile, 'r') as f: |
| 114 | logging.info('--- BEGINNING OF DEVSERVER LOG ---') |
| 115 | logging.info(f.read()) |
| 116 | logging.info('--- ENDING OF DEVSERVER LOG ---') |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 117 | self._RemoveFile(self.logfile) |
| 118 | shutil.rmtree(self.test_data_path) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 119 | |
| 120 | # Helper methods begin here. |
| 121 | |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 122 | def _CreateLabelAndCopyUpdatePayloadFiles(self, label): |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 123 | """Creates a label location and copies an image to it.""" |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 124 | update_dir = os.path.join(self.src_dir, TEST_IMAGE_PATH) |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 125 | label_dir = os.path.join(self.test_data_path, label) |
| 126 | os.makedirs(label_dir) |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 127 | for name in (TEST_UPDATE_PAYLOAD_NAME, TEST_UPDATE_PAYLOAD_METADATA_NAME): |
| 128 | shutil.copy(os.path.join(update_dir, name), label_dir) |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 129 | |
| 130 | def _MakeTempFile(self, suffix): |
| 131 | """Return path of a newly created temporary file.""" |
| 132 | with tempfile.NamedTemporaryFile(suffix='-devserver-%s' % suffix) as f: |
| 133 | name = f.name |
| 134 | f.close() |
| 135 | |
| 136 | return name |
| 137 | |
| 138 | def _RemoveFile(self, filename): |
| 139 | """Removes a file if it is present.""" |
| 140 | if os.path.isfile(filename): |
| 141 | os.remove(filename) |
| 142 | |
| 143 | def _ReadIntValueFromFile(self, path, desc): |
| 144 | """Reads a string from file and returns its conversion into an integer.""" |
| 145 | if not os.path.isfile(path): |
| 146 | raise DevserverFailedToStart('Devserver did not drop %s (%r).' % |
| 147 | (desc, path)) |
| 148 | |
| 149 | with open(path) as f: |
| 150 | value_str = f.read() |
| 151 | |
| 152 | try: |
| 153 | return int(value_str) |
| 154 | except ValueError: |
| 155 | raise DevserverFailedToStart('Devserver did not drop a valid value ' |
| 156 | 'in %s (%r).' % (desc, value_str)) |
| 157 | |
| 158 | def _StartServer(self, port=0): |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 159 | """Attempts to start devserver on |port|. |
| 160 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 161 | In the default case where port == 0, the server will bind to an arbitrary |
Dan Shi | 2f13686 | 2016-02-11 15:38:38 -0800 | [diff] [blame] | 162 | available port. If successful, this method will set the devserver's pid |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 163 | (self.pid), actual listening port (self.port) and URL (self.devserver_url). |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 164 | |
| 165 | Raises: |
| 166 | DevserverFailedToStart: If the devserver could not be started. |
| 167 | """ |
| 168 | cmd = [ |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 169 | os.path.join(self.src_dir, 'devserver.py'), |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 170 | '--static_dir', self.test_data_path, |
| 171 | '--pidfile', self.pidfile, |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 172 | '--portfile', self.portfile, |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 173 | '--port', str(port), |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 174 | '--logfile', self.logfile, |
| 175 | ] |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 176 | |
| 177 | # Pipe all output. Use logfile to get devserver log. |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 178 | self.devserver = subprocess.Popen(cmd, stderr=subprocess.PIPE, |
| 179 | stdout=subprocess.PIPE) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 180 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 181 | # Wait for devserver to start, determining its actual serving port and URL. |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 182 | current_time = time.time() |
| 183 | deadline = current_time + DEVSERVER_START_TIMEOUT |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 184 | error = None |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 185 | while current_time < deadline: |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 186 | try: |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 187 | self.port = self._ReadIntValueFromFile(self.portfile, 'portfile') |
| 188 | self.devserver_url = 'http://127.0.0.1:%d' % self.port |
Gabe Black | 3b56720 | 2015-09-23 14:07:59 -0700 | [diff] [blame] | 189 | self._MakeRPC(CHECK_HEALTH, timeout=1) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 190 | break |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 191 | except Exception as e: |
| 192 | error = e |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 193 | time.sleep(DEVSERVER_START_SLEEP) |
| 194 | current_time = time.time() |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 195 | else: |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 196 | raise DevserverFailedToStart( |
| 197 | 'Devserver failed to start within timeout with error: %s' % error) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 198 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 199 | # Retrieve PID. |
| 200 | self.pid = self._ReadIntValueFromFile(self.pidfile, 'pidfile') |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 201 | |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 202 | def _StopServer(self): |
| 203 | """Stops the current running devserver.""" |
| 204 | if not self.pid: |
| 205 | return |
| 206 | |
| 207 | self.devserver.terminate() |
| 208 | |
| 209 | # Just to flush the stdout/stderr so python3 doesn't complain about the |
| 210 | # unclosed file. |
| 211 | self.devserver.communicate() |
| 212 | |
| 213 | self.devserver.wait() |
| 214 | |
| 215 | self.pid = None |
| 216 | self.devserver = None |
| 217 | |
Amin Hassani | 2aa3428 | 2020-11-18 01:18:19 +0000 | [diff] [blame] | 218 | |
| 219 | def VerifyHandleUpdate(self, label, use_test_payload=True, |
| 220 | appid='{DEV-BUILD}'): |
| 221 | """Verifies that we can send an update request to the devserver. |
| 222 | |
| 223 | This method verifies (using a fake update_request blob) that the devserver |
| 224 | can interpret the payload and give us back the right payload. |
| 225 | |
| 226 | Args: |
| 227 | label: Label that update is served from e.g. <board>-release/<version> |
| 228 | use_test_payload: If set to true, expects to serve payload under |
| 229 | testdata/ and does extra checks i.e. compares hash and content of |
| 230 | payload. |
| 231 | appid: The APP ID of the board. |
| 232 | |
| 233 | Returns: |
| 234 | url of the update payload if we verified the update. |
| 235 | """ |
| 236 | update_label = '/'.join([UPDATE, label]) |
| 237 | response = self._MakeRPC( |
| 238 | update_label, data=UPDATE_REQUEST.substitute({'appid': appid}), |
| 239 | critical_update=True) |
| 240 | self.assertNotEqual('', response) |
| 241 | self.assertIn('deadline="now"', response) |
| 242 | |
| 243 | # Parse the response and check if it contains the right result. |
| 244 | dom = minidom.parseString(response) |
| 245 | update = dom.getElementsByTagName('updatecheck')[0] |
| 246 | expected_static_url = '/'.join([self.devserver_url, STATIC, label]) |
| 247 | url = self.VerifyV3Response(update, expected_static_url) |
| 248 | |
| 249 | # Verify the image we download is correct since we already know what it is. |
| 250 | if use_test_payload: |
| 251 | connection = urllib.request.urlopen(url) |
| 252 | contents = connection.read().decode('utf-8') |
| 253 | connection.close() |
| 254 | self.assertEqual('Developers, developers, developers!\n', contents) |
| 255 | |
| 256 | return url |
| 257 | |
| 258 | def VerifyV3Response(self, update, expected_static_url): |
| 259 | """Verifies the update DOM from a v3 response and returns the url.""" |
| 260 | # Parse the response and check if it contains the right result. |
| 261 | urls = update.getElementsByTagName('urls')[0] |
| 262 | url = urls.getElementsByTagName('url')[0] |
| 263 | |
| 264 | static_url = url.getAttribute('codebase') |
| 265 | # Static url's end in /. |
| 266 | self.assertEqual(expected_static_url + '/', static_url) |
| 267 | |
| 268 | manifest = update.getElementsByTagName('manifest')[0] |
| 269 | packages = manifest.getElementsByTagName('packages')[0] |
| 270 | package = packages.getElementsByTagName('package')[0] |
| 271 | filename = package.getAttribute('name') |
| 272 | self.assertEqual(TEST_UPDATE_PAYLOAD_NAME, filename) |
| 273 | |
| 274 | return os.path.join(static_url, filename) |
| 275 | |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 276 | def _MakeRPC(self, rpc, data=None, timeout=None, **kwargs): |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 277 | """Makes an RPC call to the devserver. |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 278 | |
| 279 | Args: |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 280 | rpc: The function to run on the devserver, e.g. 'stage'. |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 281 | data: Optional post data to send. |
| 282 | timeout: Optional timeout to pass to urlopen. |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 283 | kwargs: Optional arguments to the function, e.g. artifact_url='foo/bar'. |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 284 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 285 | Returns: |
| 286 | The function output. |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 287 | """ |
| 288 | request = '/'.join([self.devserver_url, rpc]) |
| 289 | if kwargs: |
| 290 | # Join the kwargs to the URL. |
Amin Hassani | 6eec879 | 2020-01-09 14:06:48 -0800 | [diff] [blame] | 291 | request += '?' + '&'.join('%s=%s' % (k, v) for k, v in kwargs.items()) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 292 | |
Amin Hassani | 6eec879 | 2020-01-09 14:06:48 -0800 | [diff] [blame] | 293 | response = (requests.post(request, data=data, timeout=timeout) if data |
| 294 | else requests.get(request, timeout=timeout)) |
| 295 | response.raise_for_status() |
| 296 | return response.text |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 297 | |
| 298 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 299 | class AutoStartDevserverTestBase(DevserverTestBase): |
| 300 | """Test base class that automatically starts the devserver.""" |
| 301 | |
| 302 | def setUp(self): |
| 303 | """Initialize everything, then start the server.""" |
| 304 | super(AutoStartDevserverTestBase, self).setUp() |
| 305 | self._StartServer() |
| 306 | |
| 307 | |
Gilad Arnold | 7de05f7 | 2014-02-14 13:14:20 -0800 | [diff] [blame] | 308 | class DevserverStartTests(DevserverTestBase): |
| 309 | """Test that devserver starts up correctly.""" |
| 310 | |
| 311 | def testStartAnyPort(self): |
| 312 | """Starts the devserver, have it bind to an arbitrary available port.""" |
| 313 | self._StartServer() |
| 314 | |
| 315 | def testStartSpecificPort(self): |
| 316 | """Starts the devserver with a specific port.""" |
| 317 | for _ in range(MAX_START_ATTEMPTS): |
| 318 | # This is a cheap hack to find an arbitrary unused port: we open a socket |
| 319 | # and bind it to port zero, then pull out the actual port number and |
| 320 | # close the socket. In all likelihood, this will leave us with an |
| 321 | # available port number that we can use for starting the devserver. |
| 322 | # However, this heuristic is susceptible to race conditions, hence the |
| 323 | # retry loop. |
| 324 | s = socket.socket() |
| 325 | s.bind(('', 0)) |
Gilad Arnold | 7de05f7 | 2014-02-14 13:14:20 -0800 | [diff] [blame] | 326 | _, port = s.getsockname() |
| 327 | s.close() |
| 328 | |
| 329 | self._StartServer(port=port) |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 330 | self._StopServer() |
Gilad Arnold | 7de05f7 | 2014-02-14 13:14:20 -0800 | [diff] [blame] | 331 | |
| 332 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 333 | class DevserverBasicTests(AutoStartDevserverTestBase): |
| 334 | """Short running tests for the devserver (no remote deps). |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 335 | |
| 336 | These are technically not unittests because they depend on being able to |
| 337 | start a devserver locally which technically requires external resources so |
| 338 | they are lumped with the remote tests here. |
| 339 | """ |
| 340 | |
Amin Hassani | 2aa3428 | 2020-11-18 01:18:19 +0000 | [diff] [blame] | 341 | def testHandleUpdateV3(self): |
| 342 | self.VerifyHandleUpdate(label=LABEL) |
| 343 | |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 344 | def testXBuddyLocalAlias(self): |
| 345 | """Extensive local image xbuddy unittest. |
| 346 | |
| 347 | This test verifies all the local xbuddy logic by creating a new local folder |
| 348 | with the necessary update items and verifies we can use all of them. |
| 349 | """ |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 350 | build_id = 'x86-generic/R32-9999.0.0-a1' |
| 351 | xbuddy_path = 'x86-generic/R32-9999.0.0-a1/test' |
| 352 | build_dir = os.path.join(self.test_data_path, build_id) |
| 353 | os.makedirs(build_dir) |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 354 | |
| 355 | # Writing dummy files. |
| 356 | image_data = 'TEST IMAGE' |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 357 | test_image_file = os.path.join(build_dir, |
| 358 | devserver_constants.TEST_IMAGE_FILE) |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 359 | with open(test_image_file, 'w') as f: |
| 360 | f.write(image_data) |
| 361 | |
| 362 | stateful_data = 'STATEFUL STUFFS' |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 363 | stateful_file = os.path.join(build_dir, devserver_constants.STATEFUL_FILE) |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 364 | with open(stateful_file, 'w') as f: |
| 365 | f.write(stateful_data) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 366 | |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 367 | update_dir = os.path.join(self.src_dir, TEST_IMAGE_PATH) |
| 368 | for name in (TEST_UPDATE_PAYLOAD_NAME, TEST_UPDATE_PAYLOAD_METADATA_NAME): |
| 369 | shutil.copy(os.path.join(update_dir, name), build_dir) |
| 370 | with open(os.path.join(build_dir, TEST_UPDATE_PAYLOAD_NAME), 'r') as f: |
| 371 | update_data = f.read() |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 372 | |
Amin Hassani | 0b4843b | 2019-09-25 16:38:56 -0700 | [diff] [blame] | 373 | for item, data in zip(['full_payload', 'test', 'stateful'], |
| 374 | [update_data, image_data, stateful_data]): |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 375 | |
| 376 | xbuddy_path = '/'.join([build_id, item]) |
| 377 | logging.info('Testing xbuddy path %s', xbuddy_path) |
Amin Hassani | 2aa3428 | 2020-11-18 01:18:19 +0000 | [diff] [blame] | 378 | response = self._MakeRPC('/'.join([XBUDDY, xbuddy_path])) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 379 | self.assertEqual(response, data) |
| 380 | |
| 381 | expected_dir = '/'.join([self.devserver_url, STATIC, build_id]) |
| 382 | response = self._MakeRPC('/'.join([XBUDDY, xbuddy_path]), return_dir=True) |
| 383 | self.assertEqual(response, expected_dir) |
| 384 | |
Yu-Ju Hong | 51495eb | 2013-12-12 17:08:43 -0800 | [diff] [blame] | 385 | response = self._MakeRPC('/'.join([XBUDDY, xbuddy_path]), |
| 386 | relative_path=True) |
| 387 | self.assertEqual(response, build_id) |
| 388 | |
Amin Hassani | 2aa3428 | 2020-11-18 01:18:19 +0000 | [diff] [blame] | 389 | logging.info('Verifying the actual payload data') |
| 390 | url = self.VerifyHandleUpdate(build_id, use_test_payload=False) |
| 391 | logging.info('Verify the actual content of the update payload') |
| 392 | connection = urllib.request.urlopen(url) |
| 393 | contents = connection.read().decode('utf-8') |
| 394 | connection.close() |
| 395 | self.assertEqual(update_data, contents) |
| 396 | |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 397 | def testPidFile(self): |
| 398 | """Test that using a pidfile works correctly.""" |
| 399 | with open(self.pidfile, 'r') as f: |
| 400 | pid = f.read() |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 401 | # Let's assert some process information about the devserver. |
Dan Shi | 2f13686 | 2016-02-11 15:38:38 -0800 | [diff] [blame] | 402 | self.assertTrue(pid.strip().isdigit()) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 403 | process = psutil.Process(int(pid)) |
| 404 | self.assertTrue(process.is_running()) |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 405 | self.assertIn('./devserver.py', process.cmdline()) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 406 | |
Gilad Arnold | 0851611 | 2014-02-14 13:14:03 -0800 | [diff] [blame] | 407 | class DevserverExtendedTests(AutoStartDevserverTestBase): |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 408 | """Longer running integration tests that test interaction with Google Storage. |
| 409 | |
| 410 | Note: due to the interaction with Google Storage, these tests both require |
| 411 | 1) runner has access to the Google Storage bucket where builders store builds. |
| 412 | 2) time. These tests actually download the artifacts needed. |
| 413 | """ |
| 414 | |
Amin Hassani | 2aa3428 | 2020-11-18 01:18:19 +0000 | [diff] [blame] | 415 | def testStageAndUpdate(self): |
| 416 | """Tests core stage/update autotest workflow where with a test payload.""" |
| 417 | build_id = 'eve-release/R78-12499.0.0' |
| 418 | archive_url = 'gs://chromeos-image-archive/%s' % build_id |
| 419 | |
| 420 | response = self._MakeRPC(IS_STAGED, archive_url=archive_url, |
| 421 | artifacts='full_payload,stateful') |
| 422 | self.assertEqual(response, 'False') |
| 423 | |
| 424 | logging.info('Staging update artifacts') |
| 425 | self._MakeRPC(STAGE, archive_url=archive_url, |
| 426 | artifacts='full_payload,stateful') |
| 427 | logging.info('Staging complete. ' |
| 428 | 'Verifying files exist and are staged in the staging ' |
| 429 | 'directory.') |
| 430 | response = self._MakeRPC(IS_STAGED, archive_url=archive_url, |
| 431 | artifacts='full_payload,stateful') |
| 432 | self.assertEqual(response, 'True') |
| 433 | staged_dir = os.path.join(self.test_data_path, build_id) |
| 434 | self.assertTrue(os.path.isdir(staged_dir)) |
| 435 | self.assertTrue(os.path.exists( |
| 436 | os.path.join(staged_dir, devserver_constants.UPDATE_FILE))) |
| 437 | self.assertTrue(os.path.exists( |
| 438 | os.path.join(staged_dir, devserver_constants.UPDATE_METADATA_FILE))) |
| 439 | self.assertTrue(os.path.exists( |
| 440 | os.path.join(staged_dir, devserver_constants.STATEFUL_FILE))) |
| 441 | |
| 442 | logging.info('Verifying we can update using the stage update artifacts.') |
| 443 | self.VerifyHandleUpdate(build_id, use_test_payload=False, |
| 444 | appid='{01906EA2-3EB2-41F1-8F62-F0B7120EFD2E}') |
| 445 | |
Luis Hector Chavez | a151805 | 2018-06-14 08:19:34 -0700 | [diff] [blame] | 446 | @unittest.skip('crbug.com/640063 Broken test.') |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 447 | def testStageAutotestAndGetPackages(self): |
Luis Hector Chavez | a151805 | 2018-06-14 08:19:34 -0700 | [diff] [blame] | 448 | """Another stage/update autotest workflow test with a test payload.""" |
| 449 | build_id = 'eve-release/R69-10782.0.0' |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 450 | archive_url = 'gs://chromeos-image-archive/%s' % build_id |
| 451 | autotest_artifacts = 'autotest,test_suites,au_suite' |
| 452 | logging.info('Staging autotest artifacts (may take a while).') |
| 453 | self._MakeRPC(STAGE, archive_url=archive_url, artifacts=autotest_artifacts) |
| 454 | |
| 455 | response = self._MakeRPC(IS_STAGED, archive_url=archive_url, |
| 456 | artifacts=autotest_artifacts) |
| 457 | self.assertEqual(response, 'True') |
| 458 | |
| 459 | # Verify the files exist and are staged in the staging directory. |
| 460 | logging.info('Checking directories exist after we staged the files.') |
| 461 | staged_dir = os.path.join(self.test_data_path, build_id) |
| 462 | autotest_dir = os.path.join(staged_dir, 'autotest') |
| 463 | package_dir = os.path.join(autotest_dir, 'packages') |
| 464 | self.assertTrue(os.path.isdir(staged_dir)) |
| 465 | self.assertTrue(os.path.isdir(autotest_dir)) |
| 466 | self.assertTrue(os.path.isdir(package_dir)) |
| 467 | |
| 468 | control_files = self._MakeRPC(CONTROL_FILES, build=build_id, |
| 469 | suite_name='bvt') |
| 470 | logging.info('Checking for known control file in bvt suite.') |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 471 | self.assertIn('client/site_tests/platform_FilePerms/control', control_files) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 472 | |
| 473 | def testRemoteXBuddyAlias(self): |
Luis Hector Chavez | a151805 | 2018-06-14 08:19:34 -0700 | [diff] [blame] | 474 | """Another stage/update autotest workflow test with a test payload.""" |
| 475 | build_id = 'eve-release/R69-10782.0.0' |
| 476 | xbuddy_path = 'remote/eve/R69-10782.0.0/full_payload' |
| 477 | xbuddy_bad_path = 'remote/eve/R32-9999.9999.9999' |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 478 | logging.info('Staging artifacts using xbuddy.') |
| 479 | response = self._MakeRPC('/'.join([XBUDDY, xbuddy_path]), return_dir=True) |
| 480 | |
| 481 | logging.info('Verifying static url returned is valid.') |
| 482 | expected_static_url = '/'.join([self.devserver_url, STATIC, build_id]) |
| 483 | self.assertEqual(response, expected_static_url) |
| 484 | |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 485 | logging.info('Now give xbuddy a bad path.') |
Amin Hassani | 6eec879 | 2020-01-09 14:06:48 -0800 | [diff] [blame] | 486 | self.assertRaises(requests.exceptions.RequestException, |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 487 | self._MakeRPC, |
| 488 | '/'.join([XBUDDY, xbuddy_bad_path])) |
| 489 | |
Prashanth B | a06d2d2 | 2014-03-07 15:35:19 -0800 | [diff] [blame] | 490 | def testListImageDir(self): |
| 491 | """Verifies that we can list the contents of the image directory.""" |
| 492 | build_id = 'x86-mario-release/R32-4810.0.0' |
| 493 | archive_url = 'gs://chromeos-image-archive/%s' % build_id |
| 494 | build_dir = os.path.join(self.test_data_path, build_id) |
| 495 | shutil.rmtree(build_dir, ignore_errors=True) |
| 496 | |
| 497 | logging.info('checking for %s on an unstaged build.', LIST_IMAGE_DIR) |
| 498 | response = self._MakeRPC(LIST_IMAGE_DIR, archive_url=archive_url) |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 499 | self.assertIn(archive_url, response) |
| 500 | self.assertIn('not been staged', response) |
Prashanth B | a06d2d2 | 2014-03-07 15:35:19 -0800 | [diff] [blame] | 501 | |
| 502 | logging.info('Checking for %s on a staged build.', LIST_IMAGE_DIR) |
| 503 | fake_file_name = 'fake_file' |
| 504 | try: |
| 505 | os.makedirs(build_dir) |
| 506 | open(os.path.join(build_dir, fake_file_name), 'w').close() |
| 507 | except OSError: |
| 508 | logging.error('Could not create files to imitate staged content. ' |
| 509 | 'Build dir %s, file %s', build_dir, fake_file_name) |
| 510 | raise |
| 511 | response = self._MakeRPC(LIST_IMAGE_DIR, archive_url=archive_url) |
Amin Hassani | ba847c8 | 2019-12-06 16:30:56 -0800 | [diff] [blame] | 512 | self.assertIn(fake_file_name, response) |
Prashanth B | a06d2d2 | 2014-03-07 15:35:19 -0800 | [diff] [blame] | 513 | shutil.rmtree(build_dir, ignore_errors=True) |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 514 | |
Amin Hassani | 28df421 | 2019-10-28 10:16:50 -0700 | [diff] [blame] | 515 | |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 516 | if __name__ == '__main__': |
Chris Sosa | 7cd2320 | 2013-10-15 17:22:57 -0700 | [diff] [blame] | 517 | unittest.main() |