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

JPOS Framework and JMeter is works #4

Open
wants to merge 47 commits into
base: master
Choose a base branch
from

Conversation

erlanggaelfallujah
Copy link
Contributor

I have been success integration jpos framework and jmeter, and it is work. I am created only one channel (NACChannel), maybe u or everyone can add other channel. And then, i will create wiki or how to run the plugin :).

@yciabaud
Copy link
Owner

I have to test it in jmeter before merging because it seems that your UI does not fit JMeter good practices. You have to provide input fields for the file and for the content in order to let the user be able to use variables.

I will have a closer look and I will let you know.

@erlanggaelfallujah
Copy link
Contributor Author

Which section my UI does not fit JMeter good practices? Oh, I have provide input fields for file, because i think user can using custom packager, and using input file for data fields for make easy user for dynamic field bit. I think no problem for using input fields for file :)

@yciabaud
Copy link
Owner

I did not manage to have the time to test it yet, I am not sure but I saw you suppressed the text fields and this is very useful for the user to put some raw data or jmeter variables.

In your revision, can the user set the data with variables?
I will try to test soon.

@erlanggaelfallujah
Copy link
Contributor Author

Ok, I will waiting to you for test it. And you can read "README.md" on the section "See Mom" for how to input field :) .

@yciabaud
Copy link
Owner

I have tested your repo on jmeter 2.11 and I came across some issues.

First, TCP data is lost on save, so I cannot run the test.
Secondly, like I suggested, I would add a textarea in order to be able to add JMeter variable to the field content (look at TCP sample for example).
To finish, the file path should also be a text field in order to be able to set it from a variable (think that the unit test can be launched in command line on multiple remote machines for instance).

Do you understand what I mean?

@erlanggaelfallujah
Copy link
Contributor Author

Hmmm.. OK, i am understand, but can you explain more detail TCP data is lost on save ?

@yciabaud
Copy link
Owner

Yes, the server name, the port and the timeout is cleared from the GUI on
save, the data is well persisted in the jmx file but is not in the
component and therefor the execution does nothing. I will try to see what
is going on if you need.

Le Tue Jan 20 2015 at 04:09:02, Erlangga [email protected] a
écrit :

Hmmm.. OK, i am understand, but can you explain more detail TCP data is
lost on save ?


Reply to this email directly or view it on GitHub
#4 (comment)
.

@erlanggaelfallujah
Copy link
Contributor Author

Hi Yoann, how about this link:
https://jmeter.apache.org/api/org/apache/jmeter/gui/util/FileDialoger.html

JMeter provide FileDialoger class, which is provide promptToOpenFile(), promptToOpenFile(String existingFileName), and promptToOpenFile(String[] exts) methods.

And, how can if user input custom packager, whether by providing text area, or ... ???

@yciabaud
Copy link
Owner

yciabaud commented Feb 1, 2015

Hi,
you can look at the BSF sampler for example.
image

There is a text field where the path is stored and a browse button.
If the user selects a file, the path is set in the text field.
This allows the user to set an expression like ${path} if he needs.

For the data, you should provide a textarea (just like BSFSampler) in addition to the file path. Just tell the user that setting the path overrides de data text area content.

Do you see what I mean?

Rangga Labs and others added 26 commits October 31, 2016 10:29
display iso message request in view result on tree controller
…-jPOS-components

# Conflicts:
#	src/main/java/org/apache/jmeter/gui/custom/CustomTCPConfigGui.java
#	src/main/java/org/apache/jmeter/samplers/jpos/JPOSSampler.java
#	src/test/java/org/apache/jmeter/samplers/jpos/MockJPOSSampler.java
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.

4 participants