[jira] [Created] (YARN-6450) TestContainerManagerWithLCE requires override for each new test added to ContainerManagerTest

["Jason Lowe (JIRA)" <ji...@apache.org>]

Jason Lowe created YARN-6450:
--------------------------------

             Summary: TestContainerManagerWithLCE requires override for each new test added to ContainerManagerTest
                 Key: YARN-6450
                 URL: https://issues.apache.org/jira/browse/YARN-6450
             Project: Hadoop YARN
          Issue Type: Bug
          Components: test
            Reporter: Jason Lowe
            Assignee: Jason Lowe


Every test in TestContainerManagerWithLCE looks like this:
{code}
  @Override
  public void testSomething() throws Exception {
    // Don't run the test if the binary is not available.
    if (!shouldRunTest()) {
      LOG.info("LCE binary path is not passed. Not running the test");
      return;
    }
    LOG.info("Running something");
    super.testSomething();
  }
{code}

If  a new test is added to ContainerManagerTest then by default ContainerManagerTestWithLCE will fail when the LCE has not been configured.  This is an unnecessary maintenance burden.




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-dev-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-dev-help@hadoop.apache.org