db-jdo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michelle Caisse (JIRA)" <j...@apache.org>
Subject [jira] Closed: (JDO-611) GetObjectIdForPersistentInterface - Implementation Class / PICompany / ICompany Mess
Date Fri, 14 Nov 2008 21:55:53 GMT

     [ https://issues.apache.org/jira/browse/JDO-611?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Michelle Caisse closed JDO-611.
-------------------------------

       Resolution: Fixed
    Fix Version/s: JDO 2 maintenance release 2 update 1
                   JDO 2 maintenance release 3

Fixed in trunk and 2.2.1

> GetObjectIdForPersistentInterface - Implementation Class / PICompany / ICompany  Mess
> -------------------------------------------------------------------------------------
>
>                 Key: JDO-611
>                 URL: https://issues.apache.org/jira/browse/JDO-611
>             Project: JDO
>          Issue Type: Test
>          Components: tck2
>    Affects Versions: JDO 2 maintenance release 2
>            Reporter: Ilan Kirsh
>            Assignee: Michelle Caisse
>             Fix For: JDO 2 maintenance release 3, JDO 2 maintenance release 2 update
1
>
>
> I have two issues with this test:
> (1)
> IMO, PICompany should be written instead of ICompany:
> (a)
> ICompany icompany = (ICompany)pm.newInstance(PICompany.class);
>   instead of:
> ICompany icompany = (ICompany)pm.newInstance(ICompany.class);
> and:
> (b)
> addTearDownClass(PICompany.class);
>   instead of:
> addTearDownClass(ICompany.class);
> ICompany is not persistence capable, so the implementation does not have to handle it.
> (2)
> I think that the following check is wrong:
> if (icompanyOidClass != icompany.getClass())
> icompany.getClass()  => the physical internal implementation class
> icompanyOidClass  => the logical interface (PICompany)
> Actually and exception should be thrown if (icompanyOidClass == icompany.getClass())

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message