RFR: 8193660 Check SOURCE line in "release" file for closedjdk

Randy Crihfield Randy.Crihfield at Oracle.com
Wed Feb 28 18:15:37 UTC 2018


On 02/28/18 11:58 AM, Alan Bateman wrote:
> On 28/02/2018 16:57, Randy Crihfield wrote:
>>
>> poke poke can someone please review?  Thanks
>>
> The `release` is a properties file so I assume you can simplify the 
> test by using Properties.load and then testing if the "SOURCE" key is 
> present and whether it has the expected value.
>
> -Alan.

I could though I actually already wrote the test as requested initially 
and read the file as desired at that time, which is in the workspace.

The current incantation is to expand the test to also check a closed 
build and streamline, rather than write two tests.

I would like to get approval as it is, but if I need to go back and 
change existing behavior I will.

Randy


More information about the core-libs-dev mailing list