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

[WFCORE-770] Add handler parameters #833

Merged
merged 1 commit into from Jun 23, 2015

Conversation

kabir
Copy link
Contributor

@kabir kabir commented Jun 22, 2015

https://issues.jboss.org/browse/WFCORE-770

This is not really necessary yet, but it is currently hard to add more fields to be initialised in constructors due to the wide variety of constructors available in this class. So this is an attempt to pre-empt that.
e.g. when I was working on the ordered child resources, in the first iteration I ended up relying on overriding things from methods rather than adding more constructors due to the complexity.

@wildfly-ci
Copy link

Core - Full Integration Build 1161 is now running using a merge of f803e02

@wildfly-ci
Copy link

Linux Build 2024 is now running using a merge of f803e02

@wildfly-ci
Copy link

Windows Build 1568 is now running using a merge of f803e02

@wildfly-ci
Copy link

Windows Build 1568 outcome was SUCCESS using a merge of f803e02
Summary: Tests passed: 3212, ignored: 71 Build time: 0:22:34

@wildfly-ci
Copy link

Linux Build 2024 outcome was SUCCESS using a merge of f803e02
Summary: Tests passed: 3212, ignored: 71 Build time: 0:26:07

@wildfly-ci
Copy link

Core - Full Integration Build 1161 outcome was FAILURE using a merge of f803e02
Summary: Tests failed: 3 (3 new), passed: 2875, ignored: 395, muted: 2 Build time: 0:53:27

Build problems:

Failed tests detected

Failed tests

org.jboss.as.test.integration.ejb.client.descriptor.EJBClientDescriptorTestCase.testClientInvocationTimeout: <no details avaliable>

org.jboss.as.test.integration.ejb.timerservice.mgmt.PersistentCalendarTimerManagementTestCase.testResourceExistence: <no details avaliable>

org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

@wildfly-ci
Copy link

Core - Full Integration Build 1163 is now running using a merge of f803e02

@wildfly-ci
Copy link

Core - Full Integration Build 1163 outcome was SUCCESS using a merge of f803e02
Summary: Tests passed: 2880, ignored: 395 Build time: 0:48:19

@bstansberry bstansberry added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Jun 23, 2015
bstansberry added a commit that referenced this pull request Jun 23, 2015
@bstansberry bstansberry merged commit 87b5394 into wildfly:master Jun 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
3 participants