Author: aramirez
Date: Tue Mar 28 23:52:47 2006
New Revision: 389702

URL: http://svn.apache.org/viewcvs?rev=389702&view=rev
Log:
PR:MSUREFIREREP-15

-checks if the group attribute in the xml exists. For testng support

Modified:
    
maven/plugins/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestSuite.java

Modified: 
maven/plugins/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestSuite.java
URL: 
http://svn.apache.org/viewcvs/maven/plugins/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestSuite.java?rev=389702&r1=389701&r2=389702&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestSuite.java
 (original)
+++ 
maven/plugins/trunk/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/ReportTestSuite.java
 Tue Mar 28 23:52:47 2006
@@ -88,20 +88,32 @@
                 
                 timeElapsed = time.floatValue();
 
-                fullClassName = attrs.getValue( "name" );
-    
-                name = fullClassName.substring( fullClassName.lastIndexOf( "." 
) + 1,
-                                        fullClassName.length(  ) );
-
-                int lastDotPosition = fullClassName.lastIndexOf( "." );
-                if ( lastDotPosition < 0 )
+                //check if group attribute is existing
+                if( attrs.getValue( "group" ) != null && !attrs.getValue( 
"group" ).equals( "" ) )
                 {
-                    /* no package name */
-                    packageName = "";
+                    packageName = attrs.getValue( "group" );
+
+                    name = attrs.getValue( "name" );
+
+                    fullClassName = packageName + "." + name;
                 }
                 else
                 {
-                    packageName = fullClassName.substring( 0, lastDotPosition 
);
+                    fullClassName = attrs.getValue( "name" );
+
+                    name = fullClassName.substring( fullClassName.lastIndexOf( 
"." ) + 1,
+                                            fullClassName.length(  ) );
+
+                    int lastDotPosition = fullClassName.lastIndexOf( "." );
+                    if ( lastDotPosition < 0 )
+                    {
+                        /* no package name */
+                        packageName = "";
+                    }
+                    else
+                    {
+                        packageName = fullClassName.substring( 0, 
lastDotPosition );
+                    }
                 }
 
                 testCases = new ArrayList(  );


Reply via email to