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

Using it for log analysis instead of log suppressing #6

Merged
merged 3 commits into from
Dec 18, 2018

Conversation

NoriSte
Copy link
Contributor

@NoriSte NoriSte commented Nov 13, 2018

As explained here #5
An example of using this library to read the logs instead of suppressing them, useful when you're fighting with a hidden (by Jest) logs and a failing test.

Readme.md Outdated
@@ -100,6 +101,32 @@ describe(...
)
```

## Analyze all logs

In some circumstances Jest is infamouse for logging because it overwrites the bash output. If you're experiencing this kind of logging frustration and you need to analyze all the `console.log` call arguments (tipically to understand why a test is failing) you can temporary add an expect statement on the console itself.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few spelling errors.
"infamouse" should be "infamous"
"tipically" should be "typically"
"temporary" should be "temporarily"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh sorry, I've just fixed them!

@bpedersen
Copy link
Owner

I will fix these tests this weeknd so we can get this in

typos fixing
@codecov-io
Copy link

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #6   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines          25     25           
  Branches        3      3           
=====================================
  Hits           25     25

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 859b20d...0da9153. Read the comment docs.

@bpedersen bpedersen merged commit 2482012 into bpedersen:master Dec 18, 2018
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

Successfully merging this pull request may close these issues.

3 participants