diff options
| author | Mads Kiilerich <madski@unity3d.com> | 2013-02-28 17:54:42 +0100 |
|---|---|---|
| committer | Mads Kiilerich <madski@unity3d.com> | 2013-02-28 17:54:42 +0100 |
| commit | ba75d7ea8eb94830732cd036d9b04aefa922c2bf (patch) | |
| tree | b96e4f42a7ee55ecaa0d1df7a6a437d27d31356d /rhodecode/tests | |
| parent | cc28a933d9b75a22cdcd1411ae75d299f713806e (diff) | |
compare: cleanup of as_form handling
as_form is only used when loaded 'partial' from pullrequest.html. Including it
in the 'Detailed compare view' url did thus not do any harm.
as_form is never used in pullrequest_show.html or compare_diff.html and it is
thus never used together with swap_url ... which is why the wrong
'as_form=None' or 'as_form=False' in urls didn't do any harm.
--HG--
branch : beta
extra : source : d3f835437bc61e205ceb89b2b300a84739c4e121
Diffstat (limited to 'rhodecode/tests')
| -rw-r--r-- | rhodecode/tests/functional/test_compare.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/rhodecode/tests/functional/test_compare.py b/rhodecode/tests/functional/test_compare.py index 4ea7c611..d7827433 100644 --- a/rhodecode/tests/functional/test_compare.py +++ b/rhodecode/tests/functional/test_compare.py @@ -120,7 +120,7 @@ class TestCompareController(TestController): ## files response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?other_repo=%s#C--826e8142e6ba">file1</a>""" % (repo1.repo_name, rev2, rev1, repo2.repo_name)) #swap - response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?as_form=None&other_repo=%s">[swap]</a>""" % (repo2.repo_name, rev1, rev2, repo1.repo_name)) + response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?other_repo=%s">[swap]</a>""" % (repo2.repo_name, rev1, rev2, repo1.repo_name)) def test_compare_forks_on_branch_extra_commits_origin_has_incomming_hg(self): self.log_user() @@ -173,7 +173,7 @@ class TestCompareController(TestController): ## files response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?other_repo=%s#C--826e8142e6ba">file1</a>""" % (repo1.repo_name, rev2, rev1, repo2.repo_name)) #swap - response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?as_form=None&other_repo=%s">[swap]</a>""" % (repo2.repo_name, rev1, rev2, repo1.repo_name)) + response.mustcontain("""<a href="/%s/compare/branch@%s...branch@%s?other_repo=%s">[swap]</a>""" % (repo2.repo_name, rev1, rev2, repo1.repo_name)) def test_compare_cherry_pick_changesets_from_bottom(self): |