TC should fail on AssertionError in logs

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

TC should fail on AssertionError in logs

yzhdanov
Guys, I see couple of tests which have assertion error in some background
Ignite threads that may not cause test to fail.

https://issues.apache.org/jira/browse/IGNITE-4895

Anton V, can you please take a look?

--Yakov
Reply | Threaded
Open this post in threaded view
|

Re: TC should fail on AssertionError in logs

Anton Vinogradov
Yakov,

I found following at first opened suite logs (Basic)

1) [INFO ][test-runner-#4966%future.GridEmbeddedFutureSelfTest%][root]
Failed with unhandled error (normal behaviour): java.lang.AssertionError:
Test assertion (should be ignored).
2) [INFO ][test-runner-#4883%lang.GridByteArrayListSelfTest%][root] Caught
expected error: java.lang.AssertionError

Not sure we can just fail in case "AssertionError" found.

On Fri, Mar 31, 2017 at 1:02 PM, Yakov Zhdanov <[hidden email]> wrote:

> Guys, I see couple of tests which have assertion error in some background
> Ignite threads that may not cause test to fail.
>
> https://issues.apache.org/jira/browse/IGNITE-4895
>
> Anton V, can you please take a look?
>
> --Yakov
>
Reply | Threaded
Open this post in threaded view
|

Re: TC should fail on AssertionError in logs

yzhdanov
Change this to RuntimeException. I think it would be fine. Assertion in
logs should mean something terrible from now and on.

--Yakov
Reply | Threaded
Open this post in threaded view
|

Re: TC should fail on AssertionError in logs

Anton Vinogradov
Additional task created (https://issues.apache.org/jira/browse/IGNITE-4909)

On Mon, Apr 3, 2017 at 7:47 PM, Yakov Zhdanov <[hidden email]> wrote:

> Change this to RuntimeException. I think it would be fine. Assertion in
> logs should mean something terrible from now and on.
>
> --Yakov
>