Merge ~racb/git-ubuntu:test-structure-consistency into git-ubuntu:master

Proposed by Robie Basak
Status: Merged
Merged at revision: f8843744fee9b04d3150113b025d2ffc800b0db7
Proposed branch: ~racb/git-ubuntu:test-structure-consistency
Merge into: git-ubuntu:master
Diff against target: 113 lines (+40/-34)
3 files modified
gitubuntu/git_repository.py (+0/-33)
gitubuntu/git_repository_test.py (+37/-0)
gitubuntu/importer_test.py (+3/-1)
Reviewer Review Type Date Requested Status
Nish Aravamudan Approve
Server Team CI bot continuous-integration Approve
Review via email: mp+347971@code.launchpad.net

Commit message

Make Jenkins happy

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

FAILED: Continuous integration, rev:f8843744fee9b04d3150113b025d2ffc800b0db7
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/13/
Executed test runs:
    SUCCESS: VM Setup
    SUCCESS: Build
    FAILED: Unit Tests

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/13/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

PASSED: Continuous integration, rev:f8843744fee9b04d3150113b025d2ffc800b0db7
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/14/
Executed test runs:
    SUCCESS: VM Setup
    SUCCESS: Build
    SUCCESS: Unit Tests
    SUCCESS: Integration Tests
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/14/rebuild

review: Approve (continuous-integration)
Revision history for this message
Nish Aravamudan (nacc) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/gitubuntu/git_repository.py b/gitubuntu/git_repository.py
2index 9d2ca66..2a7d4dd 100644
3--- a/gitubuntu/git_repository.py
4+++ b/gitubuntu/git_repository.py
5@@ -676,39 +676,6 @@ class Changelog:
6 return ret
7 return self._shell_srcpkg
8
9-@pytest.mark.parametrize('changelog_name, expected', [
10- ('test_versions_1', ['1.0', None]),
11- ('test_versions_2', ['2.0', '1.0']),
12- ('test_versions_3', ['4.0', '3.0']),
13- ('test_versions_unknown', ['ss-970814-1', None]),
14-])
15-def test_changelog_versions(changelog_name, expected):
16- test_changelog = get_test_changelog(changelog_name)
17- assert [test_changelog.version, test_changelog.previous_version] == expected
18-
19-
20-@pytest.mark.parametrize('changelog_name, expected', [
21- ('test_versions_unknown', ['ss-970814-1',]),
22-])
23-def test_changelog_all_versions(changelog_name, expected):
24- test_changelog = get_test_changelog(changelog_name)
25- assert test_changelog.all_versions == expected
26-
27-
28-def test_changelog_distribution():
29- test_changelog = get_test_changelog('test_distribution')
30- assert test_changelog.distribution == 'xenial'
31-
32-def test_changelog_date():
33- test_changelog = get_test_changelog('test_date_1')
34- assert test_changelog.date == 'Mon, 12 May 2016 08:14:34 -0700'
35- test_changelog = get_test_changelog('test_date_2')
36- assert test_changelog.date == 'Mon, 12 May 2016 08:14:34 -0700'
37-
38-
39-def test_changelog_utf8():
40- test_changelog = get_test_changelog('test_utf8_error')
41- assert test_changelog.version == '1.0.3-2'
42
43
44 class GitUbuntuChangelogError(Exception):
45diff --git a/gitubuntu/test_git_repository.py b/gitubuntu/git_repository_test.py
46index 2bbb25d..8b5b692 100644
47--- a/gitubuntu/test_git_repository.py
48+++ b/gitubuntu/git_repository_test.py
49@@ -24,6 +24,43 @@ from gitubuntu.test_fixtures import (
50 repo,
51 pygit2_repo,
52 )
53+from gitubuntu.test_util import get_test_changelog
54+
55+
56+@pytest.mark.parametrize('changelog_name, expected', [
57+ ('test_versions_1', ['1.0', None]),
58+ ('test_versions_2', ['2.0', '1.0']),
59+ ('test_versions_3', ['4.0', '3.0']),
60+ ('test_versions_unknown', ['ss-970814-1', None]),
61+])
62+def test_changelog_versions(changelog_name, expected):
63+ test_changelog = get_test_changelog(changelog_name)
64+ assert [test_changelog.version, test_changelog.previous_version] == expected
65+
66+
67+@pytest.mark.parametrize('changelog_name, expected', [
68+ ('test_versions_unknown', ['ss-970814-1',]),
69+])
70+def test_changelog_all_versions(changelog_name, expected):
71+ test_changelog = get_test_changelog(changelog_name)
72+ assert test_changelog.all_versions == expected
73+
74+
75+def test_changelog_distribution():
76+ test_changelog = get_test_changelog('test_distribution')
77+ assert test_changelog.distribution == 'xenial'
78+
79+
80+def test_changelog_date():
81+ test_changelog = get_test_changelog('test_date_1')
82+ assert test_changelog.date == 'Mon, 12 May 2016 08:14:34 -0700'
83+ test_changelog = get_test_changelog('test_date_2')
84+ assert test_changelog.date == 'Mon, 12 May 2016 08:14:34 -0700'
85+
86+
87+def test_changelog_utf8():
88+ test_changelog = get_test_changelog('test_utf8_error')
89+ assert test_changelog.version == '1.0.3-2'
90
91
92 @pytest.mark.parametrize('tree_func', [
93diff --git a/gitubuntu/test_importer.py b/gitubuntu/importer_test.py
94index df011e3..f9537c4 100644
95--- a/gitubuntu/test_importer.py
96+++ b/gitubuntu/importer_test.py
97@@ -9,13 +9,15 @@ import pytest
98 import shutil
99 import tempfile
100
101+import pygit2
102+
103 import gitubuntu.repo_builder as repo_builder
104 import gitubuntu.repo_comparator as repo_comparator
105 import gitubuntu.source_builder as source_builder
106
107 import gitubuntu.importer as target
108
109-from gitubuntu.test_git_repository import pygit2, repo
110+from gitubuntu.test_fixtures import repo
111
112
113 @pytest.mark.parametrize('head_versions,expected', [

Subscribers

People subscribed via source and target branches