kuhyx/testsAndMisc
1
0
Fork 0
mirror of https://github.com/kuhyx/testsAndMisc.git synced 2026-07-04 14:23:16 +02:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

refactor(randomize_numbers): use list comprehension to achieve 100% coverage

Browse Source 
Replaced for loop with implicit continue with a list comprehension
using walrus operator. This avoids the coverage.py limitation with
partial branch detection on for loop continue statements.

Coverage: 99% -> 100%
This commit is contained in:
Krzysztof kuhy Rudnicki 2025-12-01 20:19:02 +01:00
parent 57233aa8d5
commit 1f20f727d0
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
python_pkg/randomize_numbers/random_digits.py
View File

@ -39,15 +39,14 @@ def parse_input(text: str) -> tuple[list[float], list[int]]:
cleaned_input = re.sub(r"[^\d.,\s]", "", text).replace(",", ".")
# Split the cleaned input into individual numbers
number_strings = cleaned_input.split()
# Convert the number strings to floats
nums: list[float] = []
decimals: list[int] = []
for num_str in number_strings:
parsed = _parse_single_number(num_str)
if parsed is not None:
float_num, digits_count = parsed
nums.append(float_num)
decimals.append(digits_count)
# Convert the number strings to floats, filtering out invalid ones
parsed_results = [
result
for num_str in number_strings
if (result := _parse_single_number(num_str)) is not None
]
nums = [float_num for float_num, _ in parsed_results]
decimals = [digits_count for _, digits_count in parsed_results]
return nums, decimals