Skip to content

Commit 97e24ce

Browse files
committed
fix for duplicated err count print
1 parent 9466f3c commit 97e24ce

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

nodescraper/plugins/inband/package/package_analyzer.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ def analyze_data(
236236

237237
total_errors = len(not_found_keys) + len(regex_errors) + len(version_mismatches)
238238
if total_errors > 0:
239-
self.result.message = f"{'; '.join(error_parts)} ({total_errors} error{'s' if total_errors != 1 else ''})"
239+
self.result.message = f"{'; '.join(error_parts)}"
240240
self.result.status = ExecutionStatus.ERROR
241241
else:
242242
self.result.message = "All packages found and versions matched"

test/unit/plugin/test_package_analyzer.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,5 +107,5 @@ def test_data_multiple_errors_regex(package_analyzer, default_data_lib):
107107
assert res.status == ExecutionStatus.ERROR
108108
assert "missing-package" in res.message
109109
assert "another-missing" in res.message
110-
assert "test-ubuntu-package.x86_64" in res.message
111-
assert "3 error" in res.message
110+
# Note: error count is automatically added by TaskResult.finalize() based on logged events
111+
assert len(res.events) == 3 # Should have 3 error events logged

0 commit comments

Comments
 (0)