test_compare.py
441 lines
| 20.4 KiB
| text/x-python
|
PythonLexer
r2241 | from rhodecode.tests import * | |||
r2684 | from rhodecode.model.repo import RepoModel | |||
from rhodecode.model.meta import Session | ||||
from rhodecode.model.db import Repository | ||||
from rhodecode.model.scm import ScmModel | ||||
from rhodecode.lib.vcs.backends.base import EmptyChangeset | ||||
r2241 | ||||
r2434 | ||||
r3023 | def _fork_repo(fork_name, vcs_type, parent=None): | |||
if vcs_type =='hg': | ||||
_REPO = HG_REPO | ||||
elif vcs_type == 'git': | ||||
_REPO = GIT_REPO | ||||
if parent: | ||||
_REPO = parent | ||||
form_data = dict( | ||||
repo_name=fork_name, | ||||
repo_name_full=fork_name, | ||||
repo_group=None, | ||||
repo_type=vcs_type, | ||||
description='', | ||||
private=False, | ||||
copy_permissions=False, | ||||
landing_rev='tip', | ||||
update_after_clone=False, | ||||
fork_parent_id=Repository.get_by_repo_name(_REPO), | ||||
) | ||||
r3380 | RepoModel().create_fork(form_data, cur_user=TEST_USER_ADMIN_LOGIN) | |||
r3023 | ||||
Session().commit() | ||||
return Repository.get_by_repo_name(fork_name) | ||||
def _commit_change(repo, filename, content, message, vcs_type, parent=None, newfile=False): | ||||
repo = Repository.get_by_repo_name(repo) | ||||
_cs = parent | ||||
if not parent: | ||||
_cs = EmptyChangeset(alias=vcs_type) | ||||
if newfile: | ||||
cs = ScmModel().create_node( | ||||
repo=repo.scm_instance, repo_name=repo.repo_name, | ||||
cs=_cs, user=TEST_USER_ADMIN_LOGIN, | ||||
author=TEST_USER_ADMIN_LOGIN, | ||||
message=message, | ||||
content=content, | ||||
f_path=filename | ||||
) | ||||
else: | ||||
cs = ScmModel().commit_change( | ||||
repo=repo.scm_instance, repo_name=repo.repo_name, | ||||
cs=parent, user=TEST_USER_ADMIN_LOGIN, | ||||
author=TEST_USER_ADMIN_LOGIN, | ||||
message=message, | ||||
content=content, | ||||
f_path=filename | ||||
) | ||||
return cs | ||||
r2241 | class TestCompareController(TestController): | |||
r3379 | def setUp(self): | |||
self.r1_id = None | ||||
self.r2_id = None | ||||
def tearDown(self): | ||||
if self.r2_id: | ||||
RepoModel().delete(self.r2_id) | ||||
if self.r1_id: | ||||
RepoModel().delete(self.r1_id) | ||||
Session().commit() | ||||
Session.remove() | ||||
r3023 | def test_compare_forks_on_branch_extra_commits_hg(self): | |||
r2684 | self.log_user() | |||
r3023 | repo1 = RepoModel().create_repo(repo_name='one', repo_type='hg', | |||
description='diff-test', | ||||
owner=TEST_USER_ADMIN_LOGIN) | ||||
Session().commit() | ||||
r3379 | self.r1_id = repo1.repo_id | |||
r3023 | #commit something ! | |||
cs0 = _commit_change(repo1.repo_name, filename='file1', content='line1\n', | ||||
message='commit1', vcs_type='hg', parent=None, newfile=True) | ||||
#fork this repo | ||||
repo2 = _fork_repo('one-fork', 'hg', parent='one') | ||||
r3379 | self.r2_id = repo2.repo_id | |||
r2684 | ||||
r3023 | #add two extra commit into fork | |||
cs1 = _commit_change(repo2.repo_name, filename='file1', content='line1\nline2\n', | ||||
message='commit2', vcs_type='hg', parent=cs0) | ||||
r2684 | ||||
r3023 | cs2 = _commit_change(repo2.repo_name, filename='file1', content='line1\nline2\nline3\n', | |||
message='commit3', vcs_type='hg', parent=cs1) | ||||
r3015 | ||||
r3023 | rev1 = 'default' | |||
rev2 = 'default' | ||||
r3331 | ||||
r3379 | response = self.app.get(url(controller='compare', action='index', | |||
repo_name=repo1.repo_name, | ||||
org_ref_type="branch", | ||||
org_ref=rev2, | ||||
other_repo=repo2.repo_name, | ||||
other_ref_type="branch", | ||||
other_ref=rev1, | ||||
Mads Kiilerich
|
r3486 | merge='1', | ||
r3379 | )) | |||
r3331 | ||||
r3379 | response.mustcontain('%s@%s -> %s@%s' % (repo1.repo_name, rev2, repo2.repo_name, rev1)) | |||
response.mustcontain("""Showing 2 commits""") | ||||
response.mustcontain("""1 file changed with 2 insertions and 0 deletions""") | ||||
r3015 | ||||
r3379 | response.mustcontain("""<div class="message tooltip" title="commit2" style="white-space:normal">commit2</div>""") | |||
response.mustcontain("""<div class="message tooltip" title="commit3" style="white-space:normal">commit3</div>""") | ||||
r3015 | ||||
r3379 | response.mustcontain("""<a href="/%s/changeset/%s">r1:%s</a>""" % (repo2.repo_name, cs1.raw_id, cs1.short_id)) | |||
response.mustcontain("""<a href="/%s/changeset/%s">r2:%s</a>""" % (repo2.repo_name, cs2.raw_id, cs2.short_id)) | ||||
## files | ||||
Mads Kiilerich
|
r3486 | response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?other_repo=%s&merge=1#C--826e8142e6ba">file1</a>""" % (repo1.repo_name, rev2, rev1, repo2.repo_name)) | ||
r3379 | #swap | |||
Mads Kiilerich
|
r3486 | response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?other_repo=%s&merge=True">[swap]</a>""" % (repo2.repo_name, rev1, rev2, repo1.repo_name)) | ||
r3029 | ||||
r3023 | def test_compare_forks_on_branch_extra_commits_origin_has_incomming_hg(self): | |||
r3015 | self.log_user() | |||
r3023 | repo1 = RepoModel().create_repo(repo_name='one', repo_type='hg', | |||
description='diff-test', | ||||
owner=TEST_USER_ADMIN_LOGIN) | ||||
Session().commit() | ||||
r3379 | self.r1_id = repo1.repo_id | |||
r3023 | #commit something ! | |||
cs0 = _commit_change(repo1.repo_name, filename='file1', content='line1\n', | ||||
message='commit1', vcs_type='hg', parent=None, newfile=True) | ||||
r3015 | ||||
r3023 | #fork this repo | |||
repo2 = _fork_repo('one-fork', 'hg', parent='one') | ||||
r3379 | self.r2_id = repo2.repo_id | |||
r3015 | ||||
r3023 | #now commit something to origin repo | |||
cs1_prim = _commit_change(repo1.repo_name, filename='file2', content='line1file2\n', | ||||
message='commit2', vcs_type='hg', parent=cs0, newfile=True) | ||||
#add two extra commit into fork | ||||
cs1 = _commit_change(repo2.repo_name, filename='file1', content='line1\nline2\n', | ||||
message='commit2', vcs_type='hg', parent=cs0) | ||||
cs2 = _commit_change(repo2.repo_name, filename='file1', content='line1\nline2\nline3\n', | ||||
message='commit3', vcs_type='hg', parent=cs1) | ||||
rev1 = 'default' | ||||
rev2 = 'default' | ||||
r2684 | ||||
r3379 | response = self.app.get(url(controller='compare', action='index', | |||
repo_name=repo1.repo_name, | ||||
org_ref_type="branch", | ||||
org_ref=rev2, | ||||
other_repo=repo2.repo_name, | ||||
other_ref_type="branch", | ||||
other_ref=rev1, | ||||
Mads Kiilerich
|
r3486 | merge='x', | ||
r3379 | )) | |||
response.mustcontain('%s@%s -> %s@%s' % (repo1.repo_name, rev2, repo2.repo_name, rev1)) | ||||
response.mustcontain("""Showing 2 commits""") | ||||
response.mustcontain("""1 file changed with 2 insertions and 0 deletions""") | ||||
r2684 | ||||
r3379 | response.mustcontain("""<div class="message tooltip" title="commit2" style="white-space:normal">commit2</div>""") | |||
response.mustcontain("""<div class="message tooltip" title="commit3" style="white-space:normal">commit3</div>""") | ||||
r2684 | ||||
r3379 | response.mustcontain("""<a href="/%s/changeset/%s">r1:%s</a>""" % (repo2.repo_name, cs1.raw_id, cs1.short_id)) | |||
response.mustcontain("""<a href="/%s/changeset/%s">r2:%s</a>""" % (repo2.repo_name, cs2.raw_id, cs2.short_id)) | ||||
## files | ||||
Mads Kiilerich
|
r3486 | response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?other_repo=%s&merge=x#C--826e8142e6ba">file1</a>""" % (repo1.repo_name, rev2, rev1, repo2.repo_name)) | ||
r3379 | #swap | |||
Mads Kiilerich
|
r3486 | response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?other_repo=%s&merge=True">[swap]</a>""" % (repo2.repo_name, rev1, rev2, repo1.repo_name)) | ||
r2684 | ||||
r3331 | def test_compare_cherry_pick_changesets_from_bottom(self): | |||
r3380 | ||||
# repo1: | ||||
# cs0: | ||||
# cs1: | ||||
# repo1-fork- in which we will cherry pick bottom changesets | ||||
# cs0: | ||||
# cs1: | ||||
# cs2: x | ||||
# cs3: x | ||||
# cs4: x | ||||
# cs5: | ||||
r3331 | #make repo1, and cs1+cs2 | |||
self.log_user() | ||||
r3023 | ||||
r3331 | repo1 = RepoModel().create_repo(repo_name='repo1', repo_type='hg', | |||
description='diff-test', | ||||
owner=TEST_USER_ADMIN_LOGIN) | ||||
Session().commit() | ||||
r3379 | self.r1_id = repo1.repo_id | |||
r3331 | #commit something ! | |||
r3380 | cs0 = _commit_change(repo1.repo_name, filename='file1', content='line1\n', | |||
r3331 | message='commit1', vcs_type='hg', parent=None, | |||
newfile=True) | ||||
r3380 | cs1 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\n', | |||
message='commit2', vcs_type='hg', parent=cs0) | ||||
r3331 | #fork this repo | |||
repo2 = _fork_repo('repo1-fork', 'hg', parent='repo1') | ||||
r3379 | self.r2_id = repo2.repo_id | |||
r3331 | #now make cs3-6 | |||
r3380 | cs2 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\n', | |||
message='commit3', vcs_type='hg', parent=cs1) | ||||
cs3 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\n', | ||||
message='commit4', vcs_type='hg', parent=cs2) | ||||
cs4 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\nline5\n', | ||||
message='commit5', vcs_type='hg', parent=cs3) | ||||
cs5 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\nline5\nline6\n', | ||||
message='commit6', vcs_type='hg', parent=cs4) | ||||
response = self.app.get(url(controller='compare', action='index', | ||||
repo_name=repo2.repo_name, | ||||
Mads Kiilerich
|
r3484 | org_ref_type="rev", | ||
org_ref=cs1.short_id, # parent of cs2, in repo2 | ||||
r3380 | other_repo=repo1.repo_name, | |||
Mads Kiilerich
|
r3484 | other_ref_type="rev", | ||
other_ref=cs4.short_id, | ||||
Mads Kiilerich
|
r3486 | merge='True', | ||
r3380 | )) | |||
Mads Kiilerich
|
r3484 | response.mustcontain('%s@%s -> %s@%s' % (repo2.repo_name, cs1.short_id, repo1.repo_name, cs4.short_id)) | ||
r3380 | response.mustcontain("""Showing 3 commits""") | |||
response.mustcontain("""1 file changed with 3 insertions and 0 deletions""") | ||||
response.mustcontain("""<div class="message tooltip" title="commit3" style="white-space:normal">commit3</div>""") | ||||
response.mustcontain("""<div class="message tooltip" title="commit4" style="white-space:normal">commit4</div>""") | ||||
response.mustcontain("""<div class="message tooltip" title="commit5" style="white-space:normal">commit5</div>""") | ||||
response.mustcontain("""<a href="/%s/changeset/%s">r2:%s</a>""" % (repo1.repo_name, cs2.raw_id, cs2.short_id)) | ||||
response.mustcontain("""<a href="/%s/changeset/%s">r3:%s</a>""" % (repo1.repo_name, cs3.raw_id, cs3.short_id)) | ||||
response.mustcontain("""<a href="/%s/changeset/%s">r4:%s</a>""" % (repo1.repo_name, cs4.raw_id, cs4.short_id)) | ||||
## files | ||||
response.mustcontain("""#C--826e8142e6ba">file1</a>""") | ||||
r3331 | ||||
r3380 | def test_compare_cherry_pick_changesets_from_top(self): | |||
# repo1: | ||||
# cs0: | ||||
# cs1: | ||||
# repo1-fork- in which we will cherry pick bottom changesets | ||||
# cs0: | ||||
# cs1: | ||||
# cs2: | ||||
# cs3: x | ||||
# cs4: x | ||||
# cs5: x | ||||
# | ||||
#make repo1, and cs1+cs2 | ||||
self.log_user() | ||||
repo1 = RepoModel().create_repo(repo_name='repo1', repo_type='hg', | ||||
description='diff-test', | ||||
owner=TEST_USER_ADMIN_LOGIN) | ||||
Session().commit() | ||||
self.r1_id = repo1.repo_id | ||||
#commit something ! | ||||
cs0 = _commit_change(repo1.repo_name, filename='file1', content='line1\n', | ||||
message='commit1', vcs_type='hg', parent=None, | ||||
newfile=True) | ||||
cs1 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\n', | ||||
message='commit2', vcs_type='hg', parent=cs0) | ||||
#fork this repo | ||||
repo2 = _fork_repo('repo1-fork', 'hg', parent='repo1') | ||||
self.r2_id = repo2.repo_id | ||||
#now make cs3-6 | ||||
cs2 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\n', | ||||
message='commit3', vcs_type='hg', parent=cs1) | ||||
cs3 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\n', | ||||
message='commit4', vcs_type='hg', parent=cs2) | ||||
cs4 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\nline5\n', | ||||
message='commit5', vcs_type='hg', parent=cs3) | ||||
cs5 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\nline5\nline6\n', | ||||
message='commit6', vcs_type='hg', parent=cs4) | ||||
r3379 | response = self.app.get(url(controller='compare', action='index', | |||
Mads Kiilerich
|
r3484 | repo_name=repo1.repo_name, | ||
org_ref_type="rev", | ||||
org_ref=cs2.short_id, # parent of cs3, not in repo2 | ||||
other_ref_type="rev", | ||||
other_ref=cs5.short_id, | ||||
Mads Kiilerich
|
r3486 | merge='1', | ||
r3379 | )) | |||
r3380 | ||||
Mads Kiilerich
|
r3484 | response.mustcontain('%s@%s -> %s@%s' % (repo1.repo_name, cs2.short_id, repo1.repo_name, cs5.short_id)) | ||
r3379 | response.mustcontain("""Showing 3 commits""") | |||
response.mustcontain("""1 file changed with 3 insertions and 0 deletions""") | ||||
r3331 | ||||
r3379 | response.mustcontain("""<div class="message tooltip" title="commit4" style="white-space:normal">commit4</div>""") | |||
response.mustcontain("""<div class="message tooltip" title="commit5" style="white-space:normal">commit5</div>""") | ||||
r3380 | response.mustcontain("""<div class="message tooltip" title="commit6" style="white-space:normal">commit6</div>""") | |||
r3379 | ||||
r3380 | response.mustcontain("""<a href="/%s/changeset/%s">r3:%s</a>""" % (repo1.repo_name, cs3.raw_id, cs3.short_id)) | |||
response.mustcontain("""<a href="/%s/changeset/%s">r4:%s</a>""" % (repo1.repo_name, cs4.raw_id, cs4.short_id)) | ||||
response.mustcontain("""<a href="/%s/changeset/%s">r5:%s</a>""" % (repo1.repo_name, cs5.raw_id, cs5.short_id)) | ||||
r3379 | ## files | |||
r3380 | response.mustcontain("""#C--826e8142e6ba">file1</a>""") | |||
r3331 | ||||
def test_compare_cherry_pick_changeset_mixed_branches(self): | ||||
""" | ||||
""" | ||||
pass | ||||
#TODO write this tastecase | ||||
def test_compare_remote_branches_hg(self): | ||||
self.log_user() | ||||
r3380 | repo2 = _fork_repo(HG_FORK, 'hg') | |||
self.r2_id = repo2.repo_id | ||||
r3331 | rev1 = '56349e29c2af' | |||
rev2 = '7d4bc8ec6be5' | ||||
r3379 | response = self.app.get(url(controller='compare', action='index', | |||
repo_name=HG_REPO, | ||||
org_ref_type="rev", | ||||
org_ref=rev1, | ||||
other_ref_type="rev", | ||||
other_ref=rev2, | ||||
other_repo=HG_FORK, | ||||
Mads Kiilerich
|
r3486 | merge='1', | ||
r3379 | )) | |||
response.mustcontain('%s@%s -> %s@%s' % (HG_REPO, rev1, HG_FORK, rev2)) | ||||
## outgoing changesets between those revisions | ||||
r3331 | ||||
r3379 | response.mustcontain("""<a href="/%s/changeset/2dda4e345facb0ccff1a191052dd1606dba6781d">r4:2dda4e345fac</a>""" % (HG_FORK)) | |||
response.mustcontain("""<a href="/%s/changeset/6fff84722075f1607a30f436523403845f84cd9e">r5:6fff84722075</a>""" % (HG_FORK)) | ||||
response.mustcontain("""<a href="/%s/changeset/7d4bc8ec6be56c0f10425afb40b6fc315a4c25e7">r6:%s</a>""" % (HG_FORK, rev2)) | ||||
r3331 | ||||
r3379 | ## files | |||
Mads Kiilerich
|
r3486 | response.mustcontain("""<a href="/%s/compare/rev@%s...rev@%s?other_repo=%s&merge=1#C--9c390eb52cd6">vcs/backends/hg.py</a>""" % (HG_REPO, rev1, rev2, HG_FORK)) | ||
response.mustcontain("""<a href="/%s/compare/rev@%s...rev@%s?other_repo=%s&merge=1#C--41b41c1f2796">vcs/backends/__init__.py</a>""" % (HG_REPO, rev1, rev2, HG_FORK)) | ||||
response.mustcontain("""<a href="/%s/compare/rev@%s...rev@%s?other_repo=%s&merge=1#C--2f574d260608">vcs/backends/base.py</a>""" % (HG_REPO, rev1, rev2, HG_FORK)) | ||||
r3331 | ||||
def test_org_repo_new_commits_after_forking_simple_diff(self): | ||||
self.log_user() | ||||
repo1 = RepoModel().create_repo(repo_name='one', repo_type='hg', | ||||
description='diff-test', | ||||
owner=TEST_USER_ADMIN_LOGIN) | ||||
Session().commit() | ||||
r3379 | self.r1_id = repo1.repo_id | |||
r3331 | r1_name = repo1.repo_name | |||
#commit something initially ! | ||||
cs0 = ScmModel().create_node( | ||||
repo=repo1.scm_instance, repo_name=r1_name, | ||||
cs=EmptyChangeset(alias='hg'), user=TEST_USER_ADMIN_LOGIN, | ||||
author=TEST_USER_ADMIN_LOGIN, | ||||
message='commit1', | ||||
content='line1', | ||||
f_path='file1' | ||||
) | ||||
Session().commit() | ||||
self.assertEqual(repo1.scm_instance.revisions, [cs0.raw_id]) | ||||
#fork the repo1 | ||||
repo2 = RepoModel().create_repo(repo_name='one-fork', repo_type='hg', | ||||
description='compare-test', | ||||
clone_uri=repo1.repo_full_path, | ||||
owner=TEST_USER_ADMIN_LOGIN, fork_of='one') | ||||
Session().commit() | ||||
self.assertEqual(repo2.scm_instance.revisions, [cs0.raw_id]) | ||||
r3379 | self.r2_id = repo2.repo_id | |||
r3331 | r2_name = repo2.repo_name | |||
#make 3 new commits in fork | ||||
cs1 = ScmModel().create_node( | ||||
repo=repo2.scm_instance, repo_name=r2_name, | ||||
cs=repo2.scm_instance[-1], user=TEST_USER_ADMIN_LOGIN, | ||||
author=TEST_USER_ADMIN_LOGIN, | ||||
message='commit1-fork', | ||||
content='file1-line1-from-fork', | ||||
f_path='file1-fork' | ||||
) | ||||
cs2 = ScmModel().create_node( | ||||
repo=repo2.scm_instance, repo_name=r2_name, | ||||
cs=cs1, user=TEST_USER_ADMIN_LOGIN, | ||||
author=TEST_USER_ADMIN_LOGIN, | ||||
message='commit2-fork', | ||||
content='file2-line1-from-fork', | ||||
f_path='file2-fork' | ||||
) | ||||
cs3 = ScmModel().create_node( | ||||
repo=repo2.scm_instance, repo_name=r2_name, | ||||
cs=cs2, user=TEST_USER_ADMIN_LOGIN, | ||||
author=TEST_USER_ADMIN_LOGIN, | ||||
message='commit3-fork', | ||||
content='file3-line1-from-fork', | ||||
f_path='file3-fork' | ||||
) | ||||
#compare ! | ||||
rev1 = 'default' | ||||
rev2 = 'default' | ||||
r3379 | response = self.app.get(url(controller='compare', action='index', | |||
repo_name=r2_name, | ||||
org_ref_type="branch", | ||||
org_ref=rev1, | ||||
other_ref_type="branch", | ||||
other_ref=rev2, | ||||
other_repo=r1_name, | ||||
Mads Kiilerich
|
r3486 | merge='1', | ||
r3379 | )) | |||
response.mustcontain('%s@%s -> %s@%s' % (r2_name, rev1, r1_name, rev2)) | ||||
response.mustcontain('No files') | ||||
response.mustcontain('No changesets') | ||||
r3331 | ||||
r3379 | #add new commit into parent ! | |||
cs0 = ScmModel().create_node( | ||||
repo=repo1.scm_instance, repo_name=r1_name, | ||||
cs=EmptyChangeset(alias='hg'), user=TEST_USER_ADMIN_LOGIN, | ||||
author=TEST_USER_ADMIN_LOGIN, | ||||
message='commit2-parent', | ||||
content='line1-added-after-fork', | ||||
f_path='file2' | ||||
) | ||||
#compare ! | ||||
rev1 = 'default' | ||||
rev2 = 'default' | ||||
response = self.app.get(url(controller='compare', action='index', | ||||
repo_name=r2_name, | ||||
org_ref_type="branch", | ||||
org_ref=rev1, | ||||
other_ref_type="branch", | ||||
other_ref=rev2, | ||||
other_repo=r1_name, | ||||
Mads Kiilerich
|
r3486 | merge='1', | ||
r3379 | )) | |||
r3331 | ||||
r3379 | response.mustcontain('%s@%s -> %s@%s' % (r2_name, rev1, r1_name, rev2)) | |||
response.mustcontain("""commit2-parent""") | ||||
response.mustcontain("""1 file changed with 1 insertions and 0 deletions""") | ||||
response.mustcontain("""line1-added-after-fork""") | ||||