Normalize test titles

Issue #2774

Author: KillyMXI

lib/parse-test-args.js

@@ -1,5 +1,5 @@
 export default function parseTestArgs(args) {
-	const rawTitle = typeof args[0] === 'string' ? args.shift() : undefined;
+	const rawTitle = typeof args[0] === 'string' ? args.shift().trim().replace(/\s+/g, ' ') : undefined;
 	const receivedImplementationArray = Array.isArray(args[0]);
 	const implementations = receivedImplementationArray ? args.shift() : args.splice(0, 1);
 

test/snapshot-tests/fixtures/normalized-title-in-snapshots/package.json

@@ -0,0 +1 @@
+{}

test/snapshot-tests/fixtures/normalized-title-in-snapshots/test.js

@@ -0,0 +1,5 @@
+const test = require(process.env.TEST_AVA_IMPORT_FROM);
+
+test(process.env.TEMPLATE ? 'test\r\n\ttitle' : '  test title  ', t => {
+	t.snapshot('Hello, World!');
+});

test/snapshot-tests/fixtures/normalized-title-in-snapshots/test.js.md

@@ -0,0 +1,11 @@
+# Snapshot report for `test.js`
+
+The actual snapshot is saved in `test.js.snap`.
+
+Generated by [AVA](https://avajs.dev).
+
+## test title
+
+> Snapshot 1
+
+    'Hello, World!'

test/snapshot-tests/fixtures/normalized-title-in-snapshots/test.js.snap

@@ -0,0 +1 @@
+{}

test/snapshot-tests/fixtures/normalized-title-in-stdout/package.json

@@ -0,0 +1,15 @@
+const test = require(process.env.TEST_AVA_IMPORT_FROM);
+
+test(`a rather wordy test title that is wrapped
+			to  meet  line  length  requirements   in
+			an unconventional way  that may interfere
+			with   report   formatting               `, t => {
+	t.pass();
+});
+
+test(`multiline try assertion title`, async t => {
+	const firstTry = await t.try(`try assertions
+																can have titles too`, (tt) => tt.pass());
+	firstTry.commit();
+	t.log(firstTry.title);
+});

test/snapshot-tests/fixtures/normalized-title-in-stdout/test.js

@@ -5,6 +5,7 @@ import test from '@ava/test';
 
 import {cwd, fixture} from '../helpers/exec.js';
 import {withTemporaryFixture} from '../helpers/with-temporary-fixture.js';
+import {beforeAndAfter} from '../snapshot-workflow/helpers/macros.js';
 
 test('multiline snapshot label should be formatted correctly in the report', async t => {
 	await withTemporaryFixture(cwd('multiline-snapshot-label'), async cwd => {
@@ -22,3 +23,28 @@ test('multiline snapshot label should be formatted correctly in the report', asy
 		t.snapshot(report, 'resulting snapshot report');
 	});
 });
+
+test('test title should be normalized in stdout', async t => {
+	await withTemporaryFixture(cwd('normalized-title-in-stdout'), async cwd => {
+		// Run test fixture
+		const result = await fixture(['--update-snapshots'], {
+			cwd,
+			env: {
+				AVA_FORCE_CI: 'not-ci'
+			}
+		});
+
+		// Assert stdout is unchanged
+		t.snapshot(result.stdout, 'stdout');
+	});
+});
+
+test(
+	'test title should be normalized in snapshot',
+	beforeAndAfter,
+	{
+		cwd: cwd('normalized-title-in-snapshots'),
+		cli: ['--update-snapshots'],
+		expectChanged: false
+	}
+);

test/snapshot-tests/formatting.js

@@ -27,3 +27,15 @@ Generated by [AVA](https://avajs.dev).
     ␊
         'Hello, World!'␊
     `
+
+## test title should be normalized in stdout
+
+> stdout
+
+    `␍␊
+      √ a rather wordy test title that is wrapped to meet line length requirements in an unconventional way that may interfere with report formatting␍␊
+      √ multiline try assertion title␍␊
+        i multiline try assertion title ─ try assertions can have titles too␍␊
+      ─␍␊
+    ␍␊
+      2 tests passed`