Skip to content

[BUGFIX] Initialize test variables before appending to them - #2234

Merged
sebastian-meyer merged 1 commit into
kitodo:mainfrom
stweil:fix/codacy-undefined-variables
Oct 27, 2026
Merged

sebastian-meyer merged 1 commit into
kitodo:mainfrom
stweil:fix/codacy-undefined-variables

Conversation

@stweil

@stweil stweil commented Oct 26, 2026

Copy link
Copy Markdown
Member

Avoid using undefined variables in the canSearchInCollections() functional test to satisfy the UndefinedVariable code pattern (Codacy issues).

Assisted-by: OpenCode / qwen3.8-27b-thinking (Alibaba Cloud)

Avoid using undefined variables in the canSearchInCollections()
functional test to satisfy the UndefinedVariable code pattern.

Assisted-by: OpenCode / qwen3.8-27b-thinking (Alibaba Cloud)
Signed-off-by: Stefan Weil <sw@weilnetz.de>
@sebastian-meyer sebastian-meyer added the ☇ bug A non-security related bug. label Oct 27, 2026
@sebastian-meyer
sebastian-meyer merged commit 07c0077 into kitodo:main Oct 27, 2026
10 checks passed
@codecov

codecov Bot commented Oct 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (edf8b13) to head (9d862a8).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff      @@
##   main   #2234   +/-   ##
============================
============================

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@stweil
stweil deleted the fix/codacy-undefined-variables branch October 27, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

☇ bug A non-security related bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants