badges / badges/shields

Improve BaseService logging

Open
#1,887 3 comments 2 reactions 0 assignees View on GitHub
core developer-experience
Dominant language
JavaScript
Stars
27.2k
Forks
5.6k
Avg merge
1d 8h
Merged PRs (30d)
46

Description

* Show logs on test failure but suppress them if the test passes
* Make log target an injected dependency (so we can capture it, write to a file, etc)
* Pass `logDebug()` a log level and make use of different logging levels (`console.info()`, `console.error()`, `console.warn()` etc)

Contributor guide

Open the contributing guide

Research direction

Start by locating BaseService and the tests that exercise its logging behavior. Review how logs are currently emitted, then verify the three requested outcomes: failed tests show logs while passing tests suppress them, the log target can be injected, and logDebug() supports distinct logging levels.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
observability
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.