[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15498764#comment-15498764 ] Hudson commented on SUREFIRE-1254: -- SUCCESS: Integrated in Jenkins build maven-surefire #1615 (See [https://builds.apache.org/job/maven-surefire/1615/]) [SUREFIRE-1254] add color messages (tibor17: rev 7617fc089ffacda47ca3b66656ff30ba3bbf1d25) * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java [SUREFIRE-1254] add color messages (tibor17: rev b6d221295e3f0691413844cf023a56d514c035f5) * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: Improvement > Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven > Surefire Report Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > Attachments: surefire-1254-test skipped.png > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15498684#comment-15498684 ] Tibor Digana commented on SUREFIRE-1254: I fixed it. Now Skipped # is printed in yellow/warning. CI will handle this in a while. On Sat, Sep 17, 2016 at 11:53 AM, Hervé Boutemy (JIRA) > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: Improvement > Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven > Surefire Report Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > Attachments: surefire-1254-test skipped.png > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15498678#comment-15498678 ] Tibor Digana commented on SUREFIRE-1254: sometimes as warning I will check it out again. I guess your test was like this: assumeTrue( false ); or different? On Sat, Sep 17, 2016 at 11:49 AM, Hervé Boutemy (JIRA) wrote: > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: Improvement > Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven > Surefire Report Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > Attachments: surefire-1254-test skipped.png > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15498669#comment-15498669 ] Hervé Boutemy commented on SUREFIRE-1254: - I like the overall result: great work a few remarks: - given the big change in output (with log level printing), I suppose 2.19.2 could be renamed 2.20 - Skipped styling seems inconsistent: sometimes it is shown as failure, and sometimes as warning: and I'm not even sure if a skipped test is something abnormal (I'll attach a screenshot) > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: Improvement > Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven > Surefire Report Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15498624#comment-15498624 ] Hudson commented on SUREFIRE-1254: -- SUCCESS: Integrated in Jenkins build maven-surefire #1614 (See [https://builds.apache.org/job/maven-surefire/1614/]) [SUREFIRE-1254] add color messages (tibor17: rev c03f6ed5e31b16e503b56b650b98a8c333b084c7) * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/DefaultReporterFactory.java [SUREFIRE-1254] add color messages (tibor17: rev ed27af4d260a9e5dc429a3a7ddafd11188bfe5d2) * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: Improvement > Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven > Surefire Report Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15497681#comment-15497681 ] Hudson commented on SUREFIRE-1254: -- FAILURE: Integrated in Jenkins build maven-surefire #1609 (See [https://builds.apache.org/job/maven-surefire/1609/]) [SUREFIRE-1254] add color messages (tibor17: rev 6a79127ab63273f0f1f5268c65e4806761871a46) * (add) maven-surefire-common/src/test/java/org/apache/maven/surefire/spi/NoServiceInterface.java * (edit) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/jiras/Surefire735ForkFailWithRedirectConsoleOutputIT.java * (delete) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/Not2xCompatible.java * (edit) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/Junit47concurrencyIT.java * (add) surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/DefaultConsoleReporter.java * (edit) surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/SurefireReportParser.java * (edit) surefire-report-parser/src/test/java/org/apache/maven/plugins/surefire/report/SurefireReportParserTest.java * (edit) surefire-setup-integration-tests/src/test/java/org/apache/maven/surefire/its/StagedLocalRepoHelper.java * (edit) surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SharedThreadPoolStrategy.java * (edit) surefire-api/src/main/java/org/apache/maven/surefire/providerapi/ProviderParameters.java * (edit) surefire-api/src/main/java/org/apache/maven/surefire/booter/CommandReader.java * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/DefaultReporterFactory.java * (add) surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/ConsoleLogger.java * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/ConsoleReporter.java * (delete) surefire-api/src/test/java/org/apache/maven/surefire/report/MockReporter.java * (edit) maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java * (add) surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/ConsoleLoggerDecorator.java * (edit) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/fixture/HelperAssertions.java * (edit) maven-surefire-common/pom.xml * (edit) surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/SchedulingStrategy.java * (delete) surefire-providers/surefire-junit47/src/test/java/org/apache/maven/surefire/junitcore/Logger.java * (edit) surefire-integration-tests/src/test/resources/surefire-1080-parallel-fork-double-test/pom.xml * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/InPluginVMSurefireStarter.java * (edit) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1122ParallelAndFlakyTestsIT.java * (add) surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/PrintStreamLogger.java * (edit) surefire-report-parser/pom.xml * (edit) maven-surefire-common/src/main/java/org/apache/maven/surefire/report/RunStatistics.java * (edit) surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/pc/Scheduler.java * (edit) surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1136CwdPropagationInForkedModeIT.java * (edit) surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ConcurrentRunListener.java * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/DirectConsoleOutput.java * (edit) pom.xml * (edit) surefire-api/src/main/java/org/apache/maven/surefire/booter/ForkingRunListener.java * (edit) maven-failsafe-plugin/pom.xml * (edit) surefire-providers/surefire-junit47/src/main/java/org/apache/maven/surefire/junitcore/ClassesParallelRunListener.java * (edit) surefire-report-parser/src/main/java/org/apache/maven/plugins/surefire/report/TestSuiteXmlParser.java * (edit) maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/booterclient/TestSetMockReporterFactory.java * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/booterclient/output/ForkClient.java * (edit) maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetRunListener.java * (edit) surefire-integration-tests/src/test/resources/surefire-1122-parallel-and-flakyTests/pom.xml * (add) maven-surefire-common/src/test/java/org/apache/maven/surefire/spi/SPImpl1.java * (edit) surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java * (delete) surefire-api/src/main/java/org/apache/maven/surefire/report/DefaultConsoleReporter.java * (edit) maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/DefaultReporterFac
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15497676#comment-15497676 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 commented on a diff in the pull request: https://github.com/apache/maven-surefire/pull/117#discussion_r79264943 --- Diff: maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java --- @@ -34,10 +34,11 @@ import org.apache.maven.plugins.annotations.Mojo; import org.apache.maven.plugins.annotations.Parameter; import org.apache.maven.plugins.annotations.ResolutionScope; -import org.apache.maven.shared.utils.ReaderFactory; import org.apache.maven.shared.utils.StringUtils; import org.apache.maven.surefire.suite.RunResult; +import static org.apache.maven.shared.utils.ReaderFactory.FILE_ENCODING; + --- End diff -- I use it for several purposes. When line is getting to lone around 120 characters and the constant is just a storage, or assertion statement when it is ugly to read the class; otherwise the whole statement is not verbal enough if class appears. I use IntelliJ IDEA which navigates me to the class where the constant is stored. I do not know how bout Eclipse abilities. I tried to use same name of method in static import and Java compiler failed, so it's safe compiler. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: Improvement > Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven > Surefire Report Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15497662#comment-15497662 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 closed the pull request at: https://github.com/apache/maven-surefire/pull/117 > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: Improvement > Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven > Surefire Report Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[
https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15469044#comment-15469044
]
ASF GitHub Bot commented on SUREFIRE-1254:
--
Github user hboutemy commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/117#discussion_r77742105
--- Diff:
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/log/PluginConsoleLogger.java
---
@@ -0,0 +1,126 @@
+package org.apache.maven.plugin.surefire.log;
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+import org.apache.maven.plugin.logging.Log;
+import org.apache.maven.surefire.report.ConsoleLogger;
+
+/**
+ * Wrapper logger of miscellaneous (Maven 2.2.1 or 3.1) implementations of
{@link Log}.
+ * Calling {@link Log#isInfoEnabled()} before {@link
Log#info(CharSequence)} due to Maven 2.2.1.
+ *
+ * @author mailto:tibordig...@apache.org";>Tibor Digana
(tibor17)
+ * @since 2.19.2
+ * @see ConsoleLogger
+ */
+public final class PluginConsoleLogger
+implements ConsoleLogger
+{
+private final Log mojoLogger;
+
+public PluginConsoleLogger( Log mojoLogger )
+{
+this.mojoLogger = mojoLogger;
+}
+
+public boolean isDebugEnabled()
+{
+return mojoLogger.isDebugEnabled();
+}
+
+public void debug( String message )
+{
+if ( mojoLogger.isDebugEnabled() )
+{
+mojoLogger.debug( message );
--- End diff --
here, I completely missed the point: ok, now I understand that it's just
proxying log call from one API to the other
my fear is more on DefaultColoredConsoleMessageBuilder, where it seems to
add color to each log message: I'll test
> add color messages
> --
>
> Key: SUREFIRE-1254
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1254
> Project: Maven Surefire
> Issue Type: Improvement
> Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven
> Surefire Report Plugin
>Reporter: Hervé Boutemy
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> with MNG-3507 fixed, adding colors to tests results would improve readability
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15469019#comment-15469019 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user hboutemy commented on a diff in the pull request: https://github.com/apache/maven-surefire/pull/117#discussion_r77741146 --- Diff: maven-failsafe-plugin/src/main/java/org/apache/maven/plugin/failsafe/IntegrationTestMojo.java --- @@ -34,10 +34,11 @@ import org.apache.maven.plugins.annotations.Mojo; import org.apache.maven.plugins.annotations.Parameter; import org.apache.maven.plugins.annotations.ResolutionScope; -import org.apache.maven.shared.utils.ReaderFactory; import org.apache.maven.shared.utils.StringUtils; import org.apache.maven.surefire.suite.RunResult; +import static org.apache.maven.shared.utils.ReaderFactory.FILE_ENCODING; + --- End diff -- is it really intentional to reformat everything to have static import on any static access? I don't find it convenient in general. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: Improvement > Components: Maven Failsafe Plugin, Maven Surefire Plugin, Maven > Surefire Report Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15468908#comment-15468908 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/117 @hboutemy I found NPE so that consoleLogger is null. I will fix it and run the tests again. Some tests were successful. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15468793#comment-15468793 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/117 @hboutemy now we can test. All unit tests passed and I picked up first IT and passed successfully. So I am going to run all ITs. The content of binaries is expected and I compared it with previous release. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15467337#comment-15467337 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 commented on a diff in the pull request: https://github.com/apache/maven-surefire/pull/117#discussion_r77627125 --- Diff: surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java --- @@ -38,11 +38,16 @@ import org.apache.maven.surefire.suite.RunResult; import org.apache.maven.surefire.testset.TestSetFailedException; +import static java.lang.System.*; --- End diff -- My IDEA added star. I should fix it in her configuration. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15467328#comment-15467328 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 commented on a diff in the pull request: https://github.com/apache/maven-surefire/pull/117#discussion_r77626502 --- Diff: surefire-api/src/main/java/org/apache/maven/surefire/booter/CommandReader.java --- @@ -46,10 +48,8 @@ import static org.apache.maven.surefire.booter.MasterProcessCommand.SKIP_SINCE_NEXT_TEST; import static org.apache.maven.surefire.booter.MasterProcessCommand.TEST_SET_FINISHED; import static org.apache.maven.surefire.booter.MasterProcessCommand.decode; -import static org.apache.maven.surefire.util.internal.StringUtils.encodeStringForForkCommunication; -import static org.apache.maven.surefire.util.internal.StringUtils.isNotBlank; -import static org.apache.maven.surefire.util.internal.StringUtils.isBlank; import static org.apache.maven.surefire.util.internal.DaemonThreadFactory.newDaemonThread; +import static org.apache.maven.surefire.util.internal.StringUtils.*; --- End diff -- yes this is change in my local code as well. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[
https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15467324#comment-15467324
]
ASF GitHub Bot commented on SUREFIRE-1254:
--
Github user Tibor17 commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/117#discussion_r77626386
--- Diff:
surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/Level.java
---
@@ -0,0 +1,83 @@
+package org.apache.maven.plugin.surefire.log.api;
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+/**
+ * Standard log levels, {@link #DEBUG} {@link #INFO} {@link #WARN} {@link
#ERROR}, used in console.
+ * Test result levels {@link #FAILURE}, {@link #UNSTABLE}, {@link
#SUCCESS}.
+ * Writing to console without color via {@link #NO_COLOR}.
+ *
+ * @author mailto:tibordig...@apache.org";>Tibor Digana
(tibor17)
+ * @since 2.19.2
+ */
+public enum Level {
+/**
+ * call {@link ColoredConsoleMessageBuilder#toColorless(CharSequence)}
+ */
+NO_COLOR,
+
+/**
+ * defaults to bold, cyan
+ */
+DEBUG,
+
+/**
+ * defaults to bold, green
+ */
+INFO,
+
+/**
+ * defaults to bold, yellow
+ */
+WARN,
+
+/**
+ * defaults to bold, red
+ */
+ERROR,
+
+/**
+ * defaults to bold, green
+ */
+FAILURE,
+
+/**
+ * defaults to bold, yellow
+ */
+UNSTABLE,
+
+/**
+ * defaults to bold, red
+ */
+SUCCESS;
+
+public static Level resolveLevel(boolean hasSuccessful, boolean
hasFailure, boolean hasError,
+ boolean hasSkipped, boolean hasFlake)
{
+boolean isRed = hasFailure | hasError;
+if (isRed) {
--- End diff --
yes, this has benn done - yet not committed.
I am currently running build and one before last module failed
`surefire-setup-integration-tests` because of wrong shading.
The last module is `surefire-integration-tests` where I should switch off
collors and then I have some activity to compare all binaries against binaries
of previous release 2.19.1.
Then I will run manual testing and compare console with old version and
several features which alter the console like `rerun`, `assumptions` and
reports.
Maybe we should then test together.
> add color messages
> --
>
> Key: SUREFIRE-1254
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1254
> Project: Maven Surefire
> Issue Type: New Feature
> Components: Maven Failsafe Plugin, Maven Surefire Plugin
>Reporter: Hervé Boutemy
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> with MNG-3507 fixed, adding colors to tests results would improve readability
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15467078#comment-15467078 ] Tibor Digana commented on SUREFIRE-1254: Hi Herve, I just wanted to declare that the work is in progress and make the people less nervous which means that the release is still closer :) On Tue, Sep 6, 2016 at 12:15 PM, Tibor Digana > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15467051#comment-15467051 ] Tibor Digana commented on SUREFIRE-1254: I am still in progress. On Tue, Sep 6, 2016 at 8:52 AM, Hervé Boutemy (JIRA) > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15466634#comment-15466634 ] Hervé Boutemy commented on SUREFIRE-1254: - currently, after fixing errors reported by Checkstyle, I get a compile error in maven-surefire-common, which does not find package org.apache.maven.shared.utils.logging I want to see this branch running to see how the rendering looks like before eventually proposing updates in rendering code > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15466615#comment-15466615 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user hboutemy commented on a diff in the pull request: https://github.com/apache/maven-surefire/pull/117#discussion_r77579970 --- Diff: surefire-booter/src/main/java/org/apache/maven/surefire/booter/ForkedBooter.java --- @@ -38,11 +38,16 @@ import org.apache.maven.surefire.suite.RunResult; import org.apache.maven.surefire.testset.TestSetFailedException; +import static java.lang.System.*; --- End diff -- I really don't like this static import nor the next one: System.out and System.err in code is a lot more manageable that out and err with static import > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15466608#comment-15466608 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user hboutemy commented on a diff in the pull request: https://github.com/apache/maven-surefire/pull/117#discussion_r77579682 --- Diff: surefire-api/src/main/java/org/apache/maven/surefire/booter/CommandReader.java --- @@ -46,10 +48,8 @@ import static org.apache.maven.surefire.booter.MasterProcessCommand.SKIP_SINCE_NEXT_TEST; import static org.apache.maven.surefire.booter.MasterProcessCommand.TEST_SET_FINISHED; import static org.apache.maven.surefire.booter.MasterProcessCommand.decode; -import static org.apache.maven.surefire.util.internal.StringUtils.encodeStringForForkCommunication; -import static org.apache.maven.surefire.util.internal.StringUtils.isNotBlank; -import static org.apache.maven.surefire.util.internal.StringUtils.isBlank; import static org.apache.maven.surefire.util.internal.DaemonThreadFactory.newDaemonThread; +import static org.apache.maven.surefire.util.internal.StringUtils.*; --- End diff -- import static org.apache.maven.surefire.util.internal.StringUtils.encodeStringForForkCommunication; import static org.apache.maven.surefire.util.internal.StringUtils.isBlank; import static org.apache.maven.surefire.util.internal.StringUtils.isNotBlank; import static org.apache.maven.surefire.util.internal.StringUtils.requireNonNull; > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[
https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15466599#comment-15466599
]
ASF GitHub Bot commented on SUREFIRE-1254:
--
Github user hboutemy commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/117#discussion_r77579430
--- Diff:
surefire-logger-api/src/main/java/org/apache/maven/plugin/surefire/log/api/Level.java
---
@@ -0,0 +1,83 @@
+package org.apache.maven.plugin.surefire.log.api;
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+/**
+ * Standard log levels, {@link #DEBUG} {@link #INFO} {@link #WARN} {@link
#ERROR}, used in console.
+ * Test result levels {@link #FAILURE}, {@link #UNSTABLE}, {@link
#SUCCESS}.
+ * Writing to console without color via {@link #NO_COLOR}.
+ *
+ * @author mailto:tibordig...@apache.org";>Tibor Digana
(tibor17)
+ * @since 2.19.2
+ */
+public enum Level {
+/**
+ * call {@link ColoredConsoleMessageBuilder#toColorless(CharSequence)}
+ */
+NO_COLOR,
+
+/**
+ * defaults to bold, cyan
+ */
+DEBUG,
+
+/**
+ * defaults to bold, green
+ */
+INFO,
+
+/**
+ * defaults to bold, yellow
+ */
+WARN,
+
+/**
+ * defaults to bold, red
+ */
+ERROR,
+
+/**
+ * defaults to bold, green
+ */
+FAILURE,
+
+/**
+ * defaults to bold, yellow
+ */
+UNSTABLE,
+
+/**
+ * defaults to bold, red
+ */
+SUCCESS;
+
+public static Level resolveLevel(boolean hasSuccessful, boolean
hasFailure, boolean hasError,
+ boolean hasSkipped, boolean hasFlake)
{
+boolean isRed = hasFailure | hasError;
+if (isRed) {
--- End diff --
you'll need to reformat this class (and NullConsoleLogger) since they don't
use Maven formatting conventions then are detected by Checkstyle
> add color messages
> --
>
> Key: SUREFIRE-1254
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1254
> Project: Maven Surefire
> Issue Type: New Feature
> Components: Maven Failsafe Plugin, Maven Surefire Plugin
>Reporter: Hervé Boutemy
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> with MNG-3507 fixed, adding colors to tests results would improve readability
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15462522#comment-15462522 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/117 @hboutemy I will remove the SPI. The code will be simple if I just move the implementation to `surefire-common`. I thought that `surefire-common` appeared in classpath of forked jvm but I see now the classpath and no such artifact is in there. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15462495#comment-15462495 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/117 @hboutemy Hi Herve, I will amend the commit. So one commit would still appear however it is in progress. Surefire report has I think three place with logger only. Maybe important to know. I use ConsoleLogger in with methods like `debug/info/warning/errror` in in-plugin jvm and forked jvm as well. This was here all the history of Surefire but I only extended with few more methods. The implementations should be different in forked-jvm and simply, as higher design principle was, the surefire providers are slaves and they only encode and send a message to main process or receive a command to do. The main process does what it has to do; either creates a report of displays message in console. From the same principle in-plugin jvm is the only place where the ClassLoader should see ANSI and MessageBuilder of m-shared-u. So this separation is safe because such classes will never be loaded or failed to load in forked jvm and this prevents me from having ugly jira issues. So I decided to use SPI to separate implementation which is in surefire-common from surefire-api which is used everywhere. The SPI is a builder of colored messages only. So now we will have logs like `[INFO] Running MyTest` with colors instead of `Running MyTest`. Three advantages: + I fixed the issue you reported https://issues.apache.org/jira/browse/SUREFIRE-1254 + many people reported bug that Surefire writes anything in consolve, so logger will again control the output and we don't have to add configuration parameter in plugin which is good, thus we fixed https://issues.apache.org/jira/browse/SUREFIRE-725 as well + and what I like is peace in code because as I told you before developers come and go to Surefire and everybody solved his problem without maintaining for long time. And that's the reason why logger was sometimes System.out, or System.err, or ConsoleLogger or mojo logger. I like when the code has internal principles that the attitude is just one; unlike use directly worked with stream PrintWriter which was wrapped stream of System.out which disallowed you to use normal logger like in Maven with slf4j and disallowed you to see messages like `[INFO] ...` etc. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15462470#comment-15462470 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user Tibor17 commented on a diff in the pull request: https://github.com/apache/maven-surefire/pull/117#discussion_r77448156 --- Diff: maven-failsafe-plugin/pom.xml --- @@ -127,6 +127,19 @@ true + +shared-logging-generated-sources --- End diff -- Good point. I used m-shared-u:3.1.0 but there was compilation error I know from branch 3.0-rc1. One class of m-shared-u appeared in another package related to report. I don't remember the name of the class but it can be easily reproduced. This code would have very short life time because we already have branch `3.0-rc1` and one IT should be fixed and Surefire 2.x is gone. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[
https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15462465#comment-15462465
]
ASF GitHub Bot commented on SUREFIRE-1254:
--
Github user Tibor17 commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/117#discussion_r77448108
--- Diff:
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/log/PluginConsoleLogger.java
---
@@ -0,0 +1,126 @@
+package org.apache.maven.plugin.surefire.log;
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+import org.apache.maven.plugin.logging.Log;
+import org.apache.maven.surefire.report.ConsoleLogger;
+
+/**
+ * Wrapper logger of miscellaneous (Maven 2.2.1 or 3.1) implementations of
{@link Log}.
+ * Calling {@link Log#isInfoEnabled()} before {@link
Log#info(CharSequence)} due to Maven 2.2.1.
+ *
+ * @author mailto:tibordig...@apache.org";>Tibor Digana
(tibor17)
+ * @since 2.19.2
+ * @see ConsoleLogger
+ */
+public final class PluginConsoleLogger
+implements ConsoleLogger
+{
+private final Log mojoLogger;
+
+public PluginConsoleLogger( Log mojoLogger )
+{
+this.mojoLogger = mojoLogger;
+}
+
+public boolean isDebugEnabled()
+{
+return mojoLogger.isDebugEnabled();
+}
+
+public void debug( String message )
+{
+if ( mojoLogger.isDebugEnabled() )
+{
+mojoLogger.debug( message );
--- End diff --
The purpose is to separate Mojo Logger. I cannot use Mojo in forked JVM.
Many times the code uses the same classes and interfaces in in-plugin tests or
forked-jvm tests but the same classes are extended and behave different in in
these two places however the super type is used eveywhere like polymorphism.
The same happened with ConsoleLogger and it has advantage because all can be
developed independently and the logger would be same (unlike sometimes we used
System.out and Mojo logger or console logger), so now it's only one.
The important things is that `ConsoleLogger` does not need e.g.
`isDebugEnabled` method but `AbstractSurefireMojo` used this for long time, so
I decided to use concrete implementation class of `ConsoleLogger` in MOJO but
that's the second reason to have this class here :)
Unfortunately we use Java 5; otherwise you would see those annotations in
methods implemented from interface via `@Override`.
> add color messages
> --
>
> Key: SUREFIRE-1254
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1254
> Project: Maven Surefire
> Issue Type: New Feature
> Components: Maven Failsafe Plugin, Maven Surefire Plugin
>Reporter: Hervé Boutemy
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> with MNG-3507 fixed, adding colors to tests results would improve readability
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[
https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15462384#comment-15462384
]
ASF GitHub Bot commented on SUREFIRE-1254:
--
Github user hboutemy commented on a diff in the pull request:
https://github.com/apache/maven-surefire/pull/117#discussion_r77447554
--- Diff:
maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/log/PluginConsoleLogger.java
---
@@ -0,0 +1,126 @@
+package org.apache.maven.plugin.surefire.log;
+
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+import org.apache.maven.plugin.logging.Log;
+import org.apache.maven.surefire.report.ConsoleLogger;
+
+/**
+ * Wrapper logger of miscellaneous (Maven 2.2.1 or 3.1) implementations of
{@link Log}.
+ * Calling {@link Log#isInfoEnabled()} before {@link
Log#info(CharSequence)} due to Maven 2.2.1.
+ *
+ * @author mailto:tibordig...@apache.org";>Tibor Digana
(tibor17)
+ * @since 2.19.2
+ * @see ConsoleLogger
+ */
+public final class PluginConsoleLogger
+implements ConsoleLogger
+{
+private final Log mojoLogger;
+
+public PluginConsoleLogger( Log mojoLogger )
+{
+this.mojoLogger = mojoLogger;
+}
+
+public boolean isDebugEnabled()
+{
+return mojoLogger.isDebugEnabled();
+}
+
+public void debug( String message )
+{
+if ( mojoLogger.isDebugEnabled() )
+{
+mojoLogger.debug( message );
--- End diff --
wow, the whole message with debug level color, and more generally each
message with level color? this will give too much color.
I need to see the effective result, but if it's really what I fear, it was
not the intent when adding color to Maven output: the intent is just to have a
few colored words with general normal message (and nothing else than level with
level color)
> add color messages
> --
>
> Key: SUREFIRE-1254
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1254
> Project: Maven Surefire
> Issue Type: New Feature
> Components: Maven Failsafe Plugin, Maven Surefire Plugin
>Reporter: Hervé Boutemy
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> with MNG-3507 fixed, adding colors to tests results would improve readability
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15462362#comment-15462362 ] ASF GitHub Bot commented on SUREFIRE-1254: -- Github user hboutemy commented on a diff in the pull request: https://github.com/apache/maven-surefire/pull/117#discussion_r77447349 --- Diff: maven-failsafe-plugin/pom.xml --- @@ -127,6 +127,19 @@ true + +shared-logging-generated-sources --- End diff -- IMHO, a little comment to explain why the source code is rebuilt here could be useful for future maintainers > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15462168#comment-15462168 ] ASF GitHub Bot commented on SUREFIRE-1254: -- GitHub user Tibor17 opened a pull request: https://github.com/apache/maven-surefire/pull/117 [SUREFIRE-1254] add color messages You can merge this pull request into a Git repository by running: $ git pull https://github.com/Tibor17/maven-surefire SUREFIRE-1254 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/maven-surefire/pull/117.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #117 commit 87dd0e24c422c6def4f62f3b9c7eb3f1f67d2f18 Author: Tibor17 Date: 2016-09-04T03:32:31Z [SUREFIRE-1254] add color messages > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15437868#comment-15437868 ] Benno Markiewicz commented on SUREFIRE-1254: Disabling the coloring when running in batch-mode?! That makes sense. Thanks for the feedback! > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[
https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15437832#comment-15437832
]
Tibor Digana commented on SUREFIRE-1254:
Colors are disabled by {{mvn -B test}}; otherwise any other plugin @
mvn 3.4.0 would be problematic.
Jenkins does not parse std/out. The tools use target/surefire-reports/.
We can directly ask the Jenkins team.
--
Cheers
Tibor
> add color messages
> --
>
> Key: SUREFIRE-1254
> URL: https://issues.apache.org/jira/browse/SUREFIRE-1254
> Project: Maven Surefire
> Issue Type: New Feature
> Components: Maven Failsafe Plugin, Maven Surefire Plugin
>Reporter: Hervé Boutemy
>Assignee: Tibor Digana
> Fix For: 2.19.2
>
>
> with MNG-3507 fixed, adding colors to tests results would improve readability
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15437798#comment-15437798 ] Benno Markiewicz commented on SUREFIRE-1254: Please make it optional and disabled by default. Else you will break external tools like Jenkins/NetBeans, which parse the output of the surefire-plugin. Their regex patterns won't match anymore. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy >Assignee: Tibor Digana > Fix For: 2.19.2 > > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (SUREFIRE-1254) add color messages
[ https://issues.apache.org/jira/browse/SUREFIRE-1254?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15328328#comment-15328328 ] Tibor Digana commented on SUREFIRE-1254: This would be cool to have. Some users stated we should use logger instead of std/out. I will have a look into this. > add color messages > -- > > Key: SUREFIRE-1254 > URL: https://issues.apache.org/jira/browse/SUREFIRE-1254 > Project: Maven Surefire > Issue Type: New Feature > Components: Maven Failsafe Plugin, Maven Surefire Plugin >Reporter: Hervé Boutemy > > with MNG-3507 fixed, adding colors to tests results would improve readability -- This message was sent by Atlassian JIRA (v6.3.4#6332)
