Changeset 340 for trunk/EventBenchCore/src/de/ugoe/cs/eventbench/assertions/FileEqualsReplay.java

Timestamp:

12/21/11 12:25:17 (12 years ago)

Author:

sherbold

Message:

• made all fields in de.ugoe.cs.eventbench.assertions.FileEqualsReplay? and TextEqualsReplay? final and protected
• added null-checks to constructors of de.ugoe.cs.eventbench.assertions.FileEqualsReplay? and TextEqualsReplay? to forbid null values for some fields

File:

• trunk/EventBenchCore/src/de/ugoe/cs/eventbench/assertions/FileEqualsReplay.java (modified) (5 diffs)

trunk/EventBenchCore/src/de/ugoe/cs/eventbench/assertions/FileEqualsReplay.java

@@ -1 +1 @@
 package de.ugoe.cs.eventbench.assertions;
+
+import java.security.InvalidParameterException;
 
 import de.ugoe.cs.eventbench.data.IReplayable;
@@ -19 +21 @@
          * </p>
          */
-        String actualFile = null;
+        protected final String actualFile;
 
         /**
@@ -26 +28 @@
          * </p>
          */
-        String expectedFile = null;
+        protected final String expectedFile;
 
         /**
@@ -44 +46 @@
          * @param actualFile
          *            name and path of the actual file
+         * @throws InvalidParameterException
+         *             thrown if expectedFile or actualFile are null
          */
         public FileEqualsReplay(String expectedFile, String actualFile) {
+                if (expectedFile == null) {
+                        throw new InvalidParameterException(
+                                        "expected file must not be null");
+                }
+                if (actualFile == null) {
+                        throw new InvalidParameterException("actual file must not be null");
+                }
                 this.expectedFile = expectedFile;
                 this.actualFile = actualFile;
@@ -57 +68 @@
         public String getReplay() {
 
-                actualFile = StringTools.xmlEntityReplacement(actualFile);
-                expectedFile = StringTools.xmlEntityReplacement(expectedFile);
+                String actualFileTmp = StringTools.xmlEntityReplacement(actualFile);
+                String expectedFileTmp = StringTools.xmlEntityReplacement(expectedFile);
 
                 StringBuilder currentMsgStr = new StringBuilder(800);
                 currentMsgStr.append("  <fileEquals ");
-                currentMsgStr.append("actualFile=\"" + actualFile + "\" ");
-                currentMsgStr.append("expectedFile=\"" + expectedFile + "\"/>");
+                currentMsgStr.append("actualFile=\"" + actualFileTmp + "\" ");
+                currentMsgStr.append("expectedFile=\"" + expectedFileTmp + "\"/>");
                 currentMsgStr.append(StringTools.ENDLINE);
                 return currentMsgStr.toString();