Skip to content

tests: more coverage (UrlValidator, cache stats, settings) [clean]#146

Closed
RicherTunes wants to merge 1 commit intomainfrom
chore/coverage-35-push-clean
Closed

tests: more coverage (UrlValidator, cache stats, settings) [clean]#146
RicherTunes wants to merge 1 commit intomainfrom
chore/coverage-35-push-clean

Conversation

@RicherTunes
Copy link
Owner

@RicherTunes RicherTunes commented Sep 13, 2025

Reopened as clean, tests-only PR:

  • UrlValidator: dangerous schemes, local URL validation, scheme inference for local, IPv6 and port bounds.
  • ConcurrentCache: hits/misses tracking, remove/clear sizing.
  • BrainarrSettings property tests: API key trimming/length; local URL normalization; model set/get.

No production changes.

Additional coverage:

  • ConcurrentCache: added tests for hits and misses tracking.
  • ConcurrentCache: added tests for Remove and Clear methods updating cache size.

No production changes.

🌿 Generated by Terry


ℹ️ Tag @terragon-labs to ask questions and address PR feedback

📎 Task: https://www.terragonlabs.com/task/e9bb08d9-44a7-4400-b3e7-2e4495f3e2ac

@RicherTunes
Copy link
Owner Author

Closing this stale PR from September 2025. The test coverage additions have multiple build and test failures, indicating significant code drift from the current main branch. If these test improvements are still desired, please create a fresh PR based on the current codebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant