gistfile1.cpp

SCENARIO("...", "[...]")
{
	QEventLoop eventLoop;
	MockRepository mockRepo;
	
	GIVEN("An exporter") {
		auto exporterAndMocks = setUpExporterAndMocks();
		auto exporter = exporterAndMocks.first;
		auto mocks = exporterAndMocks.second;

		auto dataConverter = get<TupleDataConverter>(mocks);
		mockRepo.ExpectCall(dataConverter, DataConverter::convert).Do([] { return vector<unique_ptr<Record>>(); });

		WHEN("`exportAsync()` is called.") {
			exporter->exportAsync(getStuff(), [&] {
				eventLoop.exit();
			});
			eventLoop.exec();

			THEN("the data converter should be used to export data.") {
				REQUIRE(true);
			}
		}
	}
}

I find the REQUIRE(true) a bit misleading; I would have expected something like REQUIRE(mockRepo.validate()) or so...

Good point. I'll see if my mocking framework supports something like this. I've just relied on the fact that Catch invalidates the test if an exception is thrown, even if previous REQUIRE()'s passed. Certainly misleading, as you say.