Mercurial > hgsubversion
diff tests/test_util.py @ 1541:adba152a5177
test_util: make developer warnings cause our tests to fail
Previously hgsubversion didn't get any help from develwarn calls,
because we don't use core's testrunner. The upstream configitem
refactor was easier to hunt down using the develwarn being a failure,
which prompted this code. It also exposed several locking correctness
bugs, so it feels like it's worth keeping around long term.
We don't bother with develwarn on anything so old that we don't try to
use smartset. The smartset deprecation was not handled gracefully, so
hg complains for many versions without actually breaking anything, and
it'd pointlessly complicate our code to handle smartset living in
multiple places between when the warning was introduced and the
present.
| author | Augie Fackler <raf@durin42.com> |
|---|---|
| date | Sun, 29 Oct 2017 09:07:12 -0400 |
| parents | b3e41b0d50a2 |
| children | 8410a978c650 |
line wrap: on
line diff
--- a/tests/test_util.py +++ b/tests/test_util.py @@ -289,8 +289,17 @@ def fileurl(path): url = 'file://%s%s' % (drive, path) return url +class _testui(ui.ui): + def develwarn(self, msg, stacklevel=1, *args, **kwargs): + from hgsubversion import util + if util.smartset is not None: + config = args[0] if args else kwargs['config'] + raise Exception('flunked develwarn: %r (%r)' % (msg, config)) + return ui.ui.develwarn(self, msg, stacklevel=stacklevel, + *args, **kwargs) + def testui(stupid=False, layout='auto', startrev=0): - u = ui.ui() + u = _testui() bools = {True: 'true', False: 'false'} u.setconfig('ui', 'quiet', bools[True]) u.setconfig('ui', 'username', 'automated tests')