Mercurial > hg
diff tests/test-check-interfaces.py @ 43076:2372284d9457
formatting: blacken the codebase
This is using my patch to black
(https://github.com/psf/black/pull/826) so we don't un-wrap collection
literals.
Done with:
hg files 'set:**.py - mercurial/thirdparty/** - "contrib/python-zstandard/**"' | xargs black -S
# skip-blame mass-reformatting only
# no-check-commit reformats foo_bar functions
Differential Revision: https://phab.mercurial-scm.org/D6971
author | Augie Fackler <augie@google.com> |
---|---|
date | Sun, 06 Oct 2019 09:45:02 -0400 |
parents | d459cd8ea42d |
children | c86256bd4eb8 |
line wrap: on
line diff
--- a/tests/test-check-interfaces.py Sat Oct 05 10:29:34 2019 -0400 +++ b/tests/test-check-interfaces.py Sun Oct 06 09:45:02 2019 -0400 @@ -3,6 +3,7 @@ from __future__ import absolute_import, print_function from mercurial import encoding + encoding.environ[b'HGREALINTERFACES'] = b'1' import os @@ -10,20 +11,17 @@ import sys # Only run if tests are run in a repo -if subprocess.call(['python', '%s/hghave' % os.environ['TESTDIR'], - 'test-repo']): +if subprocess.call( + ['python', '%s/hghave' % os.environ['TESTDIR'], 'test-repo'] +): sys.exit(80) from mercurial.interfaces import ( dirstate as intdirstate, repository, ) -from mercurial.thirdparty.zope import ( - interface as zi, -) -from mercurial.thirdparty.zope.interface import ( - verify as ziverify, -) +from mercurial.thirdparty.zope import interface as zi +from mercurial.thirdparty.zope.interface import verify as ziverify from mercurial import ( bundlerepo, dirstate, @@ -49,8 +47,10 @@ sys.path[0:0] = [testdir] import simplestorerepo + del sys.path[0] + def checkzobject(o, allowextra=False): """Verify an object with a zope interface.""" ifaces = zi.providedBy(o) @@ -75,35 +75,45 @@ public = {a for a in dir(o) if not a.startswith('_')} for attr in sorted(public - allowed): - print('public attribute not declared in interfaces: %s.%s' % ( - o.__class__.__name__, attr)) + print( + 'public attribute not declared in interfaces: %s.%s' + % (o.__class__.__name__, attr) + ) + # Facilitates testing localpeer. class dummyrepo(object): def __init__(self): self.ui = uimod.ui() + def filtered(self, name): pass + def _restrictcapabilities(self, caps): pass + class dummyopener(object): handlers = [] + # Facilitates testing sshpeer without requiring a server. class badpeer(httppeer.httppeer): def __init__(self): - super(badpeer, self).__init__(None, None, None, dummyopener(), None, - None) + super(badpeer, self).__init__( + None, None, None, dummyopener(), None, None + ) self.badattribute = True def badmethod(self): pass + class dummypipe(object): def close(self): pass + def main(): ui = uimod.ui() # Needed so we can open a local repo with obsstore without a warning. @@ -117,25 +127,44 @@ ziverify.verifyClass(repository.ipeerv2, httppeer.httpv2peer) checkzobject(httppeer.httpv2peer(None, b'', b'', None, None, None)) - ziverify.verifyClass(repository.ipeerbase, - localrepo.localpeer) + ziverify.verifyClass(repository.ipeerbase, localrepo.localpeer) checkzobject(localrepo.localpeer(dummyrepo())) - ziverify.verifyClass(repository.ipeercommandexecutor, - localrepo.localcommandexecutor) + ziverify.verifyClass( + repository.ipeercommandexecutor, localrepo.localcommandexecutor + ) checkzobject(localrepo.localcommandexecutor(None)) - ziverify.verifyClass(repository.ipeercommandexecutor, - wireprotov1peer.peerexecutor) + ziverify.verifyClass( + repository.ipeercommandexecutor, wireprotov1peer.peerexecutor + ) checkzobject(wireprotov1peer.peerexecutor(None)) ziverify.verifyClass(repository.ipeerbase, sshpeer.sshv1peer) - checkzobject(sshpeer.sshv1peer(ui, b'ssh://localhost/foo', b'', dummypipe(), - dummypipe(), None, None)) + checkzobject( + sshpeer.sshv1peer( + ui, + b'ssh://localhost/foo', + b'', + dummypipe(), + dummypipe(), + None, + None, + ) + ) ziverify.verifyClass(repository.ipeerbase, sshpeer.sshv2peer) - checkzobject(sshpeer.sshv2peer(ui, b'ssh://localhost/foo', b'', dummypipe(), - dummypipe(), None, None)) + checkzobject( + sshpeer.sshv2peer( + ui, + b'ssh://localhost/foo', + b'', + dummypipe(), + dummypipe(), + None, + None, + ) + ) ziverify.verifyClass(repository.ipeerbase, bundlerepo.bundlepeer) checkzobject(bundlerepo.bundlepeer(dummyrepo())) @@ -146,21 +175,29 @@ ziverify.verifyClass(repository.ipeerbase, unionrepo.unionpeer) checkzobject(unionrepo.unionpeer(dummyrepo())) - ziverify.verifyClass(repository.ilocalrepositorymain, - localrepo.localrepository) - ziverify.verifyClass(repository.ilocalrepositoryfilestorage, - localrepo.revlogfilestorage) + ziverify.verifyClass( + repository.ilocalrepositorymain, localrepo.localrepository + ) + ziverify.verifyClass( + repository.ilocalrepositoryfilestorage, localrepo.revlogfilestorage + ) repo = localrepo.makelocalrepository(ui, rootdir) checkzobject(repo) - ziverify.verifyClass(wireprototypes.baseprotocolhandler, - wireprotoserver.sshv1protocolhandler) - ziverify.verifyClass(wireprototypes.baseprotocolhandler, - wireprotoserver.sshv2protocolhandler) - ziverify.verifyClass(wireprototypes.baseprotocolhandler, - wireprotoserver.httpv1protocolhandler) - ziverify.verifyClass(wireprototypes.baseprotocolhandler, - wireprotov2server.httpv2protocolhandler) + ziverify.verifyClass( + wireprototypes.baseprotocolhandler, wireprotoserver.sshv1protocolhandler + ) + ziverify.verifyClass( + wireprototypes.baseprotocolhandler, wireprotoserver.sshv2protocolhandler + ) + ziverify.verifyClass( + wireprototypes.baseprotocolhandler, + wireprotoserver.httpv1protocolhandler, + ) + ziverify.verifyClass( + wireprototypes.baseprotocolhandler, + wireprotov2server.httpv2protocolhandler, + ) sshv1 = wireprotoserver.sshv1protocolhandler(None, None, None) checkzobject(sshv1) @@ -174,22 +211,28 @@ ziverify.verifyClass(repository.ifilestorage, filelog.filelog) ziverify.verifyClass(repository.imanifestdict, manifest.manifestdict) - ziverify.verifyClass(repository.imanifestrevisionstored, - manifest.manifestctx) - ziverify.verifyClass(repository.imanifestrevisionwritable, - manifest.memmanifestctx) - ziverify.verifyClass(repository.imanifestrevisionstored, - manifest.treemanifestctx) - ziverify.verifyClass(repository.imanifestrevisionwritable, - manifest.memtreemanifestctx) + ziverify.verifyClass( + repository.imanifestrevisionstored, manifest.manifestctx + ) + ziverify.verifyClass( + repository.imanifestrevisionwritable, manifest.memmanifestctx + ) + ziverify.verifyClass( + repository.imanifestrevisionstored, manifest.treemanifestctx + ) + ziverify.verifyClass( + repository.imanifestrevisionwritable, manifest.memtreemanifestctx + ) ziverify.verifyClass(repository.imanifestlog, manifest.manifestlog) ziverify.verifyClass(repository.imanifeststorage, manifest.manifestrevlog) - ziverify.verifyClass(repository.irevisiondelta, - simplestorerepo.simplestorerevisiondelta) + ziverify.verifyClass( + repository.irevisiondelta, simplestorerepo.simplestorerevisiondelta + ) ziverify.verifyClass(repository.ifilestorage, simplestorerepo.filestorage) - ziverify.verifyClass(repository.iverifyproblem, - simplestorerepo.simplefilestoreproblem) + ziverify.verifyClass( + repository.iverifyproblem, simplestorerepo.simplefilestoreproblem + ) ziverify.verifyClass(intdirstate.idirstate, dirstate.dirstate) @@ -198,8 +241,9 @@ checkzobject(fl, allowextra=True) # Conforms to imanifestlog. - ml = manifest.manifestlog(vfs, repo, manifest.manifestrevlog(repo.svfs), - repo.narrowmatch()) + ml = manifest.manifestlog( + vfs, repo, manifest.manifestrevlog(repo.svfs), repo.narrowmatch() + ) checkzobject(ml) checkzobject(repo.manifestlog) @@ -217,8 +261,7 @@ mrl = manifest.manifestrevlog(vfs) checkzobject(mrl) - ziverify.verifyClass(repository.irevisiondelta, - revlog.revlogrevisiondelta) + ziverify.verifyClass(repository.irevisiondelta, revlog.revlogrevisiondelta) rd = revlog.revlogrevisiondelta( node=b'', @@ -229,11 +272,12 @@ flags=b'', baserevisionsize=None, revision=b'', - delta=None) + delta=None, + ) checkzobject(rd) - ziverify.verifyClass(repository.iverifyproblem, - revlog.revlogproblem) + ziverify.verifyClass(repository.iverifyproblem, revlog.revlogproblem) checkzobject(revlog.revlogproblem()) + main()