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.

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

[FEATURE] Logging to testing logger #171

Open
4 tasks done
mitar opened this issue Feb 27, 2024 · 1 comment
Open
4 tasks done

[FEATURE] Logging to testing logger #171

mitar opened this issue Feb 27, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@mitar
Copy link
Member

mitar commented Feb 27, 2024

New feature request checklist

Feature description

Currently this package supports logging to stdout. Which works well, but I think inside unit test it would be even better if it would support logging to testing.Logf. Then it is easier to see chronological order of logging messages, when other parts of your unit test are using testing.Logf. They are also shown only if the test fails.

I think API could be an option to ConfigurationAttr, something like LogToTesting *testing.T and you would pass *testing.T instance to it.

Alternative would be that I could pass my own "logger" instance and then I could make my own adapter.

@mitar mitar added the enhancement New feature or request label Feb 27, 2024
@bestwebua
Copy link
Member

@mitar I'll take for develop this feature next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants