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-1885. Fix for RTS InboundBridgeTestCase failures #405

Merged
merged 1 commit into from Aug 27, 2013

Conversation

Projects
None yet
4 participants
@gytis
Copy link
Member

commented Aug 22, 2013

No description provided.

@jbosstm-bot

This comment has been minimized.

Copy link

commented Aug 22, 2013

Started testing this pull request: http://172.17.131.2/job/btny-pulls-narayana/1034/

@jbosstm-bot

This comment has been minimized.

Copy link

commented Aug 22, 2013

@jbosstm-bot

This comment has been minimized.

Copy link

commented Aug 22, 2013

try {
txSupport.getTransactionInfo();
Assert.fail("Failed to rollback the transaction.");
} catch (Throwable e) {

This comment has been minimized.

Copy link
@mmusgrov

mmusgrov Aug 22, 2013

Contributor

Isn't there a specific HTTP status code you should be asserting

This comment has been minimized.

Copy link
@gytis

gytis Aug 22, 2013

Author Member

If I had a transaction before, it gives 404 here. However, if there were no transaction, it used to give some sort of exception, maybe JAXB?

This comment has been minimized.

Copy link
@mmusgrov

mmusgrov Aug 22, 2013

Contributor

Then there is a bug in txSupport.getTransactionInfo() that needs fixing

This comment has been minimized.

Copy link
@gytis

gytis Aug 22, 2013

Author Member

OK, I'll check it again

This comment has been minimized.

Copy link
@mmusgrov

mmusgrov Aug 22, 2013

Contributor

If you don't want to fix it with this PR then feel free to raise a new JIRA

This comment has been minimized.

Copy link
@gytis

gytis Aug 23, 2013

Author Member

I'll fix it for this one, it's not much of the change

@jbosstm-bot

This comment has been minimized.

Copy link

commented Aug 23, 2013

All tests passed - Job complete http://172.17.131.2/job/btny-pulls-narayana/1034/

@jbosstm-bot

This comment has been minimized.

Copy link

commented Aug 23, 2013

Started testing this pull request: http://172.17.131.2/job/btny-pulls-narayana/1042/

@jbosstm-bot

This comment has been minimized.

Copy link

commented Aug 23, 2013

@jbosstm-bot

This comment has been minimized.

Copy link

commented Aug 23, 2013

@jbosstm-bot

This comment has been minimized.

Copy link

commented Aug 23, 2013

All tests passed - Job complete http://172.17.131.2/job/btny-pulls-narayana/1042/

@paulrobinson paulrobinson merged commit d6825e2 into jbosstm:master Aug 27, 2013

@gytis gytis deleted the gytis:master-JBTM-1885 branch Aug 29, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.