Re: [jakartaee-tck-dev] Question about removing failing EJB legacy compa

[Scott Marlow <smarlow@xxxxxxxxxx>]

https://github.com/eclipse-ee4j/jakartaee-tck/pull/502 removes more than 
is mentioned in links [3][4][5] below.  The reason for removing the 
additional tests is for the same reason as discussed below.  The removed 
compatibility tests are all written to test deployment descriptors from 
before Jakarta EE 8 (e.g. J2EE 1.3-1.4).

List of removed tests:

src/com/sun/ts/tests/ejb/ee/deploy/session/stateful/compat*
src/com/sun/ts/tests/ejb/ee/deploy/session/stateless/compat*
src/com/sun/ts/tests/ejb/ee/deploy/entity/bmp/compat*
src/com/sun/ts/tests/ejb/ee/deploy/entity/cmp11/compat*

To ensure that enough time is given for feedback, the new deadline to 
request keeping any of these tests is next Wednesday morning (September 
16).  If the Jakarta Enterprise Beans team needs more time to respond, 
please speak up that you need more time.

Thanks,
Scott

On 9/11/20 12:00 PM, Scott Marlow wrote:
> We will remove the mentioned EJB legacy compatibility tests as agreed in 
> this discussion.
>
> Thanks,
> Scott
>
> On 9/8/20 11:50 AM, Scott Marlow wrote:
> > Hi,
> >
> > Does the EJB team have input to share on removing legacy EJB BMP/CMP 
> > compatibility tests that were written to test J2EE 1.2/1.3/1.4?
> >
> > As part of the conversion to use Jakarta EE 9 XSDs  [1], we updated 
> > 1.2/1.3/1.4 BMP/CMP compatibility tests.  Now that we are seeing test 
> > failures [2].  We are wondering if we should instead remove failing 
> > tests [3][4][5].
> >
> > We updated these tests to use the Jakarta EE 9 XSDs because Jakarta EE 
> > 9 implementations are only required to support Jakarta EE 8+ schemas.
> >
> > Could the EJB team give a quick answer on this (by Sept 10)?  If more 
> > than a few days is needed, please do indicate so.
> >
> > Scott
> >
> > [1] https://github.com/eclipse-ee4j/jakartaee-tck/pull/474
> > [2] 
> > https://github.com/eclipse-ee4j/jakartaee-tck/pull/474#issuecomment-688278559 
> >
> > [3] 
> > https://github.com/eclipse-ee4j/jakartaee-tck/blob/master/src/com/sun/ts/tests/ejb/ee/deploy/entity/bmp/compat13_14/Client.java#L76 
> >
> > [4] 
> > https://github.com/eclipse-ee4j/jakartaee-tck/blob/master/src/com/sun/ts/tests/ejb/ee/deploy/entity/bmp/compat12_13/Client.java#L78 
> >
> > [5] 
> > https://github.com/eclipse-ee4j/jakartaee-tck/blob/master/src/com/sun/ts/tests/ejb/ee/deploy/entity/cmp11/compat12_14/Client.java