Helios spec-6686 decouple tests (#978)
* * Remove test repository references * * Update asset_builder_tests.py docstring to include test steps * * Update test docstrings in asset_bundler_batch_tests.py to include test steps * * Update asset_processor_batch_dependency_tests.py docstrings to contain test steps * * Update asset_processor_batch_dependency_tests2.py docstrings to have test steps * * Update asset_processor_batch_tests.py docstring to include test steps * * Update asset_processor_batch_tests_2.py docstrings to include test steps * Removed a references to a JIRA ticket ID * * Update asset_processor_guit_tests.py docstrings to include test steps * * Update asset_processor_gui_tests_2.py docstrings to have Test Steps * * Update asset_relocator_tests.py docstrings to include test steps * * update missing_dependency_tests.py docstrings to have test steps * * Update auxiliary_content_tests.py docstrings to have test steps * * Update fbx_tests.py docstrings to have test steps. * * Update bank_info_parser_tests.py docstrings to have test steps * * Removed Jira issue ids from Asset Pipline owned code * * Undid two errornous code changes. * * Addressed dbbronso PR-978 feedback. * Steps declared and not populated. 1 - Removed errornous. 2 - Added missing step * Fixed line formatting by removing bad blank line in Docstring * Addressed PR-978 feedback from AMZN-stankowi * Removed commented out entry from cmakefile * Fixed several casing and spelling/typo issues caught in review * Added a missing test step in asset_bundler_batch_tests.py * Removed a test dealing with external projects, cut LYN-4116 to replace * Calarfied test steps in asset_processor_gui_tests.py * Noted test in asset_processor_gui_tests that cannot be ran manually * Clarified test steps for fbx_tests * Added in a comment and whitespace to a disguised operation. Co-authored-by: stankowi <stankowi@amazon.com>
This commit is contained in:
@@ -533,6 +533,14 @@ class TestsFBX_AllPlatforms(object):
|
||||
def test_FBXBlackboxTest_SourceFiles_Processed_ResultInExpectedProducts(self, workspace,
|
||||
ap_setup_fixture, asset_processor, project,
|
||||
blackbox_param):
|
||||
"""
|
||||
Please see run_fbx_test(...) for details
|
||||
|
||||
Test Steps:
|
||||
1. Determine if blackbox is set to none
|
||||
2. Run FBX Test
|
||||
"""
|
||||
|
||||
if blackbox_param == None:
|
||||
return
|
||||
self.run_fbx_test(workspace, ap_setup_fixture,
|
||||
@@ -544,6 +552,15 @@ class TestsFBX_AllPlatforms(object):
|
||||
workspace, ap_setup_fixture,
|
||||
asset_processor, project,
|
||||
blackbox_param):
|
||||
"""
|
||||
Please see run_fbx_test(...) for details
|
||||
|
||||
Test Steps:
|
||||
1. Determine if blackbox is set to none
|
||||
2. Run FBX Test
|
||||
2. Re-run FBX test and validate the information in override assets
|
||||
"""
|
||||
|
||||
if blackbox_param == None:
|
||||
return
|
||||
self.run_fbx_test(workspace, ap_setup_fixture,
|
||||
@@ -567,6 +584,19 @@ class TestsFBX_AllPlatforms(object):
|
||||
|
||||
def run_fbx_test(self, workspace, ap_setup_fixture, asset_processor,
|
||||
project, blackbox_params: BlackboxAssetTest, overrideAsset = False):
|
||||
"""
|
||||
These tests work by having the test case ingest the test data and determine the run pattern.
|
||||
Tests will process scene settings files and will additionally do a verification against a provided debug file
|
||||
Additionally, if an override is passed, the output is checked against the override.
|
||||
|
||||
Test Steps:
|
||||
1. Create temporary test environment
|
||||
2. Process Assets
|
||||
3. Determine what assets to validate based upon test data
|
||||
4. Validate assets were created in cache
|
||||
5. If debug file provided, verify scene files were generated correctly
|
||||
6. Verify that each given source asset resulted in the expected jobs and products
|
||||
"""
|
||||
|
||||
test_assets_folder = blackbox_params.override_asset_folder if overrideAsset else blackbox_params.asset_folder
|
||||
logger.info(f"{blackbox_params.test_name}: Processing assets in folder '"
|
||||
|
||||
Reference in New Issue
Block a user