Skip to content

Commit 00a5dc7

Browse files
committed
remove logging statements
1 parent e1fb4b8 commit 00a5dc7

2 files changed

Lines changed: 0 additions & 20 deletions

File tree

codeflash/languages/python/context/unused_definition_remover.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -748,7 +748,6 @@ def detect_unused_helper_functions(
748748
"""
749749
# Skip this analysis for non-Python languages since we use Python's ast module
750750
if not is_python():
751-
logger.debug("Skipping unused helper function detection for non-Python languages")
752751
return []
753752

754753
if isinstance(optimized_code, CodeStringsMarkdown) and len(optimized_code.code_strings) > 0:

codeflash/verification/parse_test_output.py

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -605,23 +605,16 @@ def parse_sqlite_test_results(sqlite_file_path: Path, test_files: TestFiles, tes
605605
else:
606606
# Try original_file_path first (for existing tests that were instrumented)
607607
test_type = test_files.get_test_type_by_original_file_path(test_file_path)
608-
logger.debug(f"[PARSE-DEBUG] test_module={test_module_path}, test_file_path={test_file_path}")
609-
logger.debug(f"[PARSE-DEBUG] by_original_file_path: {test_type}")
610608
# If not found, try instrumented_behavior_file_path (for generated tests)
611609
if test_type is None:
612610
test_type = test_files.get_test_type_by_instrumented_file_path(test_file_path)
613-
logger.debug(f"[PARSE-DEBUG] by_instrumented_file_path: {test_type}")
614611
# Default to GENERATED_REGRESSION for Jest/Java tests when test type can't be determined
615612
if test_type is None and (is_jest or is_java_test):
616613
test_type = TestType.GENERATED_REGRESSION
617-
logger.debug(
618-
f"[PARSE-DEBUG] defaulting to GENERATED_REGRESSION ({'Jest' if is_jest else 'Java'})"
619-
)
620614
elif test_type is None:
621615
# Skip results where test type cannot be determined
622616
logger.debug(f"Skipping result for {test_function_name}: could not determine test type")
623617
continue
624-
logger.debug(f"[PARSE-DEBUG] FINAL test_type={test_type}")
625618

626619
# Deserialize return value
627620
# For Jest/Java: Store as serialized JSON - comparison happens via language-specific comparator
@@ -699,10 +692,6 @@ def parse_test_xml(
699692
return test_results
700693
# Always use tests_project_rootdir since pytest is now the test runner for all frameworks
701694
base_dir = test_config.tests_project_rootdir
702-
logger.debug(f"[PARSE-XML] base_dir for resolution: {base_dir}")
703-
logger.debug(
704-
f"[PARSE-XML] Registered test files: {[str(tf.instrumented_behavior_file_path) for tf in test_files.test_files]}"
705-
)
706695

707696
# For Java: pre-parse fallback stdout once (not per testcase) to avoid O(n²) complexity
708697
java_fallback_stdout = None
@@ -763,13 +752,9 @@ def parse_test_xml(
763752
if test_file_name is None:
764753
if test_class_path:
765754
# TODO : This might not be true if the test is organized under a class
766-
logger.debug(f"[PARSE-XML] Resolving test_class_path={test_class_path} in base_dir={base_dir}")
767755
test_file_path = resolve_test_file_from_class_path(test_class_path, base_dir)
768756

769757
if test_file_path is None:
770-
logger.error(
771-
f"[PARSE-XML] ERROR: Could not resolve test_class_path={test_class_path}, base_dir={base_dir}"
772-
)
773758
logger.warning(f"Could not find the test for file name - {test_class_path} ")
774759
continue
775760
else:
@@ -782,15 +767,11 @@ def parse_test_xml(
782767
logger.warning(f"Could not find the test for file name - {test_file_path} ")
783768
continue
784769
# Try to match by instrumented file path first (for generated/instrumented tests)
785-
logger.debug(f"[PARSE-XML] Looking up test_type by instrumented_file_path: {test_file_path}")
786770
test_type = test_files.get_test_type_by_instrumented_file_path(test_file_path)
787-
logger.debug(f"[PARSE-XML] Lookup by instrumented path result: {test_type}")
788771
if test_type is None:
789772
# Fallback: try to match by original file path (for existing unit tests that were instrumented)
790773
# JUnit XML may reference the original class name, resolving to the original file path
791-
logger.debug(f"[PARSE-XML] Looking up test_type by original_file_path: {test_file_path}")
792774
test_type = test_files.get_test_type_by_original_file_path(test_file_path)
793-
logger.debug(f"[PARSE-XML] Lookup by original path result: {test_type}")
794775
if test_type is None:
795776
# Log registered paths for debugging
796777
registered_paths = [str(tf.instrumented_behavior_file_path) for tf in test_files.test_files]

0 commit comments

Comments
 (0)