You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by ma...@apache.org on 2010/04/23 21:36:23 UTC

svn commit: r937482 - /ant/ivy/core/trunk/test/java/org/apache/ivy/plugins/resolver/PackagerResolverTest.java

Author: maartenc
Date: Fri Apr 23 19:36:22 2010
New Revision: 937482

URL: http://svn.apache.org/viewvc?rev=937482&view=rev
Log:
Code style

Modified:
    ant/ivy/core/trunk/test/java/org/apache/ivy/plugins/resolver/PackagerResolverTest.java

Modified: ant/ivy/core/trunk/test/java/org/apache/ivy/plugins/resolver/PackagerResolverTest.java
URL: http://svn.apache.org/viewvc/ant/ivy/core/trunk/test/java/org/apache/ivy/plugins/resolver/PackagerResolverTest.java?rev=937482&r1=937481&r2=937482&view=diff
==============================================================================
--- ant/ivy/core/trunk/test/java/org/apache/ivy/plugins/resolver/PackagerResolverTest.java (original)
+++ ant/ivy/core/trunk/test/java/org/apache/ivy/plugins/resolver/PackagerResolverTest.java Fri Apr 23 19:36:22 2010
@@ -46,44 +46,43 @@ import org.apache.ivy.util.Message;
  */
 public class PackagerResolverTest extends AbstractDependencyResolverTest {
 
-    private IvySettings _settings;
+    private IvySettings settings;
 
-    private ResolveEngine _engine;
+    private ResolveData data;
 
-    private ResolveData _data;
+    private File cache;
 
-    private File _cache;
-
-    private File _workdir;
-    private File _builddir;
-    private File _cachedir;
+    private File workdir;
+    private File builddir;
+    private File cachedir;
 
     protected void setUp() throws Exception {
-        _settings = new IvySettings();
         Message.setDefaultLogger(new DefaultMessageLogger(99));
-        _engine = new ResolveEngine(_settings, new EventManager(), new SortEngine(_settings));
-        _cache = new File("build/cache");
-        _data = new ResolveData(_engine, new ResolveOptions());
-        _cache.mkdirs();
-        _settings.setDefaultCache(_cache);
+
+        settings = new IvySettings();
+        ResolveEngine engine = new ResolveEngine(settings, new EventManager(), new SortEngine(settings));
+        cache = new File("build/cache");
+        data = new ResolveData(engine, new ResolveOptions());
+        cache.mkdirs();
+        settings.setDefaultCache(cache);
 
         // Create work space with build and resource cache directories
-        _workdir = new File("build/test/PackagerResolverTest");
-        _builddir = new File(_workdir, "build");
-        _cachedir = new File(_workdir, "resources");
+        workdir = new File("build/test/PackagerResolverTest");
+        builddir = new File(workdir, "build");
+        cachedir = new File(workdir, "resources");
         cleanupTempDirs();
-        if (!_builddir.mkdirs() || !_cachedir.mkdirs()) {
-            throw new Exception("can't create directories under " + _workdir);
+        if (!builddir.mkdirs() || !cachedir.mkdirs()) {
+            throw new Exception("can't create directories under " + workdir);
         }
     }
 
     protected void tearDown() throws Exception {
-        FileUtil.forceDelete(_cache);
+        FileUtil.forceDelete(cache);
         cleanupTempDirs();
     }
 
     protected void cleanupTempDirs() throws Exception {
-        FileUtil.forceDelete(_workdir);
+        FileUtil.forceDelete(workdir);
     }
 
     public void testFile() throws Exception {
@@ -95,14 +94,14 @@ public class PackagerResolverTest extend
     
             // Create and configure resolver
             PackagerResolver resolver = new PackagerResolver();
-            resolver.setSettings(_settings);
+            resolver.setSettings(settings);
             File repoRoot = new File("test/repositories/packager/repo");
             resolver.addIvyPattern(
-              "" + new File(repoRoot, "[organisation]/[module]/[revision]/ivy.xml").getAbsoluteFile().toURL().toExternalForm());
+                    new File(repoRoot, "[organisation]/[module]/[revision]/ivy.xml").getAbsoluteFile().toURL().toExternalForm());
             resolver.setPackagerPattern(
-              "" + new File(repoRoot, "[organisation]/[module]/[revision]/packager.xml").getAbsoluteFile().toURL().toExternalForm());
-            resolver.setBuildRoot(_builddir);
-            resolver.setResourceCache(_cachedir);
+                    new File(repoRoot, "[organisation]/[module]/[revision]/packager.xml").getAbsoluteFile().toURL().toExternalForm());
+            resolver.setBuildRoot(builddir);
+            resolver.setResourceCache(cachedir);
             resolver.setPreserveBuildDirectories(true);
             resolver.setVerbose(true);
             
@@ -115,7 +114,7 @@ public class PackagerResolverTest extend
             // Get module descriptor
             ModuleRevisionId mrid = ModuleRevisionId.newInstance("org", "mod", "1.0");
             ResolvedModuleRevision rmr = resolver.getDependency(
-              new DefaultDependencyDescriptor(mrid, false), _data);
+              new DefaultDependencyDescriptor(mrid, false), data);
             assertNotNull(rmr);
     
             assertEquals(mrid, rmr.getId());
@@ -137,7 +136,7 @@ public class PackagerResolverTest extend
             assertEquals(DownloadStatus.SUCCESSFUL, ar.getDownloadStatus());
     
             // Verify resource cache now contains the distribution archive
-            assertTrue(new File(_cachedir, "mod-1.0.tar.gz").exists());
+            assertTrue(new File(cachedir, "mod-1.0.tar.gz").exists());
     
             // Download again, should use Ivy cache this time
             report = resolver.download(new Artifact[] {artifact}, downloadOptions());
@@ -177,14 +176,14 @@ public class PackagerResolverTest extend
     
             // Create and configure resolver
             PackagerResolver resolver = new PackagerResolver();
-            resolver.setSettings(_settings);
+            resolver.setSettings(settings);
             File repoRoot = new File("test/repositories/IVY-1179/repo");
             resolver.addIvyPattern(
-              "" + new File(repoRoot, "[organisation]/[module]/[revision]/ivy.xml").getAbsoluteFile().toURL().toExternalForm());
+                    new File(repoRoot, "[organisation]/[module]/[revision]/ivy.xml").getAbsoluteFile().toURL().toExternalForm());
             resolver.setPackagerPattern(
-              "" + new File(repoRoot, "[organisation]/[module]/[revision]/packager.xml").getAbsoluteFile().toURL().toExternalForm());
-            resolver.setBuildRoot(_builddir);
-            resolver.setResourceCache(_cachedir);
+                    new File(repoRoot, "[organisation]/[module]/[revision]/packager.xml").getAbsoluteFile().toURL().toExternalForm());
+            resolver.setBuildRoot(builddir);
+            resolver.setResourceCache(cachedir);
             resolver.setPreserveBuildDirectories(true);
             resolver.setVerbose(true);
             
@@ -196,18 +195,18 @@ public class PackagerResolverTest extend
             // Get module descriptor
             ModuleRevisionId mrid = ModuleRevisionId.newInstance("org", "A", "1.0");
             ResolvedModuleRevision rmr = resolver.getDependency(
-              new DefaultDependencyDescriptor(mrid, false), _data);
+              new DefaultDependencyDescriptor(mrid, false), data);
     
             // Download artifact
             Artifact artifact = new DefaultArtifact(mrid, rmr.getPublicationDate(), "A", "jar", "jar");
             resolver.download(new Artifact[] {artifact}, downloadOptions());
             
             // assert that the file A.jar is extracted from the archive
-            File jar = new File(_builddir, "org/A/1.0/artifacts/jars/A.jar");
+            File jar = new File(builddir, "org/A/1.0/artifacts/jars/A.jar");
             assertTrue(jar.exists());
             
             // assert that the file README is not extracted from the archive
-            File readme = new File(_builddir, "org/A/1.0/extract/A-1.0/README");
+            File readme = new File(builddir, "org/A/1.0/extract/A-1.0/README");
             assertFalse(readme.exists());
         } finally {
             Locale.setDefault(oldLocale);
@@ -223,14 +222,14 @@ public class PackagerResolverTest extend
 
             // Create and configure resolver
             PackagerResolver resolver = new PackagerResolver();
-            resolver.setSettings(_settings);
+            resolver.setSettings(settings);
             File repoRoot = new File("test/repositories/IVY-1179/repo");
             resolver.addIvyPattern(
-              "" + new File(repoRoot, "[organisation]/[module]/[revision]/ivy.xml").getAbsoluteFile().toURL().toExternalForm());
+                    new File(repoRoot, "[organisation]/[module]/[revision]/ivy.xml").getAbsoluteFile().toURL().toExternalForm());
             resolver.setPackagerPattern(
-              "" + new File(repoRoot, "[organisation]/[module]/[revision]/packager.xml").getAbsoluteFile().toURL().toExternalForm());
-            resolver.setBuildRoot(_builddir);
-            resolver.setResourceCache(_cachedir);
+                    new File(repoRoot, "[organisation]/[module]/[revision]/packager.xml").getAbsoluteFile().toURL().toExternalForm());
+            resolver.setBuildRoot(builddir);
+            resolver.setResourceCache(cachedir);
             resolver.setPreserveBuildDirectories(true);
             resolver.setVerbose(true);
             
@@ -242,18 +241,18 @@ public class PackagerResolverTest extend
             // Get module descriptor
             ModuleRevisionId mrid = ModuleRevisionId.newInstance("org", "B", "1.0");
             ResolvedModuleRevision rmr = resolver.getDependency(
-              new DefaultDependencyDescriptor(mrid, false), _data);
+              new DefaultDependencyDescriptor(mrid, false), data);
     
             // Download artifact
             Artifact artifact = new DefaultArtifact(mrid, rmr.getPublicationDate(), "B", "jar", "jar");
             resolver.download(new Artifact[] {artifact}, downloadOptions());
             
             // assert that the file B.jar is extracted from the archive
-            File jar = new File(_builddir, "org/B/1.0/artifacts/jars/B.jar");
+            File jar = new File(builddir, "org/B/1.0/artifacts/jars/B.jar");
             assertTrue(jar.exists());
             
             // assert that the file README is not extracted from the archive
-            File readme = new File(_builddir, "org/B/1.0/extract/B-1.0/README");
+            File readme = new File(builddir, "org/B/1.0/extract/B-1.0/README");
             assertFalse(readme.exists());
         } finally {
             Locale.setDefault(oldLocale);