GUI editor for Beanshell doesn't always pick up changes, code is lost

[asfimport]

David Martinez (Bug 42947): Most of the time, copying code and pasting it into the beanshell script areas does not work. Pasting content then using control-S to save makes the content get cleared immediately. Pasting content then running the script uses the old content. Making small changes to the content then saving also keeps the old content.

It's annoying because It's much easier to use an editor without proper syntax highlighting, completion, etcetera (especially since most of JMeter's beanshell nodes do not support just making reference to a file, but that's a request for another day). This makes it impossible to use any editor other than the jmeter included editor (and even then small edits are usually lost). I'm much more productive in (insert actual programmer's editor here).

To reproduce, try this:

1. Add a beanshell postprocessor
2. Type "See me dissapear!" in the Script: Area
3. WIthout moving focus away, press Control-S to save.

Expected: The text does not dissapear. Actual: Text dissapears

As a workaround you can move away to another node then come back. But THIS DOES NOT ALWAYS WORK on longer scripts for some reason.

Severity: major OS: Mac OS X 10.4

[asfimport]

Scott Marlowe (migrated from Bugzilla): Note that I get the exact same goofy and unpredictable behaviour when editing a regular expression dialog in a regex postprocessor. The text is in the box, but when I hit run, jmeter doesn't actually run, and when I click out of the dialog and back in, my old regex is there staring back at me, laughing.

Using Jmeter 2.3RC3

[asfimport]

Sebb (migrated from Bugzilla): It appears that using Control+S to save a test plan has always been broken for GUI items that use TestBeanGUI.

For example, using CSV Dataset: enter a value in the filename field, and save the plan using control+S. Exit and reload the plan - the value has not been saved.

The difference in 2.3RC3 is that this behaviour has become obvious, as the saved version is now reflected in the GUI.

The same problem applies to other short-cut keys such as Control+R.

Normally the value in the GUI field is saved whenever there is a focus-lost event however this does not happen for short-cut keys.

[asfimport]

Alf Hogemark (migrated from Bugzilla): (In reply to comment 2)

It appears that using Control+S to save a test plan has always been broken for GUI items that use TestBeanGUI.

For example, using CSV Dataset: enter a value in the filename field, and save the plan using control+S. Exit and reload the plan - the value has not been saved.

The difference in 2.3RC3 is that this behaviour has become obvious, as the saved version is now reflected in the GUI.

The same problem applies to other short-cut keys such as Control+R.

Normally the value in the GUI field is saved whenever there is a focus-lost event however this does not happen for short-cut keys.

The problem seems to be that the "propertyChange" event for the PropertyEditor in the TestBeanGui is not fired until after the changes have been stored. If you change the comment of the BeanShell Postprocessor and hit Ctrl+S, that change is saved.

The problem is that the PropertyChange events have not been processed before the
TestBeanGui.modifyTestElement method is moving changes from gui to model. Therefore, the changes are not saved.

I do not see an easy fix at the moment, will have to investigate more on how the java PropertyEditor works in conjunction with keyboard shortcuts.

[asfimport]

Alf Hogemark (migrated from Bugzilla): (In reply to comment 1)

Note that I get the exact same goofy and unpredictable behaviour when editing a regular expression dialog in a regex postprocessor. The text is in the box, but when I hit run, jmeter doesn't actually run, and when I click out of the dialog and back in, my old regex is there staring back at me, laughing.

Using Jmeter 2.3RC3

I think you are in fact observing https://github.com/apache/jmeter/issues/1992, which I have just made a patch for.

[asfimport]

Sebb (migrated from Bugzilla): Added code to TestBeanGUI.modifyTestElement() to copy the values of the fields from the GUI.

Applied to SVN in r571348.

[asfimport]

Sebb (migrated from Bugzilla): There was a slight problem with the previous fix - only one change was saved, as there was a return in the loop.

This has been corrected in r571407.

BTW, I looked into propertyChange events - the event was not being fired for short-cut keys. This appears to be a feature of Swing/AWT.

The propertyChange code became redundant; now both TestBeanGUI and other GUI classes save the field contents in modifyTestElement().