Merge #15943: tests: Fail if RPC has been added without tests

fad0ce59e9 tests: Fail if RPC has been added without tests (MarcoFalke)

Need to be run with --coverage

Backport of Core [PR15943](bitcoin/bitcoin#15943)

Test Plan:
```
test_runner.py --coverage --extended
```

Verify `All RPC commands covered.` is printed to the console.

Differential Revision: https://reviews.bitcoinabc.org/D5743

test/functional/rpc_misc.py

@@ -49,6 +49,13 @@ def run_test(self):
         assert_raises_rpc_error(-8, "unknown mode foobar",
                                 node.getmemoryinfo, mode="foobar")
 
+        self.log.info("test logging")
+        assert_equal(node.logging()['qt'], True)
+        node.logging(exclude=['qt'])
+        assert_equal(node.logging()['qt'], False)
+        node.logging(include=['qt'])
+        assert_equal(node.logging()['qt'], True)
+
 
 if __name__ == '__main__':
     RpcMiscTest().main()

test/functional/test_runner.py

@@ -389,17 +389,19 @@ def run_tests(test_list, build_dir, tests_dir, junitoutput, tmpdir, num_jobs, te
         build_timings.save_timings(test_results)
 
     if coverage:
-        coverage.report_rpc_coverage()
+        coverage_passed = coverage.report_rpc_coverage()
 
         logging.debug("Cleaning up coverage data")
         coverage.cleanup()
+    else:
+        coverage_passed = True
 
     # Clear up the temp directory if all subdirectories are gone
     if not os.listdir(tmpdir):
         os.rmdir(tmpdir)
 
-    all_passed = all(
-        map(lambda test_result: test_result.was_successful, test_results))
+    all_passed = all(map(
+        lambda test_result: test_result.was_successful, test_results)) and coverage_passed
 
     sys.exit(not all_passed)
 
@@ -692,8 +694,10 @@ def report_rpc_coverage(self):
         if uncovered:
             print("Uncovered RPC commands:")
             print("".join(("  - {}\n".format(i)) for i in sorted(uncovered)))
+            return False
         else:
             print("All RPC commands covered.")
+            return True
 
     def cleanup(self):
         return shutil.rmtree(self.dir)