[
https://issues.apache.org/jira/browse/JDO-639?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12760106#action_12760106
]
Craig Russell commented on JDO-639:
-----------------------------------
Assuming that Michelle's analysis is correct (without verifying anything), let
me suggest that there are three bugs:
1. NPE instead of "you poor sod, the file name you gave me isn't accessible" in
DataNucleus.
2. javax.jdo.Enhancer fails to pass full path name for the files.
3. EnhancerTest case didn't catch this error.
I will take the second and third errors.
> tck enhancement should make use of feature to enhance an entire directory
> -------------------------------------------------------------------------
>
> Key: JDO-639
> URL: https://issues.apache.org/jira/browse/JDO-639
> Project: JDO
> Issue Type: Improvement
> Components: tck2
> Affects Versions: JDO 2 maintenance release 2
> Reporter: Michael Bouschen
> Assignee: Michelle Caisse
> Attachments: enhance.txt
>
>
> The current enhancer call as part of the runtck goal takes a list of .jdo
> files as an argument (see property jdo.tck.jdometadata.files in
> project.properties and maven.xml). The new enhancer invocation API allows to
> enhance all files of a given directory. We should investigate to make use of
> this feature in order to get rid of listing all the .jdo in the property
> jdo.tck.jdometadata.files.
> There is a similar issue with the properties jdo.tck.pcclasses.sources and
> jdo.tck.pcclasses.files:
> - Property jdo.tck.pcclasses.sources is used when checking whether
> (re)enhancing is required. Maybe a pattern
> org/apache/jdo/tck/api/**/*.java, org/apache/jdo/tck/pc/**/*.java
> can be used instead of listing all the persistent capable classes explicitly.
> - Property jdo.tck.pcclasses.files is used when copying the class file into
> the identitytype specific subdirectories before running the enhancer. Maybe a
> similar pattern could replace the list of class files.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
