PIM testing: ensure that no ERROR messages are printed

This covers D-Bus log messages and stdout. Adding the check revealed
several issues which were fixed in the previous commits.

src/dbus/server/pim/testpim.py

@@ -474,6 +474,16 @@ END:VCARD(\r|\n)*''',
                               'addressbook'])
 
     def run(self, result):
+        # No errors must be logged. During testRead, libphonenumber used to print
+        #   [ERROR] Number too short to be viable: 8
+        #   [ERROR] The string supplied did not seem to be a phone number.
+        # to stdout until we reduced the log level.
+        #
+        # We check both D-Bus messages (which did not contain that
+        # text, but some other error messages) and the servers stdout.
+        self.runTestDBusCheck = lambda test, log: test.assertNotIn('ERROR', log)
+        self.runTestOutputCheck = self.runTestDBusCheck
+
         # Runtime varies a lot when using valgrind, because
         # of the need to check an additional process. Allow
         # a lot more time when running under valgrind.
@@ -862,6 +872,11 @@ END:VCARD
         expected = sources.copy()
         peers = {}
 
+        # Disable the default checking because
+        # we trigger one ERROR message.
+        self.runTestDBusCheck = None
+        self.runTestOutputCheck = None
+
         # dummy peer directory
         contacts = os.path.abspath(os.path.join(xdg_root, 'contacts'))
         os.makedirs(contacts)