Use ConsulException in keep_alive method.

* there was a change and now they python-consul raises exception instead
of `False`

src/ianitor/service.py

@@ -22,8 +22,7 @@
 import logging
 from requests import ConnectionError
 
-from consul import Check
-
+from consul import Check, ConsulException
 
 logger = logging.getLogger(__name__)
 
@@ -122,7 +121,9 @@ def keep_alive(self):
         :return: None
         """
         with ignore_connection_errors("ttl_pass"):
-            if not self.session.agent.check.ttl_pass(self.check_id):
+            try:
+                self.session.agent.check.ttl_pass(self.check_id)
+            except ConsulException:
                 # register and ttl_pass again if it failed
                 logger.warning("service keep-alive failed, re-registering")
                 self.register()

tests/test_service.py

@@ -76,7 +76,7 @@ def test_remove_services():
         if description["ID"] != 'consul':
             agent.service.deregister(description["ID"])
 
-    # this is consul 0.4.1 behavior - consul is one of services
+    # this is consul behavior - consul is one of services
     services = agent.services()
     if 'consul' in services:
         services.pop('consul')
@@ -142,7 +142,6 @@ def _get_service_status(session, service_obj):
         # return none because check does not even exist
         return
 
-    # from pprint import pprint; pprint(checks)
     service_check = list(filter(
         lambda check: check["ServiceName"] == service_obj.service_name, checks
     )).pop()