You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jdo-dev@db.apache.org by "Michael Bouschen (JIRA)" <ji...@apache.org> on 2006/09/10 15:17:22 UTC
[jira] Commented: (JDO-423) Missing addTearDownClass in
org.apache.jdo.tck.query.jdoql.variables.VariablesWithoutExtent
[ http://issues.apache.org/jira/browse/JDO-423?page=comments#action_12433704 ]
Michael Bouschen commented on JDO-423:
--------------------------------------
Method localSetUp registers the NoExtent instance for removal during tearDown by calling addTearDownInstance(noExtent) at the end. We cannot use method addTearDownClass for class NoExtent. Method tearDown gets the extent for all classes registered using addTearDownClass and this would fail for class NoExtent, because the metatdata defines requires-extent="false".
I propose to close this issue as invalid. What do you think?
> Missing addTearDownClass in org.apache.jdo.tck.query.jdoql.variables.VariablesWithoutExtent
> -------------------------------------------------------------------------------------------
>
> Key: JDO-423
> URL: http://issues.apache.org/jira/browse/JDO-423
> Project: JDO
> Issue Type: Bug
> Components: tck20
> Affects Versions: JDO 2 final
> Reporter: Ilan Kirsh
> Priority: Minor
>
> Lines 101- 108, instead of:
> protected void localSetUp() {
> addTearDownClass(CompanyModelReader.getTearDownClasses());
> loadAndPersistCompanyModel(getPM());
> NoExtent noExtent = new NoExtent(1);
> makePersistent(noExtent);
> addTearDownInstance(noExtent);
> }
> should be:
> protected void localSetUp() {
> addTearDownClass(CompanyModelReader.getTearDownClasses());
> addTearDownClass(NoExtent.class); // Added missing addTearDownClass
> loadAndPersistCompanyModel(getPM());
> NoExtent noExtent = new NoExtent(1);
> makePersistent(noExtent);
> addTearDownInstance(noExtent);
> }
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira