Fix broken Travis unit tests

alignak/daemon.py

@@ -821,6 +821,7 @@ def do_daemon_init_and_start(self):
         logger.info("Created")
 
         logger.info("Starting http_daemon thread..")
+        # pylint: disable=bad-thread-instantiation
         self.http_thread = threading.Thread(None, self.http_daemon_thread, 'http_thread')
         self.http_thread.daemon = True
         self.http_thread.start()

alignak/daemons/arbiterdaemon.py

@@ -549,9 +549,9 @@ def manage_missing_daemons(self):
                 if ret is not None:
                     logger.error("*** %s exited on start!", daemon_name)
                     for line in iter(self.my_satellites[daemon_name].stdout.readline, b''):
-                        logger.error(">>> " + line.rstrip())
+                        logger.error(">>> %s", line.rstrip())
                     for line in iter(self.my_satellites[daemon_name].stderr.readline, b''):
-                        logger.error(">>> " + line.rstrip())
+                        logger.error(">>> %s", line.rstrip())
                     result = False
                 else:
                     logger.info("%s running (pid=%d)",

alignak/macroresolver.py

@@ -442,6 +442,7 @@ def _get_type_of_macro(macros, objs):
                     continue
 
     @staticmethod
+    # pylint: disable=inconsistent-return-statements
     def _resolve_argn(macro, args):
         """Get argument from macro name
         ie : $ARG3$ -> args[2]

alignak/objects/satellitelink.py

@@ -306,6 +306,7 @@ def add_failed_check_attempt(self, reason=''):
         if self.attempt == self.max_check_attempts:
             self.set_dead()
 
+    # pylint: disable=inconsistent-return-statements
     def update_infos(self, now):
         """Update satellite info each self.check_interval seconds
         so we smooth arbiter actions for just useful actions.
@@ -348,6 +349,7 @@ def known_conf_managed_push(self, cfg_id, push_flavor):
         """
         self.managed_confs[cfg_id] = push_flavor
 
+    # pylint: disable=inconsistent-return-statements
     def ping(self):
         """Send a HTTP request to the satellite (GET /ping)
         Add failed attempt if an error occurs
@@ -463,6 +465,7 @@ def have_conf(self, magic_hash=None):
 
         return False
 
+    # pylint: disable=inconsistent-return-statements
     def remove_from_conf(self, sched_id):  # pragma: no cover, no more used
         """Send a HTTP request to the satellite (GET /remove_from_conf)
         Tell a satellite to remove a scheduler from conf
@@ -502,6 +505,7 @@ def remove_from_conf(self, sched_id):  # pragma: no cover, no more used
 
         return False
 
+    # pylint: disable=inconsistent-return-statements
     def update_managed_conf(self):
         """Send a HTTP request to the satellite (GET /what_i_managed)
         and update managed_conf attribute with dict (cleaned)

alignak/objects/service.py

@@ -1597,6 +1597,7 @@ def explode_services_from_hosts(self, hosts, service, hnames):
             new_s.host_name = hname
             self.add_item(new_s)
 
+    # pylint: disable=inconsistent-return-statements
     def _local_create_service(self, hosts, host_name, service):
         """Create a new service based on a host_name and service instance.
 

alignak/property.py

@@ -374,6 +374,7 @@ class DictProp(Property):
     """Dict property
 
     """
+    # pylint: disable=keyword-arg-before-vararg
     def __init__(self, elts_prop=None, *args, **kwargs):
         """Dictionary of values.
              If elts_prop is not None, must be a Property subclass

test/test_modules.py

@@ -376,7 +376,7 @@ def test_modulemanager_several_modules(self):
         for mod in self.arbiter.conf.modules:
             print (mod.__dict__)
 
-        time_hacker.set_real_time()
+        # time_hacker.set_real_time()
 
         # Create an Alignak module
         mod = Module({