Hey Tyler, assertNotNull returns void whereas I needed something testable for assumeTrue (since apparently gdal doesn’t always print out the Files output on all systems and versions which I found out yesterday).
Make sense? Cheers, Chris ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Chris Mattmann, Ph.D. Chief Architect Instrument Software and Science Data Systems Section (398) NASA Jet Propulsion Laboratory Pasadena, CA 91109 USA Office: 168-519, Mailstop: 168-527 Email: [email protected] WWW: http://sunset.usc.edu/~mattmann/ ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Adjunct Associate Professor, Computer Science Department University of Southern California, Los Angeles, CA 90089 USA ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ -----Original Message----- From: Tyler Palsulich <[email protected]> Reply-To: "[email protected]" <[email protected]> Date: Thursday, October 1, 2015 at 6:39 AM To: "[email protected]" <[email protected]>, "[email protected]" <[email protected]> Subject: Re: svn commit: r1706077 - /tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/gdal/TestGDAL Parser.java >Hi Chris, > >It looks like these two lines are equivalent (assert not null versus >assert >true not null). Right? > >Tyler > >On Wed, Sep 30, 2015, 9:45 AM <[email protected]> wrote: > >> Author: mattmann >> Date: Wed Sep 30 16:45:32 2015 >> New Revision: 1706077 >> >> URL: http://svn.apache.org/viewvc?rev=1706077&view=rev >> Log: >> - Files isn't always present (just found test case on older version of >> GDAL) >> >> Modified: >> >> >>tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/gdal/TestGDA >>LParser.java >> >> Modified: >> >>tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/gdal/TestGDA >>LParser.java >> URL: >> >>http://svn.apache.org/viewvc/tika/trunk/tika-parsers/src/test/java/org/ap >>ache/tika/parser/gdal/TestGDALParser.java?rev=1706077&r1=1706076&r2=17060 >>77&view=diff >> >> >>========================================================================= >>===== >> --- >> >>tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/gdal/TestGDA >>LParser.java >> (original) >> +++ >> >>tika/trunk/tika-parsers/src/test/java/org/apache/tika/parser/gdal/TestGDA >>LParser.java >> Wed Sep 30 16:45:32 2015 >> @@ -69,7 +69,7 @@ public class TestGDALParser extends Tika >> assertNotNull(met); >> assertNotNull(met.get("Driver")); >> assertEquals(expectedDriver, met.get("Driver")); >> - assertNotNull(met.get("Files")); >> + assumeTrue(met.get("Files") != null); >> assertNotNull(met.get("Coordinate System")); >> assertEquals(expectedCoordinateSystem, met.get("Coordinate >> System")); >> assertNotNull(met.get("Size")); >> >> >>
