[ 
https://issues.apache.org/jira/browse/UIMA-45?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12463614
 ] 

Michael Baessler commented on UIMA-45:
--------------------------------------

I'm not done with all of that... I did #1 and some on #2 but did not 
complete... Hopefully I get this done this week since I will be on vacation 
next week.

> Review and clean up unit tests
> ------------------------------
>
>                 Key: UIMA-45
>                 URL: https://issues.apache.org/jira/browse/UIMA-45
>             Project: UIMA
>          Issue Type: Task
>          Components: Build, Packaging and Test
>            Reporter: Adam Lally
>         Assigned To: Michael Baessler
>            Priority: Minor
>             Fix For: 2.1
>
>
> Things to review and clean up:
> 1) The way that the tests find resource files is hacked up right now.  For 
> legacy reasons they use classes in the junit_extension package to locate 
> files.  The location of the files is decided by 
> TestPropertyReader.getJUnitBasePath().  Since Maven requires (or at least 
> strongly encourages) loading resources through the classpath, I hacked up 
> getJUnitBasePath() to do a ClassLoader lookup on a dummy resource and return 
> the directory in which it is found.  This ought to be cleaned up at some 
> point.
> 2) grep for TODO in src/test/java in both uimaj-core and uimaj-cpe.  There 
> are many things that need to be addressed.  Several unit tests were commented 
> out entirely because they didn't work right in Maven for some reason.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to