Re: [ejb-dev] [jakartaee-tck-dev] Question about removing failing EJB le

[Scott Marlow <smarlow@xxxxxxxxxx>]

On 9/9/20 10:26 AM, Jean-Louis Monteiro wrote:
> Hi,
>
> On top of my head, there is only Interop and JAX-RPC related 
> features/tests pruned.
> Entity CMP/BMP are part of the optional spec (as opposed to required).
>
> But one may still want to implement and therefore test it, isn't it?

As per 
https://jakarta.ee/specifications/platform/9/platform-spec-9-SNAPSHOT.html#a3447 
Jakarta EE 9 implementations should only be required to support Jakarta 
EE 8+ schemas, so we cannot really run these compatibility tests as they 
were originally written (with older J2EE schemas).

Does the EJB SPEC team want to consider updating these compat tests now 
or state now that you might want to in the future?

If they are removed and you decide to re-purpose them in the future that 
is still possible as the test source is preserved in the git repository.

Further thoughts?

Thanks,
Scott

> --
> Jean-Louis Monteiro
> http://twitter.com/jlouismonteiro
> http://www.tomitribe.com <http://www.tomitribe.com/>
>
>
> On Tue, Sep 8, 2020 at 5:51 PM Scott Marlow <smarlow@xxxxxxxxxx 
> <mailto:smarlow@xxxxxxxxxx>> 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
>
>     _______________________________________________
>     ejb-dev mailing list
>     ejb-dev@xxxxxxxxxxx <mailto:ejb-dev@xxxxxxxxxxx>
>     To unsubscribe from this list, visit
>     https://www.eclipse.org/mailman/listinfo/ejb-dev
>
>
> _______________________________________________
> jakartaee-tck-dev mailing list
> jakartaee-tck-dev@xxxxxxxxxxx
> To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartaee-tck-dev