diff --git a/tests/test_pagure_flask_internal.py b/tests/test_pagure_flask_internal.py
index 1310264..74bf72b 100644
--- a/tests/test_pagure_flask_internal.py
+++ b/tests/test_pagure_flask_internal.py
@@ -1414,9 +1414,11 @@ class PagureFlaskInternaltests(tests.Modeltests):
output = self.app.get(js_data['url'])
js_data2 = json.loads(output.data.decode('utf-8'))
- self.assertDictEqual(
- js_data2,
- {u'results': u"Reference 'refs/heads/master' not found"}
+ self.assertTrue(
+ js_data2 in [
+ {u'results': u"reference 'refs/heads/master' not found"},
+ {u'results': u"Reference 'refs/heads/master' not found"}
+ ]
)
def test_get_stats_commits_git_populated(self):
@@ -1526,9 +1528,11 @@ class PagureFlaskInternaltests(tests.Modeltests):
output = self.app.get(js_data['url'])
js_data2 = json.loads(output.data.decode('utf-8'))
- self.assertDictEqual(
- js_data2,
- {u'results': u"Reference 'refs/heads/master' not found"}
+ self.assertTrue(
+ js_data2 in [
+ {u'results': u"reference 'refs/heads/master' not found"},
+ {u'results': u"Reference 'refs/heads/master' not found"}
+ ]
)
def test_get_stats_commits_trend_git_populated(self):
diff --git a/tests/test_pagure_flask_ui_fork.py b/tests/test_pagure_flask_ui_fork.py
index ba528a2..53c1149 100644
--- a/tests/test_pagure_flask_ui_fork.py
+++ b/tests/test_pagure_flask_ui_fork.py
@@ -89,6 +89,7 @@ class PagureFlaskForktests(tests.Modeltests):
# list of binary strings representing parents of the new commit
[]
)
+ time.sleep(1)
refname = 'refs/heads/master:refs/heads/master'
ori_remote = clone_repo.remotes[0]
PagureRepo.push(ori_remote, refname)
@@ -2189,8 +2190,8 @@ index 0000000..2a552bb
commits = _get_commits(output.data)
self.assertEqual(commits, [
'Merge #1 `PR from the feature branch`',
- 'Add sources file for testing',
'A commit on branch feature',
+ 'Add sources file for testing',
])
# Check if the closing notification was added
diff --git a/tests/test_pagure_flask_ui_issues.py b/tests/test_pagure_flask_ui_issues.py
index c4acb59..8b40fd5 100644
--- a/tests/test_pagure_flask_ui_issues.py
+++ b/tests/test_pagure_flask_ui_issues.py
@@ -14,6 +14,7 @@ import pkg_resources
from unittest.case import SkipTest
import json
import unittest
+import urlparse
import shutil
import sys
import os
@@ -708,26 +709,38 @@ class PagureFlaskIssuestests(tests.Modeltests):
# down next to the Opened column
th_elements = re.findall(r'
(.*?) | ',
output.data, re.M | re.S)
- href = ('href="/test/issues?status=Open&order_key=title&'
- 'order=desc"')
- self.assertIn(href, th_elements[0])
- href = ('/test/issues?status=Open&order_key=date_created&'
- 'order=asc"')
- self.assertIn(href, th_elements[1])
+ self.assertDictEqual(
+ {'status': ['Open'], 'order_key': ['title'], 'order': ['desc']},
+ urlparse.parse_qs(urlparse.urlparse(
+ th_elements[0].split('"')[1]).query)
+ )
+ self.assertDictEqual(
+ {'status': ['Open'], 'order_key': ['date_created'], 'order': ['asc']},
+ urlparse.parse_qs(urlparse.urlparse(
+ th_elements[1].split('"')[1]).query)
+ )
arrow = ''
self.assertIn(arrow, th_elements[1])
- href = ('href="/test/issues?status=Open&order_key=last_updated&'
- 'amp;order=desc"')
- self.assertIn(href, th_elements[2])
- href = ('href="/test/issues?status=Open&order_key=priority&'
- 'order=desc"')
- self.assertIn(href, th_elements[3])
- href = ('href="/test/issues?status=Open&order_key=user&'
- 'order=desc"')
- self.assertIn(href, th_elements[4])
- href = ('href="/test/issues?status=Open&order_key=assignee&'
- 'order=desc"')
- self.assertIn(href, th_elements[5])
+ self.assertDictEqual(
+ {'status': ['Open'], 'order_key': ['last_updated'], 'order': ['desc']},
+ urlparse.parse_qs(urlparse.urlparse(
+ th_elements[2].split('"')[1]).query)
+ )
+ self.assertDictEqual(
+ {'status': ['Open'], 'order_key': ['priority'], 'order': ['desc']},
+ urlparse.parse_qs(urlparse.urlparse(
+ th_elements[3].split('"')[1]).query)
+ )
+ self.assertDictEqual(
+ {'status': ['Open'], 'order_key': ['user'], 'order': ['desc']},
+ urlparse.parse_qs(urlparse.urlparse(
+ th_elements[4].split('"')[1]).query)
+ )
+ self.assertDictEqual(
+ {'status': ['Open'], 'order_key': ['assignee'], 'order': ['desc']},
+ urlparse.parse_qs(urlparse.urlparse(
+ th_elements[5].split('"')[1]).query)
+ )
# Status = closed (all but open)
output = self.app.get('/test/issues?status=cloSED')
diff --git a/tests/test_pagure_flask_ui_repo.py b/tests/test_pagure_flask_ui_repo.py
index 13e091e..2de46a5 100644
--- a/tests/test_pagure_flask_ui_repo.py
+++ b/tests/test_pagure_flask_ui_repo.py
@@ -1863,11 +1863,13 @@ class PagureFlaskRepotests(tests.Modeltests):
tests.add_commit_git_repo(
os.path.join(self.path, 'repos', 'test.git'), ncommits=1)
c1 = repo.revparse_single('HEAD')
+ time.sleep(1)
# Add another commit to git repo
tests.add_commit_git_repo(
os.path.join(self.path, 'repos', 'test.git'), ncommits=1)
c2 = repo.revparse_single('HEAD')
+ time.sleep(1)
# Add one more commit to git repo
tests.add_commit_git_repo(
diff --git a/tests/test_pagure_lib_git_diff_pr.py b/tests/test_pagure_lib_git_diff_pr.py
index a82407e..a42af03 100644
--- a/tests/test_pagure_lib_git_diff_pr.py
+++ b/tests/test_pagure_lib_git_diff_pr.py
@@ -107,6 +107,7 @@ class PagureFlaskForkPrtests(tests.Modeltests):
# list of binary strings representing parents of the new commit
parents
)
+ time.sleep(1)
# Push to the main repo
refname = 'refs/heads/master:refs/heads/master'
@@ -142,6 +143,7 @@ class PagureFlaskForkPrtests(tests.Modeltests):
# list of binary strings representing parents of the new commit
[last_commit.oid.hex]
)
+ time.sleep(1)
# Push to the main repo
refname = 'refs/heads/master:refs/heads/master'
ori_remote = clone_repo.remotes[0]
@@ -174,6 +176,7 @@ class PagureFlaskForkPrtests(tests.Modeltests):
# list of binary strings representing parents of the new commit
[last_commit.oid.hex]
)
+ time.sleep(1)
with open(os.path.join(repopath, 'sources'), 'w') as stream:
stream.write('foo\n bar\nbaz\n')
diff --git a/tests/test_pagure_lib_gitolite_config.py b/tests/test_pagure_lib_gitolite_config.py
index fa2c845..6b64a13 100644
--- a/tests/test_pagure_lib_gitolite_config.py
+++ b/tests/test_pagure_lib_gitolite_config.py
@@ -728,7 +728,7 @@ repo requests/test
pagure.lib.SESSIONMAKER = self.session.session_factory
pagure.lib.tasks.generate_gitolite_acls(
- self=None, namespace=None, name='test', user=None, group=None)
+ namespace=None, name='test', user=None, group=None)
get_helper.assert_called_with('gitolite3')
args = helper.generate_acls.call_args