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

Issue-1175 null error message handling in Result failure #1318

Merged
merged 1 commit into from
Oct 12, 2020

Conversation

Nishant-sehgal
Copy link
Contributor

@Nishant-sehgal Nishant-sehgal commented Oct 11, 2020

Description

"null" error message handling when Java regex method internally throws StackOverflow.
if StackOverflowError is thrown it does not have error message set in it so we see [ERROR] FooRunner.testUsers foo.feature:4 - null.
Adding error as it is if error.getMessage() is null

@ptrthomas
Copy link
Member

@Nishant-sehgal build failed so can you re-confirm

@Nishant-sehgal
Copy link
Contributor Author

Yes checking. But test case ran fine in my local.Checking further
Screenshot 2020-10-12 at 10 00 32 AM

@Nishant-sehgal
Copy link
Contributor Author

Nishant-sehgal commented Oct 12, 2020

@ptrthomas i took latest and pushed again build is failing.But in local it is passing not sure what is happening. Any pointers?

Screenshot 2020-10-12 at 10 47 39 AM

@ptrthomas
Copy link
Member

image

no idea. maybe this exception is different across OS / JDK version - so maybe it is safe to keep it as it is: https://travis-ci.org/github/intuit/karate/builds/734888650#L1251

@Nishant-sehgal
Copy link
Contributor Author

@ptrthomas Working as expected now.Please check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants