Skip to content

Commit b8fc5cb

Browse files
committed
collect the same number of tests for behavorial, benchmarking and line profiler
1 parent 44e46b4 commit b8fc5cb

File tree

1 file changed

+6
-16
lines changed

1 file changed

+6
-16
lines changed

codeflash/verification/test_runner.py

Lines changed: 6 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -166,15 +166,10 @@ def run_line_profile_tests(
166166
)
167167
test_files: list[str] = []
168168
for file in test_paths.test_files:
169-
if file.test_type in {TestType.REPLAY_TEST, TestType.EXISTING_UNIT_TEST} and file.tests_in_file:
169+
if file.test_type == TestType.REPLAY_TEST:
170+
# TODO: Does this work for unittest framework?
170171
test_files.extend(
171-
[
172-
str(file.benchmarking_file_path)
173-
+ "::"
174-
+ (test.test_class + "::" if test.test_class else "")
175-
+ (test.test_function.split("[", 1)[0] if "[" in test.test_function else test.test_function)
176-
for test in file.tests_in_file
177-
]
172+
[str(file.benchmarking_file_path) + "::" + test.test_function for test in file.tests_in_file]
178173
)
179174
else:
180175
test_files.append(str(file.benchmarking_file_path))
@@ -251,15 +246,10 @@ def run_benchmarking_tests(
251246
)
252247
test_files: list[str] = []
253248
for file in test_paths.test_files:
254-
if file.test_type in {TestType.REPLAY_TEST, TestType.EXISTING_UNIT_TEST} and file.tests_in_file:
249+
if file.test_type == TestType.REPLAY_TEST:
250+
# TODO: Does this work for unittest framework?
255251
test_files.extend(
256-
[
257-
str(file.benchmarking_file_path)
258-
+ "::"
259-
+ (test.test_class + "::" if test.test_class else "")
260-
+ (test.test_function.split("[", 1)[0] if "[" in test.test_function else test.test_function)
261-
for test in file.tests_in_file
262-
]
252+
[str(file.benchmarking_file_path) + "::" + test.test_function for test in file.tests_in_file]
263253
)
264254
else:
265255
test_files.append(str(file.benchmarking_file_path))

0 commit comments

Comments
 (0)