Merge ~kissiel/checkbox-converged:autopilot-fixes into checkbox-converged:master

Proposed by Maciej Kisielewski
Status: Merged
Approved by: Maciej Kisielewski
Approved revision: 2d08563731ec6f20c88628b63d751aac3ff64376
Merged at revision: 74628d84be1be83d518a533e382f5a846c5cab9e
Proposed branch: ~kissiel/checkbox-converged:autopilot-fixes
Merge into: checkbox-converged:master
Diff against target: 59 lines (+7/-5)
3 files modified
get-libs (+2/-2)
tests/autopilot/checkbox_converged/__init__.py (+1/-0)
utils.py (+4/-3)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+332649@code.launchpad.net

Description of the change

fix autopilot failures

The following two patches fix ./get-libs crashes
9b8ff32 copy trusted.gpg only if exists
39e6eb0 bump repo used to xenial

This one fixes deployment/packaging of the app.
27a776e use new pkg name for pyotherside

This one fixes autopilot on systems that have some global providers installed
(scenarios were broken, as they were too many test plans visible on the screen, and AP couldn't find the correct one)
2d08563 ignore installed providers when running autopilot

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

looks good to me, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/get-libs b/get-libs
2index 4b3e2c9..d310b66 100755
3--- a/get-libs
4+++ b/get-libs
5@@ -113,7 +113,7 @@ def main():
6 # skip downloading rest of the libs
7 return
8 packages = [
9- 'libpython3.5', 'pyotherside', 'python3-xlsxwriter',
10+ 'libpython3.5', 'qml-module-io-thp-pyotherside', 'python3-xlsxwriter',
11 'python3-jinja2', 'python3-markupsafe', 'python3-padme',
12 'python3-requests', 'python3-urllib3', 'python3-guacamole',
13 'python3-requests-oauthlib']
14@@ -161,7 +161,7 @@ def main():
15 os.path.join('lib', 'py'),
16 preserve_symlinks=1)
17 with tempfile.TemporaryDirectory() as tmp:
18- get_package_from_url_and_extract(uris['pyotherside'], tmp)
19+ get_package_from_url_and_extract(uris['qml-module-io-thp-pyotherside'], tmp)
20 src = os.path.join(
21 tmp, 'usr', 'lib', 'arm-linux-gnueabihf', 'qt5', 'qml')
22 dest = os.path.join('lib', 'arm-linux-gnueabihf')
23diff --git a/tests/autopilot/checkbox_converged/__init__.py b/tests/autopilot/checkbox_converged/__init__.py
24index c4d2027..21f62fc 100644
25--- a/tests/autopilot/checkbox_converged/__init__.py
26+++ b/tests/autopilot/checkbox_converged/__init__.py
27@@ -183,6 +183,7 @@ class ClickAppTestCase(base.UbuntuUIToolkitAppTestCase):
28 def _launch_application_from_desktop(self):
29 app_qml_source_location = self._get_app_qml_source_path()
30 if os.path.exists(app_qml_source_location):
31+ os.environ['PROVIDERPATH'] = ''
32 self.app = self.launch_test_application(
33 base.get_qmlscene_launch_command(),
34 '-I', self._get_plainbox_qml_modules_path(),
35diff --git a/utils.py b/utils.py
36index 1fd65b5..149a254 100644
37--- a/utils.py
38+++ b/utils.py
39@@ -57,8 +57,9 @@ def prepare_uris(packages):
40 new_etc_apt = os.path.join(tmp, 'etc', 'apt')
41 os.makedirs(new_etc_apt)
42 # copy over trusted.gpg
43- shutil.copyfile('/etc/apt/trusted.gpg',
44- os.path.join(new_etc_apt, 'trusted.gpg'))
45+ if os.path.exists('/etc/apt/trusted.gpg'):
46+ shutil.copyfile('/etc/apt/trusted.gpg',
47+ os.path.join(new_etc_apt, 'trusted.gpg'))
48 # copy over additional keyrings
49 if os.path.exists('/etc/apt/trusted.gpg.d'):
50 shutil.copytree('/etc/apt/trusted.gpg.d',
51@@ -66,7 +67,7 @@ def prepare_uris(packages):
52 sources_list = open(os.path.join(new_etc_apt, 'sources.list'), "w")
53 for source in sources:
54 sources_list.write(
55- "deb [arch=armhf] {uri} wily {repositories}\n".format(
56+ "deb [arch=armhf] {uri} xenial {repositories}\n".format(
57 uri=source.uri, repositories=source.repositories))
58 sources_list.close()
59 apt_pkg.config["Apt::Architecture"] = 'armhf'

Subscribers

People subscribed via source and target branches