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

JBTM-1179 TXBridge demo test fails due to: IllegalArgumentException: nodeName is empty, or just whitespace #11

Merged
merged 1 commit into from
Jun 1, 2012

Conversation

zhfeng
Copy link
Contributor

@zhfeng zhfeng commented May 31, 2012

No description provided.

@paulrobinson
Copy link
Contributor

Amos,

I'm looking at your diff and I don't understand how this prevents the error we where seeing. Can you explain this to me?

Cheers.

.importFrom(new ZipFile(XTS_DEMO_ARCHIVE)).as(EnterpriseArchive.class);
return archive;
}

@Deployment(name = TXBRIDGE_DEMO_SERVICE_ARCHIVE, testable = false, order = 2)
public static Archive<?> createTxBridgeDemoServiceArchive() throws Exception {
Archive<?> archive = ShrinkWrap.create(ZipImporter.class, TXBRIDGE_DEMO_SERVICE_ARCHIVE)
Archive<?> archive = ShrinkWrap.create(ZipImporter.class, "txbridge-demo-service.jar")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TXBRIDGE_DEMO_SERVICE_ARCHIVE define as "../service/target/txbridge-demo-service"
at org.apache.catalina.core.ContainerBase.getLogger(ContainerBase.java:409) which get logName as following
org.apache.catalina.core.ContainerBase.[jboss.web].[default-host].[/../service/target/txbridge-demo-service] which I use byteman script to print.

at org.jboss.logmanager.LoggerNode.getOrCreate(LoggerNode.java:154)
there is ".." in the logName which cause problem when parsing this string.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's some good debugging!

Is this a known issue with JBoss logging? If not you should raise a Jira.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@paulrobinson
Copy link
Contributor

Can you tell me what the commands are to run these tests now? The problem I see is that it can no longer find the artefacts as you've modified the paths to them.

@zhfeng
Copy link
Contributor Author

zhfeng commented Jun 1, 2012

cd XTS
mvn clean install
cd TXBridge
mvn clean install

I just modified the deploy name but not the path.

@paulrobinson paulrobinson merged commit 7100a03 into jbosstm:master Jun 1, 2012
@paulrobinson
Copy link
Contributor

Thanks Amos, that worked.

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.

2 participants