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
WFLY-3825 (Build has failed because it depends on jbossas/jboss-ejb-client#73) #6683
Conversation
Can one of the admins verify this patch? |
The parameter ERROR_MESSAGE is the message at the client side against which the test is done. This param can be modified according to the error message that is needed to be verified. |
|
||
final Hashtable jndiProperties = new Hashtable(); | ||
|
||
jndiProperties.put(Context.INITIAL_CONTEXT_FACTORY, "org.jboss.naming.remote.client.InitialContextFactory"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For some reason indentation goes off the mark.
This PR relates to wildfly/jboss-ejb-client#73 |
This is OK to test. |
@@ -23,6 +23,7 @@ | |||
<jbossas.ts.integ.dir>${basedir}/..</jbossas.ts.integ.dir> | |||
<jbossas.ts.dir>${jbossas.ts.integ.dir}/..</jbossas.ts.dir> | |||
<jbossas.project.dir>${jbossas.ts.dir}/..</jbossas.project.dir> | |||
<surefire.ejbClient.memory.args>-Xms512m -Xmx1280m -XX:MaxPermSize=256m</surefire.ejbClient.memory.args> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong indentation.
this is ok to test |
Linux Build 4755 is now running using a merge of a027386 |
Linux Build 4755 outcome was FAILURE using a merge of a027386 Build problems:Failed tests detected Failed tests
|
@ctomc #6683 (comment) -> case sensitivity? :) |
We require that all contributions be made under the terms of the MIT License, http://www.opensource.org/licenses/mit-license.php. Do you agree to the these terms? |
@stuartwdouglas Panos is a new employee of the Carlo's SET team. So I think there is no signing necessary. |
Linux Build 4764 is now running using a merge of a5a803b |
@stuartwdouglas I agree with the terms of MIT License. |
Build has failed because it depends on wildfly/jboss-ejb-client#73 |
Linux Build 4764 outcome was FAILURE using a merge of a5a803b Build problems:Failed tests detected Failed tests
|
Linux Build 4766 is now running using a merge of 55914d5 |
Linux Build 4766 outcome was FAILURE using a merge of 55914d5 Build problems:Failed tests detected Failed tests
|
…sages are displayed, when large parameters are given to an EJB method invocation. WFLY-3825 Develop manual test case to verify client console error messages are displayed, when large parameters are given to an EJB method invocation.
Linux Build 4769 is now running using a merge of b770a87 |
Linux Build 4769 outcome was FAILURE using a merge of b770a87 Build problems:Failed tests detected Failed tests
|
@rhusar -- it's not case sensitivity. You are not "one of the admins" of brontes. |
Can't this be tested somehow within jboss-ejb-client itself? We can't be adding manual mode tests, much less ones with a whole new surefire execution, for such tiny things. We'd end up with a 24 hour testsuite. |
ok. It will be added at ejb-client. |
Move to EJBCLIENT-117 #80 |
Develop manual test case to verify client console error messages are displayed, when large parameters are given to an EJB method invocation.