refactor(JF-466): deactivate this feature by default & add switch to turn it on

Author: axherrm

src/main/java/org/jadice/filetype/matchers/PDFMatcher.java

@@ -68,6 +68,10 @@ public class PDFMatcher extends Matcher {
   public static final String TEXT_LENGTH_KEY = "text-length";
   public static final String TEXT_LENGTH_PER_PAGE_KEY = "text-length-per-page";
 
+  private static boolean lookForText() {
+    return "true".equalsIgnoreCase(System.getProperty(PDFMatcher.class.getName() + ".lookForText", "false"));
+  }
+
   /*
    * (non-Javadoc)
    *
@@ -129,8 +133,8 @@ public boolean matches(final Context context) {
           fileLength = getFileLength(sis);
         }
         PDFBoxSignatureUtil.addSignatureInfo(pdfDetails, document, fileLength);
-
-        addTextInfo(pdfDetails, document);
+        if (lookForText())
+          addTextInfo(pdfDetails, document);
       }
 
       return true;

src/test/java/TestPDFMatcher.java

@@ -159,6 +159,7 @@ void testSignedPDFs(final String urlString, final int expectedSignatureCount) th
   @ParameterizedTest
   @CsvFileSource(resources = "/pdf/contains-text.csv", numLinesToSkip = 1)
   void testContainsText(final String filePath, final boolean expected) throws IOException {
+    System.setProperty(PDFMatcher.class.getName() + ".lookForText", "true");
     Map<String, Object> result = ANALYZER.analyze(new File(filePath));
     assertNotNull(result);
     assertTrue(result.containsKey(PDFMatcher.DETAILS_KEY));
@@ -174,6 +175,7 @@ void testContainsText(final String filePath, final boolean expected) throws IOEx
       final int sum = textLengthPerPages.stream().mapToInt(Integer::intValue).sum();
       assertEquals(totalTextLength, sum);
     }
+    System.clearProperty(PDFMatcher.class.getName() + ".lookForText");
   }
 
   private static File[] nullSafe(final File[] filesOrNull) {