Commit fa8a394
doc: specify sorted requires in tests
And allow anchor links to the various sections, to more easily reference
them when reviewing PRs.
PR-URL: #10716
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>1 parent 168f3e4 commit fa8a394
1 file changed
Lines changed: 7 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -29,8 +29,8 @@ Let's analyze this basic test from the Node.js test suite: | |||
| 29 | 29 | 4 // This test ensures that the http-parser can handle UTF-8 characters | |
| 30 | 30 | 5 // in the http header. | |
| 31 | 31 | 6 | |
| 32 | - 7 const http = require('http'); | ||
| 33 | - 8 const assert = require('assert'); | ||
| 32 | + 7 const assert = require('assert'); | ||
| 33 | + 8 const http = require('http'); | ||
| 34 | 34 | 9 | |
| 35 | 35 | 10 const server = http.createServer(common.mustCall((req, res) => { | |
| 36 | 36 | 11 res.end('ok'); | |
@@ -46,7 +46,7 @@ Let's analyze this basic test from the Node.js test suite: | |||
| 46 | 46 | 21 }); | |
| 47 | 47 | ``` | |
| 48 | 48 | ||
| 49 | - **Lines 1-2** | ||
| 49 | + ### **Lines 1-2** | ||
| 50 | 50 | ||
| 51 | 51 | ```javascript | |
| 52 | 52 | 'use strict'; | |
@@ -70,7 +70,7 @@ assigning it to an identifier: | |||
| 70 | 70 | require('../common'); | |
| 71 | 71 | ``` | |
| 72 | 72 | ||
| 73 | - **Lines 4-5** | ||
| 73 | + ### **Lines 4-5** | ||
| 74 | 74 | ||
| 75 | 75 | ```javascript | |
| 76 | 76 | // This test ensures that the http-parser can handle UTF-8 characters | |
@@ -80,11 +80,11 @@ require('../common'); | |||
| 80 | 80 | A test should start with a comment containing a brief description of what it is | |
| 81 | 81 | designed to test. | |
| 82 | 82 | ||
| 83 | - **Lines 7-8** | ||
| 83 | + ### **Lines 7-8** | ||
| 84 | 84 | ||
| 85 | 85 | ```javascript | |
| 86 | - const http = require('http'); | ||
| 87 | 86 | const assert = require('assert'); | |
| 87 | + const http = require('http'); | ||
| 88 | 88 | ``` | |
| 89 | 89 | ||
| 90 | 90 | The test checks functionality in the `http` module. | |
@@ -95,7 +95,7 @@ The require statements are sorted in | |||
| 95 | 95 | [ASCII](http://man7.org/linux/man-pages/man7/ascii.7.html) order (digits, upper | |
| 96 | 96 | case, `_`, lower case). | |
| 97 | 97 | ||
| 98 | - **Lines 10-21** | ||
| 98 | + ### **Lines 10-21** | ||
| 99 | 99 | ||
| 100 | 100 | This is the body of the test. This test is simple, it just tests that an | |
| 101 | 101 | HTTP server accepts `non-ASCII` characters in the headers of an incoming | |
0 commit comments