Re: [eclipselink-dev] bug 297332: JNDI lookupstring for SessionBeanTests

[Kevin Yuan <kevin.yuan@xxxxxxxxxx>]

Hi Adrian,
The changes with the test code are fine, and also verified the tests on WLS. But the minimum change should add jndi lookup for NetWeaver after JBoss and before "All lookups failed", and don't need to move back all other code, just like the following:

            try {
                service = (EmployeeService) PortableRemoteObject.narrow(context.lookup("java:comp/env/ejb/EmployeeService"), EmployeeService.class);
            } catch (NameNotFoundException notFoundException) {
                try {
                    service = (EmployeeService) PortableRemoteObject.narrow(context.lookup("ejb/EmployeeService"), EmployeeService.class);
                } catch (NameNotFoundException notFoundException2) {
                    try {
                        // WLS likes this one.
                        service = (EmployeeService) PortableRemoteObject.narrow(context.lookup("EmployeeService#org.eclipse.persistence.testing.models.jpa.sessionbean.EmployeeService"), EmployeeService.class);
                    } catch (NameNotFoundException notFoundException3) {
                        try {
                             //jboss likes this one
                             service = (EmployeeService) PortableRemoteObject.narrow(context.lookup("EmployeeService/remote"), EmployeeService.class);
                        } catch (NameNotFoundException notFoundException4) {
                             try {
                                 // NetWeaver likes this one.
                                 service = (EmployeeService) PortableRemoteObject.narrow(context.lookup("JavaEE/servertest/REMOTE/EmployeeServiceBean/org.eclipse.persistence.testing.models.jpa.sessionbean.EmployeeService"), EmployeeService.class);
                             } catch (NameNotFoundException notFoundException5) {
                                 throw new Error("All lookups failed.", notFoundException);
                             }
                        }
                    }
                }
            }
        }

By the way, if you want to run nativeapitest in server-test-lrg, you also need to make the same changes with: trunk\jpa\eclipselink.jpa.test\src\org\eclipse\persistence\testing\tests\nativeapitest\NativeAPITests.java

Thanks,
Kevin

Goerler, Adrian wrote:

Hi,

 

on NetWeaver, we are having an issue with the SessionBeanTests as the JNDI lookup string used by NetWeaver is not among the strings tried by the test:

 

https://bugs.eclipse.org/bugs/show_bug.cgi?id=297332

 

I am proposing a patch, which fixes the issue on NetWeaver. Please review and give feedback.

 

 

Thanks,

 

Adrian

 

=====

 

Adrian Görler
SAP AG

Pflichtangaben/Mandatory Disclosure Statements: http://www.sap.com/company/legal/impressum.epx

 

 

 

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev