db-jdo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Watzek (JIRA)" <j...@apache.org>
Subject [jira] Updated: (JDO-147) Remove pm.deletePersistent from query tests
Date Thu, 01 Dec 2005 16:25:31 GMT
     [ http://issues.apache.org/jira/browse/JDO-147?page=all ]

Michael Watzek updated JDO-147:
-------------------------------

    Attachment: JDO-147.patch

The attached patch fixes this issue.

Methods QueryTest.cleanupCompanyModel and QueryTest.cleanupDatabase have been removed. Instead,
localSetup methods have been introduced in all JDO1 query tests.

All JDO1 query tests (except jdoql.operator tests) have been adapted to make use of the query
framework implemented for JDO2 query tests. Thus, JDO1 query tests cases are executed as API
queries and as single string queries. Furthermore, this ensures that JDO1 and JDO2 use the
same logging infrastructure.

Additionally, methods QueryTest.getCompanyModelInstanceXXX have been removed. Instead, methods
QueryTest.getPersistentCompanyModelInstanceXXX and QueryTest.getTransientCompanyModelInstanceXXX
have been introduced. The expected result of queries is initialized using QueryTest.getTransientCompanyModelInstanceXXX.
Parameters of queries are initialized using QueryTest.getPerisistentCompanyModelInstanceXXX.
The same holds true for QueryTest.getMylibInstanceXXX methods.


> Remove pm.deletePersistent from query tests
> -------------------------------------------
>
>          Key: JDO-147
>          URL: http://issues.apache.org/jira/browse/JDO-147
>      Project: JDO
>         Type: Bug
>   Components: tck20
>     Reporter: Michelle Caisse
>     Assignee: Michael Watzek
>  Attachments: JDO-147.patch
>
> The query tests delete the persistent instance that they create at the end of the test
using pm.deletePersistent().  Now that we use the new cleanup mechanism (addTearDownClass(Class)),
there is no reason to deletie the instance.  Also, the new maven option jdo.tck.cleanupaftertest
cannot override this cleanup, so you can never see the database contents for debugging.  Therefore,
we should remove the line that deletes the persistent instance.

-- 
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


Mime
View raw message