Fix jakartaee/persistence#713, TCK should not try to lookup by time value with fractional seconds

[beikov]

Fixes Issue
jakartaee/persistence#713

Describe the change
TCK should not try to lookup by time value with fractional seconds

CC @alwin-joseph @anajosep @arjantijms @cesarhernandezgt @dblevins @m0mus @edbratt @gurunrao @jansupol @jgallimore @kazumura @kwsutter @LanceAndersen @bhatpmk @RohitKumarJain @shighbar @gthoman @brideck @OndroMih @dmatej
@starksm64 @scottmarlow

[arjantijms]

Something to remember when fixing Persistence tests like these; this is the fork. We also have the original that we intend to transfer to Gavin and team. See jakartaee/persistence#702

Is this fix also needed in the original?

[scottmarlow]

Something to remember when fixing Persistence tests like these; this is the fork. We also have the original that we intend to transfer to Gavin and team. See jakartaee/persistence#702

Is this fix also needed in the original?

I think this pull request and other ones for the persistence-outside-container tests should be made to the Persistence Spec repo after jakartaee/persistence#715 is merged (either as a service 3.2 release change or for 4.0).

[scottmarlow]

Something to remember when fixing Persistence tests like these; this is the fork. We also have the original that we intend to transfer to Gavin and team. See jakartaee/persistence#702
Is this fix also needed in the original?

I think this pull request and other ones for the persistence-outside-container tests should be made to the Persistence Spec repo after jakartaee/persistence#715 is merged (either as a service 3.2 release change or for 4.0).

Although we can continue to make Persistence changes to the tcks/apis/persistence/persistence-inside-container

[scottmarlow]

Something to remember when fixing Persistence tests like these; this is the fork. We also have the original that we intend to transfer to Gavin and team. See jakartaee/persistence#702

We will keep the tcks/apis/persistence/persistence-outside-container in the Platform TCK until Persistence 4.0 is ratified. The persistence-outside-container tests are used to test persistence providers but not (Jakarta EE) Persistence containers.

The persistence-inside-container tests are still used to test (Jakarta EE) Persistence containers.

Is this fix also needed in the original?

Yes as we will keep the persistence-outside-container in the Platform TCK source repo (main branch) until Persistence 4.0 is ratified. We also need the fix in persistence-inside-container as well for EE testing.