`getName()` should be called with the test it is naming as context

[LeaVerou]

I had a file testing various utils with the structure:

import { foo, bar, baz } from "../src/util.js";

export default {
	tests: [
		{ name: "foo()", run: foo, tests: [...] },
		{ name: "bar()", run: bar, tests: [...] },
		// ...
	]
}

So I thought, 💡 I can just use this.run.name and generate their names!

import { foo, bar, baz } from "../src/util.js";

export default {
	getName() {
		return this.run.name + "()";
	}
	tests: [
		{ name: "foo()", run: foo, tests: [...] },
		{ name: "bar()", run: bar, tests: [...] },
		// ...
	]
}

However, this did not work because this is the top-level test, which is useless.

Same with all the other getXXX() functions.

[DmitrySharabin]

I'm surprised that it didn't work. We are already passing the right context to the getXXX() functions.

Here are the tests I run with hTest (and make them fail to see their names in the output):

import { stringify } from "../src/util.js";

function foo () {
	return this.getName() === "foo()";
}

function bar () {
	return this.getName() === "bar()";
}

export default {
	name: "Inherited getters",
	tests: [
		{
			name: "getName()",
			getName () {
				return this.run.name + "()";
			},
			run: function dummy () {
				return 42;
			},
			expect: false,
			tests: [
				{
					run: foo,
				},
				{
					name: "Nested test",
					tests: [
						{
							run: bar,
						},
						{
							description: "It should also work with imported functions. Make this test fail to see if the test name is correct.",
							run: stringify,
							arg: new Set([1, 2, 3]),
							expect: "Set(3) {1, 2, 42}",
						},
					],
				},
			],
		},
	],
};

What am I missing?