fix code reivewbot bug

Signed-off-by: Ndeveco_test <liguangjie1@huawei.com>

plugins/ohos/src/ohos/drivers/drivers.py

@@ -1768,9 +1768,8 @@ class DexTestDriver(IDriver):
             if not getattr("exception", "error_no", ""):
                 setattr(exception, "error_no", "03203")
             return_message = str(exception.args)
-        finally:
-            shutil.rmtree(long_command_path)
-            return return_message
+        shutil.rmtree(long_command_path)
+        return return_message
 
     @staticmethod
     def _make_long_command_file(command, longcommand_path, filename):
@@ -1785,8 +1784,7 @@ class DexTestDriver(IDriver):
         except(IOError, ValueError) as err_msg:
             LOG.exception("Error for make long command file: ", err_msg,
                           exc_info=False, error_no="03200")
-        finally:
-            return sh_file_name, file_path
+        return sh_file_name, file_path
 
     def __result__(self):
         return self.result if os.path.exists(self.result) else ""

plugins/ohos/src/ohos/environment/device.py

@@ -602,8 +602,7 @@ class Device(IDevice):
             self._proxy = OpenHarmony(port=self._h_port, addr=self.host, device=self)
         except Exception as error:
             self.log.error(' proxy init error: {}.'.format(str(error)))
-        finally:
-            return self._proxy
+        return self._proxy
 
     def start_harmony_rpc(self, port=8080, re_install_rpc=False):
         from devicetest.core.error_message import ErrorMessage
@@ -706,8 +705,6 @@ class Device(IDevice):
             return ret
         except Exception as err:
             self.log.error('DeviceTest-20013 uninstall: %s' % str(err))
-        finally:
-            return False
 
     def reconnect(self, waittime=60):
         '''
@@ -792,8 +789,7 @@ class Device(IDevice):
             self.pull_file("/data/screen.png", path)
         except Exception as error:
             self.log.error("devicetest take_picture: {}".format(str(error)))
-        finally:
-            return path
+        return path
 
     def set_device_report_path(self, path):
         self._device_log_path = path
\ No newline at end of file

src/xdevice/_core/executor/concurrent.py

@@ -273,12 +273,11 @@ class DriversThread(threading.Thread):
                 failed_list = params[ReportConst.unsuccessful_params].get(module_name, [])
         except Exception:
             failed_list = params[ReportConst.unsuccessful_params].get(module_name, [])
-        finally:
-            if not failed_list:
-                failed_list = params[ReportConst.unsuccessful_params].get(str(module_name).split(".")[0], [])
-            unpassed_test_params.extend(failed_list)
-            LOG.debug("Get unpassed test params %s", unpassed_test_params)
-            return unpassed_test_params
+        if not failed_list:
+            failed_list = params[ReportConst.unsuccessful_params].get(str(module_name).split(".")[0], [])
+        unpassed_test_params.extend(failed_list)
+        LOG.debug("Get unpassed test params %s", unpassed_test_params)
+        return unpassed_test_params
 
     @classmethod
     def _append_unpassed_test_param(cls, history_report_file,