Nick,

Fixed in r1569788.

Speaking of pdf parser tests, let me know if you come across any shareable pdfs 
that would be tests of TIKA-1232...looking for pdfs with:
1) with PDF/A conformance level 
2) PDFExtensionVersion
3) conflicting pdf version in header vs. trailer (standard PDFParser should 
handle this; NonSequentialParser is currently not reading the header version)

Cheers,

         Tim

-----Original Message-----
From: Allison, Timothy B. [mailto:talli...@mitre.org] 
Sent: Wednesday, February 19, 2014 10:18 AM
To: dev@tika.apache.org
Subject: RE: Failing test - PDFParserTest.testSequentialParser

Y. Sorry about that.  Changing to > 15 now.

-----Original Message-----
From: Nick Burch [mailto:apa...@gagravarr.org] 
Sent: Wednesday, February 19, 2014 10:10 AM
To: dev@tika.apache.org
Subject: RE: Failing test - PDFParserTest.testSequentialParser

On Wed, 19 Feb 2014, Allison, Timothy B. wrote:
> Did you, by chance, add a pdf to your local test-documents directory?

Yes! I have one uncommitted pdf in there

> There appear to be 16 pdfs in test-documents under parsers in trunk. The 
> goal of that test was to make sure that the test-documents directory was 
> set properly and there were actually x pdf documents tested.  When 
> someone adds a new pdf, that # will have to be incremented.

Hmm, if it were me, I'd probably have it check that there were at least a 
minimum number there (15?), and check that one or two long standing ones 
were found too? That ought to be less brittle WRT people prototyping new 
PDF tests

Nick

Reply via email to