jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Felix Schumacher <felix.schumac...@internetallee.de>
Subject Re: Proposition for contribution
Date Tue, 17 Jan 2017 21:01:02 GMT
Am 11.01.2017 um 09:44 schrieb Sébastien Col:
> 2017-01-11 7:38 GMT+01:00 Felix Schumacher <
> felix.schumacher@internetallee.de>:
>
>>
>> Am 10. Januar 2017 22:42:41 MEZ schrieb "Sébastien Col" <
>> sebastiencol@gmail.com>:
>>> Hi,
>>> I've been working tonight on unit testing
>>> org.apache.jmeter.core.report.
>>> I'm having troubles running the
>>> existing org.apache.jmeter.report.core.TestCsvSampleWriter with the
>>> following exception:
>>> java.lang.RuntimeException: Could not read JMeter properties
>>> file:jmeter.properties
>>> at
>>> org.apache.jmeter.util.JMeterUtils.loadJMeterProperties(
>> JMeterUtils.java:195)
>>> at
>>> org.apache.jmeter.report.core.TestCsvSampleWriter.
>> setUp(TestCsvSampleWriter.java:36)
>>> ...
>>>
>>> Anyone encountered this before?
>> How did you run the test? What is your setup?
>>
> I've setup sources using option 1 of the "Building JMeter" section of the
> documentation (the Ant task).
> Then I've imported the project in my Eclipse Neon. That's it.
>
> I have a JMETER_HOME set and I run the test from Eclipse (Run as -> JUnit
> Test).
Normally AllTests sets JMeterUtils JMeterHome. If you want to run the 
tests in isolation, you will have to set it yourself like:

Index: test/src/org/apache/jmeter/report/core/TestCsvSampleWriter.java
===================================================================
--- test/src/org/apache/jmeter/report/core/TestCsvSampleWriter.java 
(Revision 1779251)
+++ test/src/org/apache/jmeter/report/core/TestCsvSampleWriter.java 
(Arbeitskopie)
@@ -33,7 +33,10 @@
      @Before
      public void setUp() throws Exception {
          // We have to initialize JMeterUtils
-        JMeterUtils.loadJMeterProperties("jmeter.properties");
+        if (JMeterUtils.getJMeterHome() == null) {
+ JMeterUtils.setJMeterHome(System.getenv("JMETER_HOME"));
+        }
+ JMeterUtils.loadJMeterProperties(JMeterUtils.getJMeterBinDir() + 
"/jmeter.properties");
      }

      SampleMetadata metadata = new SampleMetadata(',', "a", "b");


(Which I will commit, if my tests run without failure).

Regards and sorry for the long delay.
  Felix

> Thanks for your help.
> Sebastien
>
>
>> Felix
>>
>>> Thanks.
>>> Regards,
>>> Sebastien
>>>
>>> 2017-01-06 20:35 GMT+01:00 Philippe Mouawad
>>> <philippe.mouawad@gmail.com>:
>>>
>>>> Hello,
>>>> Thanks for your proposal.
>>>> Yes it is still up to date.
>>>>
>>>> You can have a look at our Sonar report to know what needs testing:
>>>>
>>>>     - https://builds.apache.org/analysis/
>>>>
>>>> Our priorities regarding tests:
>>>>
>>>>     - JDBC protocol (more the NON Gui classes)
>>>>     - HTTP (more non gui classes)
>>>>     - org.apache.jmeter.assertions:
>>>>        - .HTMLAssertion
>>>>        - JSR223Assertion
>>>>     - org.apache.jmeter.core.report
>>>>     - org.apache.jmeter.visualizers.backend.graphite
>>>>     - core/org/apache/jmeter/control
>>>>
>>>> Note coverage is not only related to JUnit tests but also to tests
>>> builds
>>>> with JMX plans and ran with Jacoco agent.
>>>>
>>>> Regards
>>>>
>>>>
>>>>
>>>> On Fri, Jan 6, 2017 at 5:32 PM, Sébastien Col
>>> <sebastiencol@gmail.com>
>>>> wrote:
>>>>
>>>>> Hi,
>>>>> I would like to contribute to the project and I thought I could
>>> start
>>>>> working on the task #41118bb8 related to test suite enhancement and
>>>>> migration to JUnit 4.
>>>>> I found it from the helpwanted.apache.org site.
>>>>> Is it still up-to-date?
>>>>> Regards,
>>>>> Sebastien
>>>>>
>>>>
>>>>
>>>> --
>>>> Cordialement.
>>>> Philippe Mouawad.
>>>>


Mime
View raw message