Skip to content

Commit 0d77ea4

Browse files
committed
reworked tests
1 parent a8f5f1b commit 0d77ea4

File tree

2 files changed

+382
-307
lines changed

2 files changed

+382
-307
lines changed

codeflash/context/code_context_extractor.py

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -183,14 +183,16 @@ def extract_code_string_context_from_files(
183183
helpers_of_helpers_qualified_names = {
184184
func.qualified_name for func in helpers_of_helpers.get(file_path, set())
185185
}
186+
code_without_unused_defs = remove_unused_definitions_by_function_names(
187+
original_code, qualified_function_names | helpers_of_helpers_qualified_names
188+
)
186189
code_context = parse_code_and_prune_cst(
187-
original_code,
190+
code_without_unused_defs,
188191
code_context_type,
189192
qualified_function_names,
190193
helpers_of_helpers_qualified_names,
191194
remove_docstrings,
192195
)
193-
code_context = remove_unused_definitions_by_function_names(code_context, qualified_function_names | helpers_of_helpers_qualified_names)
194196
except ValueError as e:
195197
logger.debug(f"Error while getting read-only code: {e}")
196198
continue
@@ -215,10 +217,10 @@ def extract_code_string_context_from_files(
215217
continue
216218
try:
217219
qualified_helper_function_names = {func.qualified_name for func in helper_function_sources}
220+
code_without_unused_defs = remove_unused_definitions_by_function_names(original_code, qualified_helper_function_names)
218221
code_context = parse_code_and_prune_cst(
219-
original_code, code_context_type, set(), qualified_helper_function_names, remove_docstrings
222+
code_without_unused_defs, code_context_type, set(), qualified_helper_function_names, remove_docstrings
220223
)
221-
code_context = remove_unused_definitions_by_function_names(code_context, qualified_helper_function_names)
222224
except ValueError as e:
223225
logger.debug(f"Error while getting read-only code: {e}")
224226
continue
@@ -285,16 +287,16 @@ def extract_code_markdown_context_from_files(
285287
helpers_of_helpers_qualified_names = {
286288
func.qualified_name for func in helpers_of_helpers.get(file_path, set())
287289
}
290+
code_without_unused_defs = remove_unused_definitions_by_function_names(
291+
original_code, qualified_function_names | helpers_of_helpers_qualified_names
292+
)
288293
code_context = parse_code_and_prune_cst(
289-
original_code,
294+
code_without_unused_defs,
290295
code_context_type,
291296
qualified_function_names,
292297
helpers_of_helpers_qualified_names,
293298
remove_docstrings,
294299
)
295-
code_context = remove_unused_definitions_by_function_names(
296-
code_context, qualified_function_names | helpers_of_helpers_qualified_names
297-
)
298300

299301
except ValueError as e:
300302
logger.debug(f"Error while getting read-only code: {e}")
@@ -323,11 +325,9 @@ def extract_code_markdown_context_from_files(
323325
continue
324326
try:
325327
qualified_helper_function_names = {func.qualified_name for func in helper_function_sources}
328+
code_without_unused_defs = remove_unused_definitions_by_function_names(original_code, qualified_helper_function_names)
326329
code_context = parse_code_and_prune_cst(
327-
original_code, code_context_type, set(), qualified_helper_function_names, remove_docstrings
328-
)
329-
code_context = remove_unused_definitions_by_function_names(
330-
code_context, qualified_helper_function_names
330+
code_without_unused_defs, code_context_type, set(), qualified_helper_function_names, remove_docstrings
331331
)
332332
except ValueError as e:
333333
logger.debug(f"Error while getting read-only code: {e}")

0 commit comments

Comments
 (0)