test: add util.inspect test for null maxStringLength

Trott · BethGriggs · commit b53068ec0de7 · 2020-12-15T20:15:19.000Z
Add test case to cover currently-uncovered code. Refs: https://coverage.nodejs.org/coverage-39a7f7663e8f70fc/lib/internal/util/inspect.js.html#L333 PR-URL: #36086 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
diff --git a/test/parallel/test-util-inspect.js b/test/parallel/test-util-inspect.js
@@ -2905,6 +2905,7 @@ assert.strictEqual(
     util.inspect(x, { maxStringLength: 4 }),
     "'aaaa'... 999996 more characters"
   );
+  assert.match(util.inspect(x, { maxStringLength: null }), /a'$/);
 }
 
 {

Original file line number	Diff line number	Diff line change
`@@ -2905,6 +2905,7 @@ assert.strictEqual(`
`2905`	`2905`	`util.inspect(x, { maxStringLength: 4 }),`
`2906`	`2906`	`"'aaaa'... 999996 more characters"`
`2907`	`2907`	`);`
	`2908`	`+ assert.match(util.inspect(x, { maxStringLength: null }), /a'$/);`
`2908`	`2909`	`}`
`2909`	`2910`
`2910`	`2911`	`{`