Fixes incorrect build dependencies for tests (#2959)
The serialize context tools were not being auto built by CI because of this. Added missing dependency on AzTestRunner. Also moved the AssetBundlerBatch dependency directly in there No reason to add it to additional_dependnecies, since its only used for conditional inclusions. Signed-off-by: lawsonamzn <70027408+lawsonamzn@users.noreply.github.com>
This commit is contained in:
@@ -11,7 +11,6 @@ if(PAL_TRAIT_BUILD_TESTS_SUPPORTED AND PAL_TRAIT_BUILD_HOST_TOOLS)
|
||||
if (PAL_TRAIT_BUILD_SERIALIZECONTEXTTOOLS)
|
||||
list(APPEND additional_dependencies AZ::SerializeContextTools) # test_CLITool_SerializeContextTools depends on it
|
||||
endif()
|
||||
list(APPEND additional_dependencies AZ::AssetBundlerBatch) # test_CLITool_AssetBundlerBatch_Works depends on it
|
||||
|
||||
ly_add_pytest(
|
||||
NAME AutomatedTesting::SmokeTest
|
||||
@@ -26,7 +25,9 @@ if(PAL_TRAIT_BUILD_TESTS_SUPPORTED AND PAL_TRAIT_BUILD_HOST_TOOLS)
|
||||
Legacy::Editor
|
||||
AutomatedTesting.GameLauncher
|
||||
AutomatedTesting.Assets
|
||||
${aditional_dependencies}
|
||||
AZ::AzTestRunner
|
||||
AZ::AssetBundlerBatch
|
||||
${additional_dependencies}
|
||||
COMPONENT
|
||||
Smoke
|
||||
)
|
||||
|
||||
Reference in New Issue
Block a user