core: Remove stdout from SimpleAuthenticationCheck

Removed an old System.out.println call from the class - it was probably
left there by mistake during debugging, and in any event has no place in
production code.

Change-Id: I1ac20e5aa5d9a8236a2a081c5cd0be68a25f724b
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: Removed std from KDCLocatorTest

Removed printing of results with System.out.println - this is no way to
handle robust logging.

Change-Id: I37eac9646bcbabe3a80a563e41d7bd904de626cd
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: Removed std from EncryptionUtilsTest

Removed printing of results with System.out.println - and replaced it
with a proper error message in case of failure.

Change-Id: Ia1386b1ddb0e090e5c49a6c3130a3dccfb35a365
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: Synthetic emulators EcryptionUtilsTest

Changed members visibility from private to protected to solve the
synthetic emulators warning.

This may slightly increase performance, but I doubt it will be
noticeable.

Change-Id: I5d17aa584e4b8c1d858d74c155265a9f30cf86d0
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: EncryptoinUtilsTest @Before->@BeforeClass

Replaced the @Before method with @BeforeClass.
There is a single URL that won't change between test runs.

Change-Id: I89ad4e12f08af5e0ef19e486bf21c350ad7f1015
Signed-off-by: Allon Mureinik <amureini@redhat.com>
tools: Proper error testing in EngineConfigCLITest

Converted the error tests in EngineConfigCLITest to the proper JUnit
form (with the expect parameter on the @Test annotation), instead of the
awkward explicit fail before the catch block.

Since the test methods have meaningful names, this change also allowed
removing the System.out.println calls.

Change-Id: Iad25eae4fd7b4535cc1b2d9570f142d6996b79d0
Signed-off-by: Allon Mureinik <amureini@redhat.com>