Review java.time test refactoring

Stephen Colebourne scolebourne at joda.org
Mon Sep 23 15:04:26 UTC 2013


The change looks fine AFAICT. I would prefer to see the new serial
package test classes have names that end in "Serialization", eg
"TCKLocalDateSerialization", but if that does not happen it is not the
end of the world.
Stephen


On 18 September 2013 22:31, roger riggs <roger.riggs at oracle.com> wrote:
> A review of the TCK tests for java.time resulted in a number of changes
> including refactoring the serialization tests into subpackages to follow JCK
> conventions,
> discovery that a number of tests were missing for serialization, and cleanup
> of the test code to utilize common test functions for testing serialization.
> Thanks to the JCK folks for identifying the issues and doing some of the
> refactoring.
>
> Webrev:
>   http://cr.openjdk.java.net/~rriggs/webrev-serial-refactor-8024896/
>
>  * JDK-8024896 <https://bugs.openjdk.java.net/browse/JDK-8024896>
>    Refactor java.time serialization tests into separate subpackage
>
>  * JDK-8024427 <https://bugs.openjdk.java.net/browse/JDK-8024427>
>    Missing java.time.chrono serialization tests
>
> Thanks, Roger
>



More information about the core-libs-dev mailing list