[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-07-16 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 it is ready --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-07-15 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/98 I have moved portlet-app away from the core project, please update your PR, it should work now :) --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-27 Thread cnenning
Github user cnenning commented on the issue: https://github.com/apache/struts/pull/98 > I think we should move portlet-app away from Struts into Struts Examples Oh, yes! We really should! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-25 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/98 It fails the same, I was trying many different things. I think we should move `portlet-app` away from Struts into Struts Examples (as other apps as well) --- If your project is set up for it,

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-24 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 @lukaszlenart can you check please the result in jenkins because I am getting success build for Portlet here in my PC. But here it gets `unexpected status code [500] at URL:

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 Yep it was too fast; that fixed it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 Maybe It is not a good idea because this test is failing and I don't have the clue about what is wrong. Do you know why? > > Tests run: 1659, Failures: 1, Errors: 0, Skipped: 0, Time

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/98 Yes, please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 Here is the fix: ``` org.apache.maven.plugins maven-surefire-plugin 2.19.1

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 NOp :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/98 Uf... I thought I'm betting crazy ;-) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 Sorry I compiled the wrong branch, I was able to duplicate the issue --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 > openjdk version "1.8.0_91" > OpenJDK Runtime Environment (build 1.8.0_91-b14) > OpenJDK 64-Bit Server VM (build 25.91-b14, mixed mode) --- If your project is set up for it, you can

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/98 and this is related to outdated version of `org.apache.maven.plugins:maven-surefire-plugin:2.3.1` --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread victorsosa
Github user victorsosa commented on the issue: https://github.com/apache/struts/pull/98 On my end. [INFO] Installing /home/vns/workspace/struts/plugins/testng/target/struts2-testng-plugin-2.5.1-SNAPSHOT-sources.jar to

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-07 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/98 And changing in `plugins/testng/pom.xml` is not enough, this dependency is used in few other palces. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-06 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/98 I've got such an error when building locally: `INFO] Struts 2 TestNG Plugin . FAILURE [ 7.039 s]` ```

[GitHub] struts issue #98: WW-4638 - TestNG 6.9.10 dependency error

2016-06-06 Thread lukaszlenart
Github user lukaszlenart commented on the issue: https://github.com/apache/struts/pull/98 I'm just wondering about those all merge commits, did you update your fork? https://help.github.com/articles/syncing-a-fork/ --- If your project is set up for it, you can reply to this