Review java.time test refactoring

[Xueming Shen]

Looks fine.

-Sherman

On 10/01/2013 11:53 AM, roger riggs wrote:
> Ping, needs a Reviewer.
>
>   http://cr.openjdk.java.net/~rriggs/webrev-serial-refactor-8024896/
>
> Updated the webrev with the renamed test classes to distinguish them from the
> non-serialization tests in a different package.
>
> Thanks, Roger
>
> On 9/23/2013 11:04 AM, Stephen Colebourne wrote:
>> 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
>>>
>