[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2018-01-13 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16325262#comment-16325262
 ] 

Hudson commented on SUREFIRE-1416:
--

ABORTED: Integrated in Jenkins build maven-surefire-dev #103 (See 
[https://builds.apache.org/job/maven-surefire-dev/103/])
[SUREFIRE-1416] maven-surefire-parser: add new method isError in (tibor17: 
[http://git-wip-us.apache.org/repos/asf/?p=maven-surefire.git=commit=869d6f29ff81bf2116a118f8118b63f21de4aa8a])
* (edit) 
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporter.java
* (edit) 
surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/jiras/Surefire260TestWithIdenticalNamesIT.java
* (edit) 
maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/Surefire597Test.java
* (edit) 
surefire-report-parser/src/test/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParserTest.java
* (edit) 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportParser.java
* (edit) 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParser.java
* (edit) 
maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java
* (edit) 
surefire-report-parser/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportParserTest.java
* (edit) 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestCase.java
* (edit) 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestSuite.java


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Assignee: Tibor Digana
>Priority: Minor
> Fix For: 2.21.0.Jigsaw
>
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2018-01-03 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16310472#comment-16310472
 ] 

Hudson commented on SUREFIRE-1416:
--

SUCCESS: Integrated in Jenkins build maven-surefire #1815 (See 
[https://builds.apache.org/job/maven-surefire/1815/])
[SUREFIRE-1416] maven-surefire-parser: add new method isError in (tibor17: 
[http://git-wip-us.apache.org/repos/asf/?p=maven-surefire.git=commit=869d6f29ff81bf2116a118f8118b63f21de4aa8a])
* (edit) 
surefire-report-parser/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportParserTest.java
* (edit) 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestSuite.java
* (edit) 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParser.java
* (edit) 
maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportGenerator.java
* (edit) 
surefire-report-parser/src/test/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParserTest.java
* (edit) 
surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/jiras/Surefire260TestWithIdenticalNamesIT.java
* (edit) 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportParser.java
* (edit) 
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/StatelessXmlReporter.java
* (edit) 
surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestCase.java
* (edit) 
maven-surefire-report-plugin/src/test/java/org/apache/maven/plugins/surefire/report/Surefire597Test.java


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Assignee: Tibor Digana
>Priority: Minor
> Fix For: 2.21.0.Jigsaw
>
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2018-01-03 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16310450#comment-16310450
 ] 

ASF GitHub Bot commented on SUREFIRE-1416:
--

Github user surli closed the pull request at:

https://github.com/apache/maven-surefire/pull/143


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Assignee: Tibor Digana
>Priority: Minor
> Fix For: 2.21.0.Jigsaw
>
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2018-01-03 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16310350#comment-16310350
 ] 

ASF GitHub Bot commented on SUREFIRE-1416:
--

Github user Tibor17 commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@surli 
Thx for contributing. Now you can close the PR.


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Assignee: Tibor Digana
>Priority: Minor
> Fix For: 2.21.0.Jigsaw
>
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2018-01-03 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16310006#comment-16310006
 ] 

ASF GitHub Bot commented on SUREFIRE-1416:
--

Github user surli commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@Tibor17 I can confirm it's working for me and it's useful! See my PR 
https://github.com/Spirals-Team/librepair/pull/380


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Assignee: Tibor Digana
>Priority: Minor
> Fix For: 2.21.0.Jigsaw
>
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2018-01-03 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16309784#comment-16309784
 ] 

ASF GitHub Bot commented on SUREFIRE-1416:
--

Github user Tibor17 commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@surli 
Pls try to use it in your projects. I want to know if it is useful for you 
and working. Thx.


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Assignee: Tibor Digana
>Priority: Minor
> Fix For: 2.21.0.Jigsaw
>
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2018-01-03 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16309624#comment-16309624
 ] 

ASF GitHub Bot commented on SUREFIRE-1416:
--

Github user surli commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
Hi @Tibor17 

sorry I was off lately so I just saw your comments. I did not check what 
you asked before. I'm currently reviewing your code and it looks fine for me :) 
Thanks!


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Assignee: Tibor Digana
>Priority: Minor
> Fix For: 2.21.0.Jigsaw
>
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2018-01-03 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16309622#comment-16309622
 ] 

ASF GitHub Bot commented on SUREFIRE-1416:
--

Github user Tibor17 commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@surli 
If there are no objections, I would push the branch SUREFIRE-1416 to master.
Thx.


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Assignee: Tibor Digana
>Priority: Minor
> Fix For: 2.21.0.Jigsaw
>
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (SUREFIRE-1416) maven-surefire-parser: add new method isError in ReportTestCase

2017-12-27 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/SUREFIRE-1416?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16304595#comment-16304595
 ] 

ASF GitHub Bot commented on SUREFIRE-1416:
--

Github user Tibor17 commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@surli 
I had to change you code in new branch

https://git1-us-west.apache.org/repos/asf?p=maven-surefire.git;a=shortlog;h=refs/heads/SUREFIRE-1416
Can you build the version `2.21.0-SNAPSHOT` using `mvn install -DskipTests` 
and let me know if the report site works for you? I want to include this fix in 
release in several days.


> maven-surefire-parser: add new method isError in ReportTestCase
> ---
>
> Key: SUREFIRE-1416
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1416
> Project: Maven Surefire
>  Issue Type: Improvement
>  Components: Maven Surefire Plugin
>Reporter: Simon Urli
>Priority: Minor
>
> There is currently no way after parsing test suite to know if a test case is 
> a failure or an error. I need a boolean isError in ReportTestCase and 
> associated getter and setter, to be able to use properly the 
> maven-surefire-parser API to know if a test is considered as an error or as a 
> failure, without consulting the logs.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)