hgsubversion: tests/test_util.py annotate

annotate tests/test_util.py @ 1233:0d0132cba155

editor: fix edge case with in memory file-store size limit There are a few cases where we will set a single file into to the editor's FileStore object more than once. Notably, for copied and then modified files, we will set it at least twice. Three times if editing fails (which it can for symlinks). If we pass the in-memory storage limit in between the first (or second if editing fails) time we set the file and the last time we set the file, we will write the data to the in memory store the first time and the file store the last time. We didn't remove it form the in-memory store though, and we always prefer reading from the in-memory store. This means we can sometimes end up with the wrong version of a file. This is fairly unlikely to happen in normal use since you need to hit the memory limit between two writes to the store for the same file. We only write a file multiple times if a) the file (and not one of it's parent directories) is copied and then modified or b) editing fails. From what I can tell, it's only common for editing to fail for symlinks, and they ten to be relatively small data that is unlikely to push over the limit. Finally, the default limit is 100MB which I would expect to be most often either well over (source code) or well under (binaries or automated changes) the size of the changes files in a single commit. The easiest way to reproduce this is to set the in-memory cache size to 0 and then commit a copied and modified symlink. The empty-string version from the failed editing will be the one that persists. I happened to stumble upon this while trying (and failing) to test a bug-fix for a related bug with identical symptoms (empty simlink). I have seen this in the wild, once, but couldn't reproduce it at the time. The repo in question is quite large and quite active, so I am quite confident in my estimation that this is a real, but very rare, problem. The test changes attached to this was mneant to test a related bug, but turned out not to actually cover the bug in question. They did trigger this bug though, and are worthwhile to test, so I kept them.

author	David Schleimer <dschleimer@fb.com>
date	Mon, 07 Apr 2014 17:51:59 -0700
parents	ba8485b9fee0
children	d3c79072bc6a

rev	line source
492 50d77547c218 test_util: sort imports Augie Fackler <durin42@gmail.com> parents: 487 diff changeset	1 import StringIO
548 2148eb4b4da4 test_util: add assertStringEqual to output diff output for string checks Augie Fackler <durin42@gmail.com> parents: 546 diff changeset	2 import difflib
78 072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	3 import errno
492 50d77547c218 test_util: sort imports Augie Fackler <durin42@gmail.com> parents: 487 diff changeset	4 import gettext
14 d78dbf88c13d Started a meaningful test suite. Augie Fackler <durin42@gmail.com> parents: diff changeset	5 import os
78 072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	6 import shutil
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	7 import stat
492 50d77547c218 test_util: sort imports Augie Fackler <durin42@gmail.com> parents: 487 diff changeset	8 import subprocess
50d77547c218 test_util: sort imports Augie Fackler <durin42@gmail.com> parents: 487 diff changeset	9 import sys
910 312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	10 import tarfile
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	11 import tempfile
71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	12 import unittest
78 072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	13 import urllib
14 d78dbf88c13d Started a meaningful test suite. Augie Fackler <durin42@gmail.com> parents: diff changeset	14
644 95abc4cfc78f tests: improve import logic Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 626 diff changeset	15 _rootdir = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
95abc4cfc78f tests: improve import logic Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 626 diff changeset	16 sys.path.insert(0, _rootdir)
347 537de0300510 Remove the 'outgoing' wrapper, and use the Mercurial infrastructure instead. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 342 diff changeset	17
711 cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	18 from mercurial import cmdutil
331 75f082b5897e Switch to using url scheme wrappers instead of duplicating each command we wrap. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 326 diff changeset	19 from mercurial import commands
492 50d77547c218 test_util: sort imports Augie Fackler <durin42@gmail.com> parents: 487 diff changeset	20 from mercurial import context
813 f07bfd66db13 test_util: handle dispatch.dispatch() taking a request Patrick Mezard <pmezard@gmail.com> parents: 793 diff changeset	21 from mercurial import dispatch as dispatchmod
22 95d0109e495e Refactor tests so I can reuse code more. Augie Fackler <durin42@gmail.com> parents: 16 diff changeset	22 from mercurial import hg
492 50d77547c218 test_util: sort imports Augie Fackler <durin42@gmail.com> parents: 487 diff changeset	23 from mercurial import i18n
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	24 from mercurial import node
1003 6945d3359456 testutil: clear rcpath when we update HGRCPATH env var David Schleimer <dschleimer@fb.com> parents: 1001 diff changeset	25 from mercurial import scmutil
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	26 from mercurial import ui
772 f3af4fe98d37 test_util: make manifest filter use shared list of files to ignore Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 764 diff changeset	27 from mercurial import util
527 2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	28 from mercurial import extensions
22 95d0109e495e Refactor tests so I can reuse code more. Augie Fackler <durin42@gmail.com> parents: 16 diff changeset	29
1203 518a9552f7f9 test_util: call makememfilectx Sean Farley <sean.michael.farley@gmail.com> parents: 1174 diff changeset	30 from hgsubversion import compathacks
518a9552f7f9 test_util: call makememfilectx Sean Farley <sean.michael.farley@gmail.com> parents: 1174 diff changeset	31
1049 608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	32 try:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	33 from mercurial import obsolete
1174 a22d4972e01f push: fix obsolete feature detection Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 1101 diff changeset	34 obsolete._enabled
1049 608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	35 except ImportError:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	36 obsolete = None
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	37
716 38ebf7714cdf requiresoption: raise SkipTest if available Augie Fackler <durin42@gmail.com> parents: 715 diff changeset	38 try:
734 e24fb3e27ec9 test_util: use SkipTest if provided by the unittest module. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 727 diff changeset	39 SkipTest = unittest.SkipTest
e24fb3e27ec9 test_util: use SkipTest if provided by the unittest module. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 727 diff changeset	40 except AttributeError:
716 38ebf7714cdf requiresoption: raise SkipTest if available Augie Fackler <durin42@gmail.com> parents: 715 diff changeset	41 try:
734 e24fb3e27ec9 test_util: use SkipTest if provided by the unittest module. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 727 diff changeset	42 from unittest2 import SkipTest
716 38ebf7714cdf requiresoption: raise SkipTest if available Augie Fackler <durin42@gmail.com> parents: 715 diff changeset	43 except ImportError:
734 e24fb3e27ec9 test_util: use SkipTest if provided by the unittest module. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 727 diff changeset	44 try:
e24fb3e27ec9 test_util: use SkipTest if provided by the unittest module. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 727 diff changeset	45 from nose import SkipTest
e24fb3e27ec9 test_util: use SkipTest if provided by the unittest module. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 727 diff changeset	46 except ImportError:
e24fb3e27ec9 test_util: use SkipTest if provided by the unittest module. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 727 diff changeset	47 SkipTest = None
716 38ebf7714cdf requiresoption: raise SkipTest if available Augie Fackler <durin42@gmail.com> parents: 715 diff changeset	48
337 46e69be8e2c8 Reorganize to have a more conventional module structure. Augie Fackler <durin42@gmail.com> parents: 334 diff changeset	49 from hgsubversion import util
911 772280aed751 Honor SVN auto-props (solves issue #186) Ronny Voelker <ronny.voelker@googlemail.com> parents: 910 diff changeset	50 from hgsubversion import svnwrap
333 a59ab58969d9 test_util: normalize path in svnls() Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 331 diff changeset	51
479 83fcb1cf6d8f Avoid 'ValueError: close_fds is not supported on Windows platforms' exception Risto Kankkunen <risto.kankkunen@iki.fi> parents: 355 diff changeset	52 # Documentation for Subprocess.Popen() says:
83fcb1cf6d8f Avoid 'ValueError: close_fds is not supported on Windows platforms' exception Risto Kankkunen <risto.kankkunen@iki.fi> parents: 355 diff changeset	53 # "Note that on Windows, you cannot set close_fds to true and
83fcb1cf6d8f Avoid 'ValueError: close_fds is not supported on Windows platforms' exception Risto Kankkunen <risto.kankkunen@iki.fi> parents: 355 diff changeset	54 # also redirect the standard handles by setting stdin, stdout or
83fcb1cf6d8f Avoid 'ValueError: close_fds is not supported on Windows platforms' exception Risto Kankkunen <risto.kankkunen@iki.fi> parents: 355 diff changeset	55 # stderr."
832 e9af7eba88db globally: clean up whitespace around operators and commas to conform with PEP8 Yonggang Luo <luoyonggang@gmail.com> parents: 816 diff changeset	56 canCloseFds = 'win32' not in sys.platform
479 83fcb1cf6d8f Avoid 'ValueError: close_fds is not supported on Windows platforms' exception Risto Kankkunen <risto.kankkunen@iki.fi> parents: 355 diff changeset	57
483 37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	58 if not 'win32' in sys.platform:
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	59 def kill_process(popen_obj):
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	60 os.kill(popen_obj.pid, 9)
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	61 else:
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	62 import ctypes
487 6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	63 from ctypes.wintypes import BOOL, DWORD, HANDLE, UINT
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	64
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	65 def win_status_check(result, func, args):
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	66 if result == 0:
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	67 raise ctypes.WinError()
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	68 return args
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	69
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	70 def WINAPI(returns, func, *params):
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	71 assert len(params) % 2 == 0
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	72
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	73 func.argtypes = tuple(params[0::2])
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	74 func.resvalue = returns
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	75 func.errcheck = win_status_check
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	76
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	77 return func
483 37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	78
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	79 # dwDesiredAccess
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	80 PROCESS_TERMINATE = 0x0001
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	81
487 6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	82 OpenProcess = WINAPI(HANDLE, ctypes.windll.kernel32.OpenProcess,
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	83 DWORD, 'dwDesiredAccess',
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	84 BOOL, 'bInheritHandle',
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	85 DWORD, 'dwProcessId',
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	86 )
483 37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	87
832 e9af7eba88db globally: clean up whitespace around operators and commas to conform with PEP8 Yonggang Luo <luoyonggang@gmail.com> parents: 816 diff changeset	88 CloseHandle = WINAPI(BOOL, ctypes.windll.kernel32.CloseHandle,
487 6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	89 HANDLE, 'hObject'
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	90 )
483 37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	91
487 6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	92 TerminateProcess = WINAPI(BOOL, ctypes.windll.kernel32.TerminateProcess,
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	93 HANDLE, 'hProcess',
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	94 UINT, 'uExitCode'
6b481331c355 test_util: fix up line endings Augie Fackler <durin42@gmail.com> parents: 486 diff changeset	95 )
483 37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	96
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	97 def kill_process(popen_obj):
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	98 phnd = OpenProcess(PROCESS_TERMINATE, False, popen_obj.pid)
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	99 TerminateProcess(phnd, 1)
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	100 CloseHandle(phnd)
37718f514acb No os.kill() in Windows, use ctypes to call Win32 TerminateProcess() Risto Kankkunen <risto.kankkunen@iki.fi> parents: 479 diff changeset	101
194 13ae1bded5e7 Add some comprehensive tests that can be run with nose in order to make it easier to verify stupid and real replay do the same thing. Augie Fackler <durin42@gmail.com> parents: 170 diff changeset	102 # Fixtures that need to be pulled at a subdirectory of the repo path
13ae1bded5e7 Add some comprehensive tests that can be run with nose in order to make it easier to verify stupid and real replay do the same thing. Augie Fackler <durin42@gmail.com> parents: 170 diff changeset	103 subdir = {'truncatedhistory.svndump': '/project2',
13ae1bded5e7 Add some comprehensive tests that can be run with nose in order to make it easier to verify stupid and real replay do the same thing. Augie Fackler <durin42@gmail.com> parents: 170 diff changeset	104 'fetch_missing_files_subdir.svndump': '/foo',
270 2848d17eae71 Fix rebuildmeta tests for empty_dir_in_trunk_not_repo_root Augie Fackler <durin42@gmail.com> parents: 257 diff changeset	105 'empty_dir_in_trunk_not_repo_root.svndump': '/project',
351 3d5c4352a6c8 test_util: Add another missing project root. Augie Fackler <durin42@gmail.com> parents: 347 diff changeset	106 'project_root_not_repo_root.svndump': '/dummyproj',
512 c421e6bf0d95 tests: test paths with spaces Augie Fackler <durin42@gmail.com> parents: 511 diff changeset	107 'project_name_with_space.svndump': '/project name',
717 ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 716 diff changeset	108 'non_ascii_path_1.svndump': '/b\xC3\xB8b',
ae5968ffe6fe svnwrap: fix handling of quotable URLs (fixes #197, refs #132) Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 716 diff changeset	109 'non_ascii_path_2.svndump': '/b%C3%B8b',
888 c6388ed0ec0a svnmeta: only remove directory components in normalize() Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 886 diff changeset	110 'subdir_is_file_prefix.svndump': '/flaf',
194 13ae1bded5e7 Add some comprehensive tests that can be run with nose in order to make it easier to verify stupid and real replay do the same thing. Augie Fackler <durin42@gmail.com> parents: 170 diff changeset	111 }
1092 cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	112 # map defining the layouts of the fixtures we can use with custom layout
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	113 # these are really popular layouts, so I gave them names
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	114 trunk_only = {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	115 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	116 }
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	117 trunk_dev_branch = {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	118 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	119 'dev_branch': 'branches/dev_branch',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	120 }
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	121 custom = {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	122 'addspecial.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	123 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	124 'foo': 'branches/foo',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	125 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	126 'binaryfiles.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	127 'branch_create_with_dir_delete.svndump': trunk_dev_branch,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	128 'branch_delete_parent_dir.svndump': trunk_dev_branch,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	129 'branchmap.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	130 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	131 'badname': 'branches/badname',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	132 'feature': 'branches/feature',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	133 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	134 'branch_prop_edit.svndump': trunk_dev_branch,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	135 'branch_rename_to_trunk.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	136 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	137 'dev_branch': 'branches/dev_branch',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	138 'old_trunk': 'branches/old_trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	139 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	140 'copies.svndump': trunk_only,
1232 ba8485b9fee0 editor: correctly import copies of directories from non-tracked or closed branches David Schleimer <dschleimer@fb.com> parents: 1203 diff changeset	141 'copyafterclose.svndump': {
ba8485b9fee0 editor: correctly import copies of directories from non-tracked or closed branches David Schleimer <dschleimer@fb.com> parents: 1203 diff changeset	142 'default': 'trunk',
ba8485b9fee0 editor: correctly import copies of directories from non-tracked or closed branches David Schleimer <dschleimer@fb.com> parents: 1203 diff changeset	143 'test': 'branches/test'
ba8485b9fee0 editor: correctly import copies of directories from non-tracked or closed branches David Schleimer <dschleimer@fb.com> parents: 1203 diff changeset	144 },
1092 cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	145 'copybeforeclose.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	146 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	147 'test': 'branches/test'
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	148 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	149 'delentries.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	150 'delete_restore_trunk.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	151 'empty_dir_in_trunk_not_repo_root.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	152 'executebit.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	153 'filecase.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	154 'file_not_in_trunk_root.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	155 'project_name_with_space.svndump': trunk_dev_branch,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	156 'pushrenames.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	157 'rename_branch_parent_dir.svndump': trunk_dev_branch,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	158 'renamedproject.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	159 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	160 'branch': 'branches/branch',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	161 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	162 'renames.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	163 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	164 'branch1': 'branches/branch1',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	165 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	166 'replace_branch_with_branch.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	167 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	168 'branch1': 'branches/branch1',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	169 'branch2': 'branches/branch2',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	170 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	171 'replace_trunk_with_branch.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	172 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	173 'test': 'branches/test',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	174 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	175 'revert.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	176 'siblingbranchfix.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	177 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	178 'wrongbranch': 'branches/wrongbranch',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	179 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	180 'simple_branch.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	181 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	182 'the_branch': 'branches/the_branch',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	183 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	184 'spaces-in-path.svndump': trunk_dev_branch,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	185 'symlinks.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	186 'truncatedhistory.svndump': trunk_only,
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	187 'unorderedbranch.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	188 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	189 'branch': 'branches/branch',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	190 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	191 'unrelatedbranch.svndump': {
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	192 'default': 'trunk',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	193 'branch1': 'branches/branch1',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	194 'branch2': 'branches/branch2',
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	195 },
cd0d14e25757 layouts: add custom layout for those of us that need weird mappings David Schleimer <dschleimer@fb.com> parents: 1084 diff changeset	196 }
194 13ae1bded5e7 Add some comprehensive tests that can be run with nose in order to make it easier to verify stupid and real replay do the same thing. Augie Fackler <durin42@gmail.com> parents: 170 diff changeset	197
14 d78dbf88c13d Started a meaningful test suite. Augie Fackler <durin42@gmail.com> parents: diff changeset	198 FIXTURES = os.path.join(os.path.abspath(os.path.dirname(__file__)),
d78dbf88c13d Started a meaningful test suite. Augie Fackler <durin42@gmail.com> parents: diff changeset	199 'fixtures')
d78dbf88c13d Started a meaningful test suite. Augie Fackler <durin42@gmail.com> parents: diff changeset	200
930 5bacb9c63e3e Fix more peer breakage with old hg versions Patrick Mezard <patrick@mezard.eu> parents: 923 diff changeset	201 def getlocalpeer(repo):
5bacb9c63e3e Fix more peer breakage with old hg versions Patrick Mezard <patrick@mezard.eu> parents: 923 diff changeset	202 localrepo = getattr(repo, 'local', lambda: repo)()
5bacb9c63e3e Fix more peer breakage with old hg versions Patrick Mezard <patrick@mezard.eu> parents: 923 diff changeset	203 if isinstance(localrepo, bool):
5bacb9c63e3e Fix more peer breakage with old hg versions Patrick Mezard <patrick@mezard.eu> parents: 923 diff changeset	204 localrepo = repo
5bacb9c63e3e Fix more peer breakage with old hg versions Patrick Mezard <patrick@mezard.eu> parents: 923 diff changeset	205 return localrepo
719 ae52a3b30cfb test_util: refactor requiresoption to ease creation of other skip decorators Augie Fackler <durin42@gmail.com> parents: 717 diff changeset	206
1048 903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	207 def repolen(repo):
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	208 """Naively calculate the amount of available revisions in a repository.
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	209
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	210 this is usually equal to len(repo) -- except in the face of
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	211 obsolete revisions.
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	212 """
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	213 # kind of nasty way of calculating the length, but fortunately,
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	214 # our test repositories tend to be rather small
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	215 return len([r for r in repo])
903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	216
719 ae52a3b30cfb test_util: refactor requiresoption to ease creation of other skip decorators Augie Fackler <durin42@gmail.com> parents: 717 diff changeset	217 def _makeskip(name, message):
727 e830b592917b tests: fix test skipping in pure unittest Augie Fackler <durin42@gmail.com> parents: 720 diff changeset	218 if SkipTest:
e830b592917b tests: fix test skipping in pure unittest Augie Fackler <durin42@gmail.com> parents: 720 diff changeset	219 def skip(args, *kwargs):
e830b592917b tests: fix test skipping in pure unittest Augie Fackler <durin42@gmail.com> parents: 720 diff changeset	220 raise SkipTest(message)
e830b592917b tests: fix test skipping in pure unittest Augie Fackler <durin42@gmail.com> parents: 720 diff changeset	221 skip.__name__ = name
e830b592917b tests: fix test skipping in pure unittest Augie Fackler <durin42@gmail.com> parents: 720 diff changeset	222 return skip
719 ae52a3b30cfb test_util: refactor requiresoption to ease creation of other skip decorators Augie Fackler <durin42@gmail.com> parents: 717 diff changeset	223
720 a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	224 def requiresmodule(mod):
a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	225 """Skip a test if the specified module is not None."""
a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	226 def decorator(fn):
727 e830b592917b tests: fix test skipping in pure unittest Augie Fackler <durin42@gmail.com> parents: 720 diff changeset	227 if fn is None:
e830b592917b tests: fix test skipping in pure unittest Augie Fackler <durin42@gmail.com> parents: 720 diff changeset	228 return
720 a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	229 if mod is not None:
a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	230 return fn
a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	231 return _makeskip(fn.__name__, 'missing required feature')
a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	232 return decorator
a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	233
a19a208c085b test_util: new requiresmodule decorator for tests that require a feature Augie Fackler <durin42@gmail.com> parents: 719 diff changeset	234
711 cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	235 def requiresoption(option):
715 c33293a34752 requiresoption: clean up docstring Augie Fackler <durin42@gmail.com> parents: 711 diff changeset	236 '''Skip a test if commands.clone does not take the specified option.'''
711 cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	237 def decorator(fn):
cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	238 for entry in cmdutil.findcmd('clone', commands.table)[1][1]:
cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	239 if entry[1] == option:
cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	240 return fn
716 38ebf7714cdf requiresoption: raise SkipTest if available Augie Fackler <durin42@gmail.com> parents: 715 diff changeset	241 # no match found, so skip
38ebf7714cdf requiresoption: raise SkipTest if available Augie Fackler <durin42@gmail.com> parents: 715 diff changeset	242 if SkipTest:
719 ae52a3b30cfb test_util: refactor requiresoption to ease creation of other skip decorators Augie Fackler <durin42@gmail.com> parents: 717 diff changeset	243 return _makeskip(fn.__name__,
ae52a3b30cfb test_util: refactor requiresoption to ease creation of other skip decorators Augie Fackler <durin42@gmail.com> parents: 717 diff changeset	244 'test requires clone to accept %s' % option)
716 38ebf7714cdf requiresoption: raise SkipTest if available Augie Fackler <durin42@gmail.com> parents: 715 diff changeset	245 # no skipping support, so erase decorated method
38ebf7714cdf requiresoption: raise SkipTest if available Augie Fackler <durin42@gmail.com> parents: 715 diff changeset	246 return
711 cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	247 if not isinstance(option, str):
cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	248 raise TypeError('requiresoption takes a string argument')
cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	249 return decorator
cfc7df19e4dc test_util: add requiresoption decorator function. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 706 diff changeset	250
1056 0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	251 def requiresreplay(method):
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	252 '''Skip a test in stupid mode.'''
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	253 def test(self, args, *kwargs):
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	254 if self.stupid:
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	255 if SkipTest:
1080 661f4065bfbf test_util: fix missing message in SkipTest raise Augie Fackler <raf@durin42.com> parents: 1078 diff changeset	256 raise SkipTest("test requires replay mode")
1056 0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	257 else:
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	258 return method(self, args, *kwargs)
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	259
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	260 test.__name__ = method.__name__
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	261 return test
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	262
499 1fd3cfa47c5e Support for single-directory clones. Augie Fackler <durin42@gmail.com> parents: 492 diff changeset	263 def filtermanifest(manifest):
772 f3af4fe98d37 test_util: make manifest filter use shared list of files to ignore Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 764 diff changeset	264 return [f for f in manifest if f not in util.ignoredfiles]
499 1fd3cfa47c5e Support for single-directory clones. Augie Fackler <durin42@gmail.com> parents: 492 diff changeset	265
91 7d10165cf3d9 tests: Mock the mercurial.ui.ui class like we really should to capture output. Augie Fackler <durin42@gmail.com> parents: 84 diff changeset	266 def fileurl(path):
522 60bf433647e7 test_util: fix Windows test URLs Patrick Mezard <pmezard@gmail.com> parents: 516 diff changeset	267 path = os.path.abspath(path).replace(os.sep, '/')
78 072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	268 drive, path = os.path.splitdrive(path)
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	269 if drive:
923 847953806c77 test_util: make repourl() return upper-cased drive letters Patrick Mezard <patrick@mezard.eu> parents: 917 diff changeset	270 # In svn 1.7, the swig svn wrapper returns local svn URLs
847953806c77 test_util: make repourl() return upper-cased drive letters Patrick Mezard <patrick@mezard.eu> parents: 917 diff changeset	271 # with an uppercase drive letter, try to match that to
847953806c77 test_util: make repourl() return upper-cased drive letters Patrick Mezard <patrick@mezard.eu> parents: 917 diff changeset	272 # simplify svn info tests.
847953806c77 test_util: make repourl() return upper-cased drive letters Patrick Mezard <patrick@mezard.eu> parents: 917 diff changeset	273 drive = '/' + drive.upper()
342 76c833526fbc Use fallbacks in the wrappers for Subversion support, instead of prefixes. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 337 diff changeset	274 url = 'file://%s%s' % (drive, path)
78 072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	275 return url
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	276
652 c3900ad3bf69 test_util: support specifying a start revision Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 645 diff changeset	277 def testui(stupid=False, layout='auto', startrev=0):
576 d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	278 u = ui.ui()
d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	279 bools = {True: 'true', False: 'false'}
d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	280 u.setconfig('ui', 'quiet', bools[True])
696 c11bf0dd38d3 tests: enable hgsubversion in the generated UI. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 689 diff changeset	281 u.setconfig('extensions', 'hgsubversion', '')
576 d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	282 u.setconfig('hgsubversion', 'stupid', bools[stupid])
d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	283 u.setconfig('hgsubversion', 'layout', layout)
652 c3900ad3bf69 test_util: support specifying a start revision Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 645 diff changeset	284 u.setconfig('hgsubversion', 'startrev', startrev)
576 d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	285 return u
d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	286
813 f07bfd66db13 test_util: handle dispatch.dispatch() taking a request Patrick Mezard <pmezard@gmail.com> parents: 793 diff changeset	287 def dispatch(cmd):
916 7e9d805a0e1f tests: fix regressions introduced by peer classes in hg 2.3 Bryan O'Sullivan <bryano@fb.com> parents: 910 diff changeset	288 cmd = getattr(dispatchmod, 'request', lambda x: x)(cmd)
1041 70090e2ee262 tests: assert that fetches actually succeed Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 1038 diff changeset	289 return dispatchmod.dispatch(cmd)
813 f07bfd66db13 test_util: handle dispatch.dispatch() taking a request Patrick Mezard <pmezard@gmail.com> parents: 793 diff changeset	290
78 072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	291 def rmtree(path):
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	292 # Read-only files cannot be removed under Windows
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	293 for root, dirs, files in os.walk(path):
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	294 for f in files:
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	295 f = os.path.join(root, f)
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	296 try:
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	297 s = os.stat(f)
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	298 except OSError, e:
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	299 if e.errno == errno.ENOENT:
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	300 continue
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	301 raise
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	302 if (s.st_mode & stat.S_IWRITE) == 0:
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	303 os.chmod(f, s.st_mode \| stat.S_IWRITE)
072010a271c6 Fix basic issues with tests on Windows Patrick Mezard <pmezard@gmail.com> parents: 23 diff changeset	304 shutil.rmtree(path)
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	305
645 3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	306 def _verify_our_modules():
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	307 '''
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	308 Verify that hgsubversion was imported from the correct location.
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	309
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	310 The correct location is any location within the parent directory of the
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	311 directory containing this file.
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	312 '''
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	313
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	314 for modname, module in sys.modules.iteritems():
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	315 if not module or not modname.startswith('hgsubversion.'):
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	316 continue
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	317
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	318 modloc = module.__file__
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	319 cp = os.path.commonprefix((os.path.abspath(__file__), modloc))
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	320 assert cp.rstrip(os.sep) == _rootdir, (
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	321 'Module location verification failed: hgsubversion was imported '
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	322 'from the wrong path!'
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	323 )
486 5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	324
890 78db88de9622 Partial metadata rebuilding David Schleimer <dschleimer@fb.com> parents: 888 diff changeset	325 def hgclone(ui, source, dest, update=True, rev=None):
816 86d124a8768e Fix hg.clone() calls changed by d976542986d2 Patrick Mezard <pmezard@gmail.com> parents: 813 diff changeset	326 if getattr(hg, 'peer', None):
86d124a8768e Fix hg.clone() calls changed by d976542986d2 Patrick Mezard <pmezard@gmail.com> parents: 813 diff changeset	327 # Since 1.9 (d976542986d2)
890 78db88de9622 Partial metadata rebuilding David Schleimer <dschleimer@fb.com> parents: 888 diff changeset	328 src, dest = hg.clone(ui, {}, source, dest, update=update, rev=rev)
816 86d124a8768e Fix hg.clone() calls changed by d976542986d2 Patrick Mezard <pmezard@gmail.com> parents: 813 diff changeset	329 else:
890 78db88de9622 Partial metadata rebuilding David Schleimer <dschleimer@fb.com> parents: 888 diff changeset	330 src, dest = hg.clone(ui, source, dest, update=update, rev=rev)
816 86d124a8768e Fix hg.clone() calls changed by d976542986d2 Patrick Mezard <pmezard@gmail.com> parents: 813 diff changeset	331 return src, dest
86d124a8768e Fix hg.clone() calls changed by d976542986d2 Patrick Mezard <pmezard@gmail.com> parents: 813 diff changeset	332
869 db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	333 def svnls(repo_path, path, rev='HEAD'):
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	334 path = repo_path + '/' + path
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	335 path = util.normalize_url(fileurl(path))
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	336 args = ['svn', 'ls', '-r', rev, '-R', path]
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	337 p = subprocess.Popen(args,
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	338 stdout=subprocess.PIPE,
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	339 stderr=subprocess.STDOUT)
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	340 stdout, stderr = p.communicate()
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	341 if p.returncode:
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	342 raise Exception('svn ls failed on %s: %r' % (path, stderr))
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	343 entries = [e.strip('/') for e in stdout.splitlines()]
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	344 entries.sort()
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	345 return entries
db3a651494f9 test_util: pass repo_path to svnls(), turn it into a function Patrick Mezard <patrick@mezard.eu> parents: 868 diff changeset	346
871 51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	347 def svnpropget(repo_path, path, prop, rev='HEAD'):
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	348 path = repo_path + '/' + path
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	349 path = util.normalize_url(fileurl(path))
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	350 args = ['svn', 'propget', '-r', str(rev), prop, path]
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	351 p = subprocess.Popen(args,
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	352 stdout=subprocess.PIPE,
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	353 stderr=subprocess.STDOUT)
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	354 stdout, stderr = p.communicate()
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	355 if p.returncode:
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	356 raise Exception('svn ls failed on %s: %r' % (path, stderr))
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	357 return stdout.strip()
51fd75ae62b9 test_util: pass repo_path to svnpropget(), turn into a function Patrick Mezard <patrick@mezard.eu> parents: 870 diff changeset	358
1049 608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	359
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	360 def _obsolete_wrap(cls, name):
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	361 origfunc = getattr(cls, name)
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	362
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	363 if not name.startswith('test_') or not origfunc:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	364 return
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	365
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	366 if not obsolete:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	367 wrapper = _makeskip(name, 'obsolete not available')
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	368 else:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	369 def wrapper(self, args, *opts):
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	370 self.assertFalse(obsolete._enabled, 'obsolete was already active')
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	371
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	372 obsolete._enabled = True
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	373
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	374 try:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	375 origfunc(self, args, *opts)
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	376 self.assertTrue(obsolete._enabled, 'obsolete remains active')
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	377 finally:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	378 obsolete._enabled = False
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	379
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	380 if not wrapper:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	381 return
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	382
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	383 wrapper.__name__ = name + ' obsolete'
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	384 wrapper.__module__ = origfunc.__module__
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	385
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	386 if origfunc.__doc__:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	387 firstline = origfunc.__doc__.strip().splitlines()[0]
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	388 wrapper.__doc__ = firstline + ' (obsolete)'
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	389
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	390 assert getattr(cls, wrapper.__name__, None) is None
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	391
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	392 setattr(cls, wrapper.__name__, wrapper)
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	393
1056 0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	394
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	395 def _stupid_wrap(cls, name):
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	396 origfunc = getattr(cls, name)
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	397
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	398 if not name.startswith('test_') or not origfunc:
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	399 return
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	400
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	401 def wrapper(self, args, *opts):
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	402 self.assertFalse(self.stupid, 'stupid mode was already active')
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	403
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	404 self.stupid = True
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	405
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	406 try:
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	407 origfunc(self, args, *opts)
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	408 finally:
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	409 self.stupid = False
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	410
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	411 wrapper.__name__ = name + ' stupid'
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	412 wrapper.__module__ = origfunc.__module__
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	413
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	414 if origfunc.__doc__:
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	415 firstline = origfunc.__doc__.strip().splitlines()[0]
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	416 wrapper.__doc__ = firstline + ' (stupid)'
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	417
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	418 assert getattr(cls, wrapper.__name__, None) is None
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	419
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	420 setattr(cls, wrapper.__name__, wrapper)
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	421
1049 608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	422 class TestMeta(type):
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	423 def __init__(cls, args, *opts):
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	424 if cls.obsolete_mode_tests:
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	425 for origname in dir(cls):
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	426 _obsolete_wrap(cls, origname)
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	427
1056 0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	428 if cls.stupid_mode_tests:
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	429 for origname in dir(cls):
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	430 _stupid_wrap(cls, origname)
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	431
1049 608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	432 return super(TestMeta, cls).__init__(args, *opts)
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	433
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	434 class TestBase(unittest.TestCase):
1049 608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	435 __metaclass__ = TestMeta
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	436
608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	437 obsolete_mode_tests = False
1056 0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	438 stupid_mode_tests = False
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	439
0932bb4d8870 tests: add a metaclass for triggering stupid on a class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1049 diff changeset	440 stupid = False
1049 608e7c8740af tests: add a metaclass for testing obsolete mode Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1048 diff changeset	441
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	442 def setUp(self):
645 3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	443 _verify_our_modules()
1101 4a92eb1484ba layouts: fix crash when importing hgsubversion itself (issue #402) Greg Ward <greg@gerg.ca> parents: 1092 diff changeset	444 if 'hgsubversion' in sys.modules:
4a92eb1484ba layouts: fix crash when importing hgsubversion itself (issue #402) Greg Ward <greg@gerg.ca> parents: 1092 diff changeset	445 sys.modules['hgext_hgsubversion'] = sys.modules['hgsubversion']
645 3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	446
896 04dfc23688b9 allow Python 2.7 to show medium-size multiline diffs when tests fail Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 890 diff changeset	447 # the Python 2.7 default of 640 is obnoxiously low
04dfc23688b9 allow Python 2.7 to show medium-size multiline diffs when tests fail Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 890 diff changeset	448 self.maxDiff = 4096
04dfc23688b9 allow Python 2.7 to show medium-size multiline diffs when tests fail Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 890 diff changeset	449
832 e9af7eba88db globally: clean up whitespace around operators and commas to conform with PEP8 Yonggang Luo <luoyonggang@gmail.com> parents: 816 diff changeset	450 self.oldenv = dict([(k, os.environ.get(k, None),) for k in
e9af7eba88db globally: clean up whitespace around operators and commas to conform with PEP8 Yonggang Luo <luoyonggang@gmail.com> parents: 816 diff changeset	451 ('LANG', 'LC_ALL', 'HGRCPATH',)])
486 5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	452 self.oldt = i18n.t
5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	453 os.environ['LANG'] = os.environ['LC_ALL'] = 'C'
5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	454 i18n.t = gettext.translation('hg', i18n.localedir, fallback=True)
5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	455
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	456 self.oldwd = os.getcwd()
148 0c5f6420a8b5 tests: Add an environment variable (HGSUBVERSION_TEST_TEMP) which allows Augie Fackler <durin42@gmail.com> parents: 139 diff changeset	457 self.tmpdir = tempfile.mkdtemp(
0c5f6420a8b5 tests: Add an environment variable (HGSUBVERSION_TEST_TEMP) which allows Augie Fackler <durin42@gmail.com> parents: 139 diff changeset	458 'svnwrap_test', dir=os.environ.get('HGSUBVERSION_TEST_TEMP', None))
1081 c8d0f5983571 tests: cd to tempdir during setUp David Schleimer <dschleimer@fb.com> parents: 1080 diff changeset	459 os.chdir(self.tmpdir)
231 b1543f243910 tests: Start providing our own custom hgrc for testing purposes. Augie Fackler <durin42@gmail.com> parents: 194 diff changeset	460 self.hgrc = os.path.join(self.tmpdir, '.hgrc')
b1543f243910 tests: Start providing our own custom hgrc for testing purposes. Augie Fackler <durin42@gmail.com> parents: 194 diff changeset	461 os.environ['HGRCPATH'] = self.hgrc
1003 6945d3359456 testutil: clear rcpath when we update HGRCPATH env var David Schleimer <dschleimer@fb.com> parents: 1001 diff changeset	462 scmutil._rcpath = None
231 b1543f243910 tests: Start providing our own custom hgrc for testing purposes. Augie Fackler <durin42@gmail.com> parents: 194 diff changeset	463 rc = open(self.hgrc, 'w')
1001 26aa0172725b test_util: specify a username in our test-run hgrc. Augie Fackler <raf@durin42.com> parents: 952 diff changeset	464 rc.write('[ui]\nusername=test-user\n')
334 3c3c3264c362 test_util: Make the patching of ui.ui.write_err() slightly more reusable. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 333 diff changeset	465 for l in '[extensions]', 'hgsubversion=':
3c3c3264c362 test_util: Make the patching of ui.ui.write_err() slightly more reusable. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 333 diff changeset	466 print >> rc, l
148 0c5f6420a8b5 tests: Add an environment variable (HGSUBVERSION_TEST_TEMP) which allows Augie Fackler <durin42@gmail.com> parents: 139 diff changeset	467
872 a279b5838aaf test_util: remove self.repo_path, generate new paths each time Patrick Mezard <patrick@mezard.eu> parents: 871 diff changeset	468 self.repocount = 0
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	469 self.wc_path = '%s/testrepo_wc' % self.tmpdir
689 35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	470 self.svn_wc = None
331 75f082b5897e Switch to using url scheme wrappers instead of duplicating each command we wrap. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 326 diff changeset	471
911 772280aed751 Honor SVN auto-props (solves issue #186) Ronny Voelker <ronny.voelker@googlemail.com> parents: 910 diff changeset	472 self.config_dir = self.tmpdir
772280aed751 Honor SVN auto-props (solves issue #186) Ronny Voelker <ronny.voelker@googlemail.com> parents: 910 diff changeset	473 svnwrap.common._svn_config_dir = self.config_dir
772280aed751 Honor SVN auto-props (solves issue #186) Ronny Voelker <ronny.voelker@googlemail.com> parents: 910 diff changeset	474 self.setup_svn_config('')
772280aed751 Honor SVN auto-props (solves issue #186) Ronny Voelker <ronny.voelker@googlemail.com> parents: 910 diff changeset	475
331 75f082b5897e Switch to using url scheme wrappers instead of duplicating each command we wrap. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 326 diff changeset	476 # Previously, we had a MockUI class that wrapped ui, and giving access
75f082b5897e Switch to using url scheme wrappers instead of duplicating each command we wrap. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 326 diff changeset	477 # to the stream. The ui.pushbuffer() and ui.popbuffer() can be used
75f082b5897e Switch to using url scheme wrappers instead of duplicating each command we wrap. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 326 diff changeset	478 # instead. Using the regular UI class, with all stderr redirected to
75f082b5897e Switch to using url scheme wrappers instead of duplicating each command we wrap. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 326 diff changeset	479 # stdout ensures that the test setup is much more similar to usage
75f082b5897e Switch to using url scheme wrappers instead of duplicating each command we wrap. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 326 diff changeset	480 # setups.
334 3c3c3264c362 test_util: Make the patching of ui.ui.write_err() slightly more reusable. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 333 diff changeset	481 self.patch = (ui.ui.write_err, ui.ui.write)
3c3c3264c362 test_util: Make the patching of ui.ui.write_err() slightly more reusable. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 333 diff changeset	482 setattr(ui.ui, self.patch[0].func_name, self.patch[1])
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	483
911 772280aed751 Honor SVN auto-props (solves issue #186) Ronny Voelker <ronny.voelker@googlemail.com> parents: 910 diff changeset	484 def setup_svn_config(self, config):
1038 aa043f64d8b7 tests: fix syntax incompatible with Python 2.4 Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 1003 diff changeset	485 c = open(self.config_dir + '/config', 'w')
aa043f64d8b7 tests: fix syntax incompatible with Python 2.4 Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 1003 diff changeset	486 try:
911 772280aed751 Honor SVN auto-props (solves issue #186) Ronny Voelker <ronny.voelker@googlemail.com> parents: 910 diff changeset	487 c.write(config)
1038 aa043f64d8b7 tests: fix syntax incompatible with Python 2.4 Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 1003 diff changeset	488 finally:
aa043f64d8b7 tests: fix syntax incompatible with Python 2.4 Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 1003 diff changeset	489 c.close()
911 772280aed751 Honor SVN auto-props (solves issue #186) Ronny Voelker <ronny.voelker@googlemail.com> parents: 910 diff changeset	490
872 a279b5838aaf test_util: remove self.repo_path, generate new paths each time Patrick Mezard <patrick@mezard.eu> parents: 871 diff changeset	491 def _makerepopath(self):
a279b5838aaf test_util: remove self.repo_path, generate new paths each time Patrick Mezard <patrick@mezard.eu> parents: 871 diff changeset	492 self.repocount += 1
a279b5838aaf test_util: remove self.repo_path, generate new paths each time Patrick Mezard <patrick@mezard.eu> parents: 871 diff changeset	493 return '%s/testrepo-%d' % (self.tmpdir, self.repocount)
a279b5838aaf test_util: remove self.repo_path, generate new paths each time Patrick Mezard <patrick@mezard.eu> parents: 871 diff changeset	494
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	495 def tearDown(self):
486 5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	496 for var, val in self.oldenv.iteritems():
5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	497 if val is None:
5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	498 del os.environ[var]
5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	499 else:
5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	500 os.environ[var] = val
5607e81ba616 test_util: specify LANG=LC_ALL=C so l10n stops breaking tests Augie Fackler <durin42@gmail.com> parents: 483 diff changeset	501 i18n.t = self.oldt
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	502 rmtree(self.tmpdir)
71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	503 os.chdir(self.oldwd)
334 3c3c3264c362 test_util: Make the patching of ui.ui.write_err() slightly more reusable. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 333 diff changeset	504 setattr(ui.ui, self.patch[0].func_name, self.patch[0])
91 7d10165cf3d9 tests: Mock the mercurial.ui.ui class like we really should to capture output. Augie Fackler <durin42@gmail.com> parents: 84 diff changeset	505
645 3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	506 _verify_our_modules()
3cb5042531fb tests: verify locations of any imported hgsubversion modules Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 644 diff changeset	507
1078 1afd308b8f46 tests: remove ability to toggle stupidity on less than class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1056 diff changeset	508 def ui(self, layout='auto'):
1afd308b8f46 tests: remove ability to toggle stupidity on less than class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1056 diff changeset	509 return testui(self.stupid, layout)
576 d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	510
866 20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	511 def load_svndump(self, fixture_name):
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	512 '''Loads an svnadmin dump into a fresh repo. Return the svn repo
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	513 path.
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	514 '''
872 a279b5838aaf test_util: remove self.repo_path, generate new paths each time Patrick Mezard <patrick@mezard.eu> parents: 871 diff changeset	515 path = self._makerepopath()
a279b5838aaf test_util: remove self.repo_path, generate new paths each time Patrick Mezard <patrick@mezard.eu> parents: 871 diff changeset	516 assert not os.path.exists(path)
866 20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	517 subprocess.call(['svnadmin', 'create', path,],
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	518 stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	519 inp = open(os.path.join(FIXTURES, fixture_name))
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	520 proc = subprocess.Popen(['svnadmin', 'load', path,], stdin=inp,
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	521 stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	522 proc.communicate()
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	523 return path
20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	524
910 312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	525 def load_repo_tarball(self, fixture_name):
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	526 '''Extracts a tarball of an svn repo and returns the svn repo path.'''
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	527 path = self._makerepopath()
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	528 assert not os.path.exists(path)
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	529 os.mkdir(path)
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	530 tarball = tarfile.open(os.path.join(FIXTURES, fixture_name))
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	531 # This is probably somewhat fragile, but I'm not sure how to
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	532 # do better in particular, I think it assumes that the tar
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	533 # entries are in the right order and that directories appear
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	534 # before their contents. This is a valid assummption for sane
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	535 # tarballs, from what I can tell. In particular, for a simple
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	536 # tarball of a svn repo with paths relative to the repo root,
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	537 # it seems to work
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	538 for entry in tarball:
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	539 tarball.extract(entry, path)
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	540 return path
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	541
1078 1afd308b8f46 tests: remove ability to toggle stupidity on less than class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1056 diff changeset	542 def fetch(self, repo_path, subdir=None, layout='auto',
944 d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 930 diff changeset	543 startrev=0, externals=None, noupdate=True, dest=None, rev=None,
d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 930 diff changeset	544 config=None):
499 1fd3cfa47c5e Support for single-directory clones. Augie Fackler <durin42@gmail.com> parents: 492 diff changeset	545 if layout == 'single':
1fd3cfa47c5e Support for single-directory clones. Augie Fackler <durin42@gmail.com> parents: 492 diff changeset	546 if subdir is None:
1fd3cfa47c5e Support for single-directory clones. Augie Fackler <durin42@gmail.com> parents: 492 diff changeset	547 subdir = 'trunk'
1fd3cfa47c5e Support for single-directory clones. Augie Fackler <durin42@gmail.com> parents: 492 diff changeset	548 elif subdir is None:
1fd3cfa47c5e Support for single-directory clones. Augie Fackler <durin42@gmail.com> parents: 492 diff changeset	549 subdir = ''
866 20e73b5ab6f7 test_util: merge load_svndump_fixture() into TestBase Patrick Mezard <patrick@mezard.eu> parents: 865 diff changeset	550 projectpath = repo_path
865 04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	551 if subdir:
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	552 projectpath += '/' + subdir
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	553
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	554 cmd = [
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	555 'clone',
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	556 '--layout=%s' % layout,
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	557 '--startrev=%s' % startrev,
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	558 fileurl(projectpath),
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	559 self.wc_path,
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	560 ]
1078 1afd308b8f46 tests: remove ability to toggle stupidity on less than class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1056 diff changeset	561 if self.stupid:
865 04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	562 cmd.append('--stupid')
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	563 if noupdate:
04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	564 cmd.append('--noupdate')
908 c4ee11a5d04c pull: add a hgsubversion.unsafeskip option to omit unwanted revs Bryan O'Sullivan <bryano@fb.com> parents: 896 diff changeset	565 if rev is not None:
c4ee11a5d04c pull: add a hgsubversion.unsafeskip option to omit unwanted revs Bryan O'Sullivan <bryano@fb.com> parents: 896 diff changeset	566 cmd.append('--rev=%s' % rev)
944 d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 930 diff changeset	567 config = dict(config or {})
865 04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	568 if externals:
944 d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 930 diff changeset	569 config['hgsubversion.externals'] = str(externals)
d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 930 diff changeset	570 for k,v in reversed(sorted(config.iteritems())):
d6db289f1548 pull: add hgsubversion.filestoresize to control memory consumption Patrick Mezard <patrick@mezard.eu> parents: 930 diff changeset	571 cmd[:0] = ['--config', '%s=%s' % (k, v)]
865 04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	572
1041 70090e2ee262 tests: assert that fetches actually succeed Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 1038 diff changeset	573 r = dispatch(cmd)
70090e2ee262 tests: assert that fetches actually succeed Dan Villiom Podlaski Christiansen <dan@cabo.dk> parents: 1038 diff changeset	574 assert not r, 'fetch of %s failed' % projectpath
865 04729f3a3d17 test_util: merge load_fixture_and_fetch() into TestBase method Patrick Mezard <patrick@mezard.eu> parents: 832 diff changeset	575
886 d3ff5807f1bd fold test_startrev and test_verify into a new test; test_verify_and_startrev Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 874 diff changeset	576 return hg.repository(testui(), self.wc_path)
d3ff5807f1bd fold test_startrev and test_verify into a new test; test_verify_and_startrev Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 874 diff changeset	577
d3ff5807f1bd fold test_startrev and test_verify into a new test; test_verify_and_startrev Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 874 diff changeset	578 def load_and_fetch(self, fixture_name, args, *opts):
910 312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	579 if fixture_name.endswith('.svndump'):
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	580 repo_path = self.load_svndump(fixture_name)
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	581 elif fixture_name.endswith('tar.gz'):
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	582 repo_path = self.load_repo_tarball(fixture_name)
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	583 else:
312f36a425f0 Ignore invalid utf8 in commit messages David Schleimer <dschleimer@fb.com> parents: 908 diff changeset	584 assert False, 'Unknown fixture type'
886 d3ff5807f1bd fold test_startrev and test_verify into a new test; test_verify_and_startrev Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 874 diff changeset	585
d3ff5807f1bd fold test_startrev and test_verify into a new test; test_verify_and_startrev Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 874 diff changeset	586 return self.fetch(repo_path, args, *opts), repo_path
867 50c13e01c7e3 test_util: add a load_and_fetch() returning the repo_path Patrick Mezard <patrick@mezard.eu> parents: 866 diff changeset	587
50c13e01c7e3 test_util: add a load_and_fetch() returning the repo_path Patrick Mezard <patrick@mezard.eu> parents: 866 diff changeset	588 def _load_fixture_and_fetch(self, args, *kwargs):
50c13e01c7e3 test_util: add a load_and_fetch() returning the repo_path Patrick Mezard <patrick@mezard.eu> parents: 866 diff changeset	589 repo, repo_path = self.load_and_fetch(args, *kwargs)
50c13e01c7e3 test_util: add a load_and_fetch() returning the repo_path Patrick Mezard <patrick@mezard.eu> parents: 866 diff changeset	590 return repo
138 40474f6c1f84 diff_cmd: more robust, add tests. Augie Fackler <durin42@gmail.com> parents: 112 diff changeset	591
868 cc1c870f1758 test_util: pass repo_path to _add_svn_rev() explicitely Patrick Mezard <patrick@mezard.eu> parents: 867 diff changeset	592 def add_svn_rev(self, repo_path, changes):
689 35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	593 '''changes is a dict of filename -> contents'''
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	594 if self.svn_wc is None:
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	595 self.svn_wc = os.path.join(self.tmpdir, 'testsvn_wc')
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	596 subprocess.call([
868 cc1c870f1758 test_util: pass repo_path to _add_svn_rev() explicitely Patrick Mezard <patrick@mezard.eu> parents: 867 diff changeset	597 'svn', 'co', '-q', fileurl(repo_path),
689 35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	598 self.svn_wc
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	599 ],
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	600 stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	601
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	602 for filename, contents in changes.iteritems():
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	603 # filenames are / separated
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	604 filename = filename.replace('/', os.path.sep)
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	605 filename = os.path.join(self.svn_wc, filename)
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	606 open(filename, 'w').write(contents)
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	607 # may be redundant
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	608 subprocess.call(['svn', 'add', '-q', filename],
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	609 stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	610 subprocess.call([
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	611 'svn', 'commit', '-q', self.svn_wc, '-m', 'test changes'],
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	612 stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
35a1e93b6f78 tests: move _add_svn_rev to test_util for reuse Peter Arrenbrecht <peter.arrenbrecht@gmail.com> parents: 652 diff changeset	613
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	614 # define this as a property so that it reloads anytime we need it
71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	615 @property
71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	616 def repo(self):
576 d96aa92d9ad9 tests: silence test suite by using quiet UIs everywhere Dirkjan Ochtman <dirkjan@ochtman.nl> parents: 548 diff changeset	617 return hg.repository(testui(), self.wc_path)
82 71de43e9f614 Extract PushTest common code into test_util.TestBase Patrick Mezard <pmezard@gmail.com> parents: 78 diff changeset	618
1078 1afd308b8f46 tests: remove ability to toggle stupidity on less than class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1056 diff changeset	619 def pushrevisions(self, expected_extra_back=0):
1048 903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	620 before = repolen(self.repo)
1078 1afd308b8f46 tests: remove ability to toggle stupidity on less than class level Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1056 diff changeset	621 self.repo.ui.setconfig('hgsubversion', 'stupid', str(self.stupid))
626 8e621dbb82d4 push: return reasonable status codes to the end user Augie Fackler <durin42@gmail.com> parents: 621 diff changeset	622 res = commands.push(self.repo.ui, self.repo)
1048 903c9c9dfe6a tests: count revisions explicitly Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1041 diff changeset	623 after = repolen(self.repo)
355 256863a65141 test_util: make it easier to test odd push cases Augie Fackler <durin42@gmail.com> parents: 351 diff changeset	624 self.assertEqual(expected_extra_back, after - before)
626 8e621dbb82d4 push: return reasonable status codes to the end user Augie Fackler <durin42@gmail.com> parents: 621 diff changeset	625 return res
83 6c9b7cf1c5aa push_cmd: delete empty svn directories, refactor directory creation Patrick Mezard <pmezard@gmail.com> parents: 82 diff changeset	626
870 1eb2a4428c42 test_util: pass repo_path to svnco() explicitely Patrick Mezard <patrick@mezard.eu> parents: 869 diff changeset	627 def svnco(self, repo_path, svnpath, rev, path):
764 bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	628 path = os.path.join(self.wc_path, path)
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	629 subpath = os.path.dirname(path)
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	630 if not os.path.isdir(subpath):
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	631 os.makedirs(subpath)
870 1eb2a4428c42 test_util: pass repo_path to svnco() explicitely Patrick Mezard <patrick@mezard.eu> parents: 869 diff changeset	632 svnpath = fileurl(repo_path + '/' + svnpath)
764 bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	633 args = ['svn', 'co', '-r', rev, svnpath, path]
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	634 p = subprocess.Popen(args,
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	635 stdout=subprocess.PIPE,
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	636 stderr=subprocess.STDOUT)
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	637 stdout, stderr = p.communicate()
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	638 if p.returncode:
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	639 raise Exception('svn co failed on %s: %r' % (svnpath, stderr))
bc5c176b63eb svnexternals: support pushing subrepo based externals Patrick Mezard <pmezard@gmail.com> parents: 762 diff changeset	640
355 256863a65141 test_util: make it easier to test odd push cases Augie Fackler <durin42@gmail.com> parents: 351 diff changeset	641 def commitchanges(self, changes, parent='tip', message='automated test'):
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	642 """Commit changes to mercurial directory
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	643
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	644 'changes' is a sequence of tuples (source, dest, data). It can look
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	645 like:
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	646 - (source, source, data) to set source content to data
91 7d10165cf3d9 tests: Mock the mercurial.ui.ui class like we really should to capture output. Augie Fackler <durin42@gmail.com> parents: 84 diff changeset	647 - (source, dest, None) to set dest content to source one, and mark it as
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	648 copied from source.
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	649 - (source, dest, data) to set dest content to data, and mark it as copied
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	650 from source.
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	651 - (source, None, None) to remove source.
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	652 """
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	653 repo = self.repo
277 3848a7f9b983 push: Add a test that demonstrates base-text detection works Augie Fackler <durin42@gmail.com> parents: 270 diff changeset	654 parentctx = repo[parent]
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	655
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	656 changed, removed = [], []
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	657 for source, dest, newdata in changes:
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	658 if dest is None:
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	659 removed.append(source)
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	660 else:
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	661 changed.append(dest)
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	662
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	663 def filectxfn(repo, memctx, path):
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	664 if path in removed:
531 cf4fe45bf8fd Change all instantiations of IOError to set both errno and strerror. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 527 diff changeset	665 raise IOError(errno.ENOENT,
cf4fe45bf8fd Change all instantiations of IOError to set both errno and strerror. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 527 diff changeset	666 "File \"%s\" no longer exists" % path)
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	667 entry = [e for e in changes if path == e[1]][0]
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	668 source, dest, newdata = entry
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	669 if newdata is None:
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	670 newdata = parentctx[source].data()
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	671 copied = None
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	672 if source != dest:
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	673 copied = source
1203 518a9552f7f9 test_util: call makememfilectx Sean Farley <sean.michael.farley@gmail.com> parents: 1174 diff changeset	674 return compathacks.makememfilectx(repo,
518a9552f7f9 test_util: call makememfilectx Sean Farley <sean.michael.farley@gmail.com> parents: 1174 diff changeset	675 path=dest,
518a9552f7f9 test_util: call makememfilectx Sean Farley <sean.michael.farley@gmail.com> parents: 1174 diff changeset	676 data=newdata,
518a9552f7f9 test_util: call makememfilectx Sean Farley <sean.michael.farley@gmail.com> parents: 1174 diff changeset	677 islink=False,
518a9552f7f9 test_util: call makememfilectx Sean Farley <sean.michael.farley@gmail.com> parents: 1174 diff changeset	678 isexec=False,
518a9552f7f9 test_util: call makememfilectx Sean Farley <sean.michael.farley@gmail.com> parents: 1174 diff changeset	679 copied=copied)
91 7d10165cf3d9 tests: Mock the mercurial.ui.ui class like we really should to capture output. Augie Fackler <durin42@gmail.com> parents: 84 diff changeset	680
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	681 ctx = context.memctx(repo,
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	682 (parentctx.node(), node.nullid),
355 256863a65141 test_util: make it easier to test odd push cases Augie Fackler <durin42@gmail.com> parents: 351 diff changeset	683 message,
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	684 changed + removed,
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	685 filectxfn,
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	686 'an_author',
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	687 '2008-10-07 20:59:48 -0500')
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	688 nodeid = repo.commitctx(ctx)
01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	689 repo = self.repo
277 3848a7f9b983 push: Add a test that demonstrates base-text detection works Augie Fackler <durin42@gmail.com> parents: 270 diff changeset	690 hg.clean(repo, nodeid)
84 01e747937d35 test_util: add commitchanges() to TestBase Patrick Mezard <pmezard@gmail.com> parents: 83 diff changeset	691 return nodeid
96 9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	692
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	693 def assertchanges(self, changes, ctx):
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	694 """Assert that all 'changes' (as in defined in commitchanged())
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	695 went into ctx.
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	696 """
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	697 for source, dest, data in changes:
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	698 if dest is None:
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	699 self.assertTrue(source not in ctx)
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	700 continue
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	701 self.assertTrue(dest in ctx)
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	702 if data is None:
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	703 data = ctx.parents()[0][source].data()
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	704 self.assertEqual(ctx[dest].data(), data)
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	705 if dest != source:
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	706 copy = ctx[dest].renamed()
9b5e528f67f8 Add a test to check EOLs are correctly converted Patrick Mezard <pmezard@gmail.com> parents: 91 diff changeset	707 self.assertEqual(copy[0], source)
527 2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	708
621 3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	709 def assertMultiLineEqual(self, first, second, msg=None):
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	710 """Assert that two multi-line strings are equal. (Based on Py3k code.)
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	711 """
744 6d6be5284056 tests: make assertMultiLineEqual call super, if possible. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 743 diff changeset	712 try:
6d6be5284056 tests: make assertMultiLineEqual call super, if possible. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 743 diff changeset	713 return super(TestBase, self).assertMultiLineEqual(first, second,
6d6be5284056 tests: make assertMultiLineEqual call super, if possible. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 743 diff changeset	714 msg)
6d6be5284056 tests: make assertMultiLineEqual call super, if possible. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 743 diff changeset	715 except AttributeError:
6d6be5284056 tests: make assertMultiLineEqual call super, if possible. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 743 diff changeset	716 pass
6d6be5284056 tests: make assertMultiLineEqual call super, if possible. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 743 diff changeset	717
621 3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	718 self.assert_(isinstance(first, str),
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	719 ('First argument is not a string'))
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	720 self.assert_(isinstance(second, str),
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	721 ('Second argument is not a string'))
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	722
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	723 if first != second:
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	724 diff = ''.join(difflib.unified_diff(first.splitlines(True),
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	725 second.splitlines(True),
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	726 fromfile='a',
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	727 tofile='b'))
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	728 msg = '%s\n%s' % (msg or '', diff)
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	729 raise self.failureException, msg
3e18cdcb6e00 test_util: add assertMultiLineEqual() method. Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 576 diff changeset	730
952 9c3b4f59e7e6 stupid: do not close branch upon branch-wide revert Patrick Mezard <patrick@mezard.eu> parents: 944 diff changeset	731 def getgraph(self, repo):
527 2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	732 """Helper function displaying a repository graph, especially
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	733 useful when debugging comprehensive tests.
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	734 """
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	735 # Could be more elegant, but it works with stock hg
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	736 _ui = ui.ui()
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	737 _ui.setconfig('extensions', 'graphlog', '')
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	738 extensions.loadall(_ui)
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	739 graphlog = extensions.find('graphlog')
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	740 templ = """\
1084 ca36d5915dc7 test_util: include Subversion revision when drawing a pretty graph Dan Villiom Podlaski Christiansen <danchr@gmail.com> parents: 1081 diff changeset	741 changeset: {rev}:{node\|short} (r{svnrev})
527 2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	742 branch: {branches}
546 d84116dda52d test_util: add tags to output of draw Augie Fackler <durin42@gmail.com> parents: 531 diff changeset	743 tags: {tags}
527 2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	744 summary: {desc\|firstline}
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	745 files: {files}
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	746
2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	747 """
952 9c3b4f59e7e6 stupid: do not close branch upon branch-wide revert Patrick Mezard <patrick@mezard.eu> parents: 944 diff changeset	748 _ui.pushbuffer()
527 2be9f14bd23f test_util: add helper to draw repo with graphlog extension Patrick Mezard <pmezard@gmail.com> parents: 522 diff changeset	749 graphlog.graphlog(_ui, repo, rev=None, template=templ)
952 9c3b4f59e7e6 stupid: do not close branch upon branch-wide revert Patrick Mezard <patrick@mezard.eu> parents: 944 diff changeset	750 return _ui.popbuffer()
9c3b4f59e7e6 stupid: do not close branch upon branch-wide revert Patrick Mezard <patrick@mezard.eu> parents: 944 diff changeset	751
9c3b4f59e7e6 stupid: do not close branch upon branch-wide revert Patrick Mezard <patrick@mezard.eu> parents: 944 diff changeset	752 def draw(self, repo):
9c3b4f59e7e6 stupid: do not close branch upon branch-wide revert Patrick Mezard <patrick@mezard.eu> parents: 944 diff changeset	753 sys.stdout.write(self.getgraph(repo))

Mercurial > hgsubversion

annotate tests/test_util.py @ 1233:0d0132cba155