[messaging-dev] Help needed for a Messaging TCK test challenge for the E

[Scott Marlow <scott.marlow.opensource@xxxxxxxxx>]

Dear Messaging team,

Could you please respond on the https://github.com/jakartaee/platform-tck/issues/2652 TCK challenge that claims it is invalid for TCK tests to use invalid ID:

>    Under §3.5.1, a JMS property name has to be a valid message-selector identifier
>    Under §3.8.1.1, selector identifiers are Java-identifier based; and Java identifier start/part characters are limited to the categories listed by Character.isJavaIdentifierStart/Part, which do not include hyphen-minus. So an application-defined JMS property name like foo-bar is non-conforming in Jakarta Messaging 3.1. (jakarta.ee (https://jakarta.ee/specifications/messaging/3.1/jakarta-messaging-spec-3.1.html))

The challenged EE 11 tests are:

com.sun.ts.tests.jms.ee.mdb.mdb_msgHdrQ.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgHdrT.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgPropsQ.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgPropsT.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgTypesQ1.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgTypesQ2.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgTypesQ3.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgTypesT1.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgTypesT2.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_msgTypesT3.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_rec.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_sndQ.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_sndToQueue.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_sndToTopic.ClientTest
com.sun.ts.tests.jms.ee.mdb.mdb_synchrec.ClientTest
com.sun.ts.tests.jms.ee.mdb.xa.ClientTest

The above tests are contained under https://github.com/jakartaee/platform-tck/tree/11.0.x/tcks/apis/messaging/messaging-inside-container/src/main/java/com/sun/ts/tests/jms

The potential fix from the TCK challenge creator for the Jakarta EE 11 Platform TCK is https://github.com/jakartaee/platform-tck/pull/2663.

Please give your input on whether the challenge should be accepted (here or in https://github.com/jakartaee/platform-tck/issues/2652) and whether the pull request looks correct (here or in pull/2663).

Thank you,

Scott