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

Using a timeout with testng breaks everything #168

Open
ivankelly opened this issue Apr 4, 2018 · 1 comment
Open

Using a timeout with testng breaks everything #168

ivankelly opened this issue Apr 4, 2018 · 1 comment

Comments

@ivankelly
Copy link

Issue Overview

If you add a timeout to a testng test using org.jboss.arquillian.testng.Arquillian, everything breaks.

Expected Behaviour

Adding a timeout shouldn't break stuff.

Current Behaviour

It breaks, and it breaks in a very non-obvious manner.

Steps To Reproduce
  1. Change @Test to @Test(timeOut=1000) on the test ArquillianClass1
  2. Run the test
Additional Information

Tested with testng 6.13.1, but the problem is with arquillian. It runs @BeforeSuite in a different thread to the test itself. @BeforeSuite sets up a thread local context object (TestRunnerAdapter), so when the test runs, it doesn't find this object, and everything goes to hell.

ivankelly added a commit to ivankelly/pulsar that referenced this issue Apr 4, 2018
The annotation listener adds timeouts to all tests, which changes the
threading model, which breaks arquillian.

I've created an issue[1] on arquillian for this, but until that is fixed,
timeouts cannot be used with arquillian.

[1] arquillian/arquillian-core#168
ivankelly added a commit to ivankelly/pulsar that referenced this issue Apr 4, 2018
The annotation listener adds timeouts to all tests, which changes the
threading model, which breaks arquillian.

I've created an issue[1] on arquillian for this, but until that is fixed,
timeouts cannot be used with arquillian.

[1] arquillian/arquillian-core#168
merlimat pushed a commit to apache/pulsar that referenced this issue Apr 5, 2018
The annotation listener adds timeouts to all tests, which changes the
threading model, which breaks arquillian.

I've created an issue[1] on arquillian for this, but until that is fixed,
timeouts cannot be used with arquillian.

[1] arquillian/arquillian-core#168
merlimat pushed a commit to apache/pulsar that referenced this issue Apr 6, 2018
* Integration smoke test for pulsar

Boots a cluster, sends and receives some messages, shuts down the
cluster.

If anything fails, then something is very broken.

* license fix

* Disable AnnotationListener for integration tests

The annotation listener adds timeouts to all tests, which changes the
threading model, which breaks arquillian.

I've created an issue[1] on arquillian for this, but until that is fixed,
timeouts cannot be used with arquillian.

[1] arquillian/arquillian-core#168
@bartoszmajsak
Copy link
Member

Thanks for reporting this issue. Could you contribute a reproducer test?

sijie pushed a commit to apache/pulsar-release that referenced this issue Nov 12, 2020
The annotation listener adds timeouts to all tests, which changes the
threading model, which breaks arquillian.

I've created an issue[1] on arquillian for this, but until that is fixed,
timeouts cannot be used with arquillian.

[1] arquillian/arquillian-core#168
sijie pushed a commit to apache/pulsar-release that referenced this issue Nov 12, 2020
* Integration smoke test for pulsar

Boots a cluster, sends and receives some messages, shuts down the
cluster.

If anything fails, then something is very broken.

* license fix

* Disable AnnotationListener for integration tests

The annotation listener adds timeouts to all tests, which changes the
threading model, which breaks arquillian.

I've created an issue[1] on arquillian for this, but until that is fixed,
timeouts cannot be used with arquillian.

[1] arquillian/arquillian-core#168
sijie pushed a commit to apache/pulsar-release that referenced this issue Nov 12, 2020
The annotation listener adds timeouts to all tests, which changes the
threading model, which breaks arquillian.

I've created an issue[1] on arquillian for this, but until that is fixed,
timeouts cannot be used with arquillian.

[1] arquillian/arquillian-core#168
sijie pushed a commit to apache/pulsar-release that referenced this issue Nov 12, 2020
* Integration smoke test for pulsar

Boots a cluster, sends and receives some messages, shuts down the
cluster.

If anything fails, then something is very broken.

* license fix

* Disable AnnotationListener for integration tests

The annotation listener adds timeouts to all tests, which changes the
threading model, which breaks arquillian.

I've created an issue[1] on arquillian for this, but until that is fixed,
timeouts cannot be used with arquillian.

[1] arquillian/arquillian-core#168
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

No branches or pull requests

2 participants