Tests are run twice.

[Xiaoy312]

when using a separate assembly for tests and said assembly name ends with "tests",
the tests are run twice.

var testAssembliesTypes =
	from asm in AppDomain.CurrentDomain.GetAssemblies()
	where asm.GetName()?.Name?.EndsWith("tests", StringComparison.OrdinalIgnoreCase) ?? false
	from type in asm.GetTypes()
	select type;

var types = GetType().GetTypeInfo().Assembly.GetTypes().Concat(testAssembliesTypes);

^ AppDomain.CurrentDomain.GetAssemblies() definitely contains GetType().GetTypeInfo().Assembly.GetTypes()

[Xiaoy312]

		var testAssemblies = AppDomain.CurrentDomain.GetAssemblies()
			.Where(x => x.GetName()?.Name?.EndsWith("Tests", StringComparison.OrdinalIgnoreCase) ?? false)
			.Concat(new[] { GetType().GetTypeInfo().Assembly })
			.Distinct();
		var types = testAssemblies.SelectMany(x => x.GetTypes());

[jeromelaban]

Interesting, this would mean we have a similar issue inside of Uno.UI. (unless there's a distinct somewhere after as well)

[Xiaoy312]

In uno.ui, this is avoided because the UnitTestsControl is in the SampleApps, so the GetType().GetTypeInfo().Assembly (SampleApp assembly) there contains no tests. And, the tests are in Uno.UI.RuntimeTests (separate assembly).
So there is no double inclusion there.

[jeromelaban]

Makes sense!