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')