You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2017/02/11 15:20:41 UTC

svn commit: r1782611 - /jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java

Author: pmouawad
Date: Sat Feb 11 15:20:41 2017
New Revision: 1782611

URL: http://svn.apache.org/viewvc?rev=1782611&view=rev
Log:
Add debug logs

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java?rev=1782611&r1=1782610&r2=1782611&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java Sat Feb 11 15:20:41 2017
@@ -253,11 +253,23 @@ public class HtmlTemplateExporter extend
                                             || !supportsControllerDiscrimination.booleanValue()
                                             || isController
                                             || !showControllerSeriesOnly;
+                                    if(log.isDebugEnabled()) {
+                                        log.debug("name:{} matches pattern:{}, supportsControllerDiscrimination:{}, isController:{}, showControllerSeriesOnly:{}", 
+                                            name, filterPattern.pattern(), 
+                                            supportsControllerDiscrimination, isController, showControllerSeriesOnly);
+                                    }
                                 } else {
                                     // If the name does not match the pattern,
                                     // other properties can hold the series
                                     matches = filtersOnlySampleSeries
                                             && !supportsControllerDiscrimination.booleanValue();
+                                    if(log.isDebugEnabled()) {
+                                        log.debug("name:{} does not match pattern:{}, filtersOnlySampleSeries:{},"
+                                            + " supportsControllerDiscrimination:{}", 
+                                            name, filterPattern.pattern(), 
+                                            filtersOnlySampleSeries,
+                                            supportsControllerDiscrimination);
+                                    }
                                 }
                             }
                             index++;
@@ -495,4 +507,8 @@ public class HtmlTemplateExporter extend
 
     }
 
+    public static void main(String[] args) {
+        Pattern filterPattern = Pattern.compile("((^EnleverArticle)|(^ModifierQuantite)|(^VoirPanier)|(^AjoutPanier)|(^ChoixMagasin)|(^ChangerMagasin)|(^VoirPanier)|(^ClickProduit)|(^ClicTriPrix)|(^ClickPagination)|(^AjoutProduitPourComparison)|(^ClickComparer)|(^ChoixCategorie)|(^ChoixSousCategorie)|(^ClickFiltrer)|(^RechercheFullText)|(^Autocomplete)|(^Rechercher)|(^HP_Mag)|(^ChoixMag)|(^HP))(-success|-failure)?");
+        System.out.println(filterPattern.matcher("ChoixSousCategorie").matches());
+    }
 }



Re: svn commit: r1782611 - /jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java

Posted by Philippe Mouawad <ph...@gmail.com>.
wrong commit.
I just wanted to. commit the logs.

Thanks
I agree with you Felix.

On Saturday, February 11, 2017, Felix Schumacher <
felix.schumacher@internetallee.de> wrote:

>
>
> Am 11. Februar 2017 16:20:41 MEZ schrieb pmouawad@apache.org
> <javascript:;>:
> >Author: pmouawad
> >Date: Sat Feb 11 15:20:41 2017
> >New Revision: 1782611
> >
> >URL: http://svn.apache.org/viewvc?rev=1782611&view=rev
> >Log:
> >Add debug logs
> >
> >Modified:
> >jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/
> HtmlTemplateExporter.java
> >
> >Modified:
> >jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/
> HtmlTemplateExporter.java
> >URL:
> >http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/
> apache/jmeter/report/dashboard/HtmlTemplateExporter.java?rev=
> 1782611&r1=1782610&r2=1782611&view=diff
> >===========================================================
> ===================
> >---
> >jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/
> HtmlTemplateExporter.java
> >(original)
> >+++
> >jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/
> HtmlTemplateExporter.java
> >Sat Feb 11 15:20:41 2017
> >@@ -253,11 +253,23 @@ public class HtmlTemplateExporter extend
> >                    || !supportsControllerDiscrimination.booleanValue()
> >                                             || isController
> >                                          || !showControllerSeriesOnly;
> >+                                    if(log.isDebugEnabled()) {
> >+                                        log.debug("name:{} matches
> >pattern:{}, supportsControllerDiscrimination:{}, isController:{},
> >showControllerSeriesOnly:{}",
> >+                                            name,
> >filterPattern.pattern(),
> >+
> >supportsControllerDiscrimination, isController,
> >showControllerSeriesOnly);
> >+                                    }
> >                                 } else {
> >                             // If the name does not match the pattern,
> >                                // other properties can hold the series
> >                                     matches = filtersOnlySampleSeries
> >                   && !supportsControllerDiscrimination.booleanValue();
> >+                                    if(log.isDebugEnabled()) {
> >+                                        log.debug("name:{} does not
> >match pattern:{}, filtersOnlySampleSeries:{},"
> >+                                            + "
> >supportsControllerDiscrimination:{}",
> >+                                            name,
> >filterPattern.pattern(),
> >+                                            filtersOnlySampleSeries,
> >+
> >supportsControllerDiscrimination);
> >+                                    }
> >                                 }
> >                             }
> >                             index++;
> >@@ -495,4 +507,8 @@ public class HtmlTemplateExporter extend
> >
> >     }
> >
> >+    public static void main(String[] args) {
> >+        Pattern filterPattern =
> >Pattern.compile("((^EnleverArticle)|(^ModifierQuantite)|(^
> VoirPanier)|(^AjoutPanier)|(^ChoixMagasin)|(^
> ChangerMagasin)|(^VoirPanier)|(^ClickProduit)|(^ClicTriPrix)
> |(^ClickPagination)|(^AjoutProduitPourComparison)|(^ClickComparer)|(^
> ChoixCategorie)|(^ChoixSousCategorie)|(^ClickFiltrer)|(^
> RechercheFullText)|(^Autocomplete)|(^Rechercher)|(^
> HP_Mag)|(^ChoixMag)|(^HP))(-success|-failure)?");
> >+
> >System.out.println(filterPattern.matcher("ChoixSousCategorie").matches()
> );
> >+    }
>
> A main method to test something? Why not use a test.
>
> When I use such a complex regex, I usually try to generate it with some
> sort of join.
>
> Regards,
>  Felix
>
> > }
>


-- 
Cordialement.
Philippe Mouawad.

Re: svn commit: r1782611 - /jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java

Posted by Felix Schumacher <fe...@internetallee.de>.

Am 11. Februar 2017 16:20:41 MEZ schrieb pmouawad@apache.org:
>Author: pmouawad
>Date: Sat Feb 11 15:20:41 2017
>New Revision: 1782611
>
>URL: http://svn.apache.org/viewvc?rev=1782611&view=rev
>Log:
>Add debug logs
>
>Modified:
>jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java
>
>Modified:
>jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java
>URL:
>http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java?rev=1782611&r1=1782610&r2=1782611&view=diff
>==============================================================================
>---
>jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java
>(original)
>+++
>jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/HtmlTemplateExporter.java
>Sat Feb 11 15:20:41 2017
>@@ -253,11 +253,23 @@ public class HtmlTemplateExporter extend
>                    || !supportsControllerDiscrimination.booleanValue()
>                                             || isController
>                                          || !showControllerSeriesOnly;
>+                                    if(log.isDebugEnabled()) {
>+                                        log.debug("name:{} matches
>pattern:{}, supportsControllerDiscrimination:{}, isController:{},
>showControllerSeriesOnly:{}", 
>+                                            name,
>filterPattern.pattern(), 
>+                                           
>supportsControllerDiscrimination, isController,
>showControllerSeriesOnly);
>+                                    }
>                                 } else {
>                             // If the name does not match the pattern,
>                                // other properties can hold the series
>                                     matches = filtersOnlySampleSeries
>                   && !supportsControllerDiscrimination.booleanValue();
>+                                    if(log.isDebugEnabled()) {
>+                                        log.debug("name:{} does not
>match pattern:{}, filtersOnlySampleSeries:{},"
>+                                            + "
>supportsControllerDiscrimination:{}", 
>+                                            name,
>filterPattern.pattern(), 
>+                                            filtersOnlySampleSeries,
>+                                           
>supportsControllerDiscrimination);
>+                                    }
>                                 }
>                             }
>                             index++;
>@@ -495,4 +507,8 @@ public class HtmlTemplateExporter extend
> 
>     }
> 
>+    public static void main(String[] args) {
>+        Pattern filterPattern =
>Pattern.compile("((^EnleverArticle)|(^ModifierQuantite)|(^VoirPanier)|(^AjoutPanier)|(^ChoixMagasin)|(^ChangerMagasin)|(^VoirPanier)|(^ClickProduit)|(^ClicTriPrix)|(^ClickPagination)|(^AjoutProduitPourComparison)|(^ClickComparer)|(^ChoixCategorie)|(^ChoixSousCategorie)|(^ClickFiltrer)|(^RechercheFullText)|(^Autocomplete)|(^Rechercher)|(^HP_Mag)|(^ChoixMag)|(^HP))(-success|-failure)?");
>+       
>System.out.println(filterPattern.matcher("ChoixSousCategorie").matches());
>+    }

A main method to test something? Why not use a test.

When I use such a complex regex, I usually try to generate it with some sort of join.

Regards,
 Felix

> }