[HADOOP-15853] TestConfigurationDeprecation leaves behind a temp file, resulting in a license issue

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.3.0
• Fix Version/s: 3.3.0
• Component/s: test
• Labels: newbie
• Target Version/s: 3.3.0
• Hadoop Flags: Reviewed
• Assignee: Ayush Saxena
• Reporter: Robert Kanter
• Created: 15/Oct/18 19:51
• Updated: 15/Oct/18 22:43
• Resolved: 15/Oct/18 22:02

Description

HADOOP-15708 made some changes to TestConfigurationDeprecation. One of them was adding

  final static String CONFIG4 = new File("./test-config4" +
      "-TestConfigurationDeprecation.xml").getAbsolutePath();

which we never clean up in the tearDown method:

  @After
  public void tearDown() throws Exception {
    new File(CONFIG).delete();
    new File(CONFIG2).delete();
    new File(CONFIG3).delete();
  }

This results in that file being left behind, and causing a license warning in test runs.

Issue Links

• is broken by: HADOOP-15708 Reading values from Configuration before adding deprecations make it impossible to read value with deprecated key