Author: brett
Date: Wed Jul 26 22:25:49 2006
New Revision: 425965

URL: http://svn.apache.org/viewvc?rev=425965&view=rev
Log:
[MNG-127] change record to use lists instead of concatenated strings (index 
representation is the same, except that the trailing \n is removed)

Modified:
    
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalIndexRecordConverter.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneStandardIndexRecordConverter.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecord.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactory.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecord.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactory.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalArtifactIndexTest.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneStandardArtifactIndexTest.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactoryTest.java
    
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactoryTest.java

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalIndexRecordConverter.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalIndexRecordConverter.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalIndexRecordConverter.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalIndexRecordConverter.java
 Wed Jul 26 22:25:49 2006
@@ -22,6 +22,7 @@
 import org.apache.lucene.document.NumberTools;
 import org.apache.maven.repository.indexing.record.MinimalArtifactIndexRecord;
 import org.apache.maven.repository.indexing.record.RepositoryIndexRecord;
+import org.codehaus.plexus.util.StringUtils;
 
 /**
  * Convert the minimal index record to a Lucene document.
@@ -51,7 +52,8 @@
                                                                                
   DateTools.Resolution.SECOND ) );
         addUntokenizedField( document, FLD_FILE_SIZE, 
NumberTools.longToString( standardIndexRecord.getSize() ) );
         addUntokenizedField( document, FLD_MD5, 
standardIndexRecord.getMd5Checksum() );
-        addTokenizedField( document, FLD_CLASSES, 
standardIndexRecord.getClasses() );
+        addTokenizedField( document, FLD_CLASSES,
+                           StringUtils.join( 
standardIndexRecord.getClasses().iterator(), "\n" ) );
 
         return document;
     }

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneStandardIndexRecordConverter.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneStandardIndexRecordConverter.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneStandardIndexRecordConverter.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/lucene/LuceneStandardIndexRecordConverter.java
 Wed Jul 26 22:25:49 2006
@@ -22,6 +22,7 @@
 import org.apache.lucene.document.NumberTools;
 import org.apache.maven.repository.indexing.record.RepositoryIndexRecord;
 import org.apache.maven.repository.indexing.record.StandardArtifactIndexRecord;
+import org.codehaus.plexus.util.StringUtils;
 
 /**
  * Convert the standard index record to a Lucene document.
@@ -86,8 +87,16 @@
         addUntokenizedField( document, FLD_FILE_SIZE, 
NumberTools.longToString( standardIndexRecord.getSize() ) );
         addUntokenizedField( document, FLD_MD5, 
standardIndexRecord.getMd5Checksum() );
         addUntokenizedField( document, FLD_SHA1, 
standardIndexRecord.getSha1Checksum() );
-        addTokenizedField( document, FLD_CLASSES, 
standardIndexRecord.getClasses() );
-        addTokenizedField( document, FLD_FILES, standardIndexRecord.getFiles() 
);
+        if ( standardIndexRecord.getClasses() != null )
+        {
+            addTokenizedField( document, FLD_CLASSES,
+                               StringUtils.join( 
standardIndexRecord.getClasses().iterator(), "\n" ) );
+        }
+        if ( standardIndexRecord.getFiles() != null )
+        {
+            addTokenizedField( document, FLD_FILES,
+                               StringUtils.join( 
standardIndexRecord.getFiles().iterator(), "\n" ) );
+        }
         addTokenizedField( document, FLD_PLUGINPREFIX, 
standardIndexRecord.getPluginPrefix() );
         addUntokenizedField( document, FLD_INCEPTION_YEAR, 
standardIndexRecord.getInceptionYear() );
         addTokenizedField( document, FLD_PROJECT_NAME, 
standardIndexRecord.getProjectName() );

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecord.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecord.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecord.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecord.java
 Wed Jul 26 22:25:49 2006
@@ -1,6 +1,7 @@
 package org.apache.maven.repository.indexing.record;
 
 import java.util.Date;
+import java.util.List;
 
 /*
  * Copyright 2005-2006 The Apache Software Foundation.
@@ -29,7 +30,7 @@
     /**
      * The classes in the archive for the artifact, if it is a JAR.
      */
-    private String classes;
+    private List classes;
 
     /**
      * The MD5 checksum of the artifact file.
@@ -51,7 +52,7 @@
      */
     private long size;
 
-    public void setClasses( String classes )
+    public void setClasses( List classes )
     {
         this.classes = classes;
     }
@@ -76,7 +77,7 @@
         this.size = size;
     }
 
-    public String getClasses()
+    public List getClasses()
     {
         return classes;
     }

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactory.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactory.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactory.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactory.java
 Wed Jul 26 22:25:49 2006
@@ -21,6 +21,7 @@
 
 import java.io.File;
 import java.io.IOException;
+import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashSet;
 import java.util.Iterator;
@@ -72,9 +73,9 @@
         return record;
     }
 
-    private String getClassesFromFiles( List files )
+    private List getClassesFromFiles( List files )
     {
-        StringBuffer classes = new StringBuffer();
+        List classes = new ArrayList();
 
         for ( Iterator i = files.iterator(); i.hasNext(); )
         {
@@ -82,10 +83,10 @@
 
             if ( isClass( name ) )
             {
-                classes.append( name.substring( 0, name.length() - 6 
).replace( '/', '.' ) ).append( "\n" );
+                classes.add( name.substring( 0, name.length() - 6 ).replace( 
'/', '.' ) );
             }
         }
 
-        return classes.toString();
+        return classes;
     }
 }

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecord.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecord.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecord.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecord.java
 Wed Jul 26 22:25:49 2006
@@ -1,5 +1,7 @@
 package org.apache.maven.repository.indexing.record;
 
+import java.util.List;
+
 /*
  * Copyright 2005-2006 The Apache Software Foundation.
  *
@@ -57,7 +59,7 @@
     /**
      * A list of files (separated by '\n') in the artifact if it is an archive.
      */
-    private String files;
+    private List files;
 
     /**
      * The identifier of the repository that the artifact came from.
@@ -119,7 +121,7 @@
         this.type = type;
     }
 
-    public void setFiles( String files )
+    public void setFiles( List files )
     {
         this.files = files;
     }
@@ -255,7 +257,7 @@
         return type;
     }
 
-    public String getFiles()
+    public List getFiles()
     {
         return files;
     }

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactory.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactory.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactory.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactory.java
 Wed Jul 26 22:25:49 2006
@@ -32,6 +32,7 @@
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStreamReader;
+import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashSet;
@@ -184,8 +185,8 @@
     private void populateArchiveEntries( List files, 
StandardArtifactIndexRecord record, File artifactFile )
         throws RepositoryIndexException
     {
-        StringBuffer classes = new StringBuffer();
-        StringBuffer fileBuffer = new StringBuffer();
+        List classes = new ArrayList();
+        List fileList = new ArrayList();
 
         for ( Iterator i = files.iterator(); i.hasNext(); )
         {
@@ -194,11 +195,11 @@
             // ignore directories
             if ( !name.endsWith( "/" ) )
             {
-                fileBuffer.append( name ).append( "\n" );
+                fileList.add( name );
 
                 if ( isClass( name ) )
                 {
-                    classes.append( name.substring( 0, name.length() - 6 
).replace( '/', '.' ) ).append( "\n" );
+                    classes.add( name.substring( 0, name.length() - 6 
).replace( '/', '.' ) );
                 }
                 else if ( PLUGIN_METADATA_NAME.equals( name ) )
                 {
@@ -211,13 +212,13 @@
             }
         }
 
-        if ( classes.length() > 0 )
+        if ( !classes.isEmpty() )
         {
-            record.setClasses( classes.toString() );
+            record.setClasses( classes );
         }
-        if ( fileBuffer.length() > 0 )
+        if ( !fileList.isEmpty() )
         {
-            record.setFiles( fileBuffer.toString() );
+            record.setFiles( fileList );
         }
     }
 

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalArtifactIndexTest.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalArtifactIndexTest.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalArtifactIndexTest.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneMinimalArtifactIndexTest.java
 Wed Jul 26 22:25:49 2006
@@ -157,7 +157,7 @@
         try
         {
             Document document = reader.document( 0 );
-            assertRecord( document, artifact, 
"3a0adc365f849366cd8b633cad155cb7", "A\nb.B\nb.c.C\n" );
+            assertRecord( document, artifact, 
"3a0adc365f849366cd8b633cad155cb7", "A\nb.B\nb.c.C" );
             assertEquals( "Check index size", 1, reader.numDocs() );
         }
         finally
@@ -184,9 +184,7 @@
         try
         {
             Document document = reader.document( 0 );
-            String expectedChecksum = "3a0adc365f849366cd8b633cad155cb7";
-            String expectedClasses = "A\nb.B\nb.c.C\n";
-            assertRecord( document, artifact, expectedChecksum, 
expectedClasses );
+            assertRecord( document, artifact, 
"3a0adc365f849366cd8b633cad155cb7", "A\nb.B\nb.c.C" );
             assertEquals( "Check index size", 1, reader.numDocs() );
         }
         finally
@@ -232,7 +230,7 @@
         {
             Document document = reader.document( 0 );
             assertRecord( document, artifact, 
"06f6fe25e46c4d4fb5be4f56a9bab0ee",
-                          "org.apache.maven.repository.record.MyMojo\n" );
+                          "org.apache.maven.repository.record.MyMojo" );
             assertEquals( "Check index size", 1, reader.numDocs() );
         }
         finally

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneStandardArtifactIndexTest.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneStandardArtifactIndexTest.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneStandardArtifactIndexTest.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/lucene/LuceneStandardArtifactIndexTest.java
 Wed Jul 26 22:25:49 2006
@@ -309,8 +309,8 @@
     {
         assertRecord( artifact, document, "test-jar", "jar", 
"3a0adc365f849366cd8b633cad155cb7",
                       "c66f18bf192cb613fc2febb4da541a34133eedc2" );
-        assertEquals( "Check document classes", "A\nb.B\nb.c.C\n", 
document.get( "classes" ) );
-        assertEquals( "Check document files", 
"META-INF/MANIFEST.MF\nA.class\nb/B.class\nb/c/C.class\n",
+        assertEquals( "Check document classes", "A\nb.B\nb.c.C", document.get( 
"classes" ) );
+        assertEquals( "Check document files", 
"META-INF/MANIFEST.MF\nA.class\nb/B.class\nb/c/C.class",
                       document.get( "files" ) );
         assertNull( "Check document inceptionYear", document.get( 
"inceptionYear" ) );
         assertNull( "Check document projectName", document.get( "projectName" 
) );
@@ -323,12 +323,12 @@
     {
         assertRecord( artifact, document, "test-plugin", "maven-plugin", 
"06f6fe25e46c4d4fb5be4f56a9bab0ee",
                       "382c1ebfb5d0c7d6061c2f8569fb53f8fc00fec2" );
-        assertEquals( "Check document classes", 
"org.apache.maven.repository.record.MyMojo\n",
+        assertEquals( "Check document classes", 
"org.apache.maven.repository.record.MyMojo",
                       document.get( "classes" ) );
         assertEquals( "Check document files", "META-INF/MANIFEST.MF\n" + 
"META-INF/maven/plugin.xml\n" +
             "org/apache/maven/repository/record/MyMojo.class\n" +
             
"META-INF/maven/org.apache.maven.repository.record/test-plugin/pom.xml\n" +
-            
"META-INF/maven/org.apache.maven.repository.record/test-plugin/pom.properties\n",
 document.get( "files" ) );
+            
"META-INF/maven/org.apache.maven.repository.record/test-plugin/pom.properties", 
document.get( "files" ) );
         assertEquals( "Check document pluginPrefix", "test", document.get( 
"pluginPrefix" ) );
         assertEquals( "Check document packaging", "maven-plugin", 
document.get( "packaging" ) );
         assertNull( "Check document inceptionYear", document.get( 
"inceptionYear" ) );

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactoryTest.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactoryTest.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactoryTest.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/MinimalArtifactIndexRecordFactoryTest.java
 Wed Jul 26 22:25:49 2006
@@ -27,6 +27,9 @@
 
 import java.io.File;
 import java.io.IOException;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
 
 /**
  * Test the minimal artifact index record.
@@ -44,6 +47,8 @@
 
     private static final String TEST_GROUP_ID = 
"org.apache.maven.repository.record";
 
+    private static final List JAR_CLASS_LIST = Arrays.asList( new 
String[]{"A", "b.B", "b.c.C"} );
+
     protected void setUp()
         throws Exception
     {
@@ -75,7 +80,7 @@
         expectedRecord.setFilename( repository.pathOf( artifact ) );
         expectedRecord.setLastModified( artifact.getFile().lastModified() );
         expectedRecord.setSize( artifact.getFile().length() );
-        expectedRecord.setClasses( "A\nb.B\nb.c.C\n" );
+        expectedRecord.setClasses( JAR_CLASS_LIST );
 
         assertEquals( "check record", expectedRecord, record );
     }
@@ -92,7 +97,7 @@
         expectedRecord.setFilename( repository.pathOf( artifact ) );
         expectedRecord.setLastModified( artifact.getFile().lastModified() );
         expectedRecord.setSize( artifact.getFile().length() );
-        expectedRecord.setClasses( "A\nb.B\nb.c.C\n" );
+        expectedRecord.setClasses( JAR_CLASS_LIST );
 
         assertEquals( "check record", expectedRecord, record );
     }
@@ -142,7 +147,7 @@
         expectedRecord.setFilename( repository.pathOf( artifact ) );
         expectedRecord.setLastModified( artifact.getFile().lastModified() );
         expectedRecord.setSize( artifact.getFile().length() );
-        expectedRecord.setClasses( 
"org.apache.maven.repository.record.MyMojo\n" );
+        expectedRecord.setClasses( Collections.singletonList( 
"org.apache.maven.repository.record.MyMojo" ) );
 
         assertEquals( "check record", expectedRecord, record );
     }

Modified: 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactoryTest.java
URL: 
http://svn.apache.org/viewvc/maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactoryTest.java?rev=425965&r1=425964&r2=425965&view=diff
==============================================================================
--- 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactoryTest.java
 (original)
+++ 
maven/repository-manager/trunk/maven-repository-indexer/src/test/java/org/apache/maven/repository/indexing/record/StandardArtifactIndexRecordFactoryTest.java
 Wed Jul 26 22:25:49 2006
@@ -27,6 +27,8 @@
 
 import java.io.File;
 import java.io.IOException;
+import java.util.Arrays;
+import java.util.List;
 
 /**
  * Test the minimal artifact index record.
@@ -44,6 +46,11 @@
 
     private static final String TEST_GROUP_ID = 
"org.apache.maven.repository.record";
 
+    private static final List JAR_CLASS_LIST = Arrays.asList( new 
String[]{"A", "b.B", "b.c.C"} );
+
+    private static final List JAR_FILE_LIST =
+        Arrays.asList( new String[]{"META-INF/MANIFEST.MF", "A.class", 
"b/B.class", "b/c/C.class"} );
+
     protected void setUp()
         throws Exception
     {
@@ -75,11 +82,11 @@
         expectedRecord.setFilename( repository.pathOf( artifact ) );
         expectedRecord.setLastModified( artifact.getFile().lastModified() );
         expectedRecord.setSize( artifact.getFile().length() );
-        expectedRecord.setClasses( "A\nb.B\nb.c.C\n" );
+        expectedRecord.setClasses( JAR_CLASS_LIST );
         expectedRecord.setArtifactId( "test-jar" );
         expectedRecord.setGroupId( TEST_GROUP_ID );
         expectedRecord.setVersion( "1.0" );
-        expectedRecord.setFiles( 
"META-INF/MANIFEST.MF\nA.class\nb/B.class\nb/c/C.class\n" );
+        expectedRecord.setFiles( JAR_FILE_LIST );
         expectedRecord.setSha1Checksum( 
"c66f18bf192cb613fc2febb4da541a34133eedc2" );
         expectedRecord.setType( "jar" );
         expectedRecord.setRepository( "test" );
@@ -99,11 +106,11 @@
         expectedRecord.setFilename( repository.pathOf( artifact ) );
         expectedRecord.setLastModified( artifact.getFile().lastModified() );
         expectedRecord.setSize( artifact.getFile().length() );
-        expectedRecord.setClasses( "A\nb.B\nb.c.C\n" );
+        expectedRecord.setClasses( JAR_CLASS_LIST );
         expectedRecord.setArtifactId( "test-jar-and-pom" );
         expectedRecord.setGroupId( TEST_GROUP_ID );
         expectedRecord.setVersion( "1.0" );
-        expectedRecord.setFiles( 
"META-INF/MANIFEST.MF\nA.class\nb/B.class\nb/c/C.class\n" );
+        expectedRecord.setFiles( JAR_FILE_LIST );
         expectedRecord.setSha1Checksum( 
"c66f18bf192cb613fc2febb4da541a34133eedc2" );
         expectedRecord.setType( "jar" );
         expectedRecord.setRepository( "test" );
@@ -125,11 +132,11 @@
         expectedRecord.setFilename( repository.pathOf( artifact ) );
         expectedRecord.setLastModified( artifact.getFile().lastModified() );
         expectedRecord.setSize( artifact.getFile().length() );
-        expectedRecord.setClasses( "A\nb.B\nb.c.C\n" );
+        expectedRecord.setClasses( JAR_CLASS_LIST );
         expectedRecord.setArtifactId( "test-child-pom" );
         expectedRecord.setGroupId( TEST_GROUP_ID );
         expectedRecord.setVersion( "1.0" );
-        expectedRecord.setFiles( 
"META-INF/MANIFEST.MF\nA.class\nb/B.class\nb/c/C.class\n" );
+        expectedRecord.setFiles( JAR_FILE_LIST );
         expectedRecord.setSha1Checksum( 
"c66f18bf192cb613fc2febb4da541a34133eedc2" );
         expectedRecord.setType( "jar" );
         expectedRecord.setRepository( "test" );
@@ -208,11 +215,11 @@
         expectedRecord.setSha1Checksum( 
"382c1ebfb5d0c7d6061c2f8569fb53f8fc00fec2" );
         expectedRecord.setType( "maven-plugin" );
         expectedRecord.setRepository( "test" );
-        expectedRecord.setClasses( 
"org.apache.maven.repository.record.MyMojo\n" );
-        expectedRecord.setFiles( "META-INF/MANIFEST.MF\n" + 
"META-INF/maven/plugin.xml\n" +
-            "org/apache/maven/repository/record/MyMojo.class\n" +
-            
"META-INF/maven/org.apache.maven.repository.record/test-plugin/pom.xml\n" +
-            
"META-INF/maven/org.apache.maven.repository.record/test-plugin/pom.properties\n"
 );
+        expectedRecord.setClasses( Arrays.asList( new 
String[]{"org.apache.maven.repository.record.MyMojo"} ) );
+        expectedRecord.setFiles( Arrays.asList( new 
String[]{"META-INF/MANIFEST.MF", "META-INF/maven/plugin.xml",
+            "org/apache/maven/repository/record/MyMojo.class",
+            
"META-INF/maven/org.apache.maven.repository.record/test-plugin/pom.xml",
+            
"META-INF/maven/org.apache.maven.repository.record/test-plugin/pom.properties"} 
) );
         expectedRecord.setPackaging( "maven-plugin" );
         expectedRecord.setProjectName( "Maven Mojo Archetype" );
         expectedRecord.setPluginPrefix( "test" );
@@ -238,11 +245,10 @@
         expectedRecord.setSha1Checksum( 
"5ebabafdbcd6684ae434c06e22c32844df284b05" );
         expectedRecord.setType( "maven-archetype" );
         expectedRecord.setRepository( "test" );
-        expectedRecord.setFiles( "META-INF/MANIFEST.MF\n" + 
"archetype-resources/pom.xml\n" +
-            "archetype-resources/src/main/java/App.java\n" + 
"archetype-resources/src/test/java/AppTest.java\n" +
-            "META-INF/maven/archetype.xml\n" +
-            
"META-INF/maven/org.apache.maven.repository.record/test-archetype/pom.xml\n" +
-            
"META-INF/maven/org.apache.maven.repository.record/test-archetype/pom.properties\n"
 );
+        expectedRecord.setFiles( Arrays.asList( new 
String[]{"META-INF/MANIFEST.MF", "archetype-resources/pom.xml",
+            "archetype-resources/src/main/java/App.java", 
"archetype-resources/src/test/java/AppTest.java",
+            "META-INF/maven/archetype.xml", 
"META-INF/maven/org.apache.maven.repository.record/test-archetype/pom.xml",
+            
"META-INF/maven/org.apache.maven.repository.record/test-archetype/pom.properties"}
 ) );
         expectedRecord.setPackaging( "jar" );
         expectedRecord.setProjectName( "Archetype - test-archetype" );
 


Reply via email to