mercurial-scm/hg-stable: tests/test-check-interfaces.py annotate

annotate tests/test-check-interfaces.py @ 46699:0738bc25d6ac

sshpeer: add a method to check if a doublepipe is closed So we can tell in a next commit if we're trying to close an already closed connection or not (in which case, we may warn). Differential Revision: https://phab.mercurial-scm.org/D9996

author	Valentin Gatien-Baron <valentin.gatienbaron@gmail.com>
date	Mon, 15 Feb 2021 14:11:38 -0500
parents	300f6e23882a
children	a41565bef69f

rev	line source
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	1 # Test that certain objects conform to well-defined interfaces.
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	2
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	3 from __future__ import absolute_import, print_function
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	4
37810 856f381ad74b interfaceutil: module to stub out zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37635 diff changeset	5 from mercurial import encoding
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	6
37810 856f381ad74b interfaceutil: module to stub out zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37635 diff changeset	7 encoding.environ[b'HGREALINTERFACES'] = b'1'
856f381ad74b interfaceutil: module to stub out zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37635 diff changeset	8
37183 0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	9 import os
37814 265c122b69aa tests: mark test-check-interfaces.py as requiring a repo Martin von Zweigbergk <martinvonz@google.com> parents: 37810 diff changeset	10 import subprocess
265c122b69aa tests: mark test-check-interfaces.py as requiring a repo Martin von Zweigbergk <martinvonz@google.com> parents: 37810 diff changeset	11 import sys
265c122b69aa tests: mark test-check-interfaces.py as requiring a repo Martin von Zweigbergk <martinvonz@google.com> parents: 37810 diff changeset	12
265c122b69aa tests: mark test-check-interfaces.py as requiring a repo Martin von Zweigbergk <martinvonz@google.com> parents: 37810 diff changeset	13 # Only run if tests are run in a repo
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	14 if subprocess.call(
46176 300f6e23882a test-check-interface: do not expect bare "python" executable exists Yuya Nishihara <yuya@tcha.org> parents: 44666 diff changeset	15 [sys.executable, '%s/hghave' % os.environ['TESTDIR'], 'test-repo']
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	16 ):
37814 265c122b69aa tests: mark test-check-interfaces.py as requiring a repo Martin von Zweigbergk <martinvonz@google.com> parents: 37810 diff changeset	17 sys.exit(80)
37183 0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	18
42823 268662aac075 interfaces: create a new folder for interfaces and move repository.py in it Pulkit Goyal <pulkit@yandex-team.ru> parents: 41159 diff changeset	19 from mercurial.interfaces import (
42931 d459cd8ea42d interfaces: introduce an interface for dirstate implementations Augie Fackler <augie@google.com> parents: 42823 diff changeset	20 dirstate as intdirstate,
42823 268662aac075 interfaces: create a new folder for interfaces and move repository.py in it Pulkit Goyal <pulkit@yandex-team.ru> parents: 41159 diff changeset	21 repository,
268662aac075 interfaces: create a new folder for interfaces and move repository.py in it Pulkit Goyal <pulkit@yandex-team.ru> parents: 41159 diff changeset	22 )
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	23 from mercurial.thirdparty.zope import interface as zi
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	24 from mercurial.thirdparty.zope.interface import verify as ziverify
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	25 from mercurial import (
34314 afcbc6f64d27 tests: add interface checks for bundle, statichttp, and union peers Gregory Szorc <gregory.szorc@gmail.com> parents: 33828 diff changeset	26 bundlerepo,
42931 d459cd8ea42d interfaces: introduce an interface for dirstate implementations Augie Fackler <augie@google.com> parents: 42823 diff changeset	27 dirstate,
37441 a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	28 filelog,
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	29 httppeer,
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	30 localrepo,
38532 c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	31 manifest,
37914 39c17718f311 tests: make test-check-interfaces.py work on Python 3 Augie Fackler <augie@google.com> parents: 37814 diff changeset	32 pycompat,
39259 b41d023a412a repository: establish API for emitting revision deltas Gregory Szorc <gregory.szorc@gmail.com> parents: 39258 diff changeset	33 revlog,
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	34 sshpeer,
34314 afcbc6f64d27 tests: add interface checks for bundle, statichttp, and union peers Gregory Szorc <gregory.szorc@gmail.com> parents: 33828 diff changeset	35 statichttprepo,
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	36 ui as uimod,
34314 afcbc6f64d27 tests: add interface checks for bundle, statichttp, and union peers Gregory Szorc <gregory.szorc@gmail.com> parents: 33828 diff changeset	37 unionrepo,
37441 a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	38 vfs as vfsmod,
37296 78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	39 wireprotoserver,
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	40 wireprototypes,
37630 e1b32dc4646c wireproto: implement command executor interface for version 1 peers Gregory Szorc <gregory.szorc@gmail.com> parents: 37611 diff changeset	41 wireprotov1peer,
37545 93397c4633f6 wireproto: extract HTTP version 2 code to own module Gregory Szorc <gregory.szorc@gmail.com> parents: 37441 diff changeset	42 wireprotov2server,
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	43 )
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	44
41159 23d5e03dad70 tests: add simplestorerepo to test-check-interfaces.py Augie Fackler <augie@google.com> parents: 41041 diff changeset	45 testdir = os.path.dirname(__file__)
23d5e03dad70 tests: add simplestorerepo to test-check-interfaces.py Augie Fackler <augie@google.com> parents: 41041 diff changeset	46 rootdir = pycompat.fsencode(os.path.normpath(os.path.join(testdir, '..')))
23d5e03dad70 tests: add simplestorerepo to test-check-interfaces.py Augie Fackler <augie@google.com> parents: 41041 diff changeset	47
23d5e03dad70 tests: add simplestorerepo to test-check-interfaces.py Augie Fackler <augie@google.com> parents: 41041 diff changeset	48 sys.path[0:0] = [testdir]
23d5e03dad70 tests: add simplestorerepo to test-check-interfaces.py Augie Fackler <augie@google.com> parents: 41041 diff changeset	49 import simplestorerepo
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	50
41159 23d5e03dad70 tests: add simplestorerepo to test-check-interfaces.py Augie Fackler <augie@google.com> parents: 41041 diff changeset	51 del sys.path[0]
37183 0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	52
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	53
37441 a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	54 def checkzobject(o, allowextra=False):
37183 0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	55 """Verify an object with a zope interface."""
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	56 ifaces = zi.providedBy(o)
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	57 if not ifaces:
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	58 print('%r does not provide any zope interfaces' % o)
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	59 return
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	60
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	61 # Run zope.interface's built-in verification routine. This verifies that
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	62 # everything that is supposed to be present is present.
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	63 for iface in ifaces:
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	64 ziverify.verifyObject(iface, o)
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	65
37441 a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	66 if allowextra:
a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	67 return
a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	68
37183 0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	69 # Now verify that the object provides no extra public attributes that
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	70 # aren't declared as part of interfaces.
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	71 allowed = set()
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	72 for iface in ifaces:
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	73 allowed \|= set(iface.names(all=True))
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	74
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	75 public = {a for a in dir(o) if not a.startswith('_')}
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	76
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	77 for attr in sorted(public - allowed):
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	78 print(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	79 'public attribute not declared in interfaces: %s.%s'
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	80 % (o.__class__.__name__, attr)
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	81 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	82
37183 0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	83
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	84 # Facilitates testing localpeer.
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	85 class dummyrepo(object):
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	86 def __init__(self):
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	87 self.ui = uimod.ui()
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	88
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	89 def filtered(self, name):
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	90 pass
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	91
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	92 def _restrictcapabilities(self, caps):
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	93 pass
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	94
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	95
37009 8e89c2bec1f7 httppeer: refactor how httppeer is created (API) Gregory Szorc <gregory.szorc@gmail.com> parents: 36401 diff changeset	96 class dummyopener(object):
8e89c2bec1f7 httppeer: refactor how httppeer is created (API) Gregory Szorc <gregory.szorc@gmail.com> parents: 36401 diff changeset	97 handlers = []
8e89c2bec1f7 httppeer: refactor how httppeer is created (API) Gregory Szorc <gregory.szorc@gmail.com> parents: 36401 diff changeset	98
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	99
37552 8b8a845c85fc httppeer: perform capabilities request in makepeer() Gregory Szorc <gregory.szorc@gmail.com> parents: 37547 diff changeset	100 # Facilitates testing sshpeer without requiring a server.
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	101 class badpeer(httppeer.httppeer):
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	102 def __init__(self):
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	103 super(badpeer, self).__init__(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	104 None, None, None, dummyopener(), None, None
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	105 )
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	106 self.badattribute = True
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	107
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	108 def badmethod(self):
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	109 pass
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	110
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	111
36401 11ba1a96f946 sshpeer: defer pipe buffering and stderr sidechannel binding Gregory Szorc <gregory.szorc@gmail.com> parents: 36017 diff changeset	112 class dummypipe(object):
11ba1a96f946 sshpeer: defer pipe buffering and stderr sidechannel binding Gregory Szorc <gregory.szorc@gmail.com> parents: 36017 diff changeset	113 def close(self):
11ba1a96f946 sshpeer: defer pipe buffering and stderr sidechannel binding Gregory Szorc <gregory.szorc@gmail.com> parents: 36017 diff changeset	114 pass
11ba1a96f946 sshpeer: defer pipe buffering and stderr sidechannel binding Gregory Szorc <gregory.szorc@gmail.com> parents: 36017 diff changeset	115
46699 0738bc25d6ac sshpeer: add a method to check if a doublepipe is closed Valentin Gatien-Baron <valentin.gatienbaron@gmail.com> parents: 46176 diff changeset	116 @property
0738bc25d6ac sshpeer: add a method to check if a doublepipe is closed Valentin Gatien-Baron <valentin.gatienbaron@gmail.com> parents: 46176 diff changeset	117 def closed(self):
0738bc25d6ac sshpeer: add a method to check if a doublepipe is closed Valentin Gatien-Baron <valentin.gatienbaron@gmail.com> parents: 46176 diff changeset	118 pass
0738bc25d6ac sshpeer: add a method to check if a doublepipe is closed Valentin Gatien-Baron <valentin.gatienbaron@gmail.com> parents: 46176 diff changeset	119
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	120
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	121 def main():
b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	122 ui = uimod.ui()
37183 0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	123 # Needed so we can open a local repo with obsstore without a warning.
40153 0ae20d2141ed py3: use b'' in test-check-interfaces.py Gregory Szorc <gregory.szorc@gmail.com> parents: 39871 diff changeset	124 ui.setconfig(b'experimental', b'evolution.createmarkers', True)
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	125
37320 39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	126 checkzobject(badpeer())
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	127
37635 cc8c06835097 wireproto: convert legacy commands to command executor Gregory Szorc <gregory.szorc@gmail.com> parents: 37630 diff changeset	128 ziverify.verifyClass(repository.ipeerbase, httppeer.httppeer)
37552 8b8a845c85fc httppeer: perform capabilities request in makepeer() Gregory Szorc <gregory.szorc@gmail.com> parents: 37547 diff changeset	129 checkzobject(httppeer.httppeer(None, None, None, dummyopener(), None, None))
37320 39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	130
40171 dac438b7346e httppeer: expose API descriptor on httpv2peer Gregory Szorc <gregory.szorc@gmail.com> parents: 40153 diff changeset	131 ziverify.verifyClass(repository.ipeerv2, httppeer.httpv2peer)
37914 39c17718f311 tests: make test-check-interfaces.py work on Python 3 Augie Fackler <augie@google.com> parents: 37814 diff changeset	132 checkzobject(httppeer.httpv2peer(None, b'', b'', None, None, None))
37609 01bfe5ad0c53 httppeer: implement ipeerconnection Gregory Szorc <gregory.szorc@gmail.com> parents: 37552 diff changeset	133
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	134 ziverify.verifyClass(repository.ipeerbase, localrepo.localpeer)
37320 39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	135 checkzobject(localrepo.localpeer(dummyrepo()))
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	136
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	137 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	138 repository.ipeercommandexecutor, localrepo.localcommandexecutor
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	139 )
37630 e1b32dc4646c wireproto: implement command executor interface for version 1 peers Gregory Szorc <gregory.szorc@gmail.com> parents: 37611 diff changeset	140 checkzobject(localrepo.localcommandexecutor(None))
e1b32dc4646c wireproto: implement command executor interface for version 1 peers Gregory Szorc <gregory.szorc@gmail.com> parents: 37611 diff changeset	141
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	142 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	143 repository.ipeercommandexecutor, wireprotov1peer.peerexecutor
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	144 )
37630 e1b32dc4646c wireproto: implement command executor interface for version 1 peers Gregory Szorc <gregory.szorc@gmail.com> parents: 37611 diff changeset	145 checkzobject(wireprotov1peer.peerexecutor(None))
e1b32dc4646c wireproto: implement command executor interface for version 1 peers Gregory Szorc <gregory.szorc@gmail.com> parents: 37611 diff changeset	146
37635 cc8c06835097 wireproto: convert legacy commands to command executor Gregory Szorc <gregory.szorc@gmail.com> parents: 37630 diff changeset	147 ziverify.verifyClass(repository.ipeerbase, sshpeer.sshv1peer)
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	148 checkzobject(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	149 sshpeer.sshv1peer(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	150 ui,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	151 b'ssh://localhost/foo',
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	152 b'',
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	153 dummypipe(),
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	154 dummypipe(),
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	155 None,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	156 None,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	157 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	158 )
37320 39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	159
37635 cc8c06835097 wireproto: convert legacy commands to command executor Gregory Szorc <gregory.szorc@gmail.com> parents: 37630 diff changeset	160 ziverify.verifyClass(repository.ipeerbase, sshpeer.sshv2peer)
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	161 checkzobject(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	162 sshpeer.sshv2peer(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	163 ui,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	164 b'ssh://localhost/foo',
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	165 b'',
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	166 dummypipe(),
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	167 dummypipe(),
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	168 None,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	169 None,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	170 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	171 )
37320 39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	172
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	173 ziverify.verifyClass(repository.ipeerbase, bundlerepo.bundlepeer)
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	174 checkzobject(bundlerepo.bundlepeer(dummyrepo()))
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	175
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	176 ziverify.verifyClass(repository.ipeerbase, statichttprepo.statichttppeer)
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	177 checkzobject(statichttprepo.statichttppeer(dummyrepo()))
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	178
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	179 ziverify.verifyClass(repository.ipeerbase, unionrepo.unionpeer)
39f7d4ee8bcd repository: port peer interfaces to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37296 diff changeset	180 checkzobject(unionrepo.unionpeer(dummyrepo()))
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	181
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	182 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	183 repository.ilocalrepositorymain, localrepo.localrepository
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	184 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	185 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	186 repository.ilocalrepositoryfilestorage, localrepo.revlogfilestorage
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	187 )
39703 bfeab472e3c0 localrepo: create new function for instantiating a local repo object Gregory Szorc <gregory.szorc@gmail.com> parents: 39341 diff changeset	188 repo = localrepo.makelocalrepository(ui, rootdir)
37183 0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	189 checkzobject(repo)
0dfb5672f015 repository: define interface for local repositories Gregory Szorc <gregory.szorc@gmail.com> parents: 37009 diff changeset	190
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	191 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	192 wireprototypes.baseprotocolhandler, wireprotoserver.sshv1protocolhandler
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	193 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	194 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	195 wireprototypes.baseprotocolhandler, wireprotoserver.sshv2protocolhandler
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	196 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	197 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	198 wireprototypes.baseprotocolhandler,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	199 wireprotoserver.httpv1protocolhandler,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	200 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	201 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	202 wireprototypes.baseprotocolhandler,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	203 wireprotov2server.httpv2protocolhandler,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	204 )
37296 78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	205
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	206 sshv1 = wireprotoserver.sshv1protocolhandler(None, None, None)
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	207 checkzobject(sshv1)
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	208 sshv2 = wireprotoserver.sshv2protocolhandler(None, None, None)
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	209 checkzobject(sshv2)
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	210
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	211 httpv1 = wireprotoserver.httpv1protocolhandler(None, None, None)
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	212 checkzobject(httpv1)
37545 93397c4633f6 wireproto: extract HTTP version 2 code to own module Gregory Szorc <gregory.szorc@gmail.com> parents: 37441 diff changeset	213 httpv2 = wireprotov2server.httpv2protocolhandler(None, None)
37296 78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	214 checkzobject(httpv2)
78103e4138b1 wireproto: port protocol handler to zope.interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37183 diff changeset	215
37441 a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	216 ziverify.verifyClass(repository.ifilestorage, filelog.filelog)
38532 c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	217 ziverify.verifyClass(repository.imanifestdict, manifest.manifestdict)
44666 8c66a680f396 manifest: also declare treemanifest as implementing imanifestdict Augie Fackler <augie@google.com> parents: 44309 diff changeset	218 ziverify.verifyClass(repository.imanifestdict, manifest.treemanifest)
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	219 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	220 repository.imanifestrevisionstored, manifest.manifestctx
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	221 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	222 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	223 repository.imanifestrevisionwritable, manifest.memmanifestctx
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	224 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	225 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	226 repository.imanifestrevisionstored, manifest.treemanifestctx
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	227 )
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	228 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	229 repository.imanifestrevisionwritable, manifest.memtreemanifestctx
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	230 )
38532 c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	231 ziverify.verifyClass(repository.imanifestlog, manifest.manifestlog)
39341 7f5e6d3e9032 manifest: proxy to revlog instance instead of inheriting Gregory Szorc <gregory.szorc@gmail.com> parents: 39259 diff changeset	232 ziverify.verifyClass(repository.imanifeststorage, manifest.manifestrevlog)
37441 a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	233
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	234 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	235 repository.irevisiondelta, simplestorerepo.simplestorerevisiondelta
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	236 )
41159 23d5e03dad70 tests: add simplestorerepo to test-check-interfaces.py Augie Fackler <augie@google.com> parents: 41041 diff changeset	237 ziverify.verifyClass(repository.ifilestorage, simplestorerepo.filestorage)
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	238 ziverify.verifyClass(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	239 repository.iverifyproblem, simplestorerepo.simplefilestoreproblem
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	240 )
41159 23d5e03dad70 tests: add simplestorerepo to test-check-interfaces.py Augie Fackler <augie@google.com> parents: 41041 diff changeset	241
42931 d459cd8ea42d interfaces: introduce an interface for dirstate implementations Augie Fackler <augie@google.com> parents: 42823 diff changeset	242 ziverify.verifyClass(intdirstate.idirstate, dirstate.dirstate)
d459cd8ea42d interfaces: introduce an interface for dirstate implementations Augie Fackler <augie@google.com> parents: 42823 diff changeset	243
37914 39c17718f311 tests: make test-check-interfaces.py work on Python 3 Augie Fackler <augie@google.com> parents: 37814 diff changeset	244 vfs = vfsmod.vfs(b'.')
39c17718f311 tests: make test-check-interfaces.py work on Python 3 Augie Fackler <augie@google.com> parents: 37814 diff changeset	245 fl = filelog.filelog(vfs, b'dummy.i')
37441 a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	246 checkzobject(fl, allowextra=True)
a3202fa83aff filelog: declare that filelog implements a storage interface Gregory Szorc <gregory.szorc@gmail.com> parents: 37320 diff changeset	247
38532 c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	248 # Conforms to imanifestlog.
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	249 ml = manifest.manifestlog(
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	250 vfs, repo, manifest.manifestrevlog(repo.svfs), repo.narrowmatch()
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	251 )
38532 c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	252 checkzobject(ml)
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	253 checkzobject(repo.manifestlog)
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	254
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	255 # Conforms to imanifestrevision.
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	256 mctx = ml[repo[0].manifestnode()]
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	257 checkzobject(mctx)
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	258
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	259 # Conforms to imanifestrevisionwritable.
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	260 checkzobject(mctx.copy())
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	261
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	262 # Conforms to imanifestdict.
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	263 checkzobject(mctx.read())
c82ea938efbb repository: define manifest interfaces Gregory Szorc <gregory.szorc@gmail.com> parents: 37914 diff changeset	264
39341 7f5e6d3e9032 manifest: proxy to revlog instance instead of inheriting Gregory Szorc <gregory.szorc@gmail.com> parents: 39259 diff changeset	265 mrl = manifest.manifestrevlog(vfs)
7f5e6d3e9032 manifest: proxy to revlog instance instead of inheriting Gregory Szorc <gregory.szorc@gmail.com> parents: 39259 diff changeset	266 checkzobject(mrl)
7f5e6d3e9032 manifest: proxy to revlog instance instead of inheriting Gregory Szorc <gregory.szorc@gmail.com> parents: 39259 diff changeset	267
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	268 ziverify.verifyClass(repository.irevisiondelta, revlog.revlogrevisiondelta)
39258 b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	269
39259 b41d023a412a repository: establish API for emitting revision deltas Gregory Szorc <gregory.szorc@gmail.com> parents: 39258 diff changeset	270 rd = revlog.revlogrevisiondelta(
39258 b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	271 node=b'',
b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	272 p1node=b'',
b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	273 p2node=b'',
b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	274 basenode=b'',
b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	275 linknode=b'',
b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	276 flags=b'',
b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	277 baserevisionsize=None,
b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	278 revision=b'',
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	279 delta=None,
2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	280 )
39258 b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	281 checkzobject(rd)
b518d495a560 repository: formalize interfaces for revision deltas and requests Gregory Szorc <gregory.szorc@gmail.com> parents: 38532 diff changeset	282
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	283 ziverify.verifyClass(repository.iverifyproblem, revlog.revlogproblem)
39847 97986c9c69d3 verify: start to abstract file verification Gregory Szorc <gregory.szorc@gmail.com> parents: 39780 diff changeset	284 checkzobject(revlog.revlogproblem())
97986c9c69d3 verify: start to abstract file verification Gregory Szorc <gregory.szorc@gmail.com> parents: 39780 diff changeset	285
43076 2372284d9457 formatting: blacken the codebase Augie Fackler <augie@google.com> parents: 42931 diff changeset	286
33828 b70029f355a3 tests: verify that peer instances only expose interface members Gregory Szorc <gregory.szorc@gmail.com> parents: diff changeset	287 main()

Mercurial > public > mercurial-scm > hg-stable

annotate tests/test-check-interfaces.py @ 46699:0738bc25d6ac