Browse Source

Test: rename Any* plugin test cases for clarity.

pull/527/head
Vladimír Vondruš 5 years ago
parent
commit
22d1484814
  1. 6
      src/MagnumPlugins/AnyImageConverter/Test/AnyImageConverterTest.cpp
  2. 6
      src/MagnumPlugins/AnyImageImporter/Test/AnyImageImporterTest.cpp
  3. 12
      src/MagnumPlugins/AnySceneConverter/Test/AnySceneConverterTest.cpp
  4. 6
      src/MagnumPlugins/AnySceneImporter/Test/AnySceneImporterTest.cpp

6
src/MagnumPlugins/AnyImageConverter/Test/AnyImageConverterTest.cpp

@ -48,7 +48,7 @@ struct AnyImageConverterTest: TestSuite::Tester {
void unknown(); void unknown();
void verbose(); void propagateFlags();
/* Explicitly forbid system-wide plugin dependencies */ /* Explicitly forbid system-wide plugin dependencies */
PluginManager::Manager<AbstractImageConverter> _manager{"nonexistent"}; PluginManager::Manager<AbstractImageConverter> _manager{"nonexistent"};
@ -84,7 +84,7 @@ AnyImageConverterTest::AnyImageConverterTest() {
addTests({&AnyImageConverterTest::unknown, addTests({&AnyImageConverterTest::unknown,
&AnyImageConverterTest::verbose}); &AnyImageConverterTest::propagateFlags});
/* Load the plugin directly from the build tree. Otherwise it's static and /* Load the plugin directly from the build tree. Otherwise it's static and
already loaded. */ already loaded. */
@ -155,7 +155,7 @@ void AnyImageConverterTest::unknown() {
CORRADE_COMPARE(output.str(), "Trade::AnyImageConverter::convertToFile(): cannot determine the format of image.xcf\n"); CORRADE_COMPARE(output.str(), "Trade::AnyImageConverter::convertToFile(): cannot determine the format of image.xcf\n");
} }
void AnyImageConverterTest::verbose() { void AnyImageConverterTest::propagateFlags() {
if(!(_manager.loadState("TgaImageConverter") & PluginManager::LoadState::Loaded)) if(!(_manager.loadState("TgaImageConverter") & PluginManager::LoadState::Loaded))
CORRADE_SKIP("TgaImageConverter plugin not enabled, cannot test"); CORRADE_SKIP("TgaImageConverter plugin not enabled, cannot test");

6
src/MagnumPlugins/AnyImageImporter/Test/AnyImageImporterTest.cpp

@ -49,7 +49,7 @@ struct AnyImageImporterTest: TestSuite::Tester {
void unknownSignature(); void unknownSignature();
void emptyData(); void emptyData();
void verbose(); void propagateFlags();
/* Explicitly forbid system-wide plugin dependencies */ /* Explicitly forbid system-wide plugin dependencies */
PluginManager::Manager<AbstractImporter> _manager{"nonexistent"}; PluginManager::Manager<AbstractImporter> _manager{"nonexistent"};
@ -130,7 +130,7 @@ AnyImageImporterTest::AnyImageImporterTest() {
addTests({&AnyImageImporterTest::emptyData}); addTests({&AnyImageImporterTest::emptyData});
addInstancedTests({&AnyImageImporterTest::verbose}, addInstancedTests({&AnyImageImporterTest::propagateFlags},
Containers::arraySize(LoadData)); Containers::arraySize(LoadData));
/* Load the plugin directly from the build tree. Otherwise it's static and /* Load the plugin directly from the build tree. Otherwise it's static and
@ -222,7 +222,7 @@ void AnyImageImporterTest::emptyData() {
CORRADE_COMPARE(output.str(), "Trade::AnyImageImporter::openData(): file is empty\n"); CORRADE_COMPARE(output.str(), "Trade::AnyImageImporter::openData(): file is empty\n");
} }
void AnyImageImporterTest::verbose() { void AnyImageImporterTest::propagateFlags() {
auto&& data = LoadData[testCaseInstanceId()]; auto&& data = LoadData[testCaseInstanceId()];
setTestCaseDescription(data.name); setTestCaseDescription(data.name);

12
src/MagnumPlugins/AnySceneConverter/Test/AnySceneConverterTest.cpp

@ -41,12 +41,12 @@ namespace Magnum { namespace Trade { namespace Test { namespace {
struct AnySceneConverterTest: TestSuite::Tester { struct AnySceneConverterTest: TestSuite::Tester {
explicit AnySceneConverterTest(); explicit AnySceneConverterTest();
void load(); void convert();
void detect(); void detect();
void unknown(); void unknown();
void verbose(); void propagateFlags();
/* Explicitly forbid system-wide plugin dependencies */ /* Explicitly forbid system-wide plugin dependencies */
PluginManager::Manager<AbstractSceneConverter> _manager{"nonexistent"}; PluginManager::Manager<AbstractSceneConverter> _manager{"nonexistent"};
@ -62,14 +62,14 @@ constexpr struct {
}; };
AnySceneConverterTest::AnySceneConverterTest() { AnySceneConverterTest::AnySceneConverterTest() {
addTests({&AnySceneConverterTest::load}); addTests({&AnySceneConverterTest::convert});
addInstancedTests({&AnySceneConverterTest::detect}, addInstancedTests({&AnySceneConverterTest::detect},
Containers::arraySize(DetectData)); Containers::arraySize(DetectData));
addTests({&AnySceneConverterTest::unknown, addTests({&AnySceneConverterTest::unknown,
&AnySceneConverterTest::verbose}); &AnySceneConverterTest::propagateFlags});
/* Load the plugin directly from the build tree. Otherwise it's static and /* Load the plugin directly from the build tree. Otherwise it's static and
already loaded. */ already loaded. */
@ -81,7 +81,7 @@ AnySceneConverterTest::AnySceneConverterTest() {
CORRADE_INTERNAL_ASSERT_OUTPUT(Utility::Directory::mkpath(ANYSCENECONVERTER_TEST_OUTPUT_DIR)); CORRADE_INTERNAL_ASSERT_OUTPUT(Utility::Directory::mkpath(ANYSCENECONVERTER_TEST_OUTPUT_DIR));
} }
void AnySceneConverterTest::load() { void AnySceneConverterTest::convert() {
CORRADE_SKIP("No scene converter plugin available to test."); CORRADE_SKIP("No scene converter plugin available to test.");
} }
@ -114,7 +114,7 @@ void AnySceneConverterTest::unknown() {
CORRADE_COMPARE(output.str(), "Trade::AnySceneConverter::convertToFile(): cannot determine the format of mesh.obj\n"); CORRADE_COMPARE(output.str(), "Trade::AnySceneConverter::convertToFile(): cannot determine the format of mesh.obj\n");
} }
void AnySceneConverterTest::verbose() { void AnySceneConverterTest::propagateFlags() {
CORRADE_SKIP("No plugin available to test."); CORRADE_SKIP("No plugin available to test.");
} }

6
src/MagnumPlugins/AnySceneImporter/Test/AnySceneImporterTest.cpp

@ -56,7 +56,7 @@ struct AnySceneImporterTest: TestSuite::Tester {
void unknown(); void unknown();
void verbose(); void propagateFlags();
/* Explicitly forbid system-wide plugin dependencies */ /* Explicitly forbid system-wide plugin dependencies */
PluginManager::Manager<AbstractImporter> _manager{"nonexistent"}; PluginManager::Manager<AbstractImporter> _manager{"nonexistent"};
@ -98,7 +98,7 @@ AnySceneImporterTest::AnySceneImporterTest() {
addTests({&AnySceneImporterTest::unknown, addTests({&AnySceneImporterTest::unknown,
&AnySceneImporterTest::verbose}); &AnySceneImporterTest::propagateFlags});
/* Load the plugin directly from the build tree. Otherwise it's static and /* Load the plugin directly from the build tree. Otherwise it's static and
already loaded. */ already loaded. */
@ -181,7 +181,7 @@ void AnySceneImporterTest::unknown() {
CORRADE_COMPARE(output.str(), "Trade::AnySceneImporter::openFile(): cannot determine the format of mesh.wtf\n"); CORRADE_COMPARE(output.str(), "Trade::AnySceneImporter::openFile(): cannot determine the format of mesh.wtf\n");
} }
void AnySceneImporterTest::verbose() { void AnySceneImporterTest::propagateFlags() {
if(!(_manager.loadState("ObjImporter") & PluginManager::LoadState::Loaded)) if(!(_manager.loadState("ObjImporter") & PluginManager::LoadState::Loaded))
CORRADE_SKIP("ObjImporter plugin not enabled, cannot test"); CORRADE_SKIP("ObjImporter plugin not enabled, cannot test");

Loading…
Cancel
Save