diff --git a/bindep.txt b/bindep.txt index ef605c8b..9c35486d 100644 --- a/bindep.txt +++ b/bindep.txt @@ -2,4 +2,4 @@ # install and tests; see http://docs.opendev.org/opendev/bindep/ for additional # information. -openjdk-8-jdk [test] +openjdk-11-jdk [test] diff --git a/git_review/tests/__init__.py b/git_review/tests/__init__.py index 545495e0..c274cdee 100644 --- a/git_review/tests/__init__.py +++ b/git_review/tests/__init__.py @@ -37,10 +37,10 @@ else: urlparse = urllib.parse.urlparse -WAR_URL = 'https://gerrit-releases.storage.googleapis.com/gerrit-2.13.14.war' +WAR_URL = 'https://gerrit-releases.storage.googleapis.com/gerrit-3.4.4.war' # Update GOLDEN_SITE_VER for every change altering golden site, including # WAR_URL changes. Set new value to something unique (just +1 it for example) -GOLDEN_SITE_VER = '5' +GOLDEN_SITE_VER = '6' # NOTE(yorik-sar): This function needs to be a perfect hash function for @@ -225,7 +225,7 @@ class BaseGitReviewTestCase(testtools.TestCase, GerritHelpers): pub_key = pub_key_fd.read().decode().strip() resp = requests.post( 'http://%s:%s/a/accounts/self/sshkeys' % (http_addr, http_port), - auth=requests.auth.HTTPDigestAuth('admin', 'secret'), + auth=requests.auth.HTTPBasicAuth('admin', 'secret'), headers={'Content-Type': 'text/plain'}, data=pub_key) if resp.status_code != 201: diff --git a/git_review/tests/test_git_review.py b/git_review/tests/test_git_review.py index dcb92323..0e643aa8 100644 --- a/git_review/tests/test_git_review.py +++ b/git_review/tests/test_git_review.py @@ -65,7 +65,9 @@ class GitReviewTestCase(tests.BaseGitReviewTestCase): """Test git-review on the just cloned repository.""" self._simple_change('test file modified', 'test commit message') self.assertNotIn('Change-Id:', self._run_git('log', '-1')) - self.assertIn('remote: New Changes:', self._run_git_review()) + gr = self._run_git_review() + self.assertIn('remote: SUCCESS', gr) + self.assertIn('test commit message [NEW]', gr) self.assertIn('Change-Id:', self._run_git('log', '-1')) def test_git_review_s(self): @@ -237,7 +239,10 @@ class GitReviewTestCase(tests.BaseGitReviewTestCase): self._simple_change('test file modified 4th time', 'test commit message 4') review_res = self._run_git_review('-y') - self.assertIn("Processing changes: new: 2", review_res) + self.assertIn("Processing changes: refs: 1, new: 2", review_res) + self.assertIn("remote: SUCCESS", review_res) + self.assertIn("test commit message 3 [NEW]", review_res) + self.assertIn("test commit message 4 [NEW]", review_res) def test_git_review_re(self): """Test git-review adding reviewers to changes.""" diff --git a/git_review/tests/utils.py b/git_review/tests/utils.py index 10013f70..edd9deb9 100644 --- a/git_review/tests/utils.py +++ b/git_review/tests/utils.py @@ -31,6 +31,8 @@ GERRIT_CONF_TMPL = """ listenAddress = %s:%s [httpd] listenUrl = http://%s:%s/ +[sendemail] + enable = false """ diff --git a/tox.ini b/tox.ini index 66148bc9..3543c94c 100644 --- a/tox.ini +++ b/tox.ini @@ -1,11 +1,6 @@ [tox] envlist = linters,docs,py3 ignore_basepython_conflict = true -# We skipsdist to force test-requirements and specifically pbr -# to be installed prior to installing git-review. This ensures that -# easy_install doesn't try to talk to pypi.org for pbr which fails -# on older python due to a lack of SNI support. -skipsdist = true [testenv] basepython = python3