Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Utilize Logging framework in tests #2222

Open
Crumby opened this issue Mar 12, 2025 · 0 comments
Open

Utilize Logging framework in tests #2222

Crumby opened this issue Mar 12, 2025 · 0 comments
Labels
test Test related problem or fix

Comments

@Crumby
Copy link
Collaborator

Crumby commented Mar 12, 2025

All the tests currently write directly to standard output. Migrating the message to any logging framework would help with debugging and it'd reduce overall clutter generated by tests.

Logging Requirements:

  • Time
  • Configurable Level (Debug, Info)
  • File Source

Frameworks for consideration:

  • go-logr/zap - used in the main code
  • slog - Structured Logging introduced as a standard library in Go 1.21
@Crumby Crumby added the test Test related problem or fix label Mar 12, 2025
@Crumby Crumby changed the title Utilize Logging in tests Utilize Logging framework in tests Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
test Test related problem or fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Crumby