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

testng 6.9.12+ #117

Closed
puntogil opened this issue Nov 4, 2016 · 15 comments
Closed

testng 6.9.12+ #117

puntogil opened this issue Nov 4, 2016 · 15 comments

Comments

@puntogil
Copy link
Contributor

puntogil commented Nov 4, 2016

hi
any plan for update testng to 6.9.12 or major?
thanks in advance
regards

@puntogil
Copy link
Contributor Author

There is still a problem
/testng/core/src/test/java/org/jboss/arquillian/testng/TestNGTestBaseClass.java:[211,36] incompatible types: java.lang.String cannot be converted to org.testng.xml.XmlSuite.FailurePolicy

@bartoszmajsak
Copy link
Member

bartoszmajsak commented Nov 11, 2016

Can you share mvn --version output?

With this:

mvn --version
Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5; 2015-11-10T17:41:47+01:00)
Maven home: /usr/share/maven/latest
Java version: 1.8.0_77, vendor: Oracle Corporation
Java home: /usr/java/jdk1.8.0_77/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "4.8.6-201.fc24.x86_64", arch: "amd64", family: "unix"

I have green build.

@puntogil
Copy link
Contributor Author

from http://koji.fedoraproject.org/koji/watchlogs?taskID=16403850
maven 3.4.0-0.2.20160807git9f2452a

@puntogil
Copy link
Contributor Author

the problem is testng 6.9.12

@puntogil
Copy link
Contributor Author

because on my system i use testng 6.9.10 and maven 3.3.9

@bartoszmajsak
Copy link
Member

bartoszmajsak commented Nov 11, 2016

I see. We updated to 6.9.10 because of testng-team/testng#1168
Until this is fixed I guess it's risky to go higher.

@puntogil
Copy link
Contributor Author

I dont know if is correct but seem 6.9.12 is not involved

@bartoszmajsak
Copy link
Member

Is it crucial for you to have 6.9.12? Maybe we wait until they ship new release?

@puntogil
Copy link
Contributor Author

no, i remove the arquillian-testng-core test suite from the build ...
thanks and sorry for the noise

@bartoszmajsak
Copy link
Member

It's fine, no noise. Thanks for your feedback! We will update TestNG to the newest one when they fix it. It seems this is also affecting versions prior to 6.9.10. What are you testing it for?

@puntogil
Copy link
Contributor Author

I was trying to import tool-tracker ...

@bartoszmajsak
Copy link
Member

@dipak-pawar there is a new release of TestNG 6.10 - can you update and verify if that works?

@dipak-pawar
Copy link
Contributor

sure @bartoszmajsak

@bartoszmajsak
Copy link
Member

Reopening as we are going to fix it in next release (see #128)

@bartoszmajsak
Copy link
Member

There is a new version released, but not published yet to Maven Central. https://bintray.com/cbeust/maven/testng/6.12

With this I can confirm that all works fine.

We have to wait however until the new version of TestNG lands in Maven Central :(

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

3 participants