RFR (S): 8072687: Update OutputAnalyzer to work with files

[Dmitry Fazunenko]

On 18.02.2015 12:34, Staffan Larsen wrote:
> I can only see one usage of the OutputAnalyzer(String) constructor and that is in test/gc/metaspace/CompressedClassSpaceSizeInJmapHeap.java. That usage does not require both stdout and stderr to be set.
>
> In fact, that usage would benefit from the new constructor that takes a File.
>
> I suggest we do:
>
> - Change OutputAnalyzer(String) to set stdout to the input string and stderr to the empty string ""
> - Update CompressedClassSpaceSizeInJmapHeap.java to use the new OutputAnalyzer(File) constructor

sounds good to me.

-- Dima

>
> Thanks,
> /Staffan
>
>
>> On 17 feb 2015, at 19:56, Dmitry Fazunenko <Dmitry.Fazunenko at oracle.com> wrote:
>>
>> Hi Staffan,
>>
>> For me it's not clear why the constructor taking a String argument sets both stderr and stdout to the same value...
>> But I don't see any reason to update this behavior and break compatibility.
>> So I would agree, that the new constructor should be consistent with the existing one and set both out and err to the same string.
>>
>> Thanks,
>> Dima
>>
>>
>>
>> On 17.02.2015 20:39, Staffan Larsen wrote:
>>> There is constructor that takes a String argument. That constructor sets both stderr and stdout to that same String. Your constructor behaves differently and only sets stdout to the contents of the file. I have no idea why the existing constructor does what it does, but it would be good if the new and old had the same behavior. I haven’t looked at the use cases for the existing constructor.
>>>
>>> /S
>>>
>>>> On 17 feb 2015, at 16:58, denis kononenko <denis.kononenko at oracle.com> wrote:
>>>>
>>>> Hi All,
>>>>
>>>> Could you please review a small change of OutputAnalyzer class from testlibrary.
>>>>
>>>> Webrev link: http://cr.openjdk.java.net/~dfazunen/dkononen/8072687/webrev.00/
>>>> Bug id: https://bugs.openjdk.java.net/browse/JDK-8072687
>>>> Testing: automated
>>>> Description:
>>>>
>>>> The purpose of this change is to extend the functionality of OutputAnalyzer by adding ability to read output from a file. Sometimes we have to analyze output written to a file (test data, logs, etc.). In that case we have to read it as a string and pass it into OutputAnalyzer's constructor (it could be done in several different ways). It would be very convenient to put this code directly into OutputAnalyzer. This would allow to make further tests code more cleaner and less error prone.
>>>>
>>>> The change consist of two parts:
>>>> 1) The change: OutputAnalyzer.java, added a new constructor that accepts File;
>>>> 2) The tests: FileOutputAnalyzerTests.java, very basic tests on reading functionality.
>>>>
>>>> Thank you,
>>>> Denis.
>>>>
>>>>