[HADOOP-12514] Make static fields in GenericTestUtils for assertExceptionContains() package-private and final

• Type: Task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.8.0
• Fix Version/s: 2.8.0, 3.0.0-alpha1
• Component/s: test
• Assignee: Mingliang Liu
• Reporter: Mingliang Liu
• Created: 26/Oct/15 21:43
• Updated: 30/Aug/16 01:22
• Resolved: 28/Oct/15 10:40

Description

This is a follow up of HADOOP-12472.

It makes sense to make the following static fields package private instead of protected, as they are for test purpose and TestGenericTestUtils is in the same package as GenericTestUtils.

• protected static String E_NULL_THROWABLE = "Null Throwable";
• protected static String E_NULL_THROWABLE_STRING = "Null Throwable.toString() value";
• protected static String E_UNEXPECTED_EXCEPTION = "but got unexpected exception";

Meanwhile, we may need to make them final.

Issue Links

• is related to: HADOOP-12472 Make GenericTestUtils.assertExceptionContains robust