vlsi commented on code in PR #5761:
URL: https://github.com/apache/jmeter/pull/5761#discussion_r1180333204
##########
src/core/src/main/java/org/apache/jmeter/testelement/TestPlan.java:
##########
@@ -82,12 +85,23 @@ public void prepareForPreCompile()
}
/**
- * Fetches the functional mode property
- *
+ * Fetches the functional mode property<br>
+ * Could be change for no-GUI test with jmeter property: {@code
PROP_FUNCTIONAL_MODE}
* @return functional mode
*/
public boolean isFunctionalMode() {
- return getPropertyAsBoolean(FUNCTIONAL_MODE);
+ boolean functionalModeDefault = getPropertyAsBoolean(FUNCTIONAL_MODE);
+ log.debug("functionalModeDefault=" + functionalModeDefault);
+ boolean functionalModeReturn = functionalModeDefault;
+ if (isNonGui()) {
+ String propFunctionalModeProperty =
JMeterUtils.getProperty(PROP_FUNCTIONAL_MODE);
+ if (propFunctionalModeProperty != null) {
+ functionalModeReturn =
JMeterUtils.getPropDefault(PROP_FUNCTIONAL_MODE, functionalModeDefault);
+ log.info("Change with property " + PROP_FUNCTIONAL_MODE + ",
value=" + functionalModeReturn);
Review Comment:
I mean the wording tells nothing to the end user. What should the end user
do with the log message like `Change with property
jmeter.test_plan.functional_mode, value=true`? Is it good or bad? What does it
mean?
If you think it is important to tell users that the actual property was
adjusted, then something like `Overriding the value of "functional mode" for
component "..." to "..." since JMeter property "..." was set to "..."`.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
