Use instance checks for True and False objects

Checking True or False by value makes no sense, they are not values.
Instead we should check if we have an "instance" of the True or False
objects.

Author: rwalton-arm

src/htrun/host_tests_conn_proxy/conn_proxy.py

@@ -269,7 +269,7 @@ def __send_sync(timeout=None):
             pass  # Check if target sent something
         else:
             # Return if state machine in host_test_default has finished to end process
-            if key == "__host_test_finished" and value == True:
+            if key == "__host_test_finished" and value is True:
                 logger.prn_inf(
                     "received special event '%s' value='%s', finishing" % (key, value)
                 )
@@ -351,7 +351,7 @@ def __send_sync(timeout=None):
                     else:
                         __notify_conn_lost()
                         break
-            elif last_sync == True:
+            elif last_sync is True:
                 # SYNC lost connection event : Device not responding, send sync failed
                 __notify_sync_failed()
                 break

src/htrun/host_tests_runner/host_test_default.py

@@ -612,9 +612,9 @@ def execute(self):
             # Execute test if flashing was successful or skipped
             test_result = self.run_test()
 
-            if test_result == True:
+            if test_result is True:
                 result = self.RESULT_SUCCESS
-            elif test_result == False:
+            elif test_result is False:
                 result = self.RESULT_FAILURE
             elif test_result is None:
                 result = self.RESULT_ERROR

src/htrun/host_tests_runner/target_base.py

@@ -172,10 +172,10 @@ def check_flash_error(target_id, disk, initial_remount_count):
                         "mount_point" in mbed_target
                         and mbed_target["mount_point"] is not None
                     ):
-                        if not initial_remount_count is None:
+                        if initial_remount_count is not None:
                             new_remount_count = get_remount_count(disk)
                             if (
-                                not new_remount_count is None
+                                new_remount_count is not None
                                 and new_remount_count == initial_remount_count
                             ):
                                 sleep(0.5)