summaryrefslogtreecommitdiff
path: root/doc/guides
diff options
context:
space:
mode:
authorVse Mozhet Byt <vsemozhetbyt@gmail.com>2019-03-09 13:53:31 +0200
committerVse Mozhet Byt <vsemozhetbyt@gmail.com>2019-03-09 14:17:16 +0200
commit4e9dc31817498ae03fcf7c2136d5d69b863a135c (patch)
treed1e0d380530087e9b8ae8b918b28b88583746407 /doc/guides
parent97ce5e0b4b614872ced69ba800bc726d464e92f2 (diff)
downloadandroid-node-v8-4e9dc31817498ae03fcf7c2136d5d69b863a135c.tar.gz
android-node-v8-4e9dc31817498ae03fcf7c2136d5d69b863a135c.tar.bz2
android-node-v8-4e9dc31817498ae03fcf7c2136d5d69b863a135c.zip
doc: fix nits in writing-tests.md
* Correct a typo. * Normalize spaces. * Split a paragraph with different topics. PR-URL: https://github.com/nodejs/node/pull/26543 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Diffstat (limited to 'doc/guides')
-rw-r--r--doc/guides/writing-tests.md12
1 files changed, 7 insertions, 5 deletions
diff --git a/doc/guides/writing-tests.md b/doc/guides/writing-tests.md
index 1c8d8daf74..7b1d168d85 100644
--- a/doc/guides/writing-tests.md
+++ b/doc/guides/writing-tests.md
@@ -285,19 +285,21 @@ assert.throws(
Output written by tests to stdout or stderr, such as with `console.log()` or
`console.error()`, can be useful when writing tests, as well as for debugging
-them during later maintenance. The output will be supressed by the test runner
+them during later maintenance. The output will be suppressed by the test runner
(`./tools/test.py`) unless the test fails, but will always be displayed when
running tests directly with `node`. For failing tests, the test runner will
include the output along with the failed test assertion in the test report.
Some output can help debugging by giving context to test failures. For example,
when troubleshooting tests that timeout in CI. With no log statements, we have
-no idea where the test got hung up. There have been cases where tests fail
-without `console.log()`, and then pass when its added, so be cautious about its
-use, particularly in tests of the I/O and streaming APIs.
+no idea where the test got hung up.
+
+There have been cases where tests fail without `console.log()`, and then pass
+when its added, so be cautious about its use, particularly in tests of the I/O
+and streaming APIs.
Excessive use of console output is discouraged as it can overwhelm the display,
-including the Jenkins console and test report displays. Be particularly
+including the Jenkins console and test report displays. Be particularly
cautious of output in loops, or other contexts where output may be repeated many
times in the case of failure.