mirror of
https://github.com/facebook/sapling.git
synced 2024-10-10 16:57:49 +03:00
Added some coverage pragmas to stop it from trying to cover things we can't test.
This commit is contained in:
parent
dee222b7e9
commit
90bc5e409c
@ -18,7 +18,7 @@ import svnwrap
|
|||||||
import util
|
import util
|
||||||
|
|
||||||
|
|
||||||
def print_your_svn_is_old_message(ui):
|
def print_your_svn_is_old_message(ui): #pragma: no cover
|
||||||
ui.status("In light of that, I'll fall back and do diffs, but it won't do "
|
ui.status("In light of that, I'll fall back and do diffs, but it won't do "
|
||||||
"as good a job. You should really upgrade your server.\n")
|
"as good a job. You should really upgrade your server.\n")
|
||||||
|
|
||||||
@ -34,7 +34,7 @@ def fetch_revisions(ui, svn_url, hg_repo_path, skipto_rev=0, stupid=None,
|
|||||||
skipto_rev=int(skipto_rev)
|
skipto_rev=int(skipto_rev)
|
||||||
have_replay = not stupid
|
have_replay = not stupid
|
||||||
if have_replay and not callable(delta.svn_txdelta_apply(None, None,
|
if have_replay and not callable(delta.svn_txdelta_apply(None, None,
|
||||||
None)[0]):
|
None)[0]): #pragma: no cover
|
||||||
ui.status('You are using old Subversion SWIG bindings. Replay will not'
|
ui.status('You are using old Subversion SWIG bindings. Replay will not'
|
||||||
' work until you upgrade to 1.5.0 or newer. Falling back to'
|
' work until you upgrade to 1.5.0 or newer. Falling back to'
|
||||||
' a slower method that may be buggier. Please upgrade, or'
|
' a slower method that may be buggier. Please upgrade, or'
|
||||||
@ -81,7 +81,7 @@ def fetch_revisions(ui, svn_url, hg_repo_path, skipto_rev=0, stupid=None,
|
|||||||
if have_replay:
|
if have_replay:
|
||||||
try:
|
try:
|
||||||
replay_convert_rev(hg_editor, svn, r)
|
replay_convert_rev(hg_editor, svn, r)
|
||||||
except svnwrap.SubversionRepoCanNotReplay, e:
|
except svnwrap.SubversionRepoCanNotReplay, e: #pragma: no cover
|
||||||
ui.status('%s\n' % e.message)
|
ui.status('%s\n' % e.message)
|
||||||
print_your_svn_is_old_message(ui)
|
print_your_svn_is_old_message(ui)
|
||||||
have_replay = False
|
have_replay = False
|
||||||
@ -95,7 +95,7 @@ def fetch_revisions(ui, svn_url, hg_repo_path, skipto_rev=0, stupid=None,
|
|||||||
fp.close()
|
fp.close()
|
||||||
merc_util.rename(tmpfile,
|
merc_util.rename(tmpfile,
|
||||||
hg_editor.last_revision_handled_file)
|
hg_editor.last_revision_handled_file)
|
||||||
except core.SubversionException, e:
|
except core.SubversionException, e: #pragma: no cover
|
||||||
if hasattr(e, 'message') and (
|
if hasattr(e, 'message') and (
|
||||||
'Server sent unexpected return value (502 Bad Gateway)'
|
'Server sent unexpected return value (502 Bad Gateway)'
|
||||||
' in response to PROPFIND') in e.message:
|
' in response to PROPFIND') in e.message:
|
||||||
@ -133,7 +133,7 @@ def replay_convert_rev(hg_editor, svn, r):
|
|||||||
pl = svn.proplist(p2, r.revnum, recurse=True)
|
pl = svn.proplist(p2, r.revnum, recurse=True)
|
||||||
cleanup_file_handles(svn, i)
|
cleanup_file_handles(svn, i)
|
||||||
i += 1
|
i += 1
|
||||||
except core.SubversionException, e:
|
except core.SubversionException, e: #pragma: no cover
|
||||||
pass
|
pass
|
||||||
props.update(pl)
|
props.update(pl)
|
||||||
if p[-1] == '/':
|
if p[-1] == '/':
|
||||||
@ -391,7 +391,7 @@ def stupid_svn_server_pull_rev(ui, svn, hg_editor, r):
|
|||||||
if e.apr_err == 170000 or (e.message.startswith("URL '")
|
if e.apr_err == 170000 or (e.message.startswith("URL '")
|
||||||
and e.message.endswith("' doesn't exist")):
|
and e.message.endswith("' doesn't exist")):
|
||||||
delete_all_files = True
|
delete_all_files = True
|
||||||
else:
|
else: #pragma: no cover
|
||||||
raise
|
raise
|
||||||
|
|
||||||
except BadPatchApply, e:
|
except BadPatchApply, e:
|
||||||
|
@ -11,12 +11,12 @@ from svn import core
|
|||||||
from svn import delta
|
from svn import delta
|
||||||
from svn import ra
|
from svn import ra
|
||||||
|
|
||||||
if (core.SVN_VER_MAJOR, core.SVN_VER_MINOR, core.SVN_VER_MICRO) < (1, 5, 0):
|
if (core.SVN_VER_MAJOR, core.SVN_VER_MINOR, core.SVN_VER_MICRO) < (1, 5, 0): #pragma: no cover
|
||||||
raise ImportError, 'You must have Subversion 1.5.0 or newer and matching SWIG bindings.'
|
raise ImportError, 'You must have Subversion 1.5.0 or newer and matching SWIG bindings.'
|
||||||
|
|
||||||
svn_config = core.svn_config_get_config(None)
|
svn_config = core.svn_config_get_config(None)
|
||||||
class RaCallbacks(ra.Callbacks):
|
class RaCallbacks(ra.Callbacks):
|
||||||
def open_tmp_file(self, pool):
|
def open_tmp_file(self, pool): #pragma: no cover
|
||||||
(fd, fn) = tempfile.mkstemp()
|
(fd, fn) = tempfile.mkstemp()
|
||||||
os.close(fd)
|
os.close(fd)
|
||||||
return fn
|
return fn
|
||||||
@ -25,7 +25,7 @@ class RaCallbacks(ra.Callbacks):
|
|||||||
return 'hgsubversion'
|
return 'hgsubversion'
|
||||||
|
|
||||||
|
|
||||||
def user_pass_prompt(realm, default_username, ms, pool):
|
def user_pass_prompt(realm, default_username, ms, pool): #pragma: no cover
|
||||||
creds = core.svn_auth_cred_simple_t()
|
creds = core.svn_auth_cred_simple_t()
|
||||||
creds.may_save = ms
|
creds.may_save = ms
|
||||||
if default_username:
|
if default_username:
|
||||||
@ -51,12 +51,12 @@ def _create_auth_baton(pool):
|
|||||||
client.get_ssl_server_trust_file_provider(),
|
client.get_ssl_server_trust_file_provider(),
|
||||||
]
|
]
|
||||||
# Platform-dependant authentication methods
|
# Platform-dependant authentication methods
|
||||||
if hasattr(client, 'get_windows_simple_provider'):
|
if hasattr(client, 'get_windows_simple_provider'): #pragma: no cover
|
||||||
try:
|
try:
|
||||||
providers.append(client.get_windows_simple_provider())
|
providers.append(client.get_windows_simple_provider())
|
||||||
except:
|
except:
|
||||||
pass
|
pass
|
||||||
if hasattr(client, 'get_keychain_simple_provider'):
|
if hasattr(client, 'get_keychain_simple_provider'): #pragma: no cover
|
||||||
try:
|
try:
|
||||||
providers.append(client.get_keychain_simple_provider())
|
providers.append(client.get_keychain_simple_provider())
|
||||||
except:
|
except:
|
||||||
@ -297,7 +297,7 @@ class SubversionRepo(object):
|
|||||||
elif action == 'add':
|
elif action == 'add':
|
||||||
try:
|
try:
|
||||||
baton = editor.add_file(path, parent, None, -1, pool)
|
baton = editor.add_file(path, parent, None, -1, pool)
|
||||||
except (core.SubversionException, TypeError), e:
|
except (core.SubversionException, TypeError), e: #pragma: no cover
|
||||||
print e.message
|
print e.message
|
||||||
raise
|
raise
|
||||||
elif action == 'delete':
|
elif action == 'delete':
|
||||||
@ -334,7 +334,7 @@ class SubversionRepo(object):
|
|||||||
try:
|
try:
|
||||||
ra.replay(self.ra, revision, oldest_rev_i_have, True, e_ptr,
|
ra.replay(self.ra, revision, oldest_rev_i_have, True, e_ptr,
|
||||||
e_baton, self.pool)
|
e_baton, self.pool)
|
||||||
except core.SubversionException, e:
|
except core.SubversionException, e: #pragma: no cover
|
||||||
# can I depend on this number being constant?
|
# can I depend on this number being constant?
|
||||||
if (e.message == "Server doesn't support the replay command"
|
if (e.message == "Server doesn't support the replay command"
|
||||||
or e.apr_err == 170003
|
or e.apr_err == 170003
|
||||||
|
Loading…
Reference in New Issue
Block a user