Expose current test instance to `test.meta`

[szmarczak]

In last release (1.1.0) there was added test.meta feature.

I'd like to propose test.meta.currentTest which points to the current test instance. Use case:

-const compare = (t, entries, goal) => {
+const compare = (entries, goal) => {
+   const t = test.meta.currentTest;
	if (Array.isArray(entries)) {
		entries = entries.map(entry => {
			t.true(typeof entry.expires === 'number');
			return {
				address: entry.address,
				family: entry.family
			};
		});
	} else {
		delete entries.expires;
	}

	t.deepEqual(entries, goal);
};

test('demo', t => {
	const entries = getEntries();
-	compare(t, entries, [
+   compare(entries, [
		{address: '1.2.3.4', family: 4},
		{address: }
	]);
});

[novemberborn]

I don't think that would work for asynchronous tests: multiple tests would be running concurrently so there wouldn't be any one currentTest.

Besides the code sample, could you elaborate on your use case?

[szmarczak]

I don't think that would work for asynchronous tests: multiple tests would be running concurrently so there wouldn't be any one currentTest.

Hmm, that's true, unfortunately.

Besides the code sample, could you elaborate on your use case?

You just need to pass the test instance every time, which is quite irritating. I often did compare(entries, ...) instead of compare(t, entries, ...).

[novemberborn]

We are open to being able to install additional assertions on t: #1094

[szmarczak]

@novemberborn That whould be the right thing :)