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

Add test logger #12

Open
nofun97 opened this issue Jan 30, 2020 · 1 comment
Open

Add test logger #12

nofun97 opened this issue Jan 30, 2020 · 1 comment

Comments

@nofun97
Copy link
Member

nofun97 commented Jan 30, 2020

Add test logger for testing the log messages of code. The API needs to be something that can be used to assert values instead of just recording the whole log. Current plan:

  1. Have it be able to assert Fields
  2. Assert message
  3. Assert time (not important but nice to have)
  4. Assert codelinker (after Add codelinker #10)
  5. Assert level
@nofun97
Copy link
Member Author

nofun97 commented Mar 5, 2020

When this is fixed, feature from the PR #32 will have to be reverted. SetOutput creates problems as some loggers may not have an output (e.g. null logger).

anzopensource pushed a commit that referenced this issue Feb 17, 2021
* Removes contrib.go.opencensus.io/exporter/prometheus dependency

The contrib.go.opencensus.io/exporter/[email protected] dependency transitively
includes the github.com/gogo/[email protected] dependency which has been flagged
as a potential security issue that was resolved in v1.3.2.

The contrib.go.opencensus.io/exporter/[email protected] dependency included
here is only used to provide an example code snippet and, as of the time of
writing, this repository does not provide a version that removes the offending
gogo/protobuf version.

https://github.com/gogo/protobuf/releases/tag/v1.3.2

* Updates go.mod with tidy
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

No branches or pull requests

1 participant