Mirrors/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2026-10-03 03:31:41 +03:00
Code Issues Packages Projects Releases Wiki Activity

refactor(tools): test_idf_tools.py ruff changes

Browse Source
This commit is contained in:
Marek Fiala
2025-11-20 14:16:00 +01:00
parent b8af6f5141
commit ec76ac281b
1 changed files with 91 additions and 98 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tools/test_idf_tools/test_idf_tools.py
+91 -98
View File
@@ -111,7 +111,7 @@ class TestUsageBase(unittest.TestCase):
@classmethod @classmethod
def setUpClass(cls): def setUpClass(cls):
with open(os.path.join(os.getenv('IDF_PATH'), 'tools/tools.json'), 'r') as json_file: with open(os.path.join(os.getenv('IDF_PATH'), 'tools/tools.json')) as json_file:
tools_dict = json.load(json_file) tools_dict = json.load(json_file)
cls.tools_dict = tools_dict cls.tools_dict = tools_dict
@@ -133,7 +133,7 @@ class TestUsageBase(unittest.TestCase):
cls.temp_tools_dir = tempfile.mkdtemp(prefix='idf_tools_tmp') cls.temp_tools_dir = tempfile.mkdtemp(prefix='idf_tools_tmp')
print('Using IDF_TOOLS_PATH={}'.format(cls.temp_tools_dir)) print(f'Using IDF_TOOLS_PATH={cls.temp_tools_dir}')
os.environ['IDF_TOOLS_PATH'] = cls.temp_tools_dir os.environ['IDF_TOOLS_PATH'] = cls.temp_tools_dir
cls.idf_env_json = os.path.join(cls.temp_tools_dir, 'idf-env.json') cls.idf_env_json = os.path.join(cls.temp_tools_dir, 'idf-env.json')
@@ -154,13 +154,13 @@ class TestUsageBase(unittest.TestCase):
def assert_tool_installed(self, output, tool, tool_version, tool_archive_name=None): def assert_tool_installed(self, output, tool, tool_version, tool_archive_name=None):
if tool_archive_name is None: if tool_archive_name is None:
tool_archive_name = tool tool_archive_name = tool
self.assertIn('Installing %s@' % tool + tool_version, output) self.assertIn(f'Installing {tool}@{tool_version}', output)
self.assertRegex(output, re.compile(rf'Downloading \S+{tool_archive_name}')) self.assertRegex(output, re.compile(rf'Downloading \S+{tool_archive_name}'))
def assert_tool_not_installed(self, output, tool, tool_version, tool_archive_name=None): def assert_tool_not_installed(self, output, tool, tool_version, tool_archive_name=None):
if tool_archive_name is None: if tool_archive_name is None:
tool_archive_name = tool tool_archive_name = tool
self.assertNotIn('Installing %s@' % tool + tool_version, output) self.assertNotIn(f'Installing {tool}@{tool_version}', output)
self.assertNotRegex(output, re.compile(rf'Downloading \S+{tool_archive_name}')) self.assertNotRegex(output, re.compile(rf'Downloading \S+{tool_archive_name}'))
def run_idf_tools_with_action(self, action): def run_idf_tools_with_action(self, action):
@@ -265,8 +265,8 @@ class TestUsage(TestUsageBase):
output = self.run_idf_tools_with_action(['export']) output = self.run_idf_tools_with_action(['export'])
self.assertIn('export IDF_DEACTIVATE_FILE_PATH=', output, 'No IDF_DEACTIVATE_FILE_PATH exported into environment') self.assertIn('export IDF_DEACTIVATE_FILE_PATH=', output, 'No IDF_DEACTIVATE_FILE_PATH exported into environment')
deactivate_file = re.findall(r'(?:IDF_DEACTIVATE_FILE_PATH=")(.*)(?:")', output)[0] deactivate_file = re.findall(r'(?:IDF_DEACTIVATE_FILE_PATH=")(.*)(?:")', output)[0]
self.assertTrue(os.path.isfile(deactivate_file), 'File {} was not found. '.format(deactivate_file)) self.assertTrue(os.path.isfile(deactivate_file), f'File {deactivate_file} was not found. ')
self.assertNotEqual(os.stat(self.idf_env_json).st_size, 0, 'File {} is empty. '.format(deactivate_file)) self.assertNotEqual(os.stat(self.idf_env_json).st_size, 0, f'File {deactivate_file} is empty. ')
def test_export_recommended_version(self): def test_export_recommended_version(self):
always_install_and_recommended_tools = [] always_install_and_recommended_tools = []
@@ -358,14 +358,14 @@ class TestUsageUnix(TestUsage):
def test_usage_basic(self): def test_usage_basic(self):
output = self.run_idf_tools_with_action(['list']) output = self.run_idf_tools_with_action(['list'])
self.assertIn('* %s:' % ESP32ULP, output) self.assertIn(f'* {ESP32ULP}:', output)
self.assertIn('- %s (recommended)' % ESP32ULP_VERSION, output) self.assertIn(f'- {ESP32ULP_VERSION} (recommended)', output)
self.assertIn('* %s:' % OPENOCD, output) self.assertIn(f'* {OPENOCD}:', output)
self.assertIn('- %s (recommended)' % OPENOCD_VERSION, output) self.assertIn(f'- {OPENOCD_VERSION} (recommended)', output)
self.assertIn('* %s:' % RISCV_ELF, output) self.assertIn(f'* {RISCV_ELF}:', output)
self.assertIn('- %s (recommended)' % RISCV_ELF_VERSION, output) self.assertIn(f'- {RISCV_ELF_VERSION} (recommended)', output)
self.assertIn('* %s:' % XTENSA_ELF, output) self.assertIn(f'* {XTENSA_ELF}:', output)
self.assertIn('- %s (recommended)' % XTENSA_ELF_VERSION, output) self.assertIn(f'- {XTENSA_ELF_VERSION} (recommended)', output)
required_tools_installed = 7 required_tools_installed = 7
output = self.run_idf_tools_with_action(['install']) output = self.run_idf_tools_with_action(['install'])
@@ -385,20 +385,19 @@ class TestUsageUnix(TestUsage):
self.assertIn('version installed in tools directory: ' + tool_version, output) self.assertIn('version installed in tools directory: ' + tool_version, output)
output = self.run_idf_tools_with_action(['export']) output = self.run_idf_tools_with_action(['export'])
self.assertIn('%s/tools/esp32ulp-elf/%s/esp32ulp-elf/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/esp32ulp-elf/{ESP32ULP_VERSION}/esp32ulp-elf/bin', output)
(self.temp_tools_dir, ESP32ULP_VERSION), output) self.assertIn(f'{self.temp_tools_dir}/tools/xtensa-esp-elf/{XTENSA_ELF_VERSION}/xtensa-esp-elf/bin', output)
self.assertIn('%s/tools/xtensa-esp-elf/%s/xtensa-esp-elf/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/openocd-esp32/{OPENOCD_VERSION}/openocd-esp32/bin', output)
(self.temp_tools_dir, XTENSA_ELF_VERSION), output) self.assertIn(f'{self.temp_tools_dir}/tools/riscv32-esp-elf/{RISCV_ELF_VERSION}/riscv32-esp-elf/bin', output)
self.assertIn('%s/tools/openocd-esp32/%s/openocd-esp32/bin' % self.assertIn(
(self.temp_tools_dir, OPENOCD_VERSION), output) f'{self.temp_tools_dir}/tools/xtensa-esp-elf-gdb/{XTENSA_ESP_GDB_VERSION}/xtensa-esp-elf-gdb/bin',
self.assertIn('%s/tools/riscv32-esp-elf/%s/riscv32-esp-elf/bin' % output,
(self.temp_tools_dir, RISCV_ELF_VERSION), output) )
self.assertIn('%s/tools/xtensa-esp-elf-gdb/%s/xtensa-esp-elf-gdb/bin' % self.assertIn(
(self.temp_tools_dir, XTENSA_ESP_GDB_VERSION), output) f'{self.temp_tools_dir}/tools/riscv32-esp-elf-gdb/{RISCV_ESP_GDB_VERSION}/riscv32-esp-elf-gdb/bin',
self.assertIn('%s/tools/riscv32-esp-elf-gdb/%s/riscv32-esp-elf-gdb/bin' % output,
(self.temp_tools_dir, RISCV_ESP_GDB_VERSION), output) )
self.assertIn('%s/tools/esp-rom-elfs/%s/' % self.assertIn(f'{self.temp_tools_dir}/tools/esp-rom-elfs/{ESP_ROM_ELFS_VERSION}/', output)
(self.temp_tools_dir, ESP_ROM_ELFS_VERSION), output)
output = self.run_idf_tools_with_action(['list', '--outdated']) output = self.run_idf_tools_with_action(['list', '--outdated'])
self.assertEqual('', output) self.assertEqual('', output)
@@ -450,20 +449,19 @@ class TestUsageUnix(TestUsage):
self.assertIn('version installed in tools directory: ' + tool_version, output) self.assertIn('version installed in tools directory: ' + tool_version, output)
output = self.run_idf_tools_with_action(['export']) output = self.run_idf_tools_with_action(['export'])
self.assertIn('%s/tools/esp32ulp-elf/%s/esp32ulp-elf/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/esp32ulp-elf/{ESP32ULP_VERSION}/esp32ulp-elf/bin', output)
(self.temp_tools_dir, ESP32ULP_VERSION), output) self.assertIn(f'{self.temp_tools_dir}/tools/xtensa-esp-elf/{XTENSA_ELF_VERSION}/xtensa-esp-elf/bin', output)
self.assertIn('%s/tools/xtensa-esp-elf/%s/xtensa-esp-elf/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/openocd-esp32/{OPENOCD_VERSION}/openocd-esp32/bin', output)
(self.temp_tools_dir, XTENSA_ELF_VERSION), output) self.assertIn(
self.assertIn('%s/tools/openocd-esp32/%s/openocd-esp32/bin' % f'{self.temp_tools_dir}/tools/xtensa-esp-elf-gdb/{XTENSA_ESP_GDB_VERSION}/xtensa-esp-elf-gdb/bin',
(self.temp_tools_dir, OPENOCD_VERSION), output) output,
self.assertIn('%s/tools/xtensa-esp-elf-gdb/%s/xtensa-esp-elf-gdb/bin' % )
(self.temp_tools_dir, XTENSA_ESP_GDB_VERSION), output) self.assertNotIn(f'{self.temp_tools_dir}/tools/riscv32-esp-elf/{RISCV_ELF_VERSION}/riscv32-esp-elf/bin', output)
self.assertNotIn('%s/tools/riscv32-esp-elf/%s/riscv32-esp-elf/bin' % self.assertNotIn(
(self.temp_tools_dir, RISCV_ELF_VERSION), output) f'{self.temp_tools_dir}/tools/riscv32-esp-elf-gdb/{RISCV_ESP_GDB_VERSION}/riscv32-esp-elf-gdb/bin',
self.assertNotIn('%s/tools/riscv32-esp-elf-gdb/%s/riscv32-esp-elf-gdb/bin' % output,
(self.temp_tools_dir, RISCV_ESP_GDB_VERSION), output) )
self.assertIn('%s/tools/esp-rom-elfs/%s/' % self.assertIn(f'{self.temp_tools_dir}/tools/esp-rom-elfs/{ESP_ROM_ELFS_VERSION}/', output)
(self.temp_tools_dir, ESP_ROM_ELFS_VERSION), output)
def test_tools_for_esp32c3(self): def test_tools_for_esp32c3(self):
required_tools_installed = 4 required_tools_installed = 4
@@ -483,18 +481,15 @@ class TestUsageUnix(TestUsage):
self.assertIn('version installed in tools directory: ' + tool_version, output) self.assertIn('version installed in tools directory: ' + tool_version, output)
output = self.run_idf_tools_with_action(['export']) output = self.run_idf_tools_with_action(['export'])
self.assertIn('%s/tools/openocd-esp32/%s/openocd-esp32/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/openocd-esp32/{OPENOCD_VERSION}/openocd-esp32/bin', output)
(self.temp_tools_dir, OPENOCD_VERSION), output) self.assertIn(f'{self.temp_tools_dir}/tools/riscv32-esp-elf/{RISCV_ELF_VERSION}/riscv32-esp-elf/bin', output)
self.assertIn('%s/tools/riscv32-esp-elf/%s/riscv32-esp-elf/bin' % self.assertNotIn(f'{self.temp_tools_dir}/tools/esp32ulp-elf/{ESP32ULP_VERSION}/esp32ulp-elf/bin', output)
(self.temp_tools_dir, RISCV_ELF_VERSION), output) self.assertNotIn(f'{self.temp_tools_dir}/tools/xtensa-esp-elf/{XTENSA_ELF_VERSION}/xtensa-esp-elf/bin', output)
self.assertNotIn('%s/tools/esp32ulp-elf/%s/esp32ulp-elf/bin' % self.assertNotIn(
(self.temp_tools_dir, ESP32ULP_VERSION), output) f'{self.temp_tools_dir}/tools/xtensa-esp-elf-gdb/{XTENSA_ESP_GDB_VERSION}/xtensa-esp-elf-gdb/bin',
self.assertNotIn('%s/tools/xtensa-esp-elf/%s/xtensa-esp-elf/bin' % output,
(self.temp_tools_dir, XTENSA_ELF_VERSION), output) )
self.assertNotIn('%s/tools/xtensa-esp-elf-gdb/%s/xtensa-esp-elf-gdb/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/esp-rom-elfs/{ESP_ROM_ELFS_VERSION}/', output)
(self.temp_tools_dir, XTENSA_ESP_GDB_VERSION), output)
self.assertIn('%s/tools/esp-rom-elfs/%s/' %
(self.temp_tools_dir, ESP_ROM_ELFS_VERSION), output)
def test_tools_for_esp32s2(self): def test_tools_for_esp32s2(self):
required_tools_installed = 6 required_tools_installed = 6
@@ -514,20 +509,19 @@ class TestUsageUnix(TestUsage):
self.assertIn('version installed in tools directory: ' + tool_version, output) self.assertIn('version installed in tools directory: ' + tool_version, output)
output = self.run_idf_tools_with_action(['export']) output = self.run_idf_tools_with_action(['export'])
self.assertIn('%s/tools/xtensa-esp-elf/%s/xtensa-esp-elf/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/xtensa-esp-elf/{XTENSA_ELF_VERSION}/xtensa-esp-elf/bin', output)
(self.temp_tools_dir, XTENSA_ELF_VERSION), output) self.assertIn(f'{self.temp_tools_dir}/tools/openocd-esp32/{OPENOCD_VERSION}/openocd-esp32/bin', output)
self.assertIn('%s/tools/openocd-esp32/%s/openocd-esp32/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/esp32ulp-elf/{ESP32ULP_VERSION}/esp32ulp-elf/bin', output)
(self.temp_tools_dir, OPENOCD_VERSION), output) self.assertIn(f'{self.temp_tools_dir}/tools/riscv32-esp-elf/{RISCV_ELF_VERSION}/riscv32-esp-elf/bin', output)
self.assertIn('%s/tools/esp32ulp-elf/%s/esp32ulp-elf/bin' % self.assertIn(
(self.temp_tools_dir, ESP32ULP_VERSION), output) f'{self.temp_tools_dir}/tools/xtensa-esp-elf-gdb/{XTENSA_ESP_GDB_VERSION}/xtensa-esp-elf-gdb/bin',
self.assertIn('%s/tools/riscv32-esp-elf/%s/riscv32-esp-elf/bin' % output,
(self.temp_tools_dir, RISCV_ELF_VERSION), output) )
self.assertIn('%s/tools/xtensa-esp-elf-gdb/%s/xtensa-esp-elf-gdb/bin' % self.assertNotIn(
(self.temp_tools_dir, XTENSA_ESP_GDB_VERSION), output) f'{self.temp_tools_dir}/tools/riscv32-esp-elf-gdb/{RISCV_ESP_GDB_VERSION}/riscv32-esp-elf-gdb/bin',
self.assertNotIn('%s/tools/riscv32-esp-elf-gdb/%s/riscv32-esp-elf-gdb/bin' % output,
(self.temp_tools_dir, RISCV_ESP_GDB_VERSION), output) )
self.assertIn('%s/tools/esp-rom-elfs/%s/' % self.assertIn(f'{self.temp_tools_dir}/tools/esp-rom-elfs/{ESP_ROM_ELFS_VERSION}/', output)
(self.temp_tools_dir, ESP_ROM_ELFS_VERSION), output)
def test_tools_for_esp32s3(self): def test_tools_for_esp32s3(self):
required_tools_installed = 6 required_tools_installed = 6
@@ -548,20 +542,19 @@ class TestUsageUnix(TestUsage):
self.assertIn('version installed in tools directory: ' + tool_version, output) self.assertIn('version installed in tools directory: ' + tool_version, output)
output = self.run_idf_tools_with_action(['export']) output = self.run_idf_tools_with_action(['export'])
self.assertIn('%s/tools/openocd-esp32/%s/openocd-esp32/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/openocd-esp32/{OPENOCD_VERSION}/openocd-esp32/bin', output)
(self.temp_tools_dir, OPENOCD_VERSION), output) self.assertIn(f'{self.temp_tools_dir}/tools/xtensa-esp-elf/{XTENSA_ELF_VERSION}/xtensa-esp-elf/bin', output)
self.assertIn('%s/tools/xtensa-esp-elf/%s/xtensa-esp-elf/bin' % self.assertIn(f'{self.temp_tools_dir}/tools/esp32ulp-elf/{ESP32ULP_VERSION}/esp32ulp-elf/bin', output)
(self.temp_tools_dir, XTENSA_ELF_VERSION), output) self.assertIn(f'{self.temp_tools_dir}/tools/riscv32-esp-elf/{RISCV_ELF_VERSION}/riscv32-esp-elf/bin', output)
self.assertIn('%s/tools/esp32ulp-elf/%s/esp32ulp-elf/bin' % self.assertIn(
(self.temp_tools_dir, ESP32ULP_VERSION), output) f'{self.temp_tools_dir}/tools/xtensa-esp-elf-gdb/{XTENSA_ESP_GDB_VERSION}/xtensa-esp-elf-gdb/bin',
self.assertIn('%s/tools/riscv32-esp-elf/%s/riscv32-esp-elf/bin' % output,
(self.temp_tools_dir, RISCV_ELF_VERSION), output) )
self.assertIn('%s/tools/xtensa-esp-elf-gdb/%s/xtensa-esp-elf-gdb/bin' % self.assertNotIn(
(self.temp_tools_dir, XTENSA_ESP_GDB_VERSION), output) f'{self.temp_tools_dir}/tools/riscv32-esp-elf-gdb/{RISCV_ESP_GDB_VERSION}/riscv32-esp-elf-gdb/bin',
self.assertNotIn('%s/tools/riscv32-esp-elf-gdb/%s/riscv32-esp-elf-gdb/bin' % output,
(self.temp_tools_dir, RISCV_ESP_GDB_VERSION), output) )
self.assertIn('%s/tools/esp-rom-elfs/%s/' % self.assertIn(f'{self.temp_tools_dir}/tools/esp-rom-elfs/{ESP_ROM_ELFS_VERSION}/', output)
(self.temp_tools_dir, ESP_ROM_ELFS_VERSION), output)
def test_tools_for_esp32p4(self): def test_tools_for_esp32p4(self):
required_tools_installed = 4 required_tools_installed = 4
@@ -618,14 +611,14 @@ class TestUsageWin(TestUsage):
def test_usage_basic_win(self): def test_usage_basic_win(self):
output = self.run_idf_tools_with_action(['list']) output = self.run_idf_tools_with_action(['list'])
self.assertIn('* %s:' % ESP32ULP, output) self.assertIn(f'* {ESP32ULP}:', output)
self.assertIn('- %s (recommended)' % ESP32ULP_VERSION, output) self.assertIn(f'- {ESP32ULP_VERSION} (recommended)', output)
self.assertIn('* %s:' % OPENOCD, output) self.assertIn(f'* {OPENOCD}:', output)
self.assertIn('- %s (recommended)' % OPENOCD_VERSION, output) self.assertIn(f'- {OPENOCD_VERSION} (recommended)', output)
self.assertIn('* %s:' % RISCV_ELF, output) self.assertIn(f'* {RISCV_ELF}:', output)
self.assertIn('- %s (recommended)' % RISCV_ELF_VERSION, output) self.assertIn(f'- {RISCV_ELF_VERSION} (recommended)', output)
self.assertIn('* %s:' % XTENSA_ELF, output) self.assertIn(f'* {XTENSA_ELF}:', output)
self.assertIn('- %s (recommended)' % XTENSA_ELF_VERSION, output) self.assertIn(f'- {XTENSA_ELF_VERSION} (recommended)', output)
required_tools_installed = 12 required_tools_installed = 12
output = self.run_idf_tools_with_action(['install']) output = self.run_idf_tools_with_action(['install'])
@@ -1077,16 +1070,16 @@ class TestMaintainer(unittest.TestCase):
def test_json_rewrite(self): def test_json_rewrite(self):
idf_tools.main(['rewrite']) idf_tools.main(['rewrite'])
with open(self.tools_old, 'r') as f: with open(self.tools_old) as f:
json_old = f.read() json_old = f.read()
with open(self.tools_new, 'r') as f: with open(self.tools_new) as f:
json_new = f.read() json_new = f.read()
self.assertEqual(json_old, json_new, "Please check 'tools/tools.new.json' to find a cause!") self.assertEqual(json_old, json_new, "Please check 'tools/tools.new.json' to find a cause!")
def add_version_get_expected_json(self, addition_file, replace=False): def add_version_get_expected_json(self, addition_file, replace=False):
with open(self.tools_old, 'r') as f: with open(self.tools_old) as f:
expected_json = json.load(f) expected_json = json.load(f)
with open(addition_file, 'r') as f: with open(addition_file) as f:
addition_json = json.load(f) addition_json = json.load(f)
for tool in expected_json['tools']: for tool in expected_json['tools']:
if tool['name'] == self.test_tool_name: if tool['name'] == self.test_tool_name:
@@ -1099,7 +1092,7 @@ class TestMaintainer(unittest.TestCase):
def test_add_version_artifact_addition(self): def test_add_version_artifact_addition(self):
filenames = [] filenames = []
with open('add_version/artifact_input.json', 'r') as f: with open('add_version/artifact_input.json') as f:
add_tools_info = json.load(f) add_tools_info = json.load(f)
for tool in add_tools_info: for tool in add_tools_info:
filenames.append(tool['filename']) filenames.append(tool['filename'])
@@ -1119,7 +1112,7 @@ class TestMaintainer(unittest.TestCase):
] + filenames ] + filenames
) )
expected_json = self.add_version_get_expected_json('add_version/artifact_expected_addition.json') expected_json = self.add_version_get_expected_json('add_version/artifact_expected_addition.json')
with open(self.tools_new, 'r') as f1: with open(self.tools_new) as f1:
self.assertEqual(json.load(f1), expected_json, "Please check 'tools/tools.new.json' to find a cause!") self.assertEqual(json.load(f1), expected_json, "Please check 'tools/tools.new.json' to find a cause!")
def test_add_version_checksum_addition(self): def test_add_version_checksum_addition(self):
@@ -1137,7 +1130,7 @@ class TestMaintainer(unittest.TestCase):
] ]
) )
expected_json = self.add_version_get_expected_json('add_version/checksum_expected_addition.json') expected_json = self.add_version_get_expected_json('add_version/checksum_expected_addition.json')
with open(self.tools_new, 'r') as f1: with open(self.tools_new) as f1:
self.assertEqual(json.load(f1), expected_json, "Please check 'tools/tools.new.json' to find a cause!") self.assertEqual(json.load(f1), expected_json, "Please check 'tools/tools.new.json' to find a cause!")
def test_add_version_checksum_with_override(self): def test_add_version_checksum_with_override(self):
@@ -1156,7 +1149,7 @@ class TestMaintainer(unittest.TestCase):
] ]
) )
expected_json = self.add_version_get_expected_json('add_version/checksum_expected_override.json', True) expected_json = self.add_version_get_expected_json('add_version/checksum_expected_override.json', True)
with open(self.tools_new, 'r') as f1: with open(self.tools_new) as f1:
self.assertEqual(json.load(f1), expected_json, "Please check 'tools/tools.new.json' to find a cause!") self.assertEqual(json.load(f1), expected_json, "Please check 'tools/tools.new.json' to find a cause!")