diff --git a/libs/benchmark/report/generate_report.py b/libs/benchmark/report/generate_report.py index a268bf46dc65696149d56e6de0aba53f50324ff5..4d482b6953be81ea57ad7d64e1fc01b9716a595d 100644 --- a/libs/benchmark/report/generate_report.py +++ b/libs/benchmark/report/generate_report.py @@ -160,10 +160,10 @@ class BenchmarkReport(object): module_name, testsuit_name, json_file) self.benchmark_list = mdl_summary_list - if sbs_name in system_summary_dic.keys() \ - and testsuit_name in system_summary_dic[sbs_name].keys(): + if sbs_name in system_summary_dic.keys() and testsuit_name in \ + system_summary_dic.get(sbs_name, {}).keys(): subsystem_summary_dic = \ - system_summary_dic[sbs_name][testsuit_name] + system_summary_dic.get(sbs_name).get(testsuit_name) subsystem_summary_dic["children"] += \ self._remove_iterations(mdl_summary_list) else: @@ -187,7 +187,7 @@ class BenchmarkReport(object): self._remove_iterations(mdl_summary_list) self.sbs_mdl_summary_list.append(subsystem_summary_dic) system_summary_dic[sbs_name] = {} - system_summary_dic[sbs_name][testsuit_name] = \ + system_summary_dic.get(sbs_name)[testsuit_name] = \ subsystem_summary_dic subsystem_summary_dic["pm"] = "unknown" subsystem_summary_dic["owner"] = "unknown" diff --git a/libs/fuzzlib/fuzzer_helper.py b/libs/fuzzlib/fuzzer_helper.py index b8c6217419047f2c6d31f1a22b6e03b4c5b3d2f9..3be2cd31223d7cbdb7efc9c543f5eca7a3674959 100644 --- a/libs/fuzzlib/fuzzer_helper.py +++ b/libs/fuzzlib/fuzzer_helper.py @@ -92,11 +92,11 @@ def _get_fuzzer_yaml_config(fuzzer_name): "projects", fuzzer_name, "project.yaml") - if not os.path.exists(project_yaml_path): - return None - #log run stdout to fuzzlog dir - with open(project_yaml_path) as filehandle: - yaml_config = yaml.safe_load(filehandle) + yaml_config = {} + if os.path.exists(project_yaml_path): + # log run stdout to fuzzlog dir + with open(project_yaml_path) as filehandle: + yaml_config = yaml.safe_load(filehandle) return yaml_config diff --git a/libs/fuzzlib/tools/colored.py b/libs/fuzzlib/tools/colored.py index fae3edf70cf0862e416bc765c886b01a1faf79c8..db1b1c1e9db85f0b73b366ab32f3fe2fb87582c1 100644 --- a/libs/fuzzlib/tools/colored.py +++ b/libs/fuzzlib/tools/colored.py @@ -41,7 +41,8 @@ class Colored(object): @staticmethod def get_project_logger(log_project="default"): if log_project in Colored.PROJECT_LOGGER_MAP: - return Colored.PROJECT_LOGGER_MAP[log_project] + return Colored.PROJECT_LOGGER_MAP.get(log_project) + logger = Colored(log_project) Colored.PROJECT_LOGGER_MAP[log_project] = logger return logger diff --git a/libs/fuzzlib/tools/run_result.py b/libs/fuzzlib/tools/run_result.py index 5fb2b6edc683e9f0a27568eccde9384378da423f..ff72648caac33ac24ceb255c2c7fd77e3e190fda 100644 --- a/libs/fuzzlib/tools/run_result.py +++ b/libs/fuzzlib/tools/run_result.py @@ -79,7 +79,9 @@ class RunResult(): if html_format: f.write(RunResult.filter_log(render_detail(self.crash_info))) else: - f.write(RunResult.filter_log(self.crash_info["backtrace"])) + f.write(RunResult.filter_log(self.crash_info.get( + "backtrace", "" + ))) if __name__ == "__main__": diff --git a/src/core/config/config_manager.py b/src/core/config/config_manager.py index 3b463f63ae52793e96ea81cfcf619f5715ca9f77..165a644e56bbf57e1aa5a07c48454bd018072dff 100755 --- a/src/core/config/config_manager.py +++ b/src/core/config/config_manager.py @@ -215,12 +215,12 @@ class UserConfigManager(object): node = root.find(target_name) if not node: - return None + return data_dic if sub_target != "": node = node.find(sub_target) if not node: - return None + return data_dic for sub in node: if sub.text is None: diff --git a/src/core/config/parse_parts_config.py b/src/core/config/parse_parts_config.py index 3e51417d40a1786073203d8f0d6d31845ff044ae..33fc51fd456153dc4696dd49b738d25196ba7c46 100755 --- a/src/core/config/parse_parts_config.py +++ b/src/core/config/parse_parts_config.py @@ -52,25 +52,22 @@ class ParsePartsConfig(object): def get_infos_data(self): config_filepath = self.get_config_file_path() + data_dic = {} if not os.path.exists(config_filepath): print("Error: %s is not exist." % config_filepath) - return None, None - - data_dic = None - with open(config_filepath, 'r') as file_handle: - data_dic = json.load(file_handle) - if not data_dic: - print("Error: json file load error.") - return None, None + else: + with open(config_filepath, 'r') as file_handle: + data_dic = json.load(file_handle) + if not data_dic: + print("Error: json file load error.") # open source branch, the part form of all product is "phone" if is_open_source_product(self.productform): - product_data_dic = data_dic.get("phone", None) + product_data_dic = data_dic.get("phone", {}) else: - product_data_dic = data_dic.get(self.productform, None) + product_data_dic = data_dic.get(self.productform, {}) if product_data_dic is None: print("Error: product_data_dic is None.") - return None, None subsystem_infos = product_data_dic.get("subsystem_infos", None) part_infos = product_data_dic.get("part_infos", None) diff --git a/src/core/driver/lite_driver.py b/src/core/driver/lite_driver.py index 1f755f46aec401fe2767ff81cddc6fe4157cd9f1..8a4296f0b78a1f506eeba80a119f034e309049cc 100755 --- a/src/core/driver/lite_driver.py +++ b/src/core/driver/lite_driver.py @@ -80,7 +80,6 @@ class LiteUnitTest(IDriver): def __check_failed__(self, msg): self.log.error("check failed {}".format(msg)) - return None def __check_environment__(self, device_options): pass @@ -291,14 +290,12 @@ class LiteUnitTest(IDriver): show help info. """ self.log.info("this is test driver for cpp test") - return None def show_driver_info(self): """ show driver info. """ self.log.info("this is test driver for cpp test") - return None def __result__(self): pass diff --git a/src/core/testcase/testcase_manager.py b/src/core/testcase/testcase_manager.py index d0ac52af924604fa789f25c1dff2895ea3282e7c..07f5921115fdab22ebce462e6f2f7f0c6c7104bd 100755 --- a/src/core/testcase/testcase_manager.py +++ b/src/core/testcase/testcase_manager.py @@ -59,7 +59,7 @@ class TestCaseManager(object): test_type, options) for key, value in suit_file_dic.items(): - suit_file_dic[key] = value + temp_dic[key] + suit_file_dic[key] = value + temp_dic.get(key, "") return suit_file_dic def get_test_file_data_by_test_type(self, test_case_path, @@ -108,17 +108,17 @@ class TestCaseManager(object): continue if suffix_name == ".dex": - suite_file_dictionary["DEX"].append(suite_file) + suite_file_dictionary.get("DEX").append(suite_file) elif suffix_name == ".hap": - suite_file_dictionary["JST"].append(suite_file) + suite_file_dictionary.get("JST").append(suite_file) elif suffix_name == ".py": if not self.check_python_test_file(suite_file): continue - suite_file_dictionary["PYT"].append(suite_file) + suite_file_dictionary.get("PYT").append(suite_file) elif suffix_name == "": - suite_file_dictionary["CXX"].append(suite_file) + suite_file_dictionary.get("CXX").append(suite_file) elif suffix_name == ".bin": - suite_file_dictionary["BIN"].append(suite_file) + suite_file_dictionary.get("BIN").append(suite_file) return suite_file_dictionary diff --git a/src/core/utils.py b/src/core/utils.py index 45d10b2ee3e7c809ddc73843525959631deb5ae0..26525b692451633c121bb378b9c31d25bbb044da 100755 --- a/src/core/utils.py +++ b/src/core/utils.py @@ -124,14 +124,12 @@ def parse_device_name(product_form): device_json_file = os.path.join(sys.source_code_root_path, "productdefine", "common", "products", "{}.json".format(product_form)) - if not os.path.exists(device_json_file): - return None - - with open(device_json_file, 'r') as json_file: - json_info = json.load(json_file) - if not json_info: - return None - device_name = json_info.get('product_device') + json_info = {} + if os.path.exists(device_json_file): + with open(device_json_file, 'r') as json_file: + json_info = json.load(json_file) + + device_name = json_info.get('product_device', None) return device_name