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-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org