hgsubversion: tests/test_urls.py annotate

annotate tests/test_urls.py @ 1023:7a262ecae4f3

svnwrap: use get_file instead of list_props during replay This patch has an enormous effect on the time taken to import commits which add many files. Specifically, it makes them much faster for pulls over the network because it avoids creating a new connection to subversion for every added file. In my testing, it dropped the time taken to import a revision that adds ~6500 files from approximately 1 hour to 30 seconds. I believe this test is representative of real-world performance improvements.

author	David Schleimer <dschleimer@fb.com>
date	Tue, 18 Jun 2013 17:47:37 -0700
parents	20e73b5ab6f7
children	d741f536f23a

rev	line source
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	1 import test_util
717 ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	2
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	3 import unittest
717 ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	4 import urllib
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	5
672 2cc1342d4476 svnwrap: factor out a common module Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 475 diff changeset	6 from hgsubversion.svnwrap import parse_url
468 037bba1c6736 svnrepo: expose the same svnurl than SubversionRepo Patrick Mezard <pmezard@gmail.com> parents: 467 diff changeset	7 from hgsubversion import svnrepo
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	8
2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	9 class TestSubversionUrls(test_util.TestBase):
2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	10 def test_standard_url(self):
2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	11 self.assertEqual((None, None, 'file:///var/svn/repo'),
2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	12 parse_url('file:///var/svn/repo'))
2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	13
2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	14 def test_user_url(self):
467 3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	15 self.assertEqual(
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	16 ('joe', None, 'https://svn.testurl.com/repo'),
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	17 parse_url('https://joe@svn.testurl.com/repo'))
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	18 self.assertEqual(
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	19 ('bob', None, 'https://svn.testurl.com/repo'),
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	20 parse_url('https://joe@svn.testurl.com/repo', 'bob'))
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	21
2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	22 def test_password_url(self):
467 3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	23 self.assertEqual(
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	24 (None, 't3stpw', 'svn+ssh://svn.testurl.com/repo'),
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	25 parse_url('svn+ssh://:t3stpw@svn.testurl.com/repo'))
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	26 self.assertEqual(
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	27 (None, '123abc', 'svn+ssh://svn.testurl.com/repo'),
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	28 parse_url('svn+ssh://:t3stpw@svn.testurl.com/repo', None, '123abc'))
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	29
392 35993ba9d119 urls: Make sure we preserve username in urls given to ssh. Augie Fackler <durin42@gmail.com> parents: 337 diff changeset	30 def test_svnssh_preserve_user(self):
35993ba9d119 urls: Make sure we preserve username in urls given to ssh. Augie Fackler <durin42@gmail.com> parents: 337 diff changeset	31 self.assertEqual(
832 e9af7eba88db globally: clean up whitespace around operators and commas to conform with PEP8 Yonggang Luo <luoyonggang@gmail.com> parents: 717 diff changeset	32 ('user', 't3stpw', 'svn+ssh://user@svn.testurl.com/repo',),
392 35993ba9d119 urls: Make sure we preserve username in urls given to ssh. Augie Fackler <durin42@gmail.com> parents: 337 diff changeset	33 parse_url('svn+ssh://user:t3stpw@svn.testurl.com/repo'))
467 3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	34 self.assertEqual(
832 e9af7eba88db globally: clean up whitespace around operators and commas to conform with PEP8 Yonggang Luo <luoyonggang@gmail.com> parents: 717 diff changeset	35 ('bob', '123abc', 'svn+ssh://bob@svn.testurl.com/repo',),
467 3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	36 parse_url('svn+ssh://user:t3stpw@svn.testurl.com/repo', 'bob', '123abc'))
392 35993ba9d119 urls: Make sure we preserve username in urls given to ssh. Augie Fackler <durin42@gmail.com> parents: 337 diff changeset	37 self.assertEqual(
832 e9af7eba88db globally: clean up whitespace around operators and commas to conform with PEP8 Yonggang Luo <luoyonggang@gmail.com> parents: 717 diff changeset	38 ('user2', None, 'svn+ssh://user2@svn.testurl.com/repo',),
392 35993ba9d119 urls: Make sure we preserve username in urls given to ssh. Augie Fackler <durin42@gmail.com> parents: 337 diff changeset	39 parse_url('svn+ssh://user2@svn.testurl.com/repo'))
467 3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	40 self.assertEqual(
832 e9af7eba88db globally: clean up whitespace around operators and commas to conform with PEP8 Yonggang Luo <luoyonggang@gmail.com> parents: 717 diff changeset	41 ('bob', None, 'svn+ssh://bob@svn.testurl.com/repo',),
467 3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	42 parse_url('svn+ssh://user2@svn.testurl.com/repo', 'bob'))
392 35993ba9d119 urls: Make sure we preserve username in urls given to ssh. Augie Fackler <durin42@gmail.com> parents: 337 diff changeset	43
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	44 def test_user_password_url(self):
467 3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	45 self.assertEqual(
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	46 ('joe', 't3stpw', 'https://svn.testurl.com/repo'),
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	47 parse_url('https://joe:t3stpw@svn.testurl.com/repo'))
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	48 self.assertEqual(
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	49 ('bob', '123abc', 'https://svn.testurl.com/repo'),
3941d73c262e svnwrappers: override svn+ssh credentials with supplied ones if any Patrick Mezard <pmezard@gmail.com> parents: 392 diff changeset	50 parse_url('https://joe:t3stpw@svn.testurl.com/repo', 'bob', '123abc'))
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	51
468 037bba1c6736 svnrepo: expose the same svnurl than SubversionRepo Patrick Mezard <pmezard@gmail.com> parents: 467 diff changeset	52 def test_url_rewriting(self):
037bba1c6736 svnrepo: expose the same svnurl than SubversionRepo Patrick Mezard <pmezard@gmail.com> parents: 467 diff changeset	53 ui = test_util.ui.ui()
037bba1c6736 svnrepo: expose the same svnurl than SubversionRepo Patrick Mezard <pmezard@gmail.com> parents: 467 diff changeset	54 ui.setconfig('hgsubversion', 'username', 'bob')
037bba1c6736 svnrepo: expose the same svnurl than SubversionRepo Patrick Mezard <pmezard@gmail.com> parents: 467 diff changeset	55 repo = svnrepo.svnremoterepo(ui, 'svn+ssh://joe@foo/bar')
710 db56e65906f4 svnrepo: make the svnurl property obtain the URL from Subversion. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 699 diff changeset	56 self.assertEqual('svn+ssh://bob@foo/bar', repo.svnauth[0])
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	57
469 5567af673f83 Revive svn+http(s) URLs support (issue94) Patrick Mezard <pmezard@gmail.com> parents: 468 diff changeset	58 repo = svnrepo.svnremoterepo(ui, 'svn+http://joe@foo/bar')
5567af673f83 Revive svn+http(s) URLs support (issue94) Patrick Mezard <pmezard@gmail.com> parents: 468 diff changeset	59 self.assertEqual(('http://foo/bar', 'bob', None), repo.svnauth)
5567af673f83 Revive svn+http(s) URLs support (issue94) Patrick Mezard <pmezard@gmail.com> parents: 468 diff changeset	60
475 15443c592f7a Remove the svn+ from svn+https urls before calling the Subversion API. This was already being done for svn+http urls. David Stanek <dstanek@dstanek.com> parents: 469 diff changeset	61 repo = svnrepo.svnremoterepo(ui, 'svn+https://joe@foo/bar')
15443c592f7a Remove the svn+ from svn+https urls before calling the Subversion API. This was already being done for svn+http urls. David Stanek <dstanek@dstanek.com> parents: 469 diff changeset	62 self.assertEqual(('https://foo/bar', 'bob', None), repo.svnauth)
15443c592f7a Remove the svn+ from svn+https urls before calling the Subversion API. This was already being done for svn+http urls. David Stanek <dstanek@dstanek.com> parents: 469 diff changeset	63
717 ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	64 def test_quoting(self):
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	65 ui = self.ui()
866 20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 833 diff changeset	66 repo_path = self.load_svndump('non_ascii_path_1.svndump')
717 ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	67
866 20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 833 diff changeset	68 repo_url = test_util.fileurl(repo_path)
717 ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	69 subdir = '/b\xC3\xB8b'
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	70 quoted_subdir = urllib.quote(subdir)
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	71
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	72 repo1 = svnrepo.svnremoterepo(ui, repo_url + subdir)
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	73 repo2 = svnrepo.svnremoterepo(ui, repo_url + quoted_subdir)
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	74 self.assertEqual(repo1.svnurl, repo2.svnurl)
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 710 diff changeset	75
235 2969a20e0eef Add support for user:pass@url repositories to be hg-like Daniel Tang <dytang@cs.purdue.edu> parents: diff changeset	76 def suite():
833 312b37bc5e20 tests: avoid shadowing Python builtin all() Yonggang Luo <luoyonggang@gmail.com> parents: 832 diff changeset	77 all_tests = [unittest.TestLoader().loadTestsFromTestCase(TestSubversionUrls)]
312b37bc5e20 tests: avoid shadowing Python builtin all() Yonggang Luo <luoyonggang@gmail.com> parents: 832 diff changeset	78 return unittest.TestSuite(all_tests)

Mercurial > hgsubversion

annotate tests/test_urls.py @ 1023:7a262ecae4f3