integration_tests: logger cleanup#599
Conversation
product-security-plaid-production
Bot
requested review from
ettec and
yashnevatia
|
👋 jmank88, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
|
| defer func() { | ||
| utils.CleanupCapabilitiesDir(lggr) | ||
| }() | ||
| lggr, obs := logger.TestObserved(t, zapcore.InfoLevel) // change this to debug to print all logs from the trigger/log poller if needed to debug |
There was a problem hiding this comment.
Somehow this does not have parity with the core version 🤔
|
This PR is stale because it has been open 30 days with no activity. |
|
This PR has been automatically closed because it has been stale for > 30 days. |
3 participants
Swap out core logger for common. Replace
CleanupCapabilitiesDirwithRemoveCapabilitiesDir.