Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added default jmeter.properties to the plugin #47

Closed
wants to merge 16 commits into from
Closed

Added default jmeter.properties to the plugin #47

wants to merge 16 commits into from

Conversation

foragerr
Copy link
Contributor

  1. Closes Plugin throws an error if a user properties file is not created #38
    Added default jmeter.properties from jmeter 2.13 into plugin.
    • If user sets jmeterPropertyFile, this value is used as jmeter properties
    • If user does not set , the jmeter.properties in $projectDir/build/jmeter is used
  2. Closes HTML report, jmeter-report error #45
    Set property jmeter.save.saveservice.output_format=xml in jmeterRun task, so results files are in xml format and can be transformed to HTML files for reports.

@djangofan
Copy link

I assume that a jmeter.properties file that you put in $srcDir/jmeter.properties will get copied into $projectDir/build/jmeter at runtime? If so, this sounds correct.

@foragerr
Copy link
Contributor Author

@djangofan no, if plugin user does not set jmeterPropertyFile, then I'm defaulting to using the jmeter.properties that ships with jmeter version 2.13.

Should I expect that there is always a jmeter.properties in srcDir?

@foragerr
Copy link
Contributor Author

Ok, made another change based on what you said. Which jmeter.properties to use is now determined in this order:

  1. Is jmeterPropertyFile defined in build.gradle? Then use it
  2. Otherwise, look in $srcDir/test/jmeter and use jmeter.properties from there, if it exists
  3. If neither, use default jmeter.properties - same as the one shipped with jmeter 2.13

@foragerr foragerr mentioned this pull request Apr 27, 2015
@foragerr
Copy link
Contributor Author

@kulya can you please accept this pull-req or suggest any changes you want made?

@@ -229,6 +248,7 @@ private String executeJmeterTest(String fileLocation) {
specs.getSystemProperties().put("saveservice_properties", System.getProperty("saveservice_properties"));
specs.getSystemProperties().put("upgrade_properties", System.getProperty("upgrade_properties"));
specs.getSystemProperties().put("log_file", System.getProperty("log_file"));
specs.getSystemProperties().put("jmeter.save.saveservice.output_format","xml");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this necessary, as i recall this used to be in the jmeter.properties file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the line in jmeter.properties (it is commented out)

#jmeter.save.saveservice.output_format=csv

which implies that the default format is xml. But for some reason, when using this plugin to run a jmeter test, the results are in csv format. see #45. I added this line to remedy that.

Erwin Jansen and others added 12 commits May 27, 2015 12:04
generation. For now it generates all the reports.

- Bumped version number.
existing project declares the plugins we will remove the in built ones.
- Make the extension optional, so the optional jars are only added when
  needed.
- Undid some layout changes.
- Updated documentation.
Conflicts:
	src/main/groovy/com/github/kulya/gradle/plugins/jmeter/JmeterAbstractTask.groovy
Conflicts:
	build.gradle
	src/main/groovy/com/github/kulya/gradle/plugins/jmeter/JmeterPluginConvention.java
	src/main/groovy/com/github/kulya/gradle/plugins/jmeter/JmeterRunTask.java
Conflicts:
	build.gradle
	src/main/groovy/com/github/kulya/gradle/plugins/jmeter/JmeterAbstractTask.groovy
@foragerr foragerr closed this Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTML report, jmeter-report error Plugin throws an error if a user properties file is not created
4 participants