[ http://issues.apache.org/jira/browse/JDO-188?page=comments#action_12356278 ]
Michael Bouschen commented on JDO-188: -------------------------------------- The patch looks good. Just one remark: the localSetUp method creates a second PM which is not closed in the test case. Should we add a localTearDown method coling this PM? > Add tests for static JDOHelper.getObjectIds methods > --------------------------------------------------- > > Key: JDO-188 > URL: http://issues.apache.org/jira/browse/JDO-188 > Project: JDO > Type: Test > Components: tck20 > Environment: TCK > Reporter: Craig Russell > Assignee: Craig Russell > Attachments: jdo-188.patch > > A8.3-11 [static Object[] getObjectIds (Object[] pcs); static Collection > getObjectIds (Collection pcs); These methods return the JDO identities of the > parameter instances. For each instance in the parameter, the getObjectId > method is called. They return one identity instance for each > persistence-capable instance in the parameter. The order of iteration of the > returned Collection exactly matches the order of iteration of the parameter > Collection]. > The test case api.persistencemanager.GetObjectIds needs a test method per > API. > The setUp method creates 50 persistent instances of type PCPoint and stores > the ids in an ArrayList and the instances in a separate ArrayList. The test > case call getObjectIds and verify that the object ids returned match the > objects ids that are in the list. > There are no negative tests. -- 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
