Java cleanup - static fields that should not be static

Somebody appears to have been confused about the purpose of static
in Java.
These are instance fields and should therefore not be static.

Also reduce the visibility of some methods and fields.

Change-Id: I0b5875b6cbd91ee89823e2058b87c1087dc5c92e
Reviewed-on: https://gerrit.libreoffice.org/3555
Reviewed-by: Fridrich Strba <fridrich@documentfoundation.org>
Tested-by: Fridrich Strba <fridrich@documentfoundation.org>

1 files changed, 4 insertions, 6 deletions

diff --git a/qadevOOo/runner/lib/TestCase.java b/qadevOOo/runner/lib/TestCase.java
index 0477ebef7c8c..a4895f13c4ce 100644
--- a/qadevOOo/runner/lib/TestCase.java
+++ b/qadevOOo/runner/lib/TestCase.java
@@ -19,8 +19,8 @@
 package lib;
 
 import java.io.PrintWriter;
-
 import lib.TestParameters;
+
 /**
  * <code>TestCase</code> represent a factory for <code>TestEnvironment</code>s
  * creation and disposing for a given implementation object. The
@@ -49,9 +49,7 @@ public abstract class TestCase {
     /**
      * Specifies the PrintWriter to log information.
      */
-    public PrintWriter log;
-
-    //public static TestCase tCase;
+    protected PrintWriter log;
 
     /**
      * Sets the log to write information during testing.
@@ -66,7 +64,7 @@ public abstract class TestCase {
      *
      * @param tParam test parameters.
      */
-    public void initializeTestCase( TestParameters tParam ) {
+    public final void initializeTestCase( TestParameters tParam ) {
         initialize( tParam, log );
     }
 
@@ -89,7 +87,7 @@ public abstract class TestCase {
      *
      * @param tParam test parameters
      */
-    public void cleanupTestCase( TestParameters tParam ) {
+    public final void cleanupTestCase( TestParameters tParam ) {
         cleanup( tParam, log );
     }