diff --git a/cmake/ctest/build.ctest b/cmake/ctest/build.ctest index 79b634d7..90aa096e 100644 --- a/cmake/ctest/build.ctest +++ b/cmake/ctest/build.ctest @@ -13,7 +13,6 @@ set(_ARGS_BOOL BUILD_VIEWER RUN_TESTS - TESTS_NODATA SEPARATE_TEST_SUITES RUN_MEMCHECK @@ -164,7 +163,6 @@ set(_CONFIGURE_OPTIONS "-DBUILD_TOOLS=${BUILD_TOOLS}" "-DBUILD_VIEWER=${BUILD_VIEWER}" "-DBUILD_TESTS=TRUE" - "-DTESTS_NODATA=${TESTS_NODATA}" "-DTEST_COVERAGE=${TEST_COVERAGE}" "-DSEPARATE_TEST_SUITES=${SEPARATE_TEST_SUITES}" "-DCHECK_IWYU=${CHECK_IWYU}" diff --git a/cmake/ctest/script_ci.ctest b/cmake/ctest/script_ci.ctest index 82c23fbf..4ef5b1a8 100644 --- a/cmake/ctest/script_ci.ctest +++ b/cmake/ctest/script_ci.ctest @@ -52,9 +52,8 @@ set(RUN_MEMCHECK FALSE) openrw_should_submit_ci(SUBMIT) # Build with no data and test -set(BUILDER_NAME "${BUILDER_NAME_BASE}-nodata") +set(BUILDER_NAME "${BUILDER_NAME_BASE}") set(APPEND_RESULTS FALSE) -set(TESTS_NODATA TRUE) set(RUN_TESTS TRUE) if($ENV{TEST_COVERAGE}) set(TEST_COVERAGE TRUE) @@ -66,14 +65,4 @@ set(CODECOV_FLAGS "nodata") include("${CTEST_SCRIPT_DIRECTORY}/build.ctest") -# Build with data and do not test -set(BUILDER_NAME "${BUILDER_NAME_BASE}-data") -set(APPEND_RESULTS FALSE) -set(TESTS_NODATA FALSE) -set(RUN_TESTS FALSE) - -set(CODECOV_FLAGS "data") - -include("${CTEST_SCRIPT_DIRECTORY}/build.ctest") - handle_warnings_errors() diff --git a/cmake/ctest/script_experimental.ctest b/cmake/ctest/script_experimental.ctest index 79319c93..eb942770 100644 --- a/cmake/ctest/script_experimental.ctest +++ b/cmake/ctest/script_experimental.ctest @@ -62,7 +62,6 @@ else() set(RES FALSE) endif() -set(TESTS_NODATA FALSE) set(RUN_TESTS TRUE) set(SUBMIT $ENV{SUBMIT}) diff --git a/cmake_options.cmake b/cmake_options.cmake index d754b4e1..d0d10cb0 100644 --- a/cmake_options.cmake +++ b/cmake_options.cmake @@ -7,7 +7,7 @@ option(BUILD_VIEWER "Build GUI data viewer") option(ENABLE_SCRIPT_DEBUG "Enable verbose script execution") option(ENABLE_PROFILING "Enable detailed profiling metrics") -option(TESTS_NODATA "Build tests for no-data testing") +option(TEST_DATA "Enable tests that require game data") set(FAILED_CHECK_ACTION "IGNORE" CACHE STRING "What action to perform on a failed RW_CHECK (in debug mode)") set_property(CACHE FAILED_CHECK_ACTION PROPERTY STRINGS "IGNORE" "ABORT" "BREAKPOINT") diff --git a/conanfile.py b/conanfile.py index 1bbbfbde..c4d15e29 100644 --- a/conanfile.py +++ b/conanfile.py @@ -9,14 +9,12 @@ class OpenrwConan(ConanFile): description = "OpenRW 'Open ReWrite' is an un-official open source recreation of the classic Grand Theft Auto III game executable" settings = 'os', 'compiler', 'build_type', 'arch' options = { - 'test_data': [True, False], 'viewer': [True, False], 'tools': [True, False], 'profiling': [True, False], } default_options = { - 'test_data': False, 'viewer': True, 'tools': True, 'profiling': True, @@ -68,7 +66,6 @@ class OpenrwConan(ConanFile): 'BUILD_TESTS': True, 'BUILD_VIEWER': self.options.viewer, 'BUILD_TOOLS': self.options.tools, - 'TESTS_NODATA': not self.options.test_data, 'ENABLE_PROFILING': self.options.profiling, 'USE_CONAN': True, 'BOOST_STATIC': not self.options['boost'].shared, diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt index 5aab89ec..c0d944c4 100644 --- a/tests/CMakeLists.txt +++ b/tests/CMakeLists.txt @@ -53,11 +53,6 @@ add_executable(rwtests ${TEST_SOURCES} ) -target_compile_definitions(rwtests - PRIVATE - "RW_TEST_WITH_DATA=$>" - ) - target_include_directories(rwtests PRIVATE "${PROJECT_SOURCE_DIR}/tests" @@ -97,3 +92,9 @@ else() TIMEOUT 300 ) endif() + +if(TEST_DATA) + add_test(NAME DataTests + COMMAND "$" "--run_test=@data-test" + ) +endif() diff --git a/tests/test_Animation.cpp b/tests/test_Animation.cpp index 41f346b4..3e8ea2f9 100644 --- a/tests/test_Animation.cpp +++ b/tests/test_Animation.cpp @@ -5,9 +5,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(AnimationTests) +BOOST_AUTO_TEST_SUITE(AnimationTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_matrix) { { auto animation = std::make_shared(); @@ -44,6 +43,5 @@ BOOST_AUTO_TEST_CASE(test_matrix) { glm::vec3(0.f, 1.f, 0.f)); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Archive.cpp b/tests/test_Archive.cpp index 69a32da4..482b3c6c 100644 --- a/tests/test_Archive.cpp +++ b/tests/test_Archive.cpp @@ -2,9 +2,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(ArchiveTests) +BOOST_AUTO_TEST_SUITE(ArchiveTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_open_archive) { LoaderIMG archive; @@ -28,6 +27,5 @@ BOOST_AUTO_TEST_CASE(test_open_archive) { BOOST_CHECK_EQUAL(f2.offset, f.offset); BOOST_CHECK_EQUAL(f2.size, f.size); } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Buoyancy.cpp b/tests/test_Buoyancy.cpp index b60c3a73..c85abd86 100644 --- a/tests/test_Buoyancy.cpp +++ b/tests/test_Buoyancy.cpp @@ -2,9 +2,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(BuoyancyTests) +BOOST_AUTO_TEST_SUITE(BuoyancyTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_vehicle_buoyancy) { glm::vec2 tpos(-WATER_WORLD_SIZE / 2.f + 10.f); { @@ -57,6 +56,5 @@ BOOST_AUTO_TEST_CASE(test_vehicle_buoyancy) { Global::get().e->destroyObject(vehicle); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Character.cpp b/tests/test_Character.cpp index fd4fe589..c34d819b 100644 --- a/tests/test_Character.cpp +++ b/tests/test_Character.cpp @@ -5,9 +5,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(CharacterTests) +BOOST_AUTO_TEST_SUITE(CharacterTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_create) { { auto character = @@ -164,6 +163,5 @@ BOOST_AUTO_TEST_CASE(test_cycle_animating) { static_cast(AnimCycle::ArrestGun)); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Chase.cpp b/tests/test_Chase.cpp index 3f9c7b91..f1d77998 100644 --- a/tests/test_Chase.cpp +++ b/tests/test_Chase.cpp @@ -2,9 +2,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(ChaseTests) +BOOST_AUTO_TEST_SUITE(ChaseTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_load_keyframes) { std::vector keyframes; BOOST_REQUIRE(ChaseKeyframe::load( @@ -12,6 +11,5 @@ BOOST_AUTO_TEST_CASE(test_load_keyframes) { BOOST_REQUIRE(keyframes.size() == 5400); BOOST_CHECK_CLOSE(keyframes[0].position.x, 273.5422, 0.1); } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Cutscene.cpp b/tests/test_Cutscene.cpp index 14b4399b..f580c055 100644 --- a/tests/test_Cutscene.cpp +++ b/tests/test_Cutscene.cpp @@ -4,9 +4,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(CutsceneTests) +BOOST_AUTO_TEST_SUITE(CutsceneTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_load) { { auto d = Global::get().e->data->index.openFile("intro.dat"); @@ -29,6 +28,5 @@ BOOST_AUTO_TEST_CASE(test_load) { BOOST_CHECK(tracks.duration == 64.8f); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Data.cpp b/tests/test_Data.cpp index bbf72497..65fd9e96 100644 --- a/tests/test_Data.cpp +++ b/tests/test_Data.cpp @@ -8,9 +8,8 @@ // Tests against loading various data files // These tests are bad but so are the interfaces so it cancels out. -BOOST_AUTO_TEST_SUITE(DataTests) +BOOST_AUTO_TEST_SUITE(DataTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_weapon_dat) { GenericDATLoader l; std::vector weaponData; @@ -121,6 +120,5 @@ BOOST_AUTO_TEST_CASE(test_model_archive_loaded) { e->destroyObject(inst); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_FileIndex.cpp b/tests/test_FileIndex.cpp index bc36b184..8ac9dfc0 100644 --- a/tests/test_FileIndex.cpp +++ b/tests/test_FileIndex.cpp @@ -21,8 +21,7 @@ BOOST_AUTO_TEST_CASE(test_normalizeName) { } } -#if RW_TEST_WITH_DATA -BOOST_AUTO_TEST_CASE(test_indexTree) { +BOOST_AUTO_TEST_CASE(test_indexTree, DATA_TEST_PREDICATE) { FileIndex index; index.indexTree(Global::getGamePath()); @@ -46,7 +45,7 @@ BOOST_AUTO_TEST_CASE(test_indexTree) { } } -BOOST_AUTO_TEST_CASE(test_openFile) { +BOOST_AUTO_TEST_CASE(test_openFile, DATA_TEST_PREDICATE) { FileIndex index; index.indexTree(Global::getGamePath() + "/data"); @@ -54,7 +53,7 @@ BOOST_AUTO_TEST_CASE(test_openFile) { BOOST_CHECK(handle.data != nullptr); } -BOOST_AUTO_TEST_CASE(test_indexArchive) { +BOOST_AUTO_TEST_CASE(test_indexArchive, DATA_TEST_PREDICATE) { FileIndex index; index.indexTree(Global::getGamePath()); @@ -70,6 +69,5 @@ BOOST_AUTO_TEST_CASE(test_indexArchive) { BOOST_CHECK(handle.data != nullptr); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_GameData.cpp b/tests/test_GameData.cpp index 6279faf2..92c41446 100644 --- a/tests/test_GameData.cpp +++ b/tests/test_GameData.cpp @@ -2,9 +2,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(GameDataTests) +BOOST_AUTO_TEST_SUITE(GameDataTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_object_data) { GameData gd(&Global::get().log, Global::getGamePath()); gd.load(); @@ -94,6 +93,5 @@ BOOST_AUTO_TEST_CASE(test_ped_groups) { BOOST_REQUIRE_GE(red.size(), 8); BOOST_CHECK_EQUAL(red[0], 34); } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_GameWorld.cpp b/tests/test_GameWorld.cpp index 00757216..d2c5c087 100644 --- a/tests/test_GameWorld.cpp +++ b/tests/test_GameWorld.cpp @@ -4,9 +4,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(GameWorldTests) +BOOST_AUTO_TEST_SUITE(GameWorldTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_gameobject_id) { auto& gw = *Global::get().e; @@ -67,6 +66,5 @@ BOOST_AUTO_TEST_CASE(test_offsetgametime) { BOOST_CHECK_EQUAL(9, gw.getHour()); BOOST_CHECK_EQUAL(25, gw.getMinute()); } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Garage.cpp b/tests/test_Garage.cpp index 2c715cd3..f37ddaf6 100644 --- a/tests/test_Garage.cpp +++ b/tests/test_Garage.cpp @@ -1,9 +1,8 @@ #include #include #include "test_Globals.hpp" -#if RW_TEST_WITH_DATA -BOOST_AUTO_TEST_SUITE(GarageTests) +BOOST_AUTO_TEST_SUITE(GarageTests, DATA_TEST_PREDICATE) BOOST_AUTO_TEST_CASE(test_garage_interaction) { { @@ -14,5 +13,3 @@ BOOST_AUTO_TEST_CASE(test_garage_interaction) { } BOOST_AUTO_TEST_SUITE_END() - -#endif diff --git a/tests/test_Globals.cpp b/tests/test_Globals.cpp index 70d271b4..7d57da2f 100644 --- a/tests/test_Globals.cpp +++ b/tests/test_Globals.cpp @@ -2,7 +2,6 @@ #include -#if RW_TEST_WITH_DATA std::string Global::getGamePath() { rwfs::path configPath = RWConfigParser::getDefaultConfigPath() / "openrw.ini"; RWConfigParser cfgParser; @@ -10,4 +9,3 @@ std::string Global::getGamePath() { BOOST_REQUIRE(parseResult.isValid()); return *cfgLayer.gamedataPath; } -#endif diff --git a/tests/test_Globals.hpp b/tests/test_Globals.hpp index ad74e4c2..760e0680 100644 --- a/tests/test_Globals.hpp +++ b/tests/test_Globals.hpp @@ -19,6 +19,11 @@ #include #include +#include + +#define DATA_TEST_PREDICATE * boost::unit_test_framework::label("data-test")\ + * boost::unit_test_framework::disabled() + std::ostream& operator<<(std::ostream& stream, glm::vec3 const& v); // Boost moved the print_log_value struct in version 1.59 @@ -80,12 +85,13 @@ BOOST_NS_MAGIC_CLOSING class Global { public: GameWindow window; -#if RW_TEST_WITH_DATA GameData* d; GameWorld* e; GameState* s; + std::unique_ptr d_; + std::unique_ptr e_; + std::unique_ptr s_; Logger log; -#endif Global() { if (SDL_Init(SDL_INIT_VIDEO) < 0) @@ -94,29 +100,25 @@ public: window.create("Tests", 800, 600, false); window.hideCursor(); -#if RW_TEST_WITH_DATA - d = new GameData(&log, getGamePath()); + d_ = std::make_unique(&log, getGamePath()); + d = d_.get(); d->load(); - e = new GameWorld(&log, d); - s = new GameState; + e_ = std::make_unique(&log, d); + e = e_.get(); + s_ = std::make_unique(); + s = s_.get(); e->state = s; e->dynamicsWorld->setGravity(btVector3(0.f, 0.f, 0.f)); -#endif } ~Global() { window.close(); -#if RW_TEST_WITH_DATA - delete e; -#endif } -#if RW_TEST_WITH_DATA static std::string getGamePath(); -#endif static Global& get() { static Global g; diff --git a/tests/test_Items.cpp b/tests/test_Items.cpp index 2ea9947b..743bfa56 100644 --- a/tests/test_Items.cpp +++ b/tests/test_Items.cpp @@ -2,9 +2,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(ItemsTests) +BOOST_AUTO_TEST_SUITE(ItemsTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_character_inventory) { { auto character = Global::get().e->createPedestrian(1, {0.f, 0.f, 0.f}); @@ -25,6 +24,5 @@ BOOST_AUTO_TEST_CASE(test_character_inventory) { Global::get().e->destroyObject(character); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Lifetime.cpp b/tests/test_Lifetime.cpp index b702c7fa..a58fcf7a 100644 --- a/tests/test_Lifetime.cpp +++ b/tests/test_Lifetime.cpp @@ -4,9 +4,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(LifetimeTests) +BOOST_AUTO_TEST_SUITE(LifetimeTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_cleanup) { GameObject* f = Global::get().e->createInstance(1337, glm::vec3(0.f, 0.f, 1000.f)); @@ -29,6 +28,5 @@ BOOST_AUTO_TEST_CASE(test_cleanup) { BOOST_CHECK(search != objects.end()); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_LoaderDFF.cpp b/tests/test_LoaderDFF.cpp index a16693ee..939fb57c 100644 --- a/tests/test_LoaderDFF.cpp +++ b/tests/test_LoaderDFF.cpp @@ -5,8 +5,7 @@ BOOST_AUTO_TEST_SUITE(LoaderDFFTests) -#if RW_TEST_WITH_DATA -BOOST_AUTO_TEST_CASE(test_load_dff) { +BOOST_AUTO_TEST_CASE(test_load_dff, DATA_TEST_PREDICATE) { { auto d = Global::get().e->data->index.openFile("landstal.dff"); @@ -26,8 +25,6 @@ BOOST_AUTO_TEST_CASE(test_load_dff) { } } -#endif - BOOST_AUTO_TEST_CASE(test_clump_clone) { { auto frame1 = std::make_shared(0); diff --git a/tests/test_Payphone.cpp b/tests/test_Payphone.cpp index f559e786..51399215 100644 --- a/tests/test_Payphone.cpp +++ b/tests/test_Payphone.cpp @@ -6,9 +6,8 @@ #include #include #include "test_Globals.hpp" -#if RW_TEST_WITH_DATA -BOOST_AUTO_TEST_SUITE(PayphoneTests) +BOOST_AUTO_TEST_SUITE(PayphoneTests, DATA_TEST_PREDICATE) BOOST_AUTO_TEST_CASE(test_payphone_interaction) { { @@ -68,5 +67,3 @@ BOOST_AUTO_TEST_CASE(test_payphone_interaction) { } BOOST_AUTO_TEST_SUITE_END() - -#endif diff --git a/tests/test_Pickup.cpp b/tests/test_Pickup.cpp index 90e04ae3..400ac69f 100644 --- a/tests/test_Pickup.cpp +++ b/tests/test_Pickup.cpp @@ -3,7 +3,6 @@ #include #include #include "test_Globals.hpp" -#if RW_TEST_WITH_DATA class TestPickup : public PickupObject { public: @@ -20,7 +19,7 @@ public: } }; -BOOST_AUTO_TEST_SUITE(PickupTests) +BOOST_AUTO_TEST_SUITE(PickupTests, DATA_TEST_PREDICATE) BOOST_AUTO_TEST_CASE(test_pickup_interaction) { { @@ -97,4 +96,3 @@ BOOST_AUTO_TEST_CASE(test_item_pickup) { BOOST_AUTO_TEST_SUITE_END() -#endif diff --git a/tests/test_RWBStream.cpp b/tests/test_RWBStream.cpp index 89f1d666..86098d56 100644 --- a/tests/test_RWBStream.cpp +++ b/tests/test_RWBStream.cpp @@ -3,9 +3,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(RWBStreamTests) +BOOST_AUTO_TEST_SUITE(RWBStreamTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(iterate_stream_test) { { auto d = Global::get().e->data->index.openFile("landstal.dff"); @@ -28,6 +27,5 @@ BOOST_AUTO_TEST_CASE(iterate_stream_test) { BOOST_CHECK_EQUAL(*reinterpret_cast(innerCursor), 0x10); } } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Text.cpp b/tests/test_Text.cpp index 9fc281b3..cb970fd5 100644 --- a/tests/test_Text.cpp +++ b/tests/test_Text.cpp @@ -9,8 +9,7 @@ BOOST_AUTO_TEST_SUITE(TextTests) -#if RW_TEST_WITH_DATA -BOOST_AUTO_TEST_CASE(load_test) { +BOOST_AUTO_TEST_CASE(load_test, DATA_TEST_PREDICATE) { { auto d = Global::get().e->data->index.openFileRaw("text/english.gxt"); @@ -23,7 +22,6 @@ BOOST_AUTO_TEST_CASE(load_test) { BOOST_CHECK_EQUAL(texts.text("1008"), T("BUSTED")); } } -#endif BOOST_AUTO_TEST_CASE(special_chars) { { diff --git a/tests/test_TrafficDirector.cpp b/tests/test_TrafficDirector.cpp index 8b7e996d..2546c193 100644 --- a/tests/test_TrafficDirector.cpp +++ b/tests/test_TrafficDirector.cpp @@ -17,9 +17,8 @@ std::ostream& operator<<(std::ostream& os, const ai::AIGraphNode* yt) { return os; } -BOOST_AUTO_TEST_SUITE(TrafficDirectorTests) +BOOST_AUTO_TEST_SUITE(TrafficDirectorTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_available_nodes) { ai::AIGraph graph; @@ -166,6 +165,5 @@ BOOST_AUTO_TEST_CASE(test_create_traffic) { // Global::get().e->destroyObject(created[0]); } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Vehicle.cpp b/tests/test_Vehicle.cpp index 3f415c1a..c032ea89 100644 --- a/tests/test_Vehicle.cpp +++ b/tests/test_Vehicle.cpp @@ -3,9 +3,8 @@ #include #include "test_Globals.hpp" -BOOST_AUTO_TEST_SUITE(VehicleTests) +BOOST_AUTO_TEST_SUITE(VehicleTests, DATA_TEST_PREDICATE) -#if RW_TEST_WITH_DATA BOOST_AUTO_TEST_CASE(test_create_vehicle) { VehicleObject* vehicle = Global::get().e->createVehicle(90u, glm::vec3(), glm::quat{1.0f,0.0f,0.0f,0.0f}); @@ -130,6 +129,5 @@ BOOST_AUTO_TEST_CASE(test_open_part) { Global::get().e->destroyObject(vehicle); } -#endif BOOST_AUTO_TEST_SUITE_END() diff --git a/tests/test_Weapon.cpp b/tests/test_Weapon.cpp index ff57ce27..92ec10fb 100644 --- a/tests/test_Weapon.cpp +++ b/tests/test_Weapon.cpp @@ -34,8 +34,7 @@ BOOST_FIXTURE_TEST_CASE(weapon_scan_does_injur_others, WeaponScanFixture) { BOOST_CHECK(scan.doesDamage(reinterpret_cast(0xDEADBEEF))); } -#if RW_TEST_WITH_DATA -BOOST_AUTO_TEST_CASE(TestDoWeaponScan) { +BOOST_AUTO_TEST_CASE(TestDoWeaponScan, DATA_TEST_PREDICATE) { { // Test RADIUS scan auto character = Global::get().e->createPedestrian(1, {0.f, 0.f, 0.f}); @@ -53,7 +52,7 @@ BOOST_AUTO_TEST_CASE(TestDoWeaponScan) { } } -BOOST_AUTO_TEST_CASE(TestProjectile) { +BOOST_AUTO_TEST_CASE(TestProjectile, DATA_TEST_PREDICATE) { { auto character = Global::get().e->createPedestrian(1, {25.f, 0.f, 0.f}); BOOST_REQUIRE(character != nullptr); @@ -141,6 +140,5 @@ BOOST_AUTO_TEST_CASE(TestProjectile) { Global::get().e->destroyQueuedObjects(); } } -#endif BOOST_AUTO_TEST_SUITE_END()