Skip to content

Add admin server tests and adapter interfaces#20

Open
fabricekabongo wants to merge 1 commit into
mainfrom
codex/increase-test-coverage-in-admin
Open

Add admin server tests and adapter interfaces#20
fabricekabongo wants to merge 1 commit into
mainfrom
codex/increase-test-coverage-in-admin

Conversation

@fabricekabongo
Copy link
Copy Markdown
Owner

Summary

  • add cluster and broadcast interfaces plus adapter to make the admin server testable
  • add unit tests covering admin data and UI handlers
  • update main to construct the admin server with the new adapter

Testing

  • go test ./...

Codex Task

@deepsource-io
Copy link
Copy Markdown
Contributor

deepsource-io Bot commented Nov 29, 2025

Here's the code health analysis summary for commits a8cfbf4..5f0d2fa. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Docker LogoDocker✅ SuccessView Check ↗
DeepSource Secrets LogoSecrets✅ SuccessView Check ↗
DeepSource Test coverage LogoTest coverage❌ Failure
❗ 14 occurences introduced
View Check ↗
DeepSource Go LogoGo❌ Failure
❗ 4 occurences introduced
View Check ↗

Code Coverage Report

MetricAggregateGo
Composite Coverage63.7% (up 3% from main)63.7% (up 3% from main)
Line Coverage63.7% (up 3% from main)63.7% (up 3% from main)
New Composite Coverage9.1%9.1%
New Line Coverage9.1%9.1%

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant