Skip to content

Commit

Permalink
doc: update information on test/known_issues
Browse files Browse the repository at this point in the history
* Since nodejs/node#6559 known_issues
  does run on CI.
* Add some notes to explain the expectations around tests in
  known_issues.

PR-URL: nodejs/node#12262
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Alexey Orlenko <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
  • Loading branch information
Jan Krems authored and andrew749 committed Jul 19, 2017
1 parent 7b2a4bf commit 1cb3cea
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions test/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,12 @@ On how to run tests in this direcotry, see
</tr>
<tr>
<td>known_issues</td>
<td>No</td>
<td>Tests reproducing known issues within the system.</td>
<td>Yes</td>
<td>
Tests reproducing known issues within the system. All tests inside of
this directory are expected to fail consistently. If a test doesn't fail
on certain platforms, those should be skipped via `known_issues.status`.
</td>
</tr>
<tr>
<td>message</td>
Expand Down

0 comments on commit 1cb3cea

Please sign in to comment.