Commit 8311561
test: improve assertion messages
Print content of domain stack if it doesn't match expected values
PR-URL: #16885
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>1 parent 38ac50a commit 8311561
1 file changed
Lines changed: 6 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -20,18 +20,21 @@ | |||
| 20 | 20 | // USE OR OTHER DEALINGS IN THE SOFTWARE. | |
| 21 | 21 | ||
| 22 | 22 | 'use strict'; | |
| 23 | + require('../common'); | ||
| 23 | 24 | // Make sure the domain stack doesn't get clobbered by un-matched .exit() | |
| 24 | 25 | ||
| 25 | - require('../common'); | ||
| 26 | 26 | const assert = require('assert'); | |
| 27 | 27 | const domain = require('domain'); | |
| 28 | + const util = require('util'); | ||
| 28 | 29 | ||
| 29 | 30 | const a = domain.create(); | |
| 30 | 31 | const b = domain.create(); | |
| 31 | 32 | ||
| 32 | 33 | a.enter(); // push | |
| 33 | 34 | b.enter(); // push | |
| 34 | - assert.deepStrictEqual(domain._stack, [a, b], 'b not pushed'); | ||
| 35 | + assert.deepStrictEqual(domain._stack, [a, b], 'Unexpected stack shape ' + | ||
| 36 | + `(domain._stack = ${util.inspect(domain._stack)})`); | ||
| 35 | 37 | ||
| 36 | 38 | domain.create().exit(); // no-op | |
| 37 | - assert.deepStrictEqual(domain._stack, [a, b], 'stack mangled!'); | ||
| 39 | + assert.deepStrictEqual(domain._stack, [a, b], 'Unexpected stack shape ' + | ||
| 40 | + `(domain._stack = ${util.inspect(domain._stack)})`); | ||
0 commit comments