# HG changeset patch # User Marcin Kuzminski # Date 2013-02-05 01:02:36 # Node ID 0379e15f0b8509dd86ab5836eb9e1db7c8c8c1af # Parent dcd1c159a13244630209a922e7d7a102dfaa8d7c added some failing tests for compare using cherry pick changesets, to be fixed later diff --git a/rhodecode/tests/functional/test_compare.py b/rhodecode/tests/functional/test_compare.py --- a/rhodecode/tests/functional/test_compare.py +++ b/rhodecode/tests/functional/test_compare.py @@ -88,16 +88,18 @@ class TestCompareController(TestControll rev1 = 'default' rev2 = 'default' - 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, - )) + try: + 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, + )) + 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""") @@ -146,16 +148,16 @@ class TestCompareController(TestControll rev1 = 'default' rev2 = 'default' - 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, - )) try: + 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, + )) 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""") @@ -173,176 +175,295 @@ class TestCompareController(TestControll RepoModel().delete(r2_id) RepoModel().delete(r1_id) + def test_compare_cherry_pick_changesets_from_bottom(self): + """ + repo1: + cs1: + cs2: + repo1-fork- in which we will cherry pick bottom changesets + cs1: + cs2: + cs3: x + cs4: x + cs5: x + cs6: + """ + #make repo1, and cs1+cs2 + self.log_user() -# def test_compare_remote_repos_remote_flag_off(self): -# self.log_user() -# _fork_repo(HG_FORK, 'hg') -# -# rev1 = '56349e29c2af' -# rev2 = '7d4bc8ec6be5' -# -# 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, -# repo=HG_FORK, -# bundle=False, -# )) -# -# try: -# response.mustcontain('%s@%s -> %s@%s' % (HG_REPO, rev1, HG_FORK, rev2)) -# ## outgoing changesets between those revisions -# -# response.mustcontain("""r4:2dda4e345fac""" % (HG_REPO)) -# response.mustcontain("""r5:6fff84722075""" % (HG_REPO)) -# response.mustcontain("""r6:%s""" % (HG_REPO, rev2)) -# -# ## files -# response.mustcontain("""vcs/backends/hg.py""" % (HG_REPO, rev1, rev2)) -# response.mustcontain("""vcs/backends/__init__.py""" % (HG_REPO, rev1, rev2)) -# response.mustcontain("""vcs/backends/base.py""" % (HG_REPO, rev1, rev2)) -# finally: -# RepoModel().delete(HG_FORK) + repo1 = RepoModel().create_repo(repo_name='repo1', repo_type='hg', + description='diff-test', + owner=TEST_USER_ADMIN_LOGIN) + r1_id = repo1.repo_id + Session().commit() + #commit something ! + cs1 = _commit_change(repo1.repo_name, filename='file1', content='line1\n', + message='commit1', vcs_type='hg', parent=None, + newfile=True) + cs2 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\n', + message='commit2', vcs_type='hg', parent=cs1) + #fork this repo + repo2 = _fork_repo('repo1-fork', 'hg', parent='repo1') + Session().commit() + r2_id = repo1.repo_id + #now make cs3-6 + cs3 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\n', + message='commit3', vcs_type='hg', parent=cs2) + cs4 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\n', + message='commit4', vcs_type='hg', parent=cs3) + cs5 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\nline5\n', + message='commit5', vcs_type='hg', parent=cs4) + cs6 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\nline5\nline6\n', + message='commit6', vcs_type='hg', parent=cs5) + + rev1 = 'tip' + rev2 = 'tip' + + try: + response = self.app.get(url(controller='compare', action='index', + repo_name=repo2.repo_name, + org_ref_type="tag", + org_ref=rev1, + other_repo=repo1.repo_name, + other_ref_type="tag", + other_ref=rev2, + rev_start=cs3.raw_id, + rev_end=cs5.raw_id, + )) + response.mustcontain('%s@%s -> %s@%s' % (repo2.repo_name, rev1, repo1.repo_name, rev2)) + response.mustcontain("""Showing 3 commits""") + response.mustcontain("""1 file changed with 3 insertions and 0 deletions""") + + response.mustcontain("""
commit3
""") + response.mustcontain("""
commit4
""") + response.mustcontain("""
commit5
""") + + response.mustcontain("""r3:%s""" % (repo2.repo_name, cs3.raw_id, cs3.short_id)) + response.mustcontain("""r4:%s""" % (repo2.repo_name, cs4.raw_id, cs4.short_id)) + response.mustcontain("""r5:%s""" % (repo2.repo_name, cs5.raw_id, cs5.short_id)) + ## files + response.mustcontain("""file1""" % (repo2.repo_name, rev1, rev2, repo1.repo_name)) + #swap + response.mustcontain("""[swap]""" % (repo1.repo_name, rev1, rev2, repo2.repo_name)) + finally: + RepoModel().delete(r2_id) + RepoModel().delete(r1_id) + def test_compare_cherry_pick_changesets_from_top(self): + """ + repo1: + cs1: + cs2: + repo1-fork- in which we will cherry pick bottom changesets + cs1: + cs2: + cs3: + cs4: x + cs5: x + cs6: 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) + r1_id = repo1.repo_id + Session().commit() + #commit something ! + cs1 = _commit_change(repo1.repo_name, filename='file1', content='line1\n', + message='commit1', vcs_type='hg', parent=None, + newfile=True) + cs2 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\n', + message='commit2', vcs_type='hg', parent=cs1) + #fork this repo + repo2 = _fork_repo('repo1-fork', 'hg', parent='repo1') + Session().commit() + r2_id = repo1.repo_id + #now make cs3-6 + cs3 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\n', + message='commit3', vcs_type='hg', parent=cs2) + cs4 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\n', + message='commit4', vcs_type='hg', parent=cs3) + cs5 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\nline5\n', + message='commit5', vcs_type='hg', parent=cs4) + cs6 = _commit_change(repo1.repo_name, filename='file1', content='line1\nline2\nline3\nline4\nline5\nline6\n', + message='commit6', vcs_type='hg', parent=cs5) + + rev1 = 'tip' + rev2 = 'tip' + + try: + response = self.app.get(url(controller='compare', action='index', + repo_name=repo2.repo_name, + org_ref_type="tag", + org_ref=rev1, + other_repo=repo1.repo_name, + other_ref_type="tag", + other_ref=rev2, + rev_start=cs4.raw_id, + rev_end=cs6.raw_id, + )) + + response.mustcontain('%s@%s -> %s@%s' % (repo2.repo_name, rev1, repo1.repo_name, rev2)) + response.mustcontain("""Showing 3 commits""") + response.mustcontain("""1 file changed with 3 insertions and 0 deletions""") + + response.mustcontain("""
commit4
""") + response.mustcontain("""
commit5
""") + response.mustcontain("""
commit6
""") -# -# def test_compare_remote_branches_hg(self): -# self.log_user() -# -# _fork_repo(HG_FORK, 'hg') -# -# rev1 = '56349e29c2af' -# rev2 = '7d4bc8ec6be5' -# -# 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, -# repo=HG_FORK, -# )) -# -# try: -# response.mustcontain('%s@%s -> %s@%s' % (HG_REPO, rev1, HG_FORK, rev2)) -# ## outgoing changesets between those revisions -# -# response.mustcontain("""r4:2dda4e345fac""" % (HG_REPO)) -# response.mustcontain("""r5:6fff84722075""" % (HG_REPO)) -# response.mustcontain("""r6:%s""" % (HG_REPO, rev2)) -# -# ## files -# response.mustcontain("""vcs/backends/hg.py""" % (HG_REPO, rev1, rev2)) -# response.mustcontain("""vcs/backends/__init__.py""" % (HG_REPO, rev1, rev2)) -# response.mustcontain("""vcs/backends/base.py""" % (HG_REPO, rev1, rev2)) -# finally: -# RepoModel().delete(HG_FORK) -# -# 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() -# r1_id = repo1.repo_id -# 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]) -# r2_id = repo2.repo_id -# 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' -# 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, -# repo=r1_name, -# bundle=False, -# )) -# -# try: -# #response.mustcontain('%s@%s -> %s@%s' % (r2_name, rev1, r1_name, rev2)) -# -# #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', -# content='line1', -# 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, -# repo=r1_name, -# bundle=False -# )) -# -# response.mustcontain('%s@%s -> %s@%s' % (r2_name, rev1, r1_name, rev2)) -# response.mustcontain("""file1-line1-from-fork""") -# response.mustcontain("""file2-line1-from-fork""") -# response.mustcontain("""file3-line1-from-fork""") -# self.assertFalse("""file2""" in response.body) # new commit from parent -# self.assertFalse("""line1-from-new-parent""" in response.body) -# finally: -# RepoModel().delete(r2_id) -# RepoModel().delete(r1_id) + response.mustcontain("""r4:%s""" % (repo2.repo_name, cs4.raw_id, cs4.short_id)) + response.mustcontain("""r5:%s""" % (repo2.repo_name, cs5.raw_id, cs5.short_id)) + response.mustcontain("""r6:%s""" % (repo2.repo_name, cs6.raw_id, cs6.short_id)) + ## files + response.mustcontain("""file1""" % (repo2.repo_name, rev1, rev2, repo1.repo_name)) + #swap + response.mustcontain("""[swap]""" % (repo1.repo_name, rev1, rev2, repo2.repo_name)) + finally: + RepoModel().delete(r2_id) + RepoModel().delete(r1_id) + + def test_compare_cherry_pick_changeset_mixed_branches(self): + """ + + """ + pass + #TODO write this tastecase + + def test_compare_remote_branches_hg(self): + self.log_user() + + _fork_repo(HG_FORK, 'hg') + + rev1 = '56349e29c2af' + rev2 = '7d4bc8ec6be5' + + try: + 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, + )) + response.mustcontain('%s@%s -> %s@%s' % (HG_REPO, rev1, HG_FORK, rev2)) + ## outgoing changesets between those revisions + + response.mustcontain("""r4:2dda4e345fac""" % (HG_REPO)) + response.mustcontain("""r5:6fff84722075""" % (HG_REPO)) + response.mustcontain("""r6:%s""" % (HG_REPO, rev2)) + + ## files + response.mustcontain("""vcs/backends/hg.py""" % (HG_REPO, rev1, rev2)) + response.mustcontain("""vcs/backends/__init__.py""" % (HG_REPO, rev1, rev2)) + response.mustcontain("""vcs/backends/base.py""" % (HG_REPO, rev1, rev2)) + finally: + RepoModel().delete(HG_FORK) + + 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() + r1_id = repo1.repo_id + 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]) + r2_id = repo2.repo_id + 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' + + try: + 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, + repo=r1_name, + )) + #response.mustcontain('%s@%s -> %s@%s' % (r2_name, rev1, r1_name, rev2)) + + #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', + content='line1', + 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, + repo=r1_name, + bundle=False + )) + + response.mustcontain('%s@%s -> %s@%s' % (r2_name, rev1, r1_name, rev2)) + response.mustcontain("""file1-line1-from-fork""") + response.mustcontain("""file2-line1-from-fork""") + response.mustcontain("""file3-line1-from-fork""") + self.assertFalse("""file2""" in response.body) # new commit from parent + self.assertFalse("""line1-from-new-parent""" in response.body) + finally: + RepoModel().delete(r2_id) + RepoModel().delete(r1_id) + Session()