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#6559 known_issues
  does run on CI.
* Add some notes to explain the expectations around tests in
  known_issues.

PR-URL: nodejs#12262
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
  • Loading branch information
Jan Krems authored and italoacasas committed Apr 10, 2017
1 parent 0f4319a commit 2f9e2fc
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 2f9e2fc

Please sign in to comment.