scawful/yaze
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor OverworldTest to initialize overworld_ with ROM data and improve formatting of assertions

Browse Source
This commit is contained in:
scawful
2025-01-18 15:10:11 -05:00
parent 2cc15350aa
commit 146f85eb31
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/test/zelda3/overworld_test.cc
View File

@@ -21,7 +21,7 @@ class OverworldTest : public ::testing::Test, public SharedRom {
}
void TearDown() override {}
zelda3::Overworld overworld_;
zelda3::Overworld overworld_{*rom()};
};
TEST_F(OverworldTest, OverworldLoadNoRomDataError) {
@@ -39,17 +39,16 @@ TEST_F(OverworldTest, OverworldLoadNoRomDataError) {
TEST_F(OverworldTest, OverworldLoadRomDataOk) {
// Arrange
EXPECT_OK(rom()->LoadFromFile("zelda3.sfc"));
ASSERT_OK_AND_ASSIGN(auto gfx_data, LoadAllGraphicsData(*rom(), /*defer_render=*/true));
ASSERT_OK_AND_ASSIGN(auto gfx_data,
LoadAllGraphicsData(*rom(), /*defer_render=*/true));
// Act
auto status = overworld_.Load(*rom());
// Assert
EXPECT_TRUE(status.ok());
EXPECT_EQ(overworld_.overworld_maps().size(),
zelda3::kNumOverworldMaps);
EXPECT_EQ(overworld_.tiles16().size(),
zelda3::kNumTile16Individual);
EXPECT_EQ(overworld_.overworld_maps().size(), zelda3::kNumOverworldMaps);
EXPECT_EQ(overworld_.tiles16().size(), zelda3::kNumTile16Individual);
}
} // namespace test