[ZOOKEEPER-1503] remove redundant JAAS configuration code in SaslAuthTest and SaslAuthFailTest

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.4.4, 3.5.0

Environment

In SaslAuthTest and SaslAuthFail test, we set the JAAS configuration twice with the same text string. This is confusing and redundant, since we need only set it once.

• Hadoop Flags: Reviewed
• Assignee: Eugene Joseph Koontz
• Reporter: Eugene Joseph Koontz
• Created: 05/Jul/12 18:03
• Updated: 01/Sep/12 02:12
• Resolved: 01/Aug/12 19:23

Description

https://builds.apache.org/job/PreCommit-ZOOKEEPER-Build/1120//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html

Issue Links

• is related to: ZOOKEEPER-1524 use more standard junit annotation "@Before" in SaslXTests rather than static blocks
• is related to: ZOOKEEPER-1497 Allow server-side SASL login with JAAS configuration to be programmatically set (rather than only by reading JAAS configuration file)