Skip to content

Commit 0ed2b5b

Browse files
author
Kate Ivanova
committed
Extend iTextCore test classes
DEVSIX-3360
1 parent 7797df7 commit 0ed2b5b

File tree

78 files changed

+296
-116
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

78 files changed

+296
-116
lines changed

barcodes/src/test/java/com/itextpdf/barcodes/BarcodeMultiThreadingTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,13 +42,14 @@ This file is part of the iText (R) project.
4242
*/
4343
package com.itextpdf.barcodes;
4444

45+
import com.itextpdf.test.ExtendedITextTest;
4546
import com.itextpdf.test.annotations.type.UnitTest;
4647
import org.junit.Assert;
4748
import org.junit.Test;
4849
import org.junit.experimental.categories.Category;
4950

5051
@Category(UnitTest.class)
51-
public class BarcodeMultiThreadingTest {
52+
public class BarcodeMultiThreadingTest extends ExtendedITextTest {
5253

5354
private static class DataMatrixThread extends Thread {
5455
@Override

forms/src/main/java/com/itextpdf/forms/xfdf/XfdfConstants.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,5 +169,10 @@ private XfdfConstants() {
169169
public static final String DEFAULT_STYLE = "defaultstyle";
170170
public static final String ATTRIBUTE_NAME_OR_VALUE_MISSING = "Attribute name or value are missing";
171171
public static final String PAGE_IS_MISSING = "Required Page attribute is missing.";
172+
/**
173+
* @deprecated Use {@link LogMessageConstant#UNSUPPORTED_ANNOTATION_ATTRIBUTE} instead.
174+
* The constant will be removed in 7.2.
175+
*/
176+
@Deprecated
172177
public static final String UNSUPPORTED_ANNOTATION_ATTRIBUTE = "Unsupported attribute type";
173-
}
178+
}

forms/src/main/java/com/itextpdf/forms/xfdf/XfdfObjectFactory.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ This file is part of the iText (R) project.
4343
package com.itextpdf.forms.xfdf;
4444

4545
import com.itextpdf.forms.PdfAcroForm;
46+
import com.itextpdf.io.LogMessageConstant;
4647
import com.itextpdf.kernel.pdf.PdfArray;
4748
import com.itextpdf.kernel.pdf.PdfDictionary;
4849
import com.itextpdf.kernel.pdf.PdfDocument;
@@ -340,7 +341,7 @@ private void addAnnotObjectAttribute(AnnotObject annotObject, Node attributeNode
340341
case XfdfConstants.FRINGE:
341342
annotObject.addAttribute(new AttributeObject(attributeName, attributeNode.getNodeValue()));
342343
break;
343-
default: logger.warn(XfdfConstants.UNSUPPORTED_ANNOTATION_ATTRIBUTE);
344+
default: logger.warn(LogMessageConstant.XFDF_UNSUPPORTED_ANNOTATION_ATTRIBUTE);
344345
break;
345346
}
346347
}

forms/src/main/java/com/itextpdf/forms/xfdf/XfdfReader.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,8 @@ This file is part of the iText (R) project.
4444

4545
import com.itextpdf.forms.PdfAcroForm;
4646
import com.itextpdf.forms.fields.PdfFormField;
47+
import com.itextpdf.io.LogMessageConstant;
48+
import com.itextpdf.io.util.MessageFormatUtil;
4749
import com.itextpdf.kernel.geom.Rectangle;
4850
import com.itextpdf.kernel.pdf.PdfDocument;
4951
import com.itextpdf.kernel.pdf.PdfName;
@@ -79,10 +81,10 @@ void mergeXfdfIntoPdf(XfdfObject xfdfObject, PdfDocument pdfDocument, String pdf
7981
if (pdfDocumentName.equalsIgnoreCase(xfdfObject.getF().getHref())) {
8082
logger.info("Xfdf href and pdf name are equal. Continue merge");
8183
} else {
82-
logger.warn("Xfdf href attribute and pdfDocument name are different!");
84+
logger.warn(LogMessageConstant.XFDF_HREF_ATTRIBUTE_AND_PDF_DOCUMENT_NAME_ARE_DIFFERENT);
8385
}
8486
} else {
85-
logger.warn("No f object to compare.");
87+
logger.warn(LogMessageConstant.XFDF_NO_F_OBJECT_TO_COMPARE);
8688
}
8789
//TODO check for ids original/modified compatability with those in pdf document
8890

@@ -105,7 +107,7 @@ private void mergeFields(FieldsObject fieldsObject, PdfAcroForm form) {
105107
if (formFields.get(name) != null && xfdfField.getValue() != null) {
106108
formFields.get(name).setValue(xfdfField.getValue());
107109
} else {
108-
logger.error("No such field in pdf document!");
110+
logger.error(LogMessageConstant.XFDF_NO_SUCH_FIELD_IN_PDF_DOCUMENT);
109111
}
110112
}
111113
}
@@ -262,10 +264,10 @@ private void addAnnotationToPdf(AnnotObject annotObject, PdfDocument pdfDocument
262264
//XfdfConstants.LINK
263265
//XfdfConstants.REDACT
264266
//XfdfConstants.PROJECTION
265-
default: logger.warn("Annotation " + annotName + " is not supported.");
267+
default: logger.warn(MessageFormatUtil.format(LogMessageConstant.XFDF_ANNOTATION_IS_NOT_SUPPORTED, annotName));
266268
break;
267269
}
268270

269271
}
270272
}
271-
}
273+
}

forms/src/test/java/com/itextpdf/forms/Utf8FormsTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ This file is part of the iText (R) project.
5353
import com.itextpdf.kernel.pdf.PdfString;
5454
import com.itextpdf.kernel.pdf.PdfWriter;
5555
import com.itextpdf.kernel.utils.CompareTool;
56+
import com.itextpdf.test.ExtendedITextTest;
5657
import com.itextpdf.test.annotations.type.UnitTest;
5758
import org.junit.Assert;
5859
import org.junit.Before;
@@ -64,7 +65,7 @@ This file is part of the iText (R) project.
6465
import static com.itextpdf.test.ITextTest.createDestinationFolder;
6566

6667
@Category(UnitTest.class)
67-
public class Utf8FormsTest {
68+
public class Utf8FormsTest extends ExtendedITextTest {
6869

6970
public static final String sourceFolder = "./src/test/resources/com/itextpdf/forms/Utf8FormsTest/";
7071
public static final String destinationFolder = "./target/test/com/itextpdf/forms/Utf8FormsTest/";

0 commit comments

Comments
 (0)