Hi,
I took latest trunk from SVN and changed bin/jmeter.properties to have
jmeter.save.saveservice.thread_counts=true
Then I ran "ant clean download_jars install test". The only error that
showed is:
[java] There was 1 failure:
[java] 1)
testMaven(org.apache.jmeter.JMeterVersionTest)junit.framework.ComparisonFailure:
serializer expected:<2.7.[2]> but was:<2.7.[1]>
[java] at
org.apache.jmeter.JMeterVersionTest.testMaven(JMeterVersionTest.java:182)
[java] at sun.reflect.NativeMethodAccessorImpl.invoke0(Native
Method)
[java] at
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
[java] at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[java] at org.apache.jorphan.test.AllTests.main(AllTests.java:236)
Which seems to be unrelated to saveservice change. It also appears
without any change to properties.
So I see no tests failing after the change. Does it mean that we can do
the change right now?
Andrey Pokhilko
On 09/26/2014 04:24 PM, UBIK LOAD PACK Support wrote:
> Hi,
> Answering on behalf of Philippe as we had some discussion on this.
>
> What you can do is make the change and run the Ant task called "test".
> properties used by test are located in bin/testfiles/jmetertest.properties.
>
> You will get tests failures because all generated files will have an
> additional field (threads) , as some test compare result files with
> expected result files, they will logically fail.
> So nothing complex in the change just some work to do to fix either the
> test results or the properties used by test.
> Maybe this can be discussed here with PMC.
>
> Regards
>
> On Fri, Sep 26, 2014 at 3:18 PM, Andrey Pokhilko <apc4@ya.ru> wrote:
>
>> Thanks for the link, Philippe.
>>
>> What prevents this change from being done? Is there really complex tests
>> that break on this change? How can I help? (I'm not familiar with
>> JMeter's source tests, unfortunately).
>>
>> Andrey Pokhilko
>>
>> On 09/26/2014 02:05 PM, Philippe Mouawad wrote:
>>> Hi Andrey,
>>> This has been discussed in a previous thread, see:
>>>
>> http://mail-archives.apache.org/mod_mbox/jmeter-dev/201404.mbox/%3CCAOGo0VaiM=TdW_Fj9fE1De4W8jao8ox5ELTAFVHTyN=5SY0d+Q@mail.gmail.com%3E
>>> I think it is agreed upon by PMC members, changing this just breaks some
>>> Tests that need to be fixed.
>>>
>>>
>>>
>>
>
|