From a849008a4857d50ed109d03ad8c087889f44d401 Mon Sep 17 00:00:00 2001 From: sphrose <82213493+sphrose@users.noreply.github.com> Date: Tue, 12 Oct 2021 09:33:36 +0100 Subject: [PATCH] compile fix and add activation failure test. Signed-off-by: sphrose <82213493+sphrose@users.noreply.github.com> --- Gems/Terrain/Code/Tests/SurfaceMaterialsListTest.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/Gems/Terrain/Code/Tests/SurfaceMaterialsListTest.cpp b/Gems/Terrain/Code/Tests/SurfaceMaterialsListTest.cpp index cc1af8bf62..c2c24e300f 100644 --- a/Gems/Terrain/Code/Tests/SurfaceMaterialsListTest.cpp +++ b/Gems/Terrain/Code/Tests/SurfaceMaterialsListTest.cpp @@ -10,7 +10,6 @@ #include #include #include -#include using ::testing::NiceMock; using ::testing::AtLeast; @@ -66,14 +65,16 @@ namespace UnitTest TEST_F(TerrainSurfaceMaterialsListTest, SurfaceGradientListRequiresShapeToActivate) { + // Check that the component requires a shape service to activate: trying to Activate the entity will cause the test to fail, so + // use the EvaluateDependenciesGetDetails function to check the dependencies are met. + auto entity = CreateEntity(); AddSurfaceMaterialListComponent(entity.get()); - entity->Activate(); - - EXPECT_EQ(entity->GetState(), AZ::Entity::State::Active); - + const AZ::Entity::DependencySortOutcome sortOutcome = entity->EvaluateDependenciesGetDetails(); + EXPECT_FALSE(sortOutcome.IsSuccess()); + entity.reset(); }