On Wed, Jan 23, 2019 at 23:23:53 -0500, Georges Racinet wrote: ... > diff -r a35cfd592a90 -r 47d5458a4c32 tests/test-rust-ancestor.py > --- a/tests/test-rust-ancestor.py Wed Jan 23 07:47:04 2019 -0500 > +++ b/tests/test-rust-ancestor.py Wed Jan 23 07:49:36 2019 -0500 > @@ -1,6 +1,10 @@ > from __future__ import absolute_import > import sys > import unittest > +from mercurial import ( > + error, > + node, > + )
Is there supposed to be whitespace before the ) ? I'm not all that familiar with the python coding style. I know, such important feedback... :) Jeff. > > try: > from mercurial import rustext > @@ -153,6 +157,12 @@ > # rust-cpython issues appropriate str instances for Python 2 and 3 > self.assertEqual(exc.args, ('ParentOutOfRange', 1)) > > + def testwdirunsupported(self): > + # trying to access ancestors of the working directory raises > + # WdirUnsupported directly > + idx = self.parseindex() > + with self.assertRaises(error.WdirUnsupported): > + list(AncestorsIterator(idx, [node.wdirrev], -1, False)) > > if __name__ == '__main__': > import silenttestrunner > _______________________________________________ > Mercurial-devel mailing list > Mercurial-devel@mercurial-scm.org > https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel -- All parts should go together without forcing. You must remember that the parts you are reassembling were disassembled by you. Therefore, if you can’t get them together again, there must be a reason. By all means, do not use a hammer. — IBM Manual, 1925 _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel