hgsubversion: tests/test_fetch

annotate tests/test_fetch_renames.py @ 1450:3c4fbf59e503

svnmeta: do not add instance references to the class object This is part of a bigger plan to prevent a fd leak by making sure the sqlite connection object is garbage collected. Usually this is not a very serious issue but the testing framework will run all tests in a single process by default. The fd leak will make tests fail on systems with a low RLIMIT_NOFILE. Previously, when we are adding new properties to the SVNMeta class, we bind the current instance in the closure, which essentially prevents any SVNMeta instances from being garbage collected (and its state like revmap). This patch changed "self" from the closure one to the lambda argument to address the issue.

author	Jun Wu <quark@fb.com>
date	Fri, 27 May 2016 01:42:55 +0100
parents	6b15eeb78c1a
children	4379baf58f9e

rev	line source
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	1 import test_util
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	2
643 d2ef7220a079 tests: import test_util as the first module in all relevant tests Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 513 diff changeset	3 import sys
d2ef7220a079 tests: import test_util as the first module in all relevant tests Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 513 diff changeset	4 import unittest
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	5
91 7d10165cf3d9 tests: Mock the mercurial.ui.ui class like we really should to capture output. Augie Fackler <durin42@gmail.com> parents: 78 diff changeset	6 class TestFetchRenames(test_util.TestBase):
1064 dc516f85b6c3 test_fetch_renames: use stupid mode metaclass Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1044 diff changeset	7 stupid_mode_tests = True
dc516f85b6c3 test_fetch_renames: use stupid mode metaclass Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1044 diff changeset	8
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	9 def _debug_print_copies(self, repo):
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	10 w = sys.stderr.write
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	11 for rev in repo:
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	12 ctx = repo[rev]
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	13 w('%d - %s\n' % (ctx.rev(), ctx.branch()))
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	14 for f in ctx:
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	15 fctx = ctx[f]
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	16 w('%s: %r %r\n' % (f, fctx.data(), fctx.renamed()))
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	17
1064 dc516f85b6c3 test_fetch_renames: use stupid mode metaclass Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1044 diff changeset	18 def test_rename(self):
944 d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 833 diff changeset	19 config = {
d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 833 diff changeset	20 'hgsubversion.filestoresize': '0',
1235 6b15eeb78c1a editor: fix replay handling for copied + modified symlinks David Schleimer <dschleimer@fb.com> parents: 1234 diff changeset	21 # we set this because we expect all of the copies to be
6b15eeb78c1a editor: fix replay handling for copied + modified symlinks David Schleimer <dschleimer@fb.com> parents: 1234 diff changeset	22 # handled via replay, and we want to notice if that
6b15eeb78c1a editor: fix replay handling for copied + modified symlinks David Schleimer <dschleimer@fb.com> parents: 1234 diff changeset	23 # changes.
6b15eeb78c1a editor: fix replay handling for copied + modified symlinks David Schleimer <dschleimer@fb.com> parents: 1234 diff changeset	24 'hgsubversion.failonmissing': 'yes',
944 d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 833 diff changeset	25 }
1064 dc516f85b6c3 test_fetch_renames: use stupid mode metaclass Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1044 diff changeset	26 repo = self._load_fixture_and_fetch('renames.svndump', config=config)
1234 d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	27 self._run_assertions(repo)
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	28
1234 d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	29 def test_rename_with_prefix(self):
d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	30 config = {
d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	31 'hgsubversion.filestoresize': '0',
1235 6b15eeb78c1a editor: fix replay handling for copied + modified symlinks David Schleimer <dschleimer@fb.com> parents: 1234 diff changeset	32 'hgsubversion.failonmissing': 'yes',
1234 d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	33 }
d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	34 repo = self._load_fixture_and_fetch('renames_with_prefix.svndump',
d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	35 subdir='prefix',
d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	36 config=config)
d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	37 self._run_assertions(repo)
d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	38
d3c79072bc6a editor: correctly import symlink copy+modify with non-empty prefix David Schleimer <dschleimer@fb.com> parents: 1233 diff changeset	39 def _run_assertions(self, repo):
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	40 # Map revnum to mappings of dest name to (source name, dest content)
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	41 copies = {
68 e0c86ebe05e3 test_fetch_renames: test copy of deleted stuff from the past Patrick Mezard <pmezard@gmail.com> parents: 67 diff changeset	42 4: {
91 7d10165cf3d9 tests: Mock the mercurial.ui.ui class like we really should to capture output. Augie Fackler <durin42@gmail.com> parents: 78 diff changeset	43 'a1': ('a', 'a\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	44 'linka1': ('linka', 'a'),
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	45 'a2': ('a', 'a\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	46 'linka2': ('linka', 'a'),
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	47 'b1': ('b', 'b\nc\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	48 'linkb1': ('linkb', 'bc'),
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	49 'da1/daf': ('da/daf', 'c\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	50 'da1/dalink': ('da/dalink', 'daf'),
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	51 'da1/db/dbf': ('da/db/dbf', 'd\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	52 'da1/db/dblink': ('da/db/dblink', '../daf'),
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	53 'da2/daf': ('da/daf', 'c\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	54 'da2/dalink': ('da/dalink', 'daf'),
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	55 'da2/db/dbf': ('da/db/dbf', 'd\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	56 'da2/db/dblink': ('da/db/dblink', '../daf'),
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	57 },
68 e0c86ebe05e3 test_fetch_renames: test copy of deleted stuff from the past Patrick Mezard <pmezard@gmail.com> parents: 67 diff changeset	58 5: {
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	59 'c1': ('c', 'c\nc\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	60 'linkc1': ('linkc', 'cc'),
69 63ece4ea25c9 hg_delta_editor: register copies only if files are unchanged between source and dest Patrick Mezard <pmezard@gmail.com> parents: 68 diff changeset	61 },
63ece4ea25c9 hg_delta_editor: register copies only if files are unchanged between source and dest Patrick Mezard <pmezard@gmail.com> parents: 68 diff changeset	62 9: {
63ece4ea25c9 hg_delta_editor: register copies only if files are unchanged between source and dest Patrick Mezard <pmezard@gmail.com> parents: 68 diff changeset	63 'unchanged2': ('unchanged', 'unchanged\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	64 'unchangedlink2': ('unchangedlink', 'unchanged'),
69 63ece4ea25c9 hg_delta_editor: register copies only if files are unchanged between source and dest Patrick Mezard <pmezard@gmail.com> parents: 68 diff changeset	65 'unchangeddir2/f': ('unchangeddir/f', 'unchanged2\n'),
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	66 'unchangeddir2/link': ('unchangeddir/link', 'f'),
496 5e0dfe59d4c3 copies: fix under-reporting of copies in hg Augie Fackler <durin42@gmail.com> parents: 132 diff changeset	67 },
5e0dfe59d4c3 copies: fix under-reporting of copies in hg Augie Fackler <durin42@gmail.com> parents: 132 diff changeset	68 10: {
1233 0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	69 'groupdir2/b': ('groupdir/b', 'b\n'),
0d0132cba155 editor: fix edge case with in memory file-store size limit David Schleimer <dschleimer@fb.com> parents: 1064 diff changeset	70 'groupdir2/linkb': ('groupdir/linkb', 'b'),
496 5e0dfe59d4c3 copies: fix under-reporting of copies in hg Augie Fackler <durin42@gmail.com> parents: 132 diff changeset	71 },
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	72 }
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	73 for rev in repo:
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	74 ctx = repo[rev]
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	75 copymap = copies.get(rev, {})
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	76 for f in ctx.manifest():
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	77 cp = ctx[f].renamed()
69 63ece4ea25c9 hg_delta_editor: register copies only if files are unchanged between source and dest Patrick Mezard <pmezard@gmail.com> parents: 68 diff changeset	78 self.assertEqual(bool(cp), bool(copymap.get(f)),
63ece4ea25c9 hg_delta_editor: register copies only if files are unchanged between source and dest Patrick Mezard <pmezard@gmail.com> parents: 68 diff changeset	79 'copy records differ for %s in %d' % (f, rev))
67 e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	80 if not cp:
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	81 continue
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	82 self.assertEqual(cp[0], copymap[f][0])
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	83 self.assertEqual(ctx[f].data(), copymap[f][1])
e319c9168910 hg_delta_editor: register svn file copies Patrick Mezard <pmezard@gmail.com> parents: diff changeset	84
130 c2a84d436202 test_fetch_rename: test content of files coming from the past Patrick Mezard <pmezard@gmail.com> parents: 101 diff changeset	85 self.assertEqual(repo['tip']['changed3'].data(), 'changed\nchanged3\n')
c2a84d436202 test_fetch_rename: test content of files coming from the past Patrick Mezard <pmezard@gmail.com> parents: 101 diff changeset	86
1064 dc516f85b6c3 test_fetch_renames: use stupid mode metaclass Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1044 diff changeset	87 def test_case(self):
dc516f85b6c3 test_fetch_renames: use stupid mode metaclass Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1044 diff changeset	88 repo = self._load_fixture_and_fetch('filecase.svndump')
132 3a9d6cd18332 test_fetch_renames: test file and directory case changes Patrick Mezard <pmezard@gmail.com> parents: 130 diff changeset	89 files = {
3a9d6cd18332 test_fetch_renames: test file and directory case changes Patrick Mezard <pmezard@gmail.com> parents: 130 diff changeset	90 0: ['A', 'a', 'e/a', 'b', 'd/a', 'D/a', 'f/a', 'F'],
3a9d6cd18332 test_fetch_renames: test file and directory case changes Patrick Mezard <pmezard@gmail.com> parents: 130 diff changeset	91 1: ['A', 'a', 'E/a', 'B', 'd/A', 'D/a', 'f/a', 'F'],
3a9d6cd18332 test_fetch_renames: test file and directory case changes Patrick Mezard <pmezard@gmail.com> parents: 130 diff changeset	92 }
3a9d6cd18332 test_fetch_renames: test file and directory case changes Patrick Mezard <pmezard@gmail.com> parents: 130 diff changeset	93 for rev in repo:
3a9d6cd18332 test_fetch_renames: test file and directory case changes Patrick Mezard <pmezard@gmail.com> parents: 130 diff changeset	94 self.assertEqual(sorted(files[rev]), sorted(repo[rev].manifest()))

Mercurial > hgsubversion

annotate tests/test_fetch_renames.py @ 1450:3c4fbf59e503