Bug 58943 - Create a Better Think Time experience
Summary: Create a Better Think Time experience
Status: RESOLVED FIXED
Alias: None
Product: JMeter
Classification: Unclassified
Component: Main (show other bugs)
Version: Nightly (Please specify date)
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---
Assignee: JMeter issues mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-29 16:42 UTC by UbikLoadPack support
Modified: 2017-02-02 16:04 UTC (History)
1 user (show)



Attachments
Step 1 (73.34 KB, image/png)
2017-01-16 11:16 UTC, UbikLoadPack support
Details
Step 2 (160.53 KB, image/png)
2017-01-16 11:16 UTC, UbikLoadPack support
Details
Step 3 (113.83 KB, image/png)
2017-01-16 11:16 UTC, UbikLoadPack support
Details
Patch implementing the feature (17.96 KB, patch)
2017-01-16 11:20 UTC, UbikLoadPack support
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description UbikLoadPack support 2016-01-29 16:42:04 UTC
It should be more friendly to create think times than how it can be done today.
Also newbies and many experienced testers still make the error of thinking timer applies after the Sample.

Maybe a config element would be a solution.
Comment 1 Sebb 2016-01-29 20:45:45 UTC
It's not clear what you are proposing here.
Comment 2 benoit.wiart 2016-01-29 21:10:31 UTC
ihmo, there are 2 pbs:
- timers are executed before the sample -> when writing a functional test plan people set the timer after an action (diplay a page, wait n seconds then go to another page)
- two many non constant timers -> the timers (constant, gaussian, poisson and uniform ) should be merge into a single gui, today you need a phd to choose a timer.
Comment 3 Philippe Mouawad 2016-08-18 21:29:11 UTC
Maybe another option is to code a particular Sampler that returns no SampleResult.
This would allow this New Timer to be placer at the same level a Samplers, leading to clear positionned pauses.

I think Timer scoping is a concept highly misunderstood and not very usable whenever Think Times is very different between samplers.
Comment 4 Sebb 2016-08-18 21:41:16 UTC
(In reply to Philippe Mouawad from comment #3)
> Maybe another option is to code a particular Sampler that returns no
> SampleResult.

The Test Sampler is exactly that.
Comment 5 Sebb 2016-08-18 21:44:11 UTC
(In reply to benoit.wiart from comment #2)
> ihmo, there are 2 pbs:
> - timers are executed before the sample -> when writing a functional test
> plan people set the timer after an action (diplay a page, wait n seconds
> then go to another page)

Changing that would break too many test plans.

Also what about the last sample in a test - you don't want a pause after that.

> - two many non constant timers -> the timers (constant, gaussian, poisson
> and uniform ) should be merge into a single gui, today you need a phd to
> choose a timer.

Not sure that merging the GUIs would help; however better documentation of what to choose would be good.
Comment 6 UbikLoadPack support 2017-01-16 11:11:56 UTC
Hello,
We've implemented a solution that might be interesting for you.
Find attached screenshots showing how it works.

Think Time in screenshots are a Test Action (pause =0) containing a Uniform Random Timer (configurable) with configurable through properties Pause and Range.



Regards
Comment 7 UbikLoadPack support 2017-01-16 11:16:05 UTC
Created attachment 34627 [details]
Step 1
Comment 8 UbikLoadPack support 2017-01-16 11:16:22 UTC
Created attachment 34628 [details]
Step 2
Comment 9 UbikLoadPack support 2017-01-16 11:16:35 UTC
Created attachment 34629 [details]
Step 3
Comment 10 UbikLoadPack support 2017-01-16 11:17:02 UTC
Screenshots show how you proceed to add Think Times easily on Thread Groups and Controllers.

This highly increases productivity on this painful steps where you currently have to make repetitive clicks, copy/pastes and insertions.
Comment 11 UbikLoadPack support 2017-01-16 11:20:54 UTC
Created attachment 34630 [details]
Patch implementing the feature
Comment 12 Philippe Mouawad 2017-01-16 20:07:33 UTC
Author: pmouawad
Date: Mon Jan 16 20:07:01 2017
New Revision: 1779097

URL: http://svn.apache.org/viewvc?rev=1779097&view=rev
Log:
Bug 58943 - Create a Better Think Time experience
Contributed by UbikLoadPack
Bugzilla Id: 58943

Added:
    jmeter/trunk/src/components/org/apache/jmeter/thinktime/
    jmeter/trunk/src/components/org/apache/jmeter/thinktime/DefaultThinkTimeCreator.java   (with props)
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/AddThinkTimeBetweenEachStep.java   (with props)
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/thinktime/
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/thinktime/ThinkTimeCreator.java   (with props)
Modified:
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/ActionNames.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/util/MenuFactory.java
    jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties
    jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties
    jmeter/trunk/src/core/org/apache/jmeter/threads/gui/AbstractThreadGroupGui.java
    jmeter/trunk/xdocs/changes.xml
    jmeter/trunk/xdocs/usermanual/properties_reference.xml
Comment 13 Philippe Mouawad 2017-01-16 20:14:04 UTC
Author: pmouawad
Date: Mon Jan 16 20:09:47 2017
New Revision: 1779098

URL: http://svn.apache.org/viewvc?rev=1779098&view=rev
Log:
Bug 58943 - Create a Better Think Time experience
Contributed by UbikLoadPack
Bugzilla Id: 58943

Modified:
    jmeter/trunk/bin/jmeter.properties
Comment 14 Milamber 2017-02-02 16:04:48 UTC
URL: http://svn.apache.org/viewvc?rev=1781410&view=rev
Log:
Add French translation
Bugzilla Id: 58943

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties