diff options
author | Mike Frysinger <vapier@google.com> | 2020-06-11 20:05:58 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2020-06-11 20:05:58 +0000 |
commit | debcb62ff14f13798982f3da206ed858f6d20d46 (patch) | |
tree | c90cef2dfc824f0b7d439bdc40ca90c332928c63 | |
parent | ede9dda1e7aae5d093e83772637eaab9a354814d (diff) | |
parent | 4160208fcb733cdb3853b5dd76500f075dd24aad (diff) | |
download | repohooks-debcb62ff14f13798982f3da206ed858f6d20d46.tar.gz |
hooks: make unittest match real code am: 4160208fcb
Original change: https://android-review.googlesource.com/c/platform/tools/repohooks/+/1253218
Change-Id: I58d386046e03a21b3529e6f992be7cd35e0da9c2
-rwxr-xr-x | rh/hooks_unittest.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/rh/hooks_unittest.py b/rh/hooks_unittest.py index 4124f10..3fbe56c 100755 --- a/rh/hooks_unittest.py +++ b/rh/hooks_unittest.py @@ -262,11 +262,11 @@ class BuiltinHooksTests(unittest.TestCase): for desc in msgs: ret = func(self.project, 'commit', desc, diff, options=self.options) if accept: - self.assertEqual( - ret, None, msg='Should have accepted: {{{%s}}}' % (desc,)) + self.assertFalse( + bool(ret), msg='Should have accepted: {{{%s}}}' % (desc,)) else: - self.assertNotEqual( - ret, None, msg='Should have rejected: {{{%s}}}' % (desc,)) + self.assertTrue( + bool(ret), msg='Should have rejected: {{{%s}}}' % (desc,)) def _test_file_filter(self, mock_check, func, files): """Helper for testing hooks that filter by files and run external tools. |