More Java 7 syntax in tests and other optimisations Project: http://git-wip-us.apache.org/repos/asf/ant-ivy/repo Commit: http://git-wip-us.apache.org/repos/asf/ant-ivy/commit/973704dc Tree: http://git-wip-us.apache.org/repos/asf/ant-ivy/tree/973704dc Diff: http://git-wip-us.apache.org/repos/asf/ant-ivy/diff/973704dc
Branch: refs/heads/master Commit: 973704dc1c284a148c989e3da92a546bd0d2af0d Parents: 3cf36b8 Author: twogee <g.grigelio...@gmail.com> Authored: Tue Jun 13 21:24:03 2017 +0200 Committer: twogee <g.grigelio...@gmail.com> Committed: Tue Jun 13 21:55:52 2017 +0200 ---------------------------------------------------------------------- .../core/module/descriptor/IvyMakePomTest.java | 11 ++---- .../apache/ivy/osgi/obr/OBRXMLWriterTest.java | 10 +---- .../plugins/lock/ArtifactLockStrategyTest.java | 6 +-- .../ivy/plugins/repository/vfs/VfsURI.java | 41 +++++++++++--------- 4 files changed, 29 insertions(+), 39 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/973704dc/test/java/org/apache/ivy/core/module/descriptor/IvyMakePomTest.java ---------------------------------------------------------------------- diff --git a/test/java/org/apache/ivy/core/module/descriptor/IvyMakePomTest.java b/test/java/org/apache/ivy/core/module/descriptor/IvyMakePomTest.java index 8b4ba44..b372479 100644 --- a/test/java/org/apache/ivy/core/module/descriptor/IvyMakePomTest.java +++ b/test/java/org/apache/ivy/core/module/descriptor/IvyMakePomTest.java @@ -170,7 +170,7 @@ public class IvyMakePomTest { // move to next sibling nextChild = nextChild.getNextSibling(); } - return new PomDependency(groupId, artifactId, version, scope, classifier, optional != null ? Boolean.parseBoolean(optional) : false); + return new PomDependency(groupId, artifactId, version, scope, classifier, optional != null && Boolean.parseBoolean(optional)); } private static Node skipIfTextNode(final Node node) { @@ -182,13 +182,8 @@ public class IvyMakePomTest { @Override public String toString() { - return "PomDependency{" + - "groupId='" + groupId + '\'' + - ", artifactId='" + artifactId + '\'' + - ", version='" + version + '\'' + - ", scope='" + scope + '\'' + - ", classifier='" + classifier + '\'' + - '}'; + return String.format("PomDependency{groupId='%s', artifactId='%s', version='%s', scope='%s', classifier='%s'}", + groupId, artifactId, version, scope, classifier); } } } http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/973704dc/test/java/org/apache/ivy/osgi/obr/OBRXMLWriterTest.java ---------------------------------------------------------------------- diff --git a/test/java/org/apache/ivy/osgi/obr/OBRXMLWriterTest.java b/test/java/org/apache/ivy/osgi/obr/OBRXMLWriterTest.java index 0054656..f0c4c0d 100644 --- a/test/java/org/apache/ivy/osgi/obr/OBRXMLWriterTest.java +++ b/test/java/org/apache/ivy/osgi/obr/OBRXMLWriterTest.java @@ -62,20 +62,14 @@ public class OBRXMLWriterTest { new File("build/test-files").mkdirs(); File obrFile = new File("build/test-files/obr-sources.xml"); - FileOutputStream out = new FileOutputStream(obrFile); - try { + try (FileOutputStream out = new FileOutputStream(obrFile)) { ContentHandler handler = OBRXMLWriter.newHandler(out, "UTF-8", true); OBRXMLWriter.writeBundles(bundles, handler); - } finally { - out.close(); } - FileInputStream in = new FileInputStream(obrFile); BundleRepoDescriptor repo; - try { + try (FileInputStream in = new FileInputStream(obrFile)) { repo = OBRXMLParser.parse(new URI("file:///test"), in); - } finally { - in.close(); } assertEquals(2, CollectionUtils.toList(repo.getModules()).size()); http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/973704dc/test/java/org/apache/ivy/plugins/lock/ArtifactLockStrategyTest.java ---------------------------------------------------------------------- diff --git a/test/java/org/apache/ivy/plugins/lock/ArtifactLockStrategyTest.java b/test/java/org/apache/ivy/plugins/lock/ArtifactLockStrategyTest.java index e886469..5f0375f 100644 --- a/test/java/org/apache/ivy/plugins/lock/ArtifactLockStrategyTest.java +++ b/test/java/org/apache/ivy/plugins/lock/ArtifactLockStrategyTest.java @@ -189,11 +189,7 @@ public class ArtifactLockStrategyTest { } } catch (ParseException e) { Message.info("parse exception " + e); - } catch (RuntimeException e) { - Message.info("exception " + e); - e.printStackTrace(); - throw e; - } catch (Error e) { + } catch (RuntimeException | Error e) { Message.info("exception " + e); e.printStackTrace(); throw e; http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/973704dc/test/java/org/apache/ivy/plugins/repository/vfs/VfsURI.java ---------------------------------------------------------------------- diff --git a/test/java/org/apache/ivy/plugins/repository/vfs/VfsURI.java b/test/java/org/apache/ivy/plugins/repository/vfs/VfsURI.java index 1dac7b0..3b89002 100644 --- a/test/java/org/apache/ivy/plugins/repository/vfs/VfsURI.java +++ b/test/java/org/apache/ivy/plugins/repository/vfs/VfsURI.java @@ -60,24 +60,29 @@ public class VfsURI { */ public static VfsURI vfsURIFactory(String scheme, String resource, Ivy ivy) { VfsURI vfsURI = null; - if (scheme.equals(SCHEME_CIFS)) { - vfsURI = new VfsURI(SCHEME_CIFS, ivy.getVariable(VfsTestHelper.PROP_VFS_USER_ID), - ivy.getVariable(VfsTestHelper.PROP_VFS_USER_PASSWD), - ivy.getVariable(VfsTestHelper.PROP_VFS_HOST), - ivy.getVariable(VfsTestHelper.PROP_VFS_SAMBA_REPO) + "/" + resource); - } else if (scheme.equals(SCHEME_FILE)) { - vfsURI = new VfsURI(SCHEME_FILE, null, null, null, VfsTestHelper.CWD + "/" - + VfsTestHelper.TEST_REPO_DIR + "/" + resource); - } else if (scheme.equals(SCHEME_FTP)) { - vfsURI = new VfsURI(SCHEME_FTP, ivy.getVariable(VfsTestHelper.PROP_VFS_USER_ID), - ivy.getVariable(VfsTestHelper.PROP_VFS_USER_PASSWD), - ivy.getVariable(VfsTestHelper.PROP_VFS_HOST), VfsTestHelper.CWD + "/" - + VfsTestHelper.TEST_REPO_DIR + "/" + resource); - } else if (scheme.equals(SCHEME_SFTP)) { - vfsURI = new VfsURI(SCHEME_SFTP, ivy.getVariable(VfsTestHelper.PROP_VFS_USER_ID), - ivy.getVariable(VfsTestHelper.PROP_VFS_USER_PASSWD), - ivy.getVariable(VfsTestHelper.PROP_VFS_HOST), VfsTestHelper.CWD + "/" - + VfsTestHelper.TEST_REPO_DIR + "/" + resource); + switch (scheme) { + case SCHEME_CIFS: + vfsURI = new VfsURI(SCHEME_CIFS, ivy.getVariable(VfsTestHelper.PROP_VFS_USER_ID), + ivy.getVariable(VfsTestHelper.PROP_VFS_USER_PASSWD), + ivy.getVariable(VfsTestHelper.PROP_VFS_HOST), + ivy.getVariable(VfsTestHelper.PROP_VFS_SAMBA_REPO) + "/" + resource); + break; + case SCHEME_FILE: + vfsURI = new VfsURI(SCHEME_FILE, null, null, null, VfsTestHelper.CWD + "/" + + VfsTestHelper.TEST_REPO_DIR + "/" + resource); + break; + case SCHEME_FTP: + vfsURI = new VfsURI(SCHEME_FTP, ivy.getVariable(VfsTestHelper.PROP_VFS_USER_ID), + ivy.getVariable(VfsTestHelper.PROP_VFS_USER_PASSWD), + ivy.getVariable(VfsTestHelper.PROP_VFS_HOST), VfsTestHelper.CWD + "/" + + VfsTestHelper.TEST_REPO_DIR + "/" + resource); + break; + case SCHEME_SFTP: + vfsURI = new VfsURI(SCHEME_SFTP, ivy.getVariable(VfsTestHelper.PROP_VFS_USER_ID), + ivy.getVariable(VfsTestHelper.PROP_VFS_USER_PASSWD), + ivy.getVariable(VfsTestHelper.PROP_VFS_HOST), VfsTestHelper.CWD + "/" + + VfsTestHelper.TEST_REPO_DIR + "/" + resource); + break; } return vfsURI; }