1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
|
#!/usr/bin/env python3
# -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
# Author: Bryce Harrington <bryce@canonical.com>
#
# Copyright (C) 2019 Bryce W. Harrington
#
# Released under GNU GPLv2 or later, read the file 'LICENSE.GPLv2+' for
# more information.
"""Ppa class tests"""
import os
import sys
import time
from unittest.mock import patch
import pytest
sys.path.insert(0, os.path.realpath(
os.path.join(os.path.dirname(os.path.realpath(__file__)), "..")))
from ppa.constants import URL_AUTOPKGTEST
from ppa.ppa import Ppa, ppa_address_split, get_ppa
from ppa.result import Result
def test_object():
"""Check that PPA objects can be instantiated."""
ppa = Ppa('test-ppa-name', 'test-owner-name', ppa_description='test-desc', service='test-svc')
assert ppa
assert ppa.ppa_name == 'test-ppa-name'
assert ppa.owner_name == 'test-owner-name'
assert ppa.ppa_description == 'test-desc'
def test_repr():
"""Check Ppa object representation."""
ppa = Ppa('a', 'b', 'c', 'd')
assert repr(ppa) == "Ppa(ppa_name='a', owner_name='b')"
def test_str():
"""Check Ppa object textual presentation."""
ppa = Ppa('a', 'b', 'c', 'd')
assert f"{ppa}" == 'b/a'
def test_address():
"""Check getting the PPA address."""
ppa = Ppa('test', 'owner')
assert ppa.address == "ppa:owner/test"
def test_description():
"""Check specifying a description when creating a PPA."""
ppa = Ppa('test-ppa-name', 'test-owner-name', 'test-description')
assert 'test-description' in ppa.ppa_description
@pytest.mark.parametrize('releases, architectures, expected_num_results, expected_num_triggers', [
([], [], 0, 0),
(['x'], ['amd64'], 1, 1),
(['x', 'y', 'z'], ['amd64'], 3, 1),
(['x'], ['amd64', 'armhf', 'i386'], 1, 3),
(['x', 'y', 'z'], ['amd64', 'armhf', 'i386'], 3, 3),
])
@patch('ppa.ppa.get_results')
@patch('ppa.ppa.open_url')
def test_get_autopkgtest_results(mock_open_url, mock_get_results,
releases, architectures, expected_num_results, expected_num_triggers):
"""Check that autopkgtest results are retrieved and iterated correctly."""
owner_name = 'a'
ppa_name = 'b'
# Bypass open_url() entirely so we don't try to retrieve anything.
# Usually this returns a response that if valid is then passed to
# get_results(), but since we'll be patching get_results(), all that
# matters is that validity check passes. We can do that by having
# our mock open_url return a generic valid value, True.
mock_open_url.return_value = True
# Substitute in our fake results to be returned by get_results().
# We need to have semi-valid looking URLs to pass internal checks,
# but can provide trivially fake data.
fake_results = {}
fake_data_url = "https://fake.data"
timestamp = time.strptime('20030201_040506', "%Y%m%d_%H%M%S")
for release in releases:
k = f"{URL_AUTOPKGTEST}/results/autopkgtest-{release}-{owner_name}-{ppa_name}/"
fake_results[k] = []
for arch in architectures:
# We don't care about the triggers for the result, just the
# number of results and their basic identity, so replace the
# get_triggers() call to avoid it making any remote calls.
result = Result(url=fake_data_url, time=timestamp, series=release, arch=arch, source=None)
result.get_triggers = lambda: "x"
fake_results[k].append(result)
def fake_get_results(response, base_url, arches, sources):
return fake_results[base_url]
mock_get_results.side_effect = fake_get_results
ppa = Ppa(ppa_name, owner_name)
results = ppa.get_autopkgtest_results(releases, architectures)
assert len(results) == expected_num_results
for trigger_set in results:
assert isinstance(trigger_set, dict)
assert all(len(triggers) == expected_num_triggers for triggers in trigger_set.values())
@pytest.mark.parametrize('address, expected', [
# Successful cases
('bb', (None, 'bb')),
('123', (None, '123')),
('a/123', ('a', '123')),
('ppa:A/bb', ('a', 'bb')),
('ppa:a/bb', ('a', 'bb')),
('ppa:ç/bb', ('ç', 'bb')),
('https://launchpad.net/~a/+archive/ubuntu/bb', ('a', 'bb')),
('https://launchpad.net/~a/+archive/ubuntu/bb/', ('a', 'bb')),
('https://launchpad.net/~a/+archive/ubuntu/bb////', ('a', 'bb')),
('https://launchpad.net/~a/+archive/ubuntu/bb/+packages', ('a', 'bb')),
('https://launchpad.net/~a/+archive/ubuntu/bb/+x', ('a', 'bb')),
# Expected failure cases
('ppa:', (None, None)),
(None, (None, None)),
('', (None, None)),
('/', (None, None)),
(':/', (None, None)),
('////', (None, None)),
('ppa:/', (None, None)),
('ppa:a/', (None, None)),
('ppa:/bb', (None, None)),
('ppa:a/bç', (None, None)),
('ppa/a/bb', (None, None)),
('ppa:a/bb/c', (None, None)),
('ppa:a/bB', (None, None)),
('http://launchpad.net/~a/+archive/ubuntu/bb', (None, None)),
('https://example.com/~a/+archive/ubuntu/bb', (None, None)),
('https://launchpad.net/~a/+archive/nobuntu/bb', (None, None)),
('https://launchpad.net/~a/+archive/ubuntu/bb/x', (None, None)),
('https://launchpad.net/~a/+archive/ubuntu/bb/+', (None, None)),
])
def test_ppa_address_split(address, expected):
"""Check ppa address input strings can be parsed properly."""
result = ppa_address_split(address)
assert result == expected
def test_get_ppa():
ppa = get_ppa(None, {'owner_name': 'a', 'ppa_name': 'bb'})
assert type(ppa) is Ppa
assert ppa.owner_name == 'a'
assert ppa.ppa_name == 'bb'
|
