YARN-10849 Clarify testcase documentation for TestServiceAM#testContainersReleasedWhenPreLaunchFails. Contributed by Szilard Nemeth

This commit is contained in:
Gergely Pollak 2021-08-04 15:43:07 +02:00
parent 0ba6f35dc2
commit 44bab51be4
1 changed files with 11 additions and 3 deletions

View File

@ -480,9 +480,17 @@ public class TestServiceAM extends ServiceTestUtils{
am.stop();
}
// Test to verify that the containers are released and the
// component instance is added to the pending queue when building the launch
// context fails.
/**
This test verifies that the containers are released and the
component instance is added to the pending queue when building the launch
context fails.
Here, we intentionally have an artifact that doesn't have an id.
This will cause TarballProviderService.processArtifact
to throw an IllegalArgumentException because the Path object is
constructed from the id of the artifact.
In case the id is set to null or unset so it is effectively null,
Path.checkPathArg throws an IllegalArgumentException.
**/
@Test(timeout = 30000)
public void testContainersReleasedWhenPreLaunchFails()
throws Exception {