You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by gi...@apache.org on 2017/08/19 04:50:56 UTC
[1/3] ant-ivy git commit: add a target to generate javadoc for test
cases; fix javadoc for doclint
Repository: ant-ivy
Updated Branches:
refs/heads/master 186fd1077 -> c1f910cc5
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/util/url/HttpclientURLHandlerTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/util/url/HttpclientURLHandlerTest.java b/test/java/org/apache/ivy/util/url/HttpclientURLHandlerTest.java
index de82269..d63cf6d 100644
--- a/test/java/org/apache/ivy/util/url/HttpclientURLHandlerTest.java
+++ b/test/java/org/apache/ivy/util/url/HttpclientURLHandlerTest.java
@@ -80,10 +80,15 @@ public class HttpclientURLHandlerTest {
assertFalse("URL resource was expected to be unreachable", handler.isReachable(new URL("http://www.google.fr/unknownpage.html"), defaultTimeoutConstraint));
}
+ /**
+ * Test case for IVY-390.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-390">IVY-390</a>
+ */
@SuppressWarnings("resource")
@Test
public void testGetURLInfo() throws Exception {
- // IVY-390
URLHandler handler = new HttpClientHandler();
URLInfo info = handler
.getURLInfo(new URL(
@@ -115,7 +120,7 @@ public class HttpclientURLHandlerTest {
* Tests that the {@link HttpClientHandler}, backed by {@link CredentialsStore Ivy credentials store}
* works as expected when it interacts with a HTTP server which requires authentication for accessing resources.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1336">IVY-1336</a>
*/
@Test
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/util/url/IvyAuthenticatorTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/util/url/IvyAuthenticatorTest.java b/test/java/org/apache/ivy/util/url/IvyAuthenticatorTest.java
index 9817576..002e68d 100644
--- a/test/java/org/apache/ivy/util/url/IvyAuthenticatorTest.java
+++ b/test/java/org/apache/ivy/util/url/IvyAuthenticatorTest.java
@@ -67,7 +67,7 @@ public class IvyAuthenticatorTest {
* relevant requesting information, including the {@link Authenticator#getRequestingURL() requesting URL} and
* {@link Authenticator#getRequestorType() request type}
*
- * @throws Exception
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1557">IVY-1557</a>
*/
@Test
[3/3] ant-ivy git commit: add a target to generate javadoc for test
cases; fix javadoc for doclint
Posted by gi...@apache.org.
add a target to generate javadoc for test cases; fix javadoc for doclint
Project: http://git-wip-us.apache.org/repos/asf/ant-ivy/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant-ivy/commit/c1f910cc
Tree: http://git-wip-us.apache.org/repos/asf/ant-ivy/tree/c1f910cc
Diff: http://git-wip-us.apache.org/repos/asf/ant-ivy/diff/c1f910cc
Branch: refs/heads/master
Commit: c1f910cc5380e3a2440f63d03d4cb4fda4e627c0
Parents: 186fd10
Author: Gintas Grigelionis <gi...@apache.org>
Authored: Sat Aug 19 06:44:37 2017 +0200
Committer: Gintas Grigelionis <gi...@apache.org>
Committed: Sat Aug 19 06:44:37 2017 +0200
----------------------------------------------------------------------
build.properties | 1 +
build.xml | 21 +-
.../ivy/core/retrieve/RetrieveOptions.java | 5 +-
.../ivy/osgi/repo/RelativeURLRepository.java | 2 +-
.../ivy/plugins/repository/ssh/SshCache.java | 1 +
src/java/org/apache/ivy/util/FileUtil.java | 2 +-
.../org/apache/ivy/util/url/URLHandler.java | 40 +-
test/java/org/apache/ivy/MainTest.java | 9 +-
test/java/org/apache/ivy/TestFixture.java | 11 +-
test/java/org/apache/ivy/TestHelper.java | 25 +-
test/java/org/apache/ivy/ant/FileUtilTest.java | 4 +-
.../ivy/ant/IvyAntSettingsBuildFileTest.java | 12 +-
.../apache/ivy/ant/IvyArtifactPropertyTest.java | 2 +-
.../org/apache/ivy/ant/IvyBuildNumberTest.java | 7 +-
.../org/apache/ivy/ant/IvyCacheFilesetTest.java | 11 +-
.../org/apache/ivy/ant/IvyCachePathTest.java | 2 +-
.../org/apache/ivy/ant/IvyCleanCacheTest.java | 2 +-
.../org/apache/ivy/ant/IvyConfigureTest.java | 13 +-
.../java/org/apache/ivy/ant/IvyDeliverTest.java | 21 +-
.../apache/ivy/ant/IvyDependencyTreeTest.java | 9 +-
.../ivy/ant/IvyDependencyUpdateCheckerTest.java | 22 +-
test/java/org/apache/ivy/ant/IvyInfoTest.java | 12 +
.../java/org/apache/ivy/ant/IvyInstallTest.java | 25 +-
.../java/org/apache/ivy/ant/IvyPublishTest.java | 26 +-
.../java/org/apache/ivy/ant/IvyResolveTest.java | 33 +-
.../org/apache/ivy/ant/IvyRetrieveTest.java | 30 +-
.../apache/ivy/core/deliver/DeliverTest.java | 6 +
.../core/module/descriptor/IvyMakePomTest.java | 10 +-
.../ivy/core/publish/PublishEventsTest.java | 6 +
.../apache/ivy/core/resolve/ResolveTest.java | 479 ++++++++++++++++---
.../apache/ivy/core/retrieve/RetrieveTest.java | 40 +-
.../core/settings/XmlSettingsParserTest.java | 31 +-
.../java/org/apache/ivy/core/sort/SortTest.java | 21 +-
.../ivy/osgi/core/OSGiManifestParserTest.java | 6 +-
.../LatestCompatibleConflictManagerTest.java | 4 +-
.../conflict/LatestConflictManagerTest.java | 49 +-
.../conflict/StrictConflictManagerTest.java | 4 +-
.../m2/PomModuleDescriptorParserTest.java | 46 +-
.../xml/XmlModuleDescriptorParserTest.java | 15 +-
.../xml/XmlModuleDescriptorWriterTest.java | 6 +-
.../parser/xml/XmlModuleUpdaterTest.java | 7 +-
.../repository/vfs/VfsRepositoryTest.java | 44 +-
.../plugins/repository/vfs/VfsResourceTest.java | 12 +
.../plugins/repository/vfs/VfsTestHelper.java | 2 +-
.../ivy/plugins/repository/vfs/VfsURI.java | 2 +-
.../ivy/plugins/resolver/ChainResolverTest.java | 22 +-
.../ivy/plugins/resolver/DualResolverTest.java | 6 +-
.../resolver/FileSystemResolverTest.java | 18 +-
.../plugins/resolver/IvyRepResolverTest.java | 5 +-
.../ivy/plugins/resolver/URLResolverTest.java | 2 +-
.../java/org/apache/ivy/util/TestXmlHelper.java | 4 +-
.../ivy/util/url/AbstractURLHandlerTest.java | 10 +-
.../ivy/util/url/ApacheURLListerTest.java | 4 +-
.../ivy/util/url/HttpclientURLHandlerTest.java | 9 +-
.../ivy/util/url/IvyAuthenticatorTest.java | 2 +-
55 files changed, 944 insertions(+), 276 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/build.properties
----------------------------------------------------------------------
diff --git a/build.properties b/build.properties
index b291a81..60606d9 100644
--- a/build.properties
+++ b/build.properties
@@ -37,6 +37,7 @@ jacoco.log=${build.dir}/jacoco.data
test.report.dir=${reports.dir}/test/html
coverage.report.dir=${reports.dir}/coverage
javadoc.build.dir=${reports.dir}/api
+test.javadoc.build.dir=${reports.dir}/test-api
ivy.report.dir=${reports.dir}/ivy
doc.src.dir=${basedir}/asciidoc
checkstyle.report.dir=${reports.dir}/checkstyle
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/build.xml
----------------------------------------------------------------------
diff --git a/build.xml b/build.xml
index 834123f..660cc8b 100644
--- a/build.xml
+++ b/build.xml
@@ -90,7 +90,6 @@
<path refid="lib.classpath"/>
</path>
<path id="test.classpath">
- <pathelement location="${coverage.classes.dir}"/>
<fileset dir="${lib.dir}">
<include name="*.jar"/>
<exclude name="ant.jar"/>
@@ -101,6 +100,10 @@
<pathelement location="${optional.classes.build.dir}"/>
<pathelement path="${test.build.dir}"/>
</path>
+ <path id="javadoc.test.classpath">
+ <path refid="run.classpath"/>
+ <pathelement path="${test.build.dir}"/>
+ </path>
</target>
<target name="prepare" depends="init">
@@ -518,6 +521,22 @@
</javadoc>
</target>
+ <target name="javadoc-test" depends="init" unless="skip.javadoc">
+ <tstamp>
+ <format pattern="2007-yyyy" property="years"/>
+ </tstamp>
+ <property name="copyright" value="Copyright &copy;${years} The Apache Software Foundation, Licensed under <a href="http://www.apache.org/licenses/LICENSE-2.0.txt">Apache License, Version 2.0</a>."/>
+ <property name="title" value="Apache Ivy&trade; dependency manager ${target.ivy.version} API"/>
+ <javadoc destdir="${test.javadoc.build.dir}"
+ useexternalfile="true"
+ windowtitle="Apache Ivy&trade;"
+ doctitle="${title}"
+ bottom="${copyright}"
+ classpathref="javadoc.test.classpath">
+ <fileset dir="${test.dir}" includes="**/*.java"/>
+ </javadoc>
+ </target>
+
<target name="sources" depends="default-version" description="Create source archive files">
<mkdir dir="${artifacts.build.dir}/sources/"/>
<jar destfile="${artifacts.build.dir}/sources/${final.name}">
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/src/java/org/apache/ivy/core/retrieve/RetrieveOptions.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/ivy/core/retrieve/RetrieveOptions.java b/src/java/org/apache/ivy/core/retrieve/RetrieveOptions.java
index 304a351..c74e13e 100644
--- a/src/java/org/apache/ivy/core/retrieve/RetrieveOptions.java
+++ b/src/java/org/apache/ivy/core/retrieve/RetrieveOptions.java
@@ -158,11 +158,12 @@ public class RetrieveOptions extends LogOptions {
return makeSymlinks || makeSymlinksInMass;
}
- @Deprecated
/**
+ * @return false
* @deprecated Starting 2.5, creating symlinks in mass is no longer supported and this
* method will always return false
*/
+ @Deprecated
public boolean isMakeSymlinksInMass() {
return false;
}
@@ -173,6 +174,8 @@ public class RetrieveOptions extends LogOptions {
}
/**
+ * @param makeSymlinksInMass ditto
+ * @return RetrieveOptions
* @deprecated Starting 2.5, creating symlinks in mass is no longer supported and this
* method plays no role in creation of symlinks. Use {@link #setMakeSymlinks(boolean)} instead
*/
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/src/java/org/apache/ivy/osgi/repo/RelativeURLRepository.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/ivy/osgi/repo/RelativeURLRepository.java b/src/java/org/apache/ivy/osgi/repo/RelativeURLRepository.java
index 8ab90cb..fbcb21d 100644
--- a/src/java/org/apache/ivy/osgi/repo/RelativeURLRepository.java
+++ b/src/java/org/apache/ivy/osgi/repo/RelativeURLRepository.java
@@ -40,7 +40,7 @@ public class RelativeURLRepository extends URLRepository {
/**
*
- * @param baseUrl
+ * @param baseUrl ditto
* @deprecated Since 2.5. Use {@link #RelativeURLRepository(URL, TimeoutConstraint)} instead
*/
@Deprecated
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/src/java/org/apache/ivy/plugins/repository/ssh/SshCache.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/ivy/plugins/repository/ssh/SshCache.java b/src/java/org/apache/ivy/plugins/repository/ssh/SshCache.java
index 04ac8b5..004b390 100644
--- a/src/java/org/apache/ivy/plugins/repository/ssh/SshCache.java
+++ b/src/java/org/apache/ivy/plugins/repository/ssh/SshCache.java
@@ -263,6 +263,7 @@ public final class SshCache {
* @param session
* to connect to
* @return channelSftp or null if not successful (channel not existent or dead)
+ * @throws IOException should never happen
*/
public ChannelSftp getChannelSftp(Session session) throws IOException {
ChannelSftp channel = null;
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/src/java/org/apache/ivy/util/FileUtil.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/ivy/util/FileUtil.java b/src/java/org/apache/ivy/util/FileUtil.java
index 138a9ce..957e8de 100644
--- a/src/java/org/apache/ivy/util/FileUtil.java
+++ b/src/java/org/apache/ivy/util/FileUtil.java
@@ -70,7 +70,7 @@ public final class FileUtil {
* @param overwrite {@code true} if any existing file at {@code link} has to be overwritten. False otherwise
* @return Returns true if the symlink was successfully created. Returns false if the symlink creation couldn't
* be done
- * @throws IOException
+ * @throws IOException if {@link Files#createSymbolicLink} fails
*/
public static boolean symlink(final File target, final File link, final boolean overwrite)
throws IOException {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/src/java/org/apache/ivy/util/url/URLHandler.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/ivy/util/url/URLHandler.java b/src/java/org/apache/ivy/util/url/URLHandler.java
index 0b31c5e..5f75479 100644
--- a/src/java/org/apache/ivy/util/url/URLHandler.java
+++ b/src/java/org/apache/ivy/util/url/URLHandler.java
@@ -109,11 +109,12 @@ public interface URLHandler {
* <p>
* Please use {@link #getURLInfo(URL, TimeoutConstraint)} if more one information about the <code>url</code>
* is needed
+ * </p>
*
* @param url The URL to access
* @param timeoutConstraint The connectivity timeout constraints. Can be null, in which case the timeouts
* are implementation specific
- * @return
+ * @return boolean
* @since 2.5
*/
boolean isReachable(URL url, TimeoutConstraint timeoutConstraint);
@@ -147,7 +148,7 @@ public interface URLHandler {
* @param url The URL to access
* @param timeoutConstraint The connectivity timeout constraints. Can be null, in which case the timeouts
* are implementation specific
- * @return
+ * @return long
* @since 2.5
*/
long getContentLength(URL url, TimeoutConstraint timeoutConstraint);
@@ -178,11 +179,12 @@ public interface URLHandler {
* <p>
* Please use {@link #getURLInfo(URL, TimeoutConstraint)} if more one information about the <code>url</code>
* is needed
+ * </p>
*
* @param url The URL to access
* @param timeoutConstraint The connectivity timeout constraints. Can be null, in which case the timeouts
* are implementation specific
- * @return
+ * @return long
* @since 2.5
*/
long getLastModified(URL url, TimeoutConstraint timeoutConstraint);
@@ -213,15 +215,15 @@ public interface URLHandler {
* @param url The URL for which the information is to be retrieved
* @param timeoutConstraint The connectivity timeout constraints. Can be null, in which case the timeouts
* are implementation specific
- * @return
+ * @return URLInfo
* @since 2.5
*/
URLInfo getURLInfo(URL url, TimeoutConstraint timeoutConstraint);
/**
- * @param url
- * @return
- * @throws IOException
+ * @param url ditto
+ * @return InputStream
+ * @throws IOException if something goes wrong
* @deprecated Use {@link #openStream(URL, TimeoutConstraint)} instead
*/
@Deprecated
@@ -233,17 +235,17 @@ public interface URLHandler {
* @param url The URL to which an {@link InputStream} has to be opened
* @param timeoutConstraint The connectivity timeout constraints. Can be null, in which case the timeouts
* are implementation specific
- * @return
- * @throws IOException
+ * @return InputStream
+ * @throws IOException if something goes wrong
* @since 2.5
*/
InputStream openStream(URL url, TimeoutConstraint timeoutConstraint) throws IOException;
/**
- * @param src
- * @param dest
- * @param l
- * @throws IOException
+ * @param src URL
+ * @param dest File
+ * @param l CopyProgressListener
+ * @throws IOException if something goes wrong
* @deprecated Use {@link #download(URL, File, CopyProgressListener, TimeoutConstraint)} instead
*/
@Deprecated
@@ -257,16 +259,16 @@ public interface URLHandler {
* @param listener The listener that will be notified of the download progress
* @param timeoutConstraint The connectivity timeout constraints. Can be null, in which case the timeouts
* are implementation specific
- * @throws IOException
+ * @throws IOException if something goes wrong
* @since 2.5
*/
void download(URL src, File dest, CopyProgressListener listener, TimeoutConstraint timeoutConstraint) throws IOException;
/**
- * @param src
- * @param dest
- * @param l
- * @throws IOException
+ * @param src File
+ * @param dest URL
+ * @param l CopyProgressListener
+ * @throws IOException if something goes wrong
* @deprecated Use {@link #upload(File, URL, CopyProgressListener, TimeoutConstraint)} instead
*/
@Deprecated
@@ -280,7 +282,7 @@ public interface URLHandler {
* @param listener The listener that will be notified of the upload progress
* @param timeoutConstraint The connectivity timeout constraints. Can be null, in which case the timeouts
* are implementation specific
- * @throws IOException
+ * @throws IOException if something goes wrong
* @since 2.5
*/
void upload(File src, URL dest, CopyProgressListener listener, TimeoutConstraint timeoutConstraint) throws IOException;
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/MainTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/MainTest.java b/test/java/org/apache/ivy/MainTest.java
index 427f97c..256906b 100644
--- a/test/java/org/apache/ivy/MainTest.java
+++ b/test/java/org/apache/ivy/MainTest.java
@@ -136,10 +136,11 @@ public class MainTest {
}
/**
- * Tests that the {@code types} argument to the command line is parsed correctly when it's passed
- * more than one value for the argument
+ * Test case for IVY-1355.
+ * {@code types} argument to the command line must be parsed correctly when it's passed
+ * more than one value for the argument.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1355">IVY-1355</a>
*/
@Test
@@ -160,7 +161,7 @@ public class MainTest {
/**
* Tests that the {@code overwriteMode} passed for the retrieve command works as expected
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testRetrieveOverwriteMode() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/TestFixture.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/TestFixture.java b/test/java/org/apache/ivy/TestFixture.java
index 7886ead..374711e 100644
--- a/test/java/org/apache/ivy/TestFixture.java
+++ b/test/java/org/apache/ivy/TestFixture.java
@@ -34,25 +34,26 @@ import org.apache.ivy.plugins.resolver.util.ResolvedResource;
/**
* Fixture easing the development of tests requiring to set up a simple repository with some
- * modules, using micro ivy format to describe the repository. <br/>
+ * modules, using micro ivy format to describe the repository.
+ * <p>
* Example of use:
- *
+ * </p>
* <pre>
* public class MyTest {
* private TestFixture fixture;
*
- * @Before
+ * @Before
* public void setUp() throws Exception {
* fixture = new TestFixture();
* // additional setup here
* }
*
- * @After
+ * @After
* public void tearDown() throws Exception {
* fixture.clean();
* }
*
- * @Test
+ * @Test
* public void testXXX() throws Exception {
* fixture.addMD("#A;1-> { #B;[1.5,1.6] #C;2.5 }").addMD("#B;1.5->#D;2.0")
* .addMD("#B;1.6->#D;2.0").addMD("#C;2.5->#D;[1.0,1.6]").addMD("#D;1.5")
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/TestHelper.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/TestHelper.java b/test/java/org/apache/ivy/TestHelper.java
index e206d3b..57303bd 100644
--- a/test/java/org/apache/ivy/TestHelper.java
+++ b/test/java/org/apache/ivy/TestHelper.java
@@ -43,7 +43,6 @@ import org.apache.tools.ant.DefaultLogger;
import org.apache.tools.ant.Project;
import org.apache.tools.ant.taskdefs.Delete;
-
import java.io.File;
import java.io.IOException;
import java.io.InputStream;
@@ -144,25 +143,19 @@ public class TestHelper {
* Parses a string representation of a module descriptor in micro ivy format.
* <p>
* Examples:
- *
+ * </p>
* <pre>
* #A;1
* </pre>
- *
- * <hr/>
- *
+ * <hr>
* <pre>
- * #A;2-> #B;[1.0,1.5]
+ * #A;2-> #B;[1.0,1.5]
* </pre>
- *
- * <hr/>
- *
+ * <hr>
* <pre>
- * #A;3-> { #B;[1.0,1.5] #C;[2.0,2.5] }
+ * #A;3-> { #B;[1.0,1.5] #C;[2.0,2.5] }
* </pre>
*
- * </p>
- *
* @param microIvy
* the micro ivy description of the module descriptor
* @return the parsed module descriptor
@@ -391,8 +384,8 @@ public class TestHelper {
* @param serverAddress The address to which the server will be bound
* @param webAppContext The context root of the application which will be handling the requests to the server
* @param localFilesystemRepoRoot The path to the root directory containing the module descriptors and artifacts
- * @return
- * @throws IOException
+ * @return AutoCloseable
+ * @throws IOException if something goes wrong
*/
public static AutoCloseable createHttpServerBackedRepository(final InetSocketAddress serverAddress, final String webAppContext,
final Path localFilesystemRepoRoot) throws IOException {
@@ -426,8 +419,8 @@ public class TestHelper {
* @param realm The realm to use for the {@code BASIC} auth mechanism
* @param validCredentials A {@link Map} of valid credentials, the key being the user name and the value being the password,
* that the server will use during the authentication process of the incoming requests
- * @return
- * @throws IOException
+ * @return AutoCloseable
+ * @throws IOException if something goes wrong
*/
public static AutoCloseable createBasicAuthHttpServerBackedRepo(final InetSocketAddress serverAddress, final String webAppContext,
final Path localFilesystemRepoRoot, final String realm,
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/FileUtilTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/FileUtilTest.java b/test/java/org/apache/ivy/ant/FileUtilTest.java
index 902d1a1..4056e86 100644
--- a/test/java/org/apache/ivy/ant/FileUtilTest.java
+++ b/test/java/org/apache/ivy/ant/FileUtilTest.java
@@ -36,7 +36,7 @@ public class FileUtilTest {
/**
* Tests that {@link FileUtil#normalize(String)} works as expected for some basic file paths
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testSimpleNormalize() throws Exception {
@@ -50,7 +50,7 @@ public class FileUtilTest {
* Tests that {@link FileUtil#normalize(String)} works as expected when passed a path that starts with
* {@link File#listRoots() filesystem roots}
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testNormalizeOfFileSystemRootPath() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyAntSettingsBuildFileTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyAntSettingsBuildFileTest.java b/test/java/org/apache/ivy/ant/IvyAntSettingsBuildFileTest.java
index 13d0cd8..22b3257 100644
--- a/test/java/org/apache/ivy/ant/IvyAntSettingsBuildFileTest.java
+++ b/test/java/org/apache/ivy/ant/IvyAntSettingsBuildFileTest.java
@@ -62,9 +62,13 @@ public class IvyAntSettingsBuildFileTest {
buildRule.getProject().getReference("ivy.instance"));
}
+ /**
+ * Test case for IVY-925.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-925">IVY-925</a>
+ */
@Test
public void testSettingsWithIdIvyInstance() {
- // IVY-925
buildRule.executeTarget("testSettingsWithPropertyAsId");
ResolveReport report = buildRule.getProject().getReference("ivy.resolved.report");
assertNotNull(report);
@@ -72,9 +76,13 @@ public class IvyAntSettingsBuildFileTest {
assertEquals(1, report.getDependencies().size());
}
+ /**
+ * Test case for IVY-924.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-924">IVY-924</a>
+ */
@Test
public void testStackOverflow() {
- // IVY-924
buildRule.configureProject("test/java/org/apache/ivy/ant/IvyAntSettingsBuildFileStackOverflow.xml");
buildRule.executeTarget("testStackOverflow");
ResolveReport report = buildRule.getProject().getReference("ivy.resolved.report");
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyArtifactPropertyTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyArtifactPropertyTest.java b/test/java/org/apache/ivy/ant/IvyArtifactPropertyTest.java
index 44365c6..60e8459 100644
--- a/test/java/org/apache/ivy/ant/IvyArtifactPropertyTest.java
+++ b/test/java/org/apache/ivy/ant/IvyArtifactPropertyTest.java
@@ -91,7 +91,7 @@ public class IvyArtifactPropertyTest {
/**
* Test must fail because no resolve was performed
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testWithResolveIdWithoutResolve() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyBuildNumberTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyBuildNumberTest.java b/test/java/org/apache/ivy/ant/IvyBuildNumberTest.java
index 73d4bff..9633326 100644
--- a/test/java/org/apache/ivy/ant/IvyBuildNumberTest.java
+++ b/test/java/org/apache/ivy/ant/IvyBuildNumberTest.java
@@ -180,9 +180,14 @@ public class IvyBuildNumberTest {
assertEquals("1", buildNumber.getProject().getProperty("ivy.new.build.number"));
}
+ /**
+ * Test case for IVY-1037.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1037">IVY-1037</a>
+ */
@Test
public void testChainResolver() throws Exception {
- // IVY-1037
Project project = TestHelper.newProject();
project.setProperty("ivy.settings.file", "test/repositories/IVY-1037/ivysettings.xml");
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyCacheFilesetTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyCacheFilesetTest.java b/test/java/org/apache/ivy/ant/IvyCacheFilesetTest.java
index cc52a66..3d8a489 100644
--- a/test/java/org/apache/ivy/ant/IvyCacheFilesetTest.java
+++ b/test/java/org/apache/ivy/ant/IvyCacheFilesetTest.java
@@ -112,7 +112,7 @@ public class IvyCacheFilesetTest {
/**
* Test must fail with default haltonfailure setting.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testFailure() throws Exception {
@@ -124,7 +124,7 @@ public class IvyCacheFilesetTest {
/**
* Test must fail with default haltonfailure setting.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testInvalidPattern() throws Exception {
@@ -202,10 +202,11 @@ public class IvyCacheFilesetTest {
}
/**
- * Tests that the {@link IvyCacheFileset} fails with an exception if it can't determine a common base directory
- * while dealing with cached artifacts
+ * Test case for IVY-1475.
+ * {@link IvyCacheFileset} must fail with an exception if it cannot determine
+ * a common base directory while dealing with cached artifacts.
*
- * @see <a href="https://issues.apache.org/jira/browse/IVY-1475">IVY-1475</a> for more details
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1475">IVY-1475</a>
*/
@Test
public void getBaseDirNoCommonBaseDir() {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyCachePathTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyCachePathTest.java b/test/java/org/apache/ivy/ant/IvyCachePathTest.java
index 00b1d4c..536022b 100644
--- a/test/java/org/apache/ivy/ant/IvyCachePathTest.java
+++ b/test/java/org/apache/ivy/ant/IvyCachePathTest.java
@@ -135,7 +135,7 @@ public class IvyCachePathTest {
/**
* Test must fail with default haltonfailure setting.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testFailure() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyCleanCacheTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyCleanCacheTest.java b/test/java/org/apache/ivy/ant/IvyCleanCacheTest.java
index e8b6932..87dc6a9 100644
--- a/test/java/org/apache/ivy/ant/IvyCleanCacheTest.java
+++ b/test/java/org/apache/ivy/ant/IvyCleanCacheTest.java
@@ -104,7 +104,7 @@ public class IvyCleanCacheTest {
/**
* clean cache must fail with unknown cache
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testUnknownCache() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyConfigureTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyConfigureTest.java b/test/java/org/apache/ivy/ant/IvyConfigureTest.java
index 11fdd69..49f62a9 100644
--- a/test/java/org/apache/ivy/ant/IvyConfigureTest.java
+++ b/test/java/org/apache/ivy/ant/IvyConfigureTest.java
@@ -222,9 +222,14 @@ public class IvyConfigureTest {
assertEquals("value", configure.getProject().getProperty("ivy.test.variable.this.id"));
}
+ /**
+ * Test case for IVY-601.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-601">IVY-601</a>
+ */
@Test
public void testIncludeTwice() throws Exception {
- // IVY-601
configure.setFile(new File("test/java/org/apache/ivy/ant/ivysettings-include-twice.xml"));
configure.execute();
@@ -270,7 +275,7 @@ public class IvyConfigureTest {
/**
* Calling settings twice with the same id with override=notallowed must fail
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testOverrideNotAllowed() throws Exception {
@@ -295,7 +300,7 @@ public class IvyConfigureTest {
/**
* Settings override with invalid value must fail.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testInvalidOverride() throws Exception {
@@ -310,7 +315,7 @@ public class IvyConfigureTest {
* Tests that if the Ivy settings file <code>include</code>s another file as <code>optional</code>,
* then the absence of that file doesn't lead to failures
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testOptionalFileInclude() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyDeliverTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyDeliverTest.java b/test/java/org/apache/ivy/ant/IvyDeliverTest.java
index 5bcc689..056e9e7 100644
--- a/test/java/org/apache/ivy/ant/IvyDeliverTest.java
+++ b/test/java/org/apache/ivy/ant/IvyDeliverTest.java
@@ -297,9 +297,14 @@ public class IvyDeliverTest {
md.getModuleRevisionId());
}
+ /**
+ * Test case for IVY-404.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-404">IVY-404</a>
+ */
@Test
public void testWithBranch() throws Exception {
- // test case for IVY-404
project.setProperty("ivy.dep.file", "test/java/org/apache/ivy/ant/ivy-latest-branch.xml");
IvyResolve res = new IvyResolve();
res.setProject(project);
@@ -358,9 +363,14 @@ public class IvyDeliverTest {
dds[0].getDynamicConstraintDependencyRevisionId());
}
+ /**
+ * Test case for IVY-415.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-415">IVY-415</a>
+ */
@Test
public void testWithExtraAttributes() throws Exception {
- // test case for IVY-415
project.setProperty("ivy.dep.file", "test/java/org/apache/ivy/ant/ivy-latest-extra.xml");
IvyResolve res = new IvyResolve();
res.setValidate(false);
@@ -432,10 +442,15 @@ public class IvyDeliverTest {
expectedFileSet, actualFileSet);
}
+ /**
+ * Test case for IVY-707.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-707">IVY-707</a>
+ */
@Test
public void testWithDynEvicted2() throws Exception {
// same as previous but dynamic dependency is placed after the one causing the conflict
- // test case for IVY-707
project.setProperty("ivy.dep.file", "test/java/org/apache/ivy/ant/ivy-dyn-evicted2.xml");
IvyResolve res = new IvyResolve();
res.setValidate(false);
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyDependencyTreeTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyDependencyTreeTest.java b/test/java/org/apache/ivy/ant/IvyDependencyTreeTest.java
index bf41b27..633f0a3 100644
--- a/test/java/org/apache/ivy/ant/IvyDependencyTreeTest.java
+++ b/test/java/org/apache/ivy/ant/IvyDependencyTreeTest.java
@@ -87,7 +87,7 @@ public class IvyDependencyTreeTest extends AntTaskTestCase {
/**
* Task must fail because no resolve was performed.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testWithResolveIdWithoutResolve() throws Exception {
@@ -118,10 +118,11 @@ public class IvyDependencyTreeTest extends AntTaskTestCase {
}
/**
- * Tests that dependency tree task doesn't run into an infinite loop due to circular dependencies
+ * Test case for IVY-1540.
+ * Dependency tree task must not run into an infinite loop due to circular dependencies.
*
- * @throws Exception
- * @see <a href="https://issues.apache.org/jira/browse/IVY-1540">IVY-1540</a> for more details
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1540">IVY-1540</a>
*/
@Test
public void testCircularDep() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyDependencyUpdateCheckerTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyDependencyUpdateCheckerTest.java b/test/java/org/apache/ivy/ant/IvyDependencyUpdateCheckerTest.java
index ed8c4ab..9501a1b 100644
--- a/test/java/org/apache/ivy/ant/IvyDependencyUpdateCheckerTest.java
+++ b/test/java/org/apache/ivy/ant/IvyDependencyUpdateCheckerTest.java
@@ -120,7 +120,7 @@ public class IvyDependencyUpdateCheckerTest extends AntTaskTestCase {
/**
* Test must fail with default haltonfailure setting.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testInlineForNonExistingModule() throws Exception {
@@ -135,7 +135,7 @@ public class IvyDependencyUpdateCheckerTest extends AntTaskTestCase {
/**
* Test must fail with default haltonfailure setting.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testFailure() throws Exception {
@@ -147,7 +147,7 @@ public class IvyDependencyUpdateCheckerTest extends AntTaskTestCase {
/**
* Test must fail because of missing configurations.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testFailureWithMissingConfigurations() throws Exception {
@@ -163,7 +163,7 @@ public class IvyDependencyUpdateCheckerTest extends AntTaskTestCase {
/**
* Test must fail with default haltonfailure setting.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testFailureOnBadDependencyIvyFile() throws Exception {
@@ -175,7 +175,7 @@ public class IvyDependencyUpdateCheckerTest extends AntTaskTestCase {
/**
* Test must fail with default haltonfailure setting.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testFailureOnBadStatusInDependencyIvyFile() throws Exception {
@@ -215,9 +215,13 @@ public class IvyDependencyUpdateCheckerTest extends AntTaskTestCase {
assertFalse(project.getProperty("ivy.resolved.configurations").contains("default"));
}
+ /**
+ * Test case for IVY-396.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-396">IVY-396</a>
+ */
@Test
public void testResolveWithAbsoluteFile() {
- // IVY-396
File ivyFile = new File("test/java/org/apache/ivy/ant/ivy-simple.xml");
dependencyUpdateChecker.getProject().setProperty("ivy.dep.file", ivyFile.getAbsolutePath());
dependencyUpdateChecker.execute();
@@ -226,9 +230,13 @@ public class IvyDependencyUpdateCheckerTest extends AntTaskTestCase {
// ModuleRevisionId.newInstance("apache", "resolve-simple", "1.0")).exists());
}
+ /**
+ * Test case for IVY-396.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-396">IVY-396</a>
+ */
@Test
public void testResolveWithRelativeFile() {
- // IVY-396
dependencyUpdateChecker.getProject().setProperty("ivy.dep.file",
"test/java/org/apache/ivy/ant/ivy-simple.xml");
dependencyUpdateChecker.execute();
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyInfoTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyInfoTest.java b/test/java/org/apache/ivy/ant/IvyInfoTest.java
index 3930a9d..5d52a22 100644
--- a/test/java/org/apache/ivy/ant/IvyInfoTest.java
+++ b/test/java/org/apache/ivy/ant/IvyInfoTest.java
@@ -75,6 +75,12 @@ public class IvyInfoTest {
assertNull(info.getProject().getProperty("ivy.configuration.private.desc"));
}
+ /**
+ * Test case for IVY-726.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-726">IVY-726</a>
+ */
@Test
public void testIVY726() throws Exception {
info.setFile(new File("test/java/org/apache/ivy/ant/ivy-info-all.xml"));
@@ -83,6 +89,12 @@ public class IvyInfoTest {
assertTrue(info.getProject().getProperty("ivy.extra.branch") == null);
}
+ /**
+ * Test case for IVY-395.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-395">IVY-395</a>
+ */
@Test
public void testIVY395() throws Exception {
info.setFile(new File("test/java/org/apache/ivy/ant/ivy-artifact-info.xml"));
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyInstallTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyInstallTest.java b/test/java/org/apache/ivy/ant/IvyInstallTest.java
index 1f24f7a..69c7873 100644
--- a/test/java/org/apache/ivy/ant/IvyInstallTest.java
+++ b/test/java/org/apache/ivy/ant/IvyInstallTest.java
@@ -127,7 +127,7 @@ public class IvyInstallTest {
}
/**
- * Normal case; no confs set (should use the default->* configuration).
+ * Normal case; no confs set (should use the default->* configuration).
*/
@Test
public void testInstallWithConfsDefaultSettings() {
@@ -147,7 +147,10 @@ public class IvyInstallTest {
}
/**
- * Test retrieving artifacts under only the master and runtime configuration.
+ * Test case for IVY-1313.
+ * Retrieve artifacts only for the master and runtime configuration.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1313">IVY-1313</a>
*/
@Test
public void testInstallWithConfsRuntimeOnly() {
@@ -167,9 +170,14 @@ public class IvyInstallTest {
assertFalse(new File("build/test/install/org1/mod3/jars/mod3-1.0.jar").exists());
}
+ /**
+ * Test case for IVY-1324.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1324">IVY-1324</a>
+ */
@Test
public void testInstallWithClassifiers() throws Exception {
- // IVY-1324
project.setProperty("ivy.settings.url", new File("test/repositories/m2/ivysettings.xml")
.toURI().toURL().toExternalForm());
install.setOrganisation("org.apache");
@@ -236,6 +244,11 @@ public class IvyInstallTest {
assertTrue(new File("build/test/install/org.apache/test/test-1.0.pom").exists());
}
+ /**
+ * Test case for IVY-843.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-843">IVY-843</a>
+ */
@Test
public void testIVY843() {
project.setProperty("ivy.settings.file", "test/repositories/ivysettings-IVY843.xml");
@@ -288,9 +301,9 @@ public class IvyInstallTest {
}
/**
- * Installing a module with namespace coordinates instead of system one should fail.
+ * Installing a module with namespace coordinates instead of system one must fail.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testInstallWithNamespace2() throws Exception {
@@ -323,7 +336,7 @@ public class IvyInstallTest {
/**
* Fail on unknown dependency when haltonfailure=true.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testDependencyNotFoundFailure() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyPublishTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyPublishTest.java b/test/java/org/apache/ivy/ant/IvyPublishTest.java
index e20c0ad..4109c2c 100644
--- a/test/java/org/apache/ivy/ant/IvyPublishTest.java
+++ b/test/java/org/apache/ivy/ant/IvyPublishTest.java
@@ -160,12 +160,17 @@ public class IvyPublishTest {
}
}
+ /**
+ * Test case for IVY-1248.
+ *
+ * @throws IOException if something goes wrong
+ * @throws ParseException if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1248">IVY-1248</a>
+ */
@Test
public void testMergeParentWithoutPublishingParent() throws IOException, ParseException {
// here we directly publish a module extending ivy-multiconf.xml,
// the module parent is not published not yet in cache
- // See : IVY-1248
-
// update=true implies merge=true
// project.setProperty("ivy.dep.file",
// "test/java/org/apache/ivy/ant/ivy-extends-multiconf.xml");
@@ -185,13 +190,18 @@ public class IvyPublishTest {
checkPublishedFile(published, "ivy-extends-merged.xml");
}
+ /**
+ * Test case for IVY-1248.
+ * Here we directly publish a module extending ivy-multiconf.xml,
+ * the module parent is not published not yet in cache.
+ *
+ * @throws IOException if something goes wrong
+ * @throws ParseException if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1248">IVY-1248</a>
+ */
@Test
public void testMergeParentWithoutPublishingParentForceDeliver()
throws IOException, ParseException {
- // here we directly publish a module extending ivy-multiconf.xml,
- // the module parent is not published not yet in cache
- // See : IVY-1248
-
IvyResolve resolve = new IvyResolve();
resolve.setProject(project);
resolve.setFile(new File("test/java/org/apache/ivy/ant/ivy-extends-multiconf.xml"));
@@ -218,8 +228,6 @@ public class IvyPublishTest {
@Test
public void testMergeParentWithoutLocationAttribute() throws IOException, ParseException {
- // See : IVY-XXX
-
IvyResolve resolve = new IvyResolve();
resolve.setProject(project);
resolve.setFile(new File("test/java/org/apache/ivy/ant/extends/child1/ivy-child1.xml"));
@@ -711,7 +719,7 @@ public class IvyPublishTest {
/**
* Test must not publish ivy file with bad revision.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testBadNoDeliver() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyResolveTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyResolveTest.java b/test/java/org/apache/ivy/ant/IvyResolveTest.java
index ee220e9..3e2ebb2 100644
--- a/test/java/org/apache/ivy/ant/IvyResolveTest.java
+++ b/test/java/org/apache/ivy/ant/IvyResolveTest.java
@@ -62,6 +62,12 @@ public class IvyResolveTest {
TestHelper.cleanCache();
}
+ /**
+ * Test case for IVY-1455.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1455">IVY-1455</a>
+ */
@Test
public void testIVY1455() throws Exception {
project.setProperty("ivy.settings.file", "test/repositories/IVY-1455/ivysettings.xml");
@@ -87,6 +93,12 @@ public class IvyResolveTest {
parallel.execute();
}
+ /**
+ * Test case for IVY-779.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-779">IVY-779</a>
+ */
@Test
public void testIVY779() throws Exception {
Project project = TestHelper.newProject();
@@ -122,9 +134,14 @@ public class IvyResolveTest {
assertTrue(getArchiveFileInCache("org1", "mod1.2", "2.0", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-630.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-630">IVY-630</a>
+ */
@Test
public void testResolveWithoutIvyFile() throws Exception {
- // IVY-630
resolve.getProject().setProperty("ivy.settings.file",
"test/repositories/IVY-630/ivysettings.xml");
@@ -444,9 +461,13 @@ public class IvyResolveTest {
assertFalse(project.getProperty("ivy.resolved.configurations").contains("default"));
}
+ /**
+ * Test case for IVY-396.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-396">IVY-396</a>
+ */
@Test
public void testResolveWithAbsoluteFile() {
- // IVY-396
File ivyFile = new File("test/java/org/apache/ivy/ant/ivy-simple.xml");
resolve.getProject().setProperty("ivy.dep.file", ivyFile.getAbsolutePath());
resolve.execute();
@@ -455,9 +476,13 @@ public class IvyResolveTest {
ModuleRevisionId.newInstance("apache", "resolve-simple", "1.0")).exists());
}
+ /**
+ * Test case for IVY-396.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-396">IVY-396</a>
+ */
@Test
public void testResolveWithRelativeFile() {
- // IVY-396
resolve.getProject().setProperty("ivy.dep.file",
"test/java/org/apache/ivy/ant/ivy-simple.xml");
resolve.execute();
@@ -615,7 +640,7 @@ public class IvyResolveTest {
/**
* Test a failing resolve.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testChildsFail() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/ant/IvyRetrieveTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/ant/IvyRetrieveTest.java b/test/java/org/apache/ivy/ant/IvyRetrieveTest.java
index 422513d..692360a 100644
--- a/test/java/org/apache/ivy/ant/IvyRetrieveTest.java
+++ b/test/java/org/apache/ivy/ant/IvyRetrieveTest.java
@@ -89,9 +89,14 @@ public class IvyRetrieveTest {
"mod3.2", "jar", "jar", "private")).exists());
}
+ /**
+ * Test case for IVY-992.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-992">IVY-992</a>
+ */
@Test
public void testValidateInIvySettings() throws Exception {
- // cfr IVY-992
project.setProperty("ivy.dep.file", "test/java/org/apache/ivy/ant/ivy-latest-extra.xml");
retrieve.getSettings().setValidate(false);
retrieve.execute();
@@ -222,9 +227,14 @@ public class IvyRetrieveTest {
"mod1.2", "jar", "jar")).exists());
}
+ /**
+ * Test case for IVY-304.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-304">IVY-304</a>
+ */
@Test
public void testUseOrigin() throws Exception {
- // test case for IVY-304
// first we do a resolve with useOrigin=true in another project
Project project = TestHelper.newProject();
project.init();
@@ -268,6 +278,12 @@ public class IvyRetrieveTest {
"ivy", "ivy", "xml")).exists());
}
+ /**
+ * Test case for IVY-631.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-631">IVY-631</a>
+ */
@Test
public void testRetrieveWithOriginalNamePattern() throws Exception {
retrieve.setFile(new File("test/java/org/apache/ivy/ant/ivy-631.xml"));
@@ -282,7 +298,7 @@ public class IvyRetrieveTest {
/**
* Retrieve without previous resolve must fail.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testFailureWithoutAPreviousResolve() throws Exception {
@@ -295,7 +311,7 @@ public class IvyRetrieveTest {
/**
* Test must fail with default haltonfailure setting.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = BuildException.class)
public void testFailure() throws Exception {
@@ -392,9 +408,13 @@ public class IvyRetrieveTest {
assertFalse(new File("build/test/lib/default/unknown").exists());
}
+ /**
+ * Test case for IVY-315.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-315">IVY-315</a>
+ */
@Test
public void testDoubleRetrieveWithDifferentConfigurations() {
- // IVY-315
project.setProperty("ivy.dep.file", "test/java/org/apache/ivy/ant/ivy-doubleretrieve.xml");
retrieve.setConf("compile");
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/core/deliver/DeliverTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/core/deliver/DeliverTest.java b/test/java/org/apache/ivy/core/deliver/DeliverTest.java
index 70dcd58..2b52e1f 100644
--- a/test/java/org/apache/ivy/core/deliver/DeliverTest.java
+++ b/test/java/org/apache/ivy/core/deliver/DeliverTest.java
@@ -70,6 +70,12 @@ public class DeliverTest {
cache.mkdirs();
}
+ /**
+ * Test case for IVY-1111.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1111">IVY-1111</a>
+ */
@Test
public void testIVY1111() throws Exception {
Project project = ivyDeliver.getProject();
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/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 04c06e4..9bad4bd 100644
--- a/test/java/org/apache/ivy/core/module/descriptor/IvyMakePomTest.java
+++ b/test/java/org/apache/ivy/core/module/descriptor/IvyMakePomTest.java
@@ -55,11 +55,13 @@ public class IvyMakePomTest {
}
/**
- * Tests that a Ivy file containing a <code>classifier</code> extra-attribute in its dependency, when converted to a
- * POM file through {@link IvyMakePom}, retains the <code>classifier</code> in the generated POM
+ * Test case for IVY-1528.
+ * An Ivy file containing a <code>classifier</code> extra attribute in its dependency,
+ * must retain the <code>classifier</code> in the generated POM when converted to
+ * a POM file through {@link IvyMakePom}.
*
- * @throws Exception
- * @see <a href="https://issues.apache.org/jira/browse/IVY-1528">IVY-1528</a> for more details
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1528">IVY-1528</a>
*/
@Test
public void testClassifier() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/core/publish/PublishEventsTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/core/publish/PublishEventsTest.java b/test/java/org/apache/ivy/core/publish/PublishEventsTest.java
index cdc5d91..1f014b1 100644
--- a/test/java/org/apache/ivy/core/publish/PublishEventsTest.java
+++ b/test/java/org/apache/ivy/core/publish/PublishEventsTest.java
@@ -178,6 +178,8 @@ public class PublishEventsTest {
/**
* Test a simple artifact publish, without errors or overwrite settings.
+ *
+ * @throws IOException if something goes wrong
*/
@Test
public void testPublishNoOverwrite() throws IOException {
@@ -197,6 +199,8 @@ public class PublishEventsTest {
/**
* Test a simple artifact publish, with overwrite set to true.
+ *
+ * @throws IOException if something goes wrong
*/
@Test
public void testPublishWithOverwrite() throws IOException {
@@ -220,6 +224,8 @@ public class PublishEventsTest {
/**
* Test an attempted publish with an invalid data file path.
+ *
+ * @throws IOException if something goes wrong
*/
@Test
public void testPublishMissingFile() throws IOException {
[2/3] ant-ivy git commit: add a target to generate javadoc for test
cases; fix javadoc for doclint
Posted by gi...@apache.org.
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/core/resolve/ResolveTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/core/resolve/ResolveTest.java b/test/java/org/apache/ivy/core/resolve/ResolveTest.java
index 194d308..0ad84b1 100644
--- a/test/java/org/apache/ivy/core/resolve/ResolveTest.java
+++ b/test/java/org/apache/ivy/core/resolve/ResolveTest.java
@@ -350,9 +350,14 @@ public class ResolveTest {
assertTrue(report.hasError());
}
+ /**
+ * Test case for IVY-258.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-258">IVY-258</a>
+ */
@Test
public void testResolveNoRevisionNowhere() throws Exception {
- // test case for IVY-258
// module1 depends on latest version of module2, which contains no revision in its ivy file,
// nor in the pattern
Ivy ivy = new Ivy();
@@ -367,9 +372,14 @@ public class ResolveTest {
assertFalse(report.hasError());
}
+ /**
+ * Test case for IVY-448.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-448">IVY-448</a>
+ */
@Test
public void testResolveWithConflictManagerPerModule() throws Exception {
- // test case for IVY-448
// all modules from myorg
// module1
// -> module2-1.0
@@ -391,9 +401,14 @@ public class ResolveTest {
.exists());
}
+ /**
+ * Test case for IVY-465.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-465">IVY-465</a>
+ */
@Test
public void testResolveWithConflictManagerDefinedInModule() throws Exception {
- // test case for IVY-465
// #M1;1.0 -> #M2;1.0
// #M2;1.0 -> {org1#mod1.2;1.1 org1#mod1.2;2.0} with
// <conflict org="org1" module="mod1.2" rev="1.1,2.0" />
@@ -410,9 +425,14 @@ public class ResolveTest {
assertEquals("2.0", adrs[1].getArtifact().getId().getRevision());
}
+ /**
+ * Test case for IVY-1404.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1404">IVY-1404</a>
+ * @throws Exception if something goes wrong
+ */
@Test
public void testResolveWithConflictManagerDefinedAtHigherLevel() throws Exception {
- // test case for IVY-1404
// #M1;1.0 -> #M2;1.0
// #M1;2.0 -> {#M2;1.0 #M3;1.0}
// #M2;1.0 -> org1#mod1.2;1.1
@@ -457,9 +477,14 @@ public class ResolveTest {
.getArtifactsNumber());
}
+ /**
+ * Test case for IVY-198.
+ *
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-198">IVY-198</a>
+ * @throws Exception if something goes wrong
+ */
@Test
public void testResolveWithSlashes() throws Exception {
- // test case for IVY-198
// module depends on mod1.2
ResolveReport report = ivy.resolve(ResolveTest.class.getResource("ivy-198.xml"),
getResolveOptions(new String[] {"*"}));
@@ -1153,10 +1178,14 @@ public class ResolveTest {
assertTrue(getArchiveFileInCache("org1", "mod1.2", "2.0", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-240.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-240">IVY-240</a>
+ */
@Test
public void testResolveMultipleExtendsAndConfs() throws Exception {
- // Test case for IVY-240
- //
// mod6.3 1.1 has four confs libraries, run (extends libraries), compile (extends run) and
// test (extends libraries)
// mod6.3 depends on mod6.2 2.0 in conf (run->default)
@@ -1189,10 +1218,14 @@ public class ResolveTest {
assertContainsArtifact("org1", "mod1.2", "2.2", "mod1.2", "jar", "jar", crr);
}
+ /**
+ * Test case for IVY-188.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-188">IVY-188</a>
+ */
@Test
public void testResolveMultipleConfsWithLatest() throws Exception {
- // Test case for IVY-188
- //
// mod6.2 has two confs compile and run
// depends on mod6.1 in conf (compile->default)
// depends on mod1.2 latest (which is 2.2) in conf (run->default)
@@ -1216,10 +1249,14 @@ public class ResolveTest {
assertTrue(getArchiveFileInCache("org1", "mod1.2", "2.2", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-173.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-173">IVY-173</a>
+ */
@Test
public void testResolveMultipleConfsWithConflicts() throws Exception {
- // Test case for IVY-173
- //
// mod6.2 has two confs compile and run
// depends on mod1.2 2.1 in conf (compile->default)
// depends on mod1.1 1.0 in conf (*->default)
@@ -1305,9 +1342,14 @@ public class ResolveTest {
assertTrue(getArchiveFileInCache("org1", "mod1.2", "2.1", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-261.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-261">IVY-261</a>
+ */
@Test
public void testResolveSeveralDefaultWithArtifacts() throws Exception {
- // test case for IVY-261
// mod1.6 depends on
// mod1.4, which depends on mod1.3 and selects one of its artifacts
// mod1.3 and selects two of its artifacts
@@ -1323,9 +1365,14 @@ public class ResolveTest {
.exists());
}
+ /**
+ * Test case for IVY-537.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-537">IVY-537</a>
+ */
@Test
public void testResolveConflictsWithArtifacts() throws Exception {
- // test case for IVY-537
// #mod2.6;0.12 -> {#mod1.6;1.0.4 #mod2.5;0.6.2 }
// #mod1.6;1.0.4 -> #mod1.3;3.0 artifacts A and B
// #mod2.5;0.6.2 -> #mod1.3;3.1 artifact C
@@ -1343,9 +1390,14 @@ public class ResolveTest {
.exists());
}
+ /**
+ * Test case for IVY-283.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-283">IVY-283</a>
+ */
@Test
public void testResolveSeveralDefaultWithArtifactsAndConfs() throws Exception {
- // test case for IVY-283
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/IVY-283/ivysettings.xml"));
ResolveReport report = ivy.resolve(new File("test/repositories/IVY-283/ivy.xml"),
@@ -1366,9 +1418,14 @@ public class ResolveTest {
.exists());
}
+ /**
+ * Second test case for IVY-283.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-283">IVY-283</a>
+ */
@Test
public void testResolveSeveralDefaultWithArtifactsAndConfs2() throws Exception {
- // second test case for IVY-283
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/IVY-283/ivysettings.xml"));
ResolveReport report = ivy.resolve(new File("test/repositories/IVY-283/ivy-d.xml"),
@@ -1583,9 +1640,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org2", "mod2.1", "0.3", "mod2.1", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-541.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-541">IVY-541</a>
+ */
@Test
public void testResolveWithIncludeArtifactsTransitive() throws Exception {
- // test case for IVY-541
// mod2.6 depends on mod2.3 and mod2.1
// mod2.3 depends on mod2.1 and selects its artifacts
ResolveReport report = ivy.resolve(new File(
@@ -1753,7 +1815,10 @@ public class ResolveTest {
}
/**
- * Testcase for IVY-1131.
+ * Test case for IVY-1131.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1131">IVY-1131</a>
*/
@Test
public void testResolveTransitiveDependenciesWithOverrideAndDynamicResolveMode()
@@ -2138,9 +2203,14 @@ public class ResolveTest {
assertTrue(getArchiveFileInCache("org1", "mod1.2", "2.1", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-264.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-264">IVY-264</a>
+ */
@Test
public void testResolveConflict3() throws Exception {
- // test case for IVY-264
// a depends on x latest, y latest, z latest
// x and z depends on commons-lang 1.0.1
// y depends on commons-lang 2.0
@@ -2190,7 +2260,10 @@ public class ResolveTest {
}
/**
- * Test IVY-618.
+ * Test case for IVY-618.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-618">IVY-618</a>
*/
@Test
public void testResolveConflictFromPoms() throws Exception {
@@ -2230,9 +2303,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org1", "mod1.2", "2.0", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-199.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-199">IVY-199</a>
+ */
@Test
public void testTransitiveEviction2() throws Exception {
- // IVY-199
// mod4.1 v 4.13 depends on
// - mod3.2 v 1.2.1 which depends on
// - mod3.1 v 1.0 which depends on mod1.2 v 2.0
@@ -2249,9 +2327,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org1", "mod1.2", "2.0", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-590.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-590">IVY-590</a>
+ */
@Test
public void testTransitiveEvictionWithExtendingConf() throws Exception {
- // IVY-590
ResolveReport report = ivy.resolve(ResolveTest.class.getResource("ivy-590.xml"),
getResolveOptions(new String[] {"*"}));
assertNotNull(report);
@@ -2385,11 +2468,16 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org5", "mod5.1", "4.0", "art51B", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-681.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-681">IVY-681</a>
+ */
@Test
public void testEvictWithConf3() throws Exception {
// same as first one but the conf requested in evicted module is no longer present in
// selected revision
- // test case for IVY-681
// mod6.1 r1.4 depends on
// mod5.1 r4.3 conf A
@@ -2413,10 +2501,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org5", "mod5.1", "4.0", "art51B", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-861.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-861">IVY-861</a>
+ */
@Test
public void testFailWithMissingConf() throws Exception {
- // test case for IVY-861
-
// mod6.1 r1.5 depends on
// mod5.1 [1.0,4.3] conf unknown which doesn't exist in mod5.1;4.3
ResolveReport report = ivy.resolve(new File("test/repositories/2/mod6.1/ivy-1.5.xml"),
@@ -2530,6 +2622,12 @@ public class ResolveTest {
crr.getDownloadReports(ModuleRevisionId.newInstance("org5", "mod5.1", "4.2")).length);
}
+ /**
+ * Test case for IVY-644.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-644">IVY-644</a>
+ */
@Test
public void testMultipleEviction() throws Exception {
@@ -2568,9 +2666,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org1", "mod1.2", "2.1", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-193.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-193">IVY-193</a>
+ */
@Test
public void testResolveForceAfterConflictSolved() throws Exception {
- // IVY-193
// mod4.1 v 4.9 depends on
// - mod3.2 v 1.1 which depends on mod1.2 v 2.0
// - mod3.1 v 1.1 which depends on mod1.2 v 2.1
@@ -2593,9 +2696,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org1", "mod1.2", "2.1", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-193.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-193">IVY-193</a>
+ */
@Test
public void testResolveForceAfterDependencyExist() throws Exception {
- // IVY-193
// mod4.1 v 4.10 depends on
// - mod3.1 v 1.0.1 which depends on mod1.2 v 2.0 and forces it
// - mod3.2 v 1.2 which depends on mod1.2 v 2.1 and on mod3.1 v1.0.1
@@ -2611,9 +2719,14 @@ public class ResolveTest {
assertTrue(getArchiveFileInCache("org1", "mod1.2", "2.1", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-193.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-193">IVY-193</a>
+ */
@Test
public void testResolveForceInDepOnly() throws Exception {
- // IVY-193
// mod4.1 v 4.11 depends on
// - mod1.2 v 2.0
// - mod3.2 v 1.3 which depends on
@@ -2633,9 +2746,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org1", "mod1.2", "1.0", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-193.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-193">IVY-193</a>
+ */
@Test
public void testResolveForceInDepOnly2() throws Exception {
- // IVY-193
// mod4.1 v 4.12 depends on
// - mod3.1 v1.0 which depends on
// - mod1.2 v 2.0
@@ -2770,9 +2888,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org1", "mod1.2", "2.2", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-182.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-182">IVY-182</a>
+ */
@Test
public void testResolveForceWithDynamicRevisionsAndCyclicDependencies() throws Exception {
- // IVY-182
// * has no revision
// * declares conf compile, test extends compile,
// * depends on
@@ -2831,9 +2954,15 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org1", "mod1.2", "2.1", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-130.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-130">IVY-130</a>
+ */
@Test
public void testResolveContradictoryConflictResolution2() throws Exception {
- // BUG IVY-130 : only mod1.2 v2.0 should be resolved and not v2.1 (because of force)
+ // Only mod1.2 v2.0 should be resolved and not v2.1 (because of force)
// mod10.1 v 1.1 depends on
// - mod1.2 v 2.0 and forces it
// - mod4.1 v 4.3
@@ -3060,12 +3189,16 @@ public class ResolveTest {
.exists());
}
+ /**
+ * Test case for IVY-602.
+ * Verify that latest.integration dependencies can be resolved when using a resolver with
+ * multiple patterns, when only the first pattern finds something
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-602">IVY-602</a>
+ */
@Test
public void testLatestWithMultiplePatterns() throws Exception {
- // The test verify that latest.integration dependencies can be resolved
- // when using a resolver with multiple patterns, when only the first pattern
- // finds something - test case for IVY-602
-
// mod9.2 depends on latest.integration of mod6.2
Ivy ivy = Ivy.newInstance();
ivy.configure(new File("test/repositories/ivysettings-IVY602.xml"));
@@ -3220,10 +3353,15 @@ public class ResolveTest {
assertTrue(getIvyFileInCache(depId).exists());
}
+ /**
+ * Test case for IVY-318.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-318">IVY-318</a>
+ */
@Test
public void testLatestMilestone2() throws Exception {
// mod9.2 depends on latest.milestone of mod6.2, but there is no milestone
- // test case for IVY-318
ResolveReport report = ivy.resolve(new File(
"test/repositories/1/org9/mod9.2/ivys/ivy-1.2.xml"),
getResolveOptions(new String[] {"default"}));
@@ -3237,6 +3375,12 @@ public class ResolveTest {
assertEquals(0, crr.getArtifactsNumber());
}
+ /**
+ * Test case for IVY-56.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-56">IVY-56</a>
+ */
@Test
public void testIVY56() throws Exception {
Ivy ivy = new Ivy();
@@ -3251,6 +3395,12 @@ public class ResolveTest {
}
}
+ /**
+ * Test case for IVY-214.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-214">IVY-214</a>
+ */
@Test
public void testIVY214() throws Exception {
ResolveReport report = ivy.resolve(ResolveTest.class.getResource("ivy-214.xml"),
@@ -3263,6 +3413,12 @@ public class ResolveTest {
.getArtifactsNumber());
}
+ /**
+ * Test case for IVY-218.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-218">IVY-218</a>
+ */
@Test
public void testIVY218() throws Exception {
ResolveReport report = ivy.resolve(ResolveTest.class.getResource("ivy-218.xml"),
@@ -3275,6 +3431,12 @@ public class ResolveTest {
.getArtifactsNumber());
}
+ /**
+ * Test case for IVY-729.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-729">IVY-729</a>
+ */
@Test
public void testIVY729() throws Exception {
Ivy ivy = new Ivy();
@@ -3289,7 +3451,7 @@ public class ResolveTest {
* Circular dependency: mod6.3 depends on mod6.2, which itself depends on mod6.3;
* circular dependency strategy set to error.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testCircular() throws Exception {
@@ -3322,7 +3484,7 @@ public class ResolveTest {
* Circular dependency: mod 9.1 (no revision) depends on mod9.2, which depends on mod9.1 2.+;
* circular dependency strategy set to error.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testCircular2() throws Exception {
@@ -3344,7 +3506,8 @@ public class ResolveTest {
* Circular dependency: mod6.3 depends on mod6.2, which itself depends on mod6.3,
* in both configuration default and test; circular dependency strategy set to error.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-400">IVY-400</a>
*/
@Test
public void testCircular3() throws Exception {
@@ -3465,6 +3628,12 @@ public class ResolveTest {
.exists());
}
+ /**
+ * Test case for IVY-1178.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1178">IVY-1178</a>
+ */
@Test
public void testIVY1178() throws Exception {
Ivy ivy = new Ivy();
@@ -3492,6 +3661,12 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("myorg", "modE", "1.1", "modE", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-1236.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1236">IVY-1236</a>
+ */
@Test
public void testIVY1236() throws Exception {
Ivy ivy = new Ivy();
@@ -3510,6 +3685,12 @@ public class ResolveTest {
assertTrue(getArchiveFileInCache("myorg", "modB", "1.0", "modB-A", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-1233.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1233">IVY-1233</a>
+ */
@Test
public void testIVY1233() throws Exception {
Ivy ivy = new Ivy();
@@ -3526,6 +3707,12 @@ public class ResolveTest {
assertTrue(modRevIds.contains(ModuleRevisionId.newInstance("test", "c", "3.0")));
}
+ /**
+ * Test case for IVY-1333.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1333">IVY-1333</a>
+ */
@Test
public void testIVY1333() throws Exception {
Ivy ivy = new Ivy();
@@ -3546,6 +3733,12 @@ public class ResolveTest {
"1.0.0.m4", extra)));
}
+ /**
+ * Test case for IVY-1347.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1347">IVY-1347</a>
+ */
@Test
public void testIVY1347() throws Exception {
Ivy ivy = new Ivy();
@@ -3566,6 +3759,12 @@ public class ResolveTest {
assertEquals(ModuleRevisionId.newInstance("foo", "parent", "1.0"), parent);
}
+ /**
+ * Test case for IVY-999.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-999">IVY-999</a>
+ */
@Test
public void testIVY999() throws Exception {
Ivy ivy = new Ivy();
@@ -3581,6 +3780,12 @@ public class ResolveTest {
assertFalse(modRevIds.contains(ModuleRevisionId.newInstance("junit", "junit", "3.8")));
}
+ /**
+ * Test case for IVY-1366.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1366">IVY-1366</a>
+ */
@Test
public void testIVY1366() throws Exception {
Ivy ivy = new Ivy();
@@ -3654,9 +3859,14 @@ public class ResolveTest {
assertFalse(getArchiveFileInCache("org1", "mod1.2", "2.0", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-105.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-105">IVY-105</a>
+ */
@Test
public void testTransitiveSetting2() throws Exception {
- // test case for IVY-105
// mod2.7 depends on mod1.1 and mod2.4
// mod2.4 depends on mod1.1 with transitive set to false
// mod1.1 depends on mod1.2
@@ -3747,6 +3957,12 @@ public class ResolveTest {
// about configuration mapping and eviction
// /////////////////////////////////////////////////////////
+ /**
+ * Test case for IVY-84.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-84">IVY-84</a>
+ */
@Test
public void testConfigurationMapping1() throws Exception {
Ivy ivy = new Ivy();
@@ -3764,6 +3980,12 @@ public class ResolveTest {
assertDoesntContainArtifact("test", "c", "1.0.2", "c-bt", "txt", "txt", conf);
}
+ /**
+ * Test case for IVY-84.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-84">IVY-84</a>
+ */
@Test
public void testConfigurationMapping2() throws Exception {
Ivy ivy = new Ivy();
@@ -3781,6 +4003,12 @@ public class ResolveTest {
assertDoesntContainArtifact("test", "c", "1.0.1", "c-bt", "txt", "txt", conf);
}
+ /**
+ * Test case for IVY-84.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-84">IVY-84</a>
+ */
@Test
public void testConfigurationMapping3() throws Exception {
Ivy ivy = new Ivy();
@@ -3798,6 +4026,12 @@ public class ResolveTest {
assertDoesntContainArtifact("test", "c", "1.0.1", "c", "txt", "txt", conf);
}
+ /**
+ * Test case for IVY-84.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-84">IVY-84</a>
+ */
@Test
public void testConfigurationMapping4() throws Exception {
Ivy ivy = new Ivy();
@@ -3815,6 +4049,12 @@ public class ResolveTest {
assertDoesntContainArtifact("test", "c", "1.0.1", "c-bt", "txt", "txt", conf);
}
+ /**
+ * Test case for IVY-84.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-84">IVY-84</a>
+ */
@Test
public void testConfigurationMapping5() throws Exception {
Ivy ivy = new Ivy();
@@ -3832,6 +4072,12 @@ public class ResolveTest {
assertDoesntContainArtifact("test", "c", "1.0.1", "c-bt", "txt", "txt", conf);
}
+ /**
+ * Test case for IVY-84.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-84">IVY-84</a>
+ */
@Test
public void testConfigurationMapping6() throws Exception {
Ivy ivy = new Ivy();
@@ -3849,6 +4095,12 @@ public class ResolveTest {
assertDoesntContainArtifact("test", "c", "1.0.1", "c-bt", "txt", "txt", conf);
}
+ /**
+ * Test case for IVY-84.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-84">IVY-84</a>
+ */
@Test
public void testConfigurationMapping7() throws Exception {
Ivy ivy = new Ivy();
@@ -3866,6 +4118,12 @@ public class ResolveTest {
assertDoesntContainArtifact("test", "c", "1.0.1", "c-bt", "txt", "txt", conf);
}
+ /**
+ * Test case for IVY-97.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-97">IVY-97</a>
+ */
@Test
public void testIVY97() throws Exception {
// mod9.2 depends on mod9.1 and mod1.2
@@ -4454,9 +4712,14 @@ public class ResolveTest {
}
}
+ /**
+ * Test case for IVY-418.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-418">IVY-418</a>
+ */
@Test
public void testResolveMaven2Classifiers() throws Exception {
- // test case for IVY-418
// test-classifier depends on test-classified with classifier asl
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/m2/ivysettings.xml"));
@@ -4482,9 +4745,14 @@ public class ResolveTest {
"1.0", "test-classified", "jar", "jar", cmap).exists());
}
+ /**
+ * Test case for IVY-1041.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1041">IVY-1041</a>
+ */
@Test
public void testResolveMaven2ClassifiersWithoutPOM() throws Exception {
- // test case for IVY-1041
// test-classifier depends on test-classified with classifier asl
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/m2/ivysettings.xml"));
@@ -4532,9 +4800,14 @@ public class ResolveTest {
assertTrue(jarFileInCache.length() != sourceFileInCache.length());
}
+ /**
+ * Test case for IVY-1138.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1138">IVY-1138</a>
+ */
@Test
public void testResolveMaven2GetSourcesWithSrcClassifier() throws Exception {
- // IVY-1138
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/m2/ivysettings.xml"));
ResolveReport report = ivy.resolve(ResolveTest.class.getResource("ivy-m2-with-src.xml"),
@@ -4663,9 +4936,14 @@ public class ResolveTest {
"jar").exists());
}
+ /**
+ * Test case for IVY-1186.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1186">IVY-1186</a>
+ */
@Test
public void testResolveMaven2ParentPomWithNamespace() throws Exception {
- // Cfr IVY-1186
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/parentPom/ivysettings-namespace.xml"));
@@ -4820,9 +5098,14 @@ public class ResolveTest {
"jar").exists());
}
+ /**
+ * Test case for IVY-1376.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1376">IVY-1376</a>
+ */
@Test
public void testResolveMaven2ParentPomDependencyManagementWithImport() throws Exception {
- // IVY-1376
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/parentPom/ivysettings.xml"));
ivy.getSettings().setDefaultResolver("parentChain");
@@ -4848,9 +5131,14 @@ public class ResolveTest {
"jar").exists());
}
+ /**
+ * Test case for IVY-501.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-501">IVY-501</a>
+ */
@Test
public void testResolveMaven2Snapshot1() throws Exception {
- // test case for IVY-501
// here we test maven SNAPSHOT versions handling,
// with m2 snapshotRepository/uniqueVersion set to true
Ivy ivy = new Ivy();
@@ -4869,9 +5157,14 @@ public class ResolveTest {
"test-SNAPSHOT1", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-1036.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1036">IVY-1036</a>
+ */
@Test
public void testResolveMaven2Snapshot1AsLatestIntegration() throws Exception {
- // test case for IVY-1036
// here we test maven SNAPSHOT versions handling,
// with m2 snapshotRepository/uniqueVersion set to true
// but retrieving by latest.integration
@@ -4891,9 +5184,14 @@ public class ResolveTest {
"test-SNAPSHOT1", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-501.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-501">IVY-501</a>
+ */
@Test
public void testResolveMaven2Snapshot2() throws Exception {
- // test case for IVY-501
// here we test maven SNAPSHOT versions handling,
// without m2 snapshotRepository/uniqueVersion set to true
Ivy ivy = new Ivy();
@@ -4912,9 +5210,14 @@ public class ResolveTest {
"test-SNAPSHOT2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-1036.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1036">IVY-1036</a>
+ */
@Test
public void testResolveMaven2Snapshot2AsLatestIntegration() throws Exception {
- // test case for IVY-1036
// here we test maven SNAPSHOT versions handling,
// with m2 snapshotRepository/uniqueVersion set to true
// but retrieving by latest.integration
@@ -5047,6 +5350,12 @@ public class ResolveTest {
assertTrue(getIvyFileInCache(mrid).exists());
}
+ /**
+ * Test case for IVY-1151.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1151">IVY-1151</a>
+ */
@Test
public void testIVY151() throws Exception {
Ivy ivy = new Ivy();
@@ -5101,9 +5410,14 @@ public class ResolveTest {
assertTrue(getArchiveFileInCache("org1", "mod1.2", "2.0", "mod1.2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-168.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-168">IVY-168</a>
+ */
@Test
public void testTransitiveConfMapping() throws Exception {
- // IVY-168
// mod13.3 depends on mod13.2 which depends on mod13.1
// each module has two confs: j2ee and compile
// each module only publishes one artifact in conf compile
@@ -5148,9 +5462,14 @@ public class ResolveTest {
assertEquals("another", resolveModRevId.getExtraAttribute("eatt2"));
}
+ /**
+ * Test case for IVY-773.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-773">IVY-773</a>
+ */
@Test
public void testExtraAttributes2() throws Exception {
- // test case for IVY-773
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/extra-attributes/ivysettings.xml"));
ivy.getSettings().setDefaultCache(cache);
@@ -5170,9 +5489,14 @@ public class ResolveTest {
assertTrue(new File(cache, "apache/module2/task2/1976/module2-linux.jar").exists());
}
+ /**
+ * Test case for IVY-745.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-745">IVY-745</a>
+ */
@Test
public void testExtraAttributes3() throws Exception {
- // test case for IVY-745
MockMessageLogger mockLogger = new MockMessageLogger();
Ivy ivy = new Ivy();
ivy.getLoggerEngine().setDefaultLogger(mockLogger);
@@ -5188,10 +5512,15 @@ public class ResolveTest {
mockLogger.assertLogContains("expected='task2' found='null'");
}
+ /**
+ * Second test case for IVY-745.ยจ
+ * Now we disable consistency checking, everything must work fine.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-745">IVY-745</a>
+ */
@Test
public void testExtraAttributes4() throws Exception {
- // second test case for IVY-745: now we disable consistency checking, everything should work
- // fine
Ivy ivy = new Ivy();
ivy.configure(new File("test/repositories/extra-attributes/ivysettings.xml"));
ivy.getSettings().setDefaultCache(cache);
@@ -5291,10 +5620,14 @@ public class ResolveTest {
assertTrue(getArchiveFileInCache(ivy, "bar#bar2#trunk;2", "bar2", "jar", "jar").exists());
}
+ /**
+ * Test case for IVY-717.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-717">IVY-717</a>
+ */
@Test
public void testBranches6() throws Exception {
- // test case for IVY-717
-
// bar1;4 -> foo#foo1#${ivy.branch};5
// foo#foo1#branch1;5 -> foo#foo2#${ivy.branch};1
// foo#foo1#trunk;5 -> {}
@@ -5597,6 +5930,12 @@ public class ResolveTest {
assertFalse(report.hasError());
}
+ /**
+ * Test case for IVY-956.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-956">IVY-956</a>
+ */
@Test(expected = StrictConflictException.class)
public void testIVY956() throws Exception {
ivy.getSettings().setDefaultConflictManager(
@@ -5605,6 +5944,12 @@ public class ResolveTest {
getResolveOptions(ivy.getSettings(), new String[] {"*"}).setValidate(false));
}
+ /**
+ * Test case for IVY-1159.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1159">IVY-1159</a>
+ */
@Test
public void testIVY1159_orderIsModAModB() throws Exception {
testIVY1159("ivy-depsorder_modA_then_modB.xml", false);
@@ -5621,6 +5966,12 @@ public class ResolveTest {
dds[1].getDependencyRevisionId());
}
+ /**
+ * Test case for IVY-1159.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1159">IVY-1159</a>
+ */
@Test
public void testIVY1159_orderIsModAModBReplaceForced() throws Exception {
testIVY1159("ivy-depsorder_modA_then_modB.xml", true);
@@ -5637,6 +5988,12 @@ public class ResolveTest {
dds[1].getDependencyRevisionId());
}
+ /**
+ * Test case for IVY-1159.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1159">IVY-1159</a>
+ */
@Test
public void testIVY1159_orderIsModBModA() throws Exception {
testIVY1159("ivy-depsorder_modB_then_modA.xml", false);
@@ -5653,6 +6010,12 @@ public class ResolveTest {
dds[1].getDependencyRevisionId());
}
+ /**
+ * Test case for IVY-1159.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1159">IVY-1159</a>
+ */
@Test
public void testIVY1159_orderIsModBModAReplaceForced() throws Exception {
testIVY1159("ivy-depsorder_modB_then_modA.xml", true);
@@ -5702,6 +6065,12 @@ public class ResolveTest {
ivy.deliver(pubrev, deliveryPattern, dopts);
}
+ /**
+ * Test case for IVY-1300.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1300">IVY-1300</a>
+ */
@Test
public void testIVY1300() throws Exception {
ivy = Ivy.newInstance();
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/core/retrieve/RetrieveTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/core/retrieve/RetrieveTest.java b/test/java/org/apache/ivy/core/retrieve/RetrieveTest.java
index 48d0fd4..d09c58e 100644
--- a/test/java/org/apache/ivy/core/retrieve/RetrieveTest.java
+++ b/test/java/org/apache/ivy/core/retrieve/RetrieveTest.java
@@ -261,7 +261,7 @@ public class RetrieveTest {
* This test is here to just test the deprecated {@code symlinkmass} option for retrieve task.
* A version or two down the line, after 2.5 release, we can remove this test and the option altogether
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@SuppressWarnings("deprecation")
@Test
@@ -293,7 +293,7 @@ public class RetrieveTest {
* {@link Files#exists(Path, LinkOption...) exists}
*
* @param filePath
- * @throws IOException
+ * @throws IOException if something goes wrong
*/
private void assertLinkOrExists(final String filePath) throws IOException {
if (systemHasSymlinkAbility) {
@@ -416,9 +416,11 @@ public class RetrieveTest {
}
/**
- * Tests that the {@link RetrieveEngine} retrieves artifacts with the correct extension if the artifact is unpacked
+ * Test case for IVY-1478.
+ * {@link RetrieveEngine} must retrieve artifacts with the correct extension if the artifact
+ * is unpacked.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1478">IVY-1478</a>
*/
@Test
@@ -445,31 +447,35 @@ public class RetrieveTest {
}
/**
- * Tests that more than one retrieve, with one having symlink enabled and the other not, doesn't lead to repository
- * cache corruption as reported in https://issues.apache.org/jira/browse/IVY-1498
+ * Test case for IVY-1498.
+ * More than one retrieve, one having symlink enabled and the other not,
+ * must not lead to repository cache corruption.
* <p>
* The test does the following:
* - Resolves a module, let's call it "irrelevant-A" which has a dependency on "org:foo-bar:1.2.3"
* <p>
* - Invokes a retrieve RT1, with {@link RetrieveOptions#setMakeSymlinks(boolean) symlinks true}
- * on that resolved module with a pattern that translates to a path "a/b/c/foo-bar.jar". When the RT1 retrieve is
- * done, the path "a/b/c/foo-bar.jar" will be a symlink to a path "repo/cache/org/foo-bar/foo-bar-1.2.3.jar" in the
- * Ivy cache.
+ * on that resolved module with a pattern that translates to a path "a/b/c/foo-bar.jar". When the
+ * RT1 retrieve is done, the path "a/b/c/foo-bar.jar" will be a symlink to a path
+ * "repo/cache/org/foo-bar/foo-bar-1.2.3.jar" in the Ivy cache.
* All fine so far.
* <p>
- * - We then resolve another module, let's call it "irrelevant-B" which has a dependency on "org:foo-bar:2.3.4"
+ * - We then resolve another module, let's call it "irrelevant-B" which has a dependency on
+ * "org:foo-bar:2.3.4"
* <p>
* - Next, do a new retrieve RT2, on this newly resolved module with
- * {@link RetrieveOptions#setMakeSymlinks(boolean) symlinks false} and {@link RetrieveOptions#getOverwriteMode()
- * overwrite semantics enabled} and with the same pattern as before), that translates to a path "a/b/c/foo-bar.jar".
+ * {@link RetrieveOptions#setMakeSymlinks(boolean) symlinks false} and
+ * {@link RetrieveOptions#getOverwriteMode() overwrite semantics enabled}
+ * and with the same pattern as before), that translates to a path "a/b/c/foo-bar.jar".
* <p>
- * When RT2 retrieve is done, we expect the path "a/b/c/foo-bar.jar" will *not* be a symlink and instead be an actual file that represents
- * the org:foo-bar:2.3.4 artifact jar.
+ * When RT2 retrieve is done, we expect the path "a/b/c/foo-bar.jar" will *not* be a symlink and
+ * instead be an actual file that represents the org:foo-bar:2.3.4 artifact jar.
* <p>
- * We also expect that this RT2 retrieve will not update/overwrite the file at path "repo/cache/org/foo-bar/foo-bar-1.2.3.jar"
- * in the Ivy cache - the file that was the end target of the symlink generated by the previous RT1.
+ * We also expect that this RT2 retrieve will not update/overwrite the file at path
+ * "repo/cache/org/foo-bar/foo-bar-1.2.3.jar" in the Ivy cache - the file that was the end
+ * target of the symlink generated by the previous RT1.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1498">IVY-1498</a>
*/
@Test
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/core/settings/XmlSettingsParserTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/core/settings/XmlSettingsParserTest.java b/test/java/org/apache/ivy/core/settings/XmlSettingsParserTest.java
index 6a271a4..71c935d 100644
--- a/test/java/org/apache/ivy/core/settings/XmlSettingsParserTest.java
+++ b/test/java/org/apache/ivy/core/settings/XmlSettingsParserTest.java
@@ -261,7 +261,7 @@ public class XmlSettingsParserTest {
/**
* Test of resolver referencing a non existent cache.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testInvalidCache() throws Exception {
@@ -400,9 +400,14 @@ public class XmlSettingsParserTest {
ivyPatterns.get(0));
}
+ /**
+ * Test case for IVY-319.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-319">IVY-319</a>
+ */
@Test
public void testMacroAndRef() throws Exception {
- // test case for IVY-319
IvySettings settings = new IvySettings();
XmlSettingsParser parser = new XmlSettingsParser(settings);
parser.parse(XmlSettingsParserTest.class.getResource("ivysettings-macro+ref.xml"));
@@ -423,9 +428,14 @@ public class XmlSettingsParserTest {
assertTrue(shared instanceof FileSystemResolver);
}
+ /**
+ * Test case for IVY-860.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-860">IVY-860</a>
+ */
@Test
public void testMacroAndRef2() throws Exception {
- // test case for IVY-860
IvySettings settings = new IvySettings();
XmlSettingsParser parser = new XmlSettingsParser(settings);
parser.parse(XmlSettingsParserTest.class.getResource("ivysettings-macro+ref2.xml"));
@@ -646,10 +656,11 @@ public class XmlSettingsParserTest {
}
/**
- * Tests that a <code><ttl></code> containing the <code>matcher</code> attribute, in a ivy settings file,
- * works as expected.
+ * Test case for IVY-1495.
+ * <code><ttl></code> containing the <code>matcher</code> attribute,
+ * in an ivy settings file, must work as expected.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1495">IVY-1495</a>
*/
@Test
@@ -673,7 +684,7 @@ public class XmlSettingsParserTest {
/**
* Tests that the <code>timeout-constraint</code> elements in a Ivy settings file are parsed correctly
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testTimeoutConstraints() throws Exception {
@@ -707,10 +718,10 @@ public class XmlSettingsParserTest {
}
/**
- * Tests that timeout constraints referenced by resolvers, in an ivy settings file, are processed correctly and the
- * corresponding resolvers use the right timeout constraints
+ * Tests that timeout constraints referenced by resolvers, in an ivy settings file, are
+ * processed correctly and the corresponding resolvers use the right timeout constraints
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testResolverTimeoutConstraintRefs() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/core/sort/SortTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/core/sort/SortTest.java b/test/java/org/apache/ivy/core/sort/SortTest.java
index 33031b8..0059775 100644
--- a/test/java/org/apache/ivy/core/sort/SortTest.java
+++ b/test/java/org/apache/ivy/core/sort/SortTest.java
@@ -90,8 +90,10 @@ public class SortTest {
/**
* Sorter does not throw circular dependency, circular dependencies are handled at resolve time
- * only. However the sort respect the transitive order when it is unambiguous. (if A depends
- * transitively of B, but B doesn't depends transitively on A then B always comes before A).
+ * only. However the sort respect the transitive order when it is unambiguous. (If A depends
+ * transitively of B, but B doesn't depends transitively on A, then B always comes before A.)
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testCircularDependency() throws Exception {
@@ -127,7 +129,12 @@ public class SortTest {
}
}
- // Test IVY-624
+ /**
+ * Test case for IVY-624
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-624">IVY-624</a>
+ */
@Test
public void testCircularDependencyInfiniteLoop() throws Exception {
addDependency(md1, "md2", "rev2");
@@ -213,8 +220,8 @@ public class SortTest {
/**
* When the version asked by a dependency is not compatible with the version declared in the
- * module to order, the two modules should be considered as independent NB: I'm sure of what
- * 'compatible' means !
+ * module to order, the two modules should be considered as independent.
+ * NB: I'm sure of what 'compatible' means !
*/
@Test
public void testDifferentVersionNotConsidered() {
@@ -339,7 +346,9 @@ public class SortTest {
fail(errorMessage.toString());
}
- /** Returns a collection of lists that contains the elements a,b,c and d */
+ /**
+ * Returns a collection of lists that contains the elements a, b, c and d
+ */
private Collection<List<ModuleDescriptor>> getAllLists(ModuleDescriptor a, ModuleDescriptor b,
ModuleDescriptor c, ModuleDescriptor d) {
final int nbOfList = 24;
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/osgi/core/OSGiManifestParserTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/osgi/core/OSGiManifestParserTest.java b/test/java/org/apache/ivy/osgi/core/OSGiManifestParserTest.java
index 60971a4..bddb05e 100644
--- a/test/java/org/apache/ivy/osgi/core/OSGiManifestParserTest.java
+++ b/test/java/org/apache/ivy/osgi/core/OSGiManifestParserTest.java
@@ -72,10 +72,10 @@ public class OSGiManifestParserTest extends AbstractModuleDescriptorParserTester
}
/**
- * Tests that the {@link OSGiManifestParser#parseDescriptor(ParserSettings, URL, Resource, boolean)} works fine for descriptors
- * that are backed by {@link FileResource}
+ * Tests that the {@link OSGiManifestParser#parseDescriptor(ParserSettings, URL, Resource, boolean)}
+ * works fine for descriptors that are backed by {@link FileResource}
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testFileResource() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/conflict/LatestCompatibleConflictManagerTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/conflict/LatestCompatibleConflictManagerTest.java b/test/java/org/apache/ivy/plugins/conflict/LatestCompatibleConflictManagerTest.java
index 104c63b..37f476c 100644
--- a/test/java/org/apache/ivy/plugins/conflict/LatestCompatibleConflictManagerTest.java
+++ b/test/java/org/apache/ivy/plugins/conflict/LatestCompatibleConflictManagerTest.java
@@ -139,7 +139,7 @@ public class LatestCompatibleConflictManagerTest {
/**
* Resolve must fail with a conflict.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = StrictConflictException.class)
public void testConflict() throws Exception {
@@ -152,7 +152,7 @@ public class LatestCompatibleConflictManagerTest {
/**
* Resolve must fail with a conflict.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = StrictConflictException.class)
public void testDynamicRootConflict() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/conflict/LatestConflictManagerTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/conflict/LatestConflictManagerTest.java b/test/java/org/apache/ivy/plugins/conflict/LatestConflictManagerTest.java
index 03a9942..3fd6d3f 100644
--- a/test/java/org/apache/ivy/plugins/conflict/LatestConflictManagerTest.java
+++ b/test/java/org/apache/ivy/plugins/conflict/LatestConflictManagerTest.java
@@ -54,7 +54,12 @@ public class LatestConflictManagerTest {
FileUtil.forceDelete(_cache);
}
- // Test case for issue IVY-388
+ /**
+ * Test case for IVY-388.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-388">IVY-388</a>
+ */
@Test
public void testIvy388() throws Exception {
ResolveReport report = ivy.resolve(
@@ -78,7 +83,12 @@ public class LatestConflictManagerTest {
}
}
- // Test case for issue IVY-383
+ /**
+ * Test case for IVY-383.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-383">IVY-383</a>
+ */
@Test
public void testIvy383() throws Exception {
ResolveReport report = ivy.resolve(
@@ -93,7 +103,12 @@ public class LatestConflictManagerTest {
}
}
- // Test case for issue IVY-407
+ /**
+ * Test case for IVY-407.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-407">IVY-407</a>
+ */
@Test
public void testLatestTime1() throws Exception {
ivy = new Ivy();
@@ -146,12 +161,19 @@ public class LatestConflictManagerTest {
}
}
- /*
- * Test case for issue IVY-407 (with transitivity) There are 5 modules A, B, C, D and E. 1)
- * publish C-1.0.0, C-1.0.1 and C-1.0.2 2) B needs C-1.0.0 : retrieve ok and publish B-1.0.0 3)
- * A needs B-1.0.0 and C-1.0.2 : retrieve ok and publish A-1.0.0 4) D needs C-1.0.1 : retrieve
- * ok and publish D-1.0.0 5) E needs D-1.0.0 and A-1.0.0 (D before A in ivy file) retrieve
- * failed to get C-1.0.2 from A (get apparently C-1.0.1 from D)
+ /**
+ * Test case for IVY-407 (with transitivity). There are 5 modules A, B, C, D and E.
+ * <ol>
+ * <li>publish C-1.0.0, C-1.0.1 and C-1.0.2</li>
+ * <li>B needs C-1.0.0 : retrieve ok and publish B-1.0.0</li>
+ * <li>A needs B-1.0.0 and C-1.0.2 : retrieve ok and publish A-1.0.0</li>
+ * <li>D needs C-1.0.1 : retrieve ok and publish D-1.0.0</li>
+ * <li>E needs D-1.0.0 and A-1.0.0 (D before A in ivy file) :
+ * retrieve failed to get C-1.0.2 from A (get apparently C-1.0.1 from D)</li>
+ * </ol>
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-407">IVY-407</a>
*/
@Test
public void testLatestTimeTransitivity() throws Exception {
@@ -187,9 +209,10 @@ public class LatestConflictManagerTest {
}
}
- /*
- * Test case for issue IVY-1399:
+ /**
+ * Test case for IVY-1399.
* Dependency tree:
+ * <pre>
* Mycompany#target;1
* MyCompany#A;1
* conflicting-dependency#dep;1
@@ -202,6 +225,10 @@ public class LatestConflictManagerTest {
* ...
* MyCompany#C;1
* conflicting-dependency#dep;1
+ * </pre>
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1399">IVY-1399</a>
*/
@Test
public void testEvictedModules() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/conflict/StrictConflictManagerTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/conflict/StrictConflictManagerTest.java b/test/java/org/apache/ivy/plugins/conflict/StrictConflictManagerTest.java
index 0851a87..5f14e9c 100644
--- a/test/java/org/apache/ivy/plugins/conflict/StrictConflictManagerTest.java
+++ b/test/java/org/apache/ivy/plugins/conflict/StrictConflictManagerTest.java
@@ -67,7 +67,7 @@ public class StrictConflictManagerTest {
/**
* Resolve must fail with a conflict.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = StrictConflictException.class)
public void testConflictResolve() throws Exception {
@@ -78,7 +78,7 @@ public class StrictConflictManagerTest {
/**
* Resolve must fail with a conflict.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = StrictConflictException.class)
public void testConflictWithDynamicRevisionResolve() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/parser/m2/PomModuleDescriptorParserTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/parser/m2/PomModuleDescriptorParserTest.java b/test/java/org/apache/ivy/plugins/parser/m2/PomModuleDescriptorParserTest.java
index 3ed867b..f51376d 100644
--- a/test/java/org/apache/ivy/plugins/parser/m2/PomModuleDescriptorParserTest.java
+++ b/test/java/org/apache/ivy/plugins/parser/m2/PomModuleDescriptorParserTest.java
@@ -376,7 +376,12 @@ public class PomModuleDescriptorParserTest extends AbstractModuleDescriptorParse
dds[1].getDependencyRevisionId());
}
- // IVY-392
+ /**
+ * Test case for IVY-392.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-392">IVY-392</a>
+ */
@Test
public void testDependenciesWithInactiveProfile() throws Exception {
ModuleDescriptor md = PomModuleDescriptorParser.getInstance().parseDescriptor(settings,
@@ -451,9 +456,14 @@ public class PomModuleDescriptorParserTest extends AbstractModuleDescriptorParse
md.getModuleRevisionId());
}
+ /**
+ * Test case for IVY-425.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-425">IVY-425</a>
+ */
@Test
public void testVariables() throws Exception {
- // test case for IVY-425
ModuleDescriptor md = PomModuleDescriptorParser.getInstance().parseDescriptor(settings,
getClass().getResource("spring-hibernate3-2.0.2.pom"), false);
assertNotNull(md);
@@ -469,9 +479,14 @@ public class PomModuleDescriptorParserTest extends AbstractModuleDescriptorParse
dds[10].getDependencyRevisionId());
}
+ /**
+ * Test case for IVY-423.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-423">IVY-423</a>
+ */
@Test
public void testDependenciesInProfile() throws Exception {
- // test case for IVY-423
ModuleDescriptor md = PomModuleDescriptorParser.getInstance().parseDescriptor(settings,
getClass().getResource("mule-module-builders-1.3.3.pom"), false);
assertNotNull(md);
@@ -481,9 +496,14 @@ public class PomModuleDescriptorParserTest extends AbstractModuleDescriptorParse
md.getModuleRevisionId());
}
+ /**
+ * Test case for IVY-424.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-424">IVY-424</a>
+ */
@Test
public void testIVY424() throws Exception {
- // test case for IVY-424
ModuleDescriptor md = PomModuleDescriptorParser.getInstance().parseDescriptor(settings,
getClass().getResource("shale-tiger-1.1.0-SNAPSHOT.pom"), false);
assertNotNull(md);
@@ -634,9 +654,14 @@ public class PomModuleDescriptorParserTest extends AbstractModuleDescriptorParse
excludeAllTransitiveDepsDescriptor.isTransitive());
}
+ /**
+ * Test case for IVY-417.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-417">IVY-417</a>
+ */
@Test
public void testWithPlugins() throws Exception {
- // test case for IVY-417
ModuleDescriptor md = PomModuleDescriptorParser.getInstance().parseDescriptor(settings,
getClass().getResource("mule-1.3.3.pom"), false);
assertNotNull(md);
@@ -683,7 +708,7 @@ public class PomModuleDescriptorParserTest extends AbstractModuleDescriptorParse
* Tests that if a module doesn't have a license specified, then parent pom's license (if any)
* is used for the child module
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testLicenseFromParent() throws Exception {
@@ -704,7 +729,7 @@ public class PomModuleDescriptorParserTest extends AbstractModuleDescriptorParse
* Tests that if a project explicitly specifies the licenses, then the licenses (if any) from
* its parent pom aren't applied to the child project
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testOverriddenLicense() throws Exception {
@@ -1059,10 +1084,11 @@ public class PomModuleDescriptorParserTest extends AbstractModuleDescriptorParse
}
/**
- * Tests that a pom.xml which has references to properties that are either set via environment variables or system
- * properties, has its properties evaluated correctly.
+ * Test case for IVY-1561.
+ * A pom.xml which has references to properties that are either set via environment variables
+ * or system properties, must have its properties evaluated correctly.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1561">IVY-1561</a>
*/
@Test
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorParserTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorParserTest.java b/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorParserTest.java
index af2a6cd..7588edb 100644
--- a/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorParserTest.java
+++ b/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorParserTest.java
@@ -200,8 +200,8 @@ public class XmlModuleDescriptorParserTest extends AbstractModuleDescriptorParse
/**
* Test must fail because of bad version.
*
- * @throws IOException
- * @throws ParseException
+ * @throws IOException if something goes wrong
+ * @throws ParseException if something goes wrong
*/
@Test(expected = ParseException.class)
public void testBadVersion() throws IOException, ParseException {
@@ -1103,7 +1103,7 @@ public class XmlModuleDescriptorParserTest extends AbstractModuleDescriptorParse
/**
* Test for parser failure.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = ParseException.class)
public void testWithNonExistingConfigInPublications() throws Exception {
@@ -1112,9 +1112,10 @@ public class XmlModuleDescriptorParserTest extends AbstractModuleDescriptorParse
}
/**
- * IVY-441: test for parser failure.
+ * Test case for IVY-441: expect parser failure.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-441">IVY-441</a>
*/
@Test(expected = ParseException.class)
public void testWithExistingConfigsInPublicationsSeparatedBySemiColon() throws Exception {
@@ -1459,8 +1460,8 @@ public class XmlModuleDescriptorParserTest extends AbstractModuleDescriptorParse
* as encoded characters (for example <code>%2F</code>) then the module descriptor and the location of
* the parent descriptor, are resolved and parsed correctly.
*
- * @throws Exception
- * @see <a href="https://issues.apache.org/jira/browse/IVY-1562">IVY-1562</a> for more details
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1562">IVY-1562</a>
*/
@Test
public void testExtendsAbsoluteLocation() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorWriterTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorWriterTest.java b/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorWriterTest.java
index 14edef1..341b569 100644
--- a/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorWriterTest.java
+++ b/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleDescriptorWriterTest.java
@@ -189,10 +189,10 @@ public class XmlModuleDescriptorWriterTest {
/**
* Test that the transitive attribute is written for non-transitive configurations.
*
- * <code><conf ... transitive="false" ... /></code>
+ * <code><conf ... transitive="false" ... /></code>
*
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1207">IVY-1207</a>
- * @throws Exception
*/
@Test
public void testTransitiveAttributeForNonTransitiveConfs() throws Exception {
@@ -219,8 +219,8 @@ public class XmlModuleDescriptorWriterTest {
* This is the default and writing it will only add noise and cause a deviation from the known
* behavior (before fixing IVY-1207).
*
+ * @throws Exception if something goes wrong
* @see <a href="https://issues.apache.org/jira/browse/IVY-1207">IVY-1207</a>
- * @throws Exception
*/
@Test
public void testTransitiveAttributeNotWrittenForTransitiveConfs() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleUpdaterTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleUpdaterTest.java b/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleUpdaterTest.java
index bb582d0..233b4e6 100644
--- a/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleUpdaterTest.java
+++ b/test/java/org/apache/ivy/plugins/parser/xml/XmlModuleUpdaterTest.java
@@ -337,7 +337,12 @@ public class XmlModuleUpdaterTest {
}
}
- // IVY-1356
+ /**
+ * Test case for IVY-1356.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-1356">IVY-1356</a>
+ */
@Test
public void testMergedUpdateWithExtendsAndExcludes() throws Exception {
URL url = XmlModuleUpdaterTest.class.getResource("test-extends-dependencies-exclude.xml");
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/repository/vfs/VfsRepositoryTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/repository/vfs/VfsRepositoryTest.java b/test/java/org/apache/ivy/plugins/repository/vfs/VfsRepositoryTest.java
index fe2fae4..e5cd9b8 100644
--- a/test/java/org/apache/ivy/plugins/repository/vfs/VfsRepositoryTest.java
+++ b/test/java/org/apache/ivy/plugins/repository/vfs/VfsRepositoryTest.java
@@ -29,19 +29,29 @@ import org.junit.Test;
import static org.junit.Assert.*;
/**
- * Testing Testing was the single biggest hurdle I faced. I have tried to provide a complete test
+ * Testing was the single biggest hurdle I faced. I have tried to provide a complete test
* suite that covers all protocols and which can be easily extended. It does differ - somewhat - in
* structure from the resolver/repository test suites. Setting up smb, ftp, sftp will undoubtedly be
- * your biggest headache (it was mine). Here are a few notes about the setup: - the VFS test suite
- * uses the build/test/repositories area - when setting samba, sftp, etc. the corresponding user
- * needs both read and write privileges. - the tests assume that the user and password is the same
- * for all services. - a limited amount of configuration is available by setting the following
- * properties in the ivy.properties file: * vfs.host * vfs.username * vfs.password * vfs.samba_share
- * Running the test requires that commons-io and ant.jar are on the classpath. Also, I would
- * recommend that at some time the tests be converted from straight junit to something which betters
- * supports functional testing. Although somewhat crude I am using jsystem
+ * your biggest headache (it was mine). Here are a few notes about the setup:
+ * <ul>
+ * <li>the VFS test suite uses the build/test/repositories area;
+ * when setting samba, sftp, etc. the corresponding user needs both read and write privileges.</li>
+ * <li>the tests assume that the user and password is the same for all services.</li>
+ * <li>a limited amount of configuration is available by setting the following properties in the
+ * <code>ivy.properties</code> file:</li>
+ * </ul>
+ * <pre>
+ * vfs.host
+ * vfs.username
+ * vfs.password
+ * vfs.samba_share
+ * </pre>
+ * Running the test requires that commons-io and ant jars are on the classpath. Also, I would
+ * recommend that at some time the tests be converted from straight junit to something which better
+ * supports functional testing. Although somewhat crude, I am using jsystem
* (http://jsystemtest.sourceforge.net/) in other projects and am finding it a much better solution
- * than straight junit. Stephen Nesbitt
+ * than straight junit.
+ * <p>Stephen Nesbitt</p>
*/
public class VfsRepositoryTest {
private VfsRepository repo = null;
@@ -70,7 +80,7 @@ public class VfsRepositoryTest {
/**
* Basic validation of happy path put - valid VFS URI and no conflict with existing file
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testPutValid() throws Exception {
@@ -98,7 +108,7 @@ public class VfsRepositoryTest {
/**
* Validate that we can overwrite an existing file
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testPutOverwriteTrue() throws Exception {
@@ -133,7 +143,7 @@ public class VfsRepositoryTest {
/**
* Validate that we put will respect a request not to overwrite an existing file
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test(expected = IOException.class)
public void testPutOverwriteFalse() throws Exception {
@@ -152,7 +162,7 @@ public class VfsRepositoryTest {
/**
* Test the retrieval of an artifact from the repository creating a new artifact
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testGetNoExisting() throws Exception {
@@ -178,7 +188,7 @@ public class VfsRepositoryTest {
/**
* Test the retrieval of an artifact from the repository overwriting an existing artifact
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testGetOverwriteExisting() throws Exception {
@@ -209,6 +219,8 @@ public class VfsRepositoryTest {
/**
* Validate that we get a non null Resource instance when passed a well-formed VfsURI pointing
* to an existing file
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testGetResourceValidExist() throws Exception {
@@ -227,6 +239,8 @@ public class VfsRepositoryTest {
/**
* Validate that we get a non null Resource instance when passed a well-formed VfsURI pointing
* to a non-existent file.
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testGetResourceValidNoExist() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/repository/vfs/VfsResourceTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/repository/vfs/VfsResourceTest.java b/test/java/org/apache/ivy/plugins/repository/vfs/VfsResourceTest.java
index 5e9cd74..99b9201 100644
--- a/test/java/org/apache/ivy/plugins/repository/vfs/VfsResourceTest.java
+++ b/test/java/org/apache/ivy/plugins/repository/vfs/VfsResourceTest.java
@@ -48,6 +48,8 @@ public class VfsResourceTest {
/**
* Validate VFSResource creation for a valid VFS URI pointing to an physically existing file
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testCreateResourceThatExists() throws Exception {
@@ -87,6 +89,8 @@ public class VfsResourceTest {
/**
* Validating that resource can be created for files which don't physically exists - e.g.
* resources that are going to created.
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testCreateResourceThatDoesntExist() throws Exception {
@@ -112,6 +116,8 @@ public class VfsResourceTest {
/**
* Validate VFSResource creation when given a poorly formed VFS identifier
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testBadURI() throws Exception {
@@ -132,6 +138,8 @@ public class VfsResourceTest {
/**
* Validate getChildren when given a VFS URI for a directory
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testListFolderChildren() throws Exception {
@@ -165,6 +173,8 @@ public class VfsResourceTest {
/**
* Validate that we don't get any results when we query a VFSResource file object for its
* children
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testListFileChildren() throws Exception {
@@ -179,6 +189,8 @@ public class VfsResourceTest {
/**
* Validate that we don't get any results if we ask an IMAGINARY VFSResource - a nonexistent
* file - for a list of its children
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testListImaginary() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/repository/vfs/VfsTestHelper.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/repository/vfs/VfsTestHelper.java b/test/java/org/apache/ivy/plugins/repository/vfs/VfsTestHelper.java
index 394dfba..a8314e4 100644
--- a/test/java/org/apache/ivy/plugins/repository/vfs/VfsTestHelper.java
+++ b/test/java/org/apache/ivy/plugins/repository/vfs/VfsTestHelper.java
@@ -73,7 +73,7 @@ public class VfsTestHelper {
*
* @param resource
* name of the resource
- * @return <class>List</class> of well-formed VFS resource identifiers
+ * @return {@link List} of well-formed VFS resource identifiers
*/
public List<VfsURI> createVFSUriSet(String resource) {
List<VfsURI> set = new ArrayList<>();
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/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 3b89002..1a5ced9 100644
--- a/test/java/org/apache/ivy/plugins/repository/vfs/VfsURI.java
+++ b/test/java/org/apache/ivy/plugins/repository/vfs/VfsURI.java
@@ -128,7 +128,7 @@ public class VfsURI {
/**
* Return a well-formed VFS Resource identifier
*
- * @return <code>String<code> representing a well formed VFS resource identifier
+ * @return <code>String</code> representing a well formed VFS resource identifier
*/
public String getVfsURI() {
StringBuilder uri = new StringBuilder();
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/resolver/ChainResolverTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/resolver/ChainResolverTest.java b/test/java/org/apache/ivy/plugins/resolver/ChainResolverTest.java
index c04894c..e9f0dd1 100644
--- a/test/java/org/apache/ivy/plugins/resolver/ChainResolverTest.java
+++ b/test/java/org/apache/ivy/plugins/resolver/ChainResolverTest.java
@@ -326,9 +326,14 @@ public class ChainResolverTest extends AbstractDependencyResolverTest {
}
}
+ /**
+ * Test case for IVY-206.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-206">IVY-206</a>
+ */
@Test
public void testFixedWithDefaultAndRealResolver() throws Exception {
- // test case for IVY-206
ChainResolver chain = new ChainResolver();
chain.setName("chain");
chain.setSettings(settings);
@@ -416,9 +421,14 @@ public class ChainResolverTest extends AbstractDependencyResolverTest {
}
}
+ /**
+ * Test case for IVY-389.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-389">IVY-389</a>
+ */
@Test
public void testReturnFirstWithDefaultAndCacheAndRealResolver() throws Exception {
- // test case for IVY-389
DefaultDependencyDescriptor dd = new DefaultDependencyDescriptor(
ModuleRevisionId.newInstance("org1", "mod1.1", "1.0"), false);
@@ -437,10 +447,14 @@ public class ChainResolverTest extends AbstractDependencyResolverTest {
assertEquals(true, rmr.getDescriptor().isDefault());
}
+ /**
+ * Test case for IVY-207.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-207">IVY-207</a>
+ */
public void testReturnFirstWithCheckModifiedAndDefaultAndCacheAndRealResolver()
throws Exception {
- // test case for IVY-207
-
DefaultDependencyDescriptor dd = new DefaultDependencyDescriptor(
ModuleRevisionId.newInstance("org1", "mod1.1", "1.0"), false);
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/resolver/DualResolverTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/resolver/DualResolverTest.java b/test/java/org/apache/ivy/plugins/resolver/DualResolverTest.java
index 8a7b5b2..6ed5853 100644
--- a/test/java/org/apache/ivy/plugins/resolver/DualResolverTest.java
+++ b/test/java/org/apache/ivy/plugins/resolver/DualResolverTest.java
@@ -87,8 +87,8 @@ public class DualResolverTest extends AbstractDependencyResolverTest {
/**
* Test fails due to bad resolver configuration.
*
- * @throws IOException
- * @throws ParseException
+ * @throws IOException if something goes wrong
+ * @throws ParseException if something goes wrong
*/
@Test(expected = ParseException.class)
public void testFromBadConf() throws IOException, ParseException {
@@ -99,7 +99,7 @@ public class DualResolverTest extends AbstractDependencyResolverTest {
/**
* Test fails due to bad resolver configuration
*
- * @throws ParseException
+ * @throws ParseException if something goes wrong
*/
@Test(expected = IllegalStateException.class)
public void testBad() throws ParseException {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/resolver/FileSystemResolverTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/resolver/FileSystemResolverTest.java b/test/java/org/apache/ivy/plugins/resolver/FileSystemResolverTest.java
index a0cbd71..6446d9c 100644
--- a/test/java/org/apache/ivy/plugins/resolver/FileSystemResolverTest.java
+++ b/test/java/org/apache/ivy/plugins/resolver/FileSystemResolverTest.java
@@ -165,9 +165,14 @@ public class FileSystemResolverTest extends AbstractDependencyResolverTest {
assertEquals(DownloadStatus.NO, ar.getDownloadStatus());
}
+ /**
+ * Test case for IVY-676.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-676">IVY-676</a>
+ */
@Test
public void testFindIvyFileRefWithMultipleIvyPatterns() throws Exception {
- // cfr IVY-676
FileSystemResolver resolver = new FileSystemResolver();
resolver.setName("test");
resolver.setSettings(settings);
@@ -263,7 +268,8 @@ public class FileSystemResolverTest extends AbstractDependencyResolverTest {
/**
* Tests that <code>SHA-256</code> algorithm can be used for checksums on resolvers
- * @throws Exception
+ *
+ * @throws Exception if something goes wrong
*/
@Test
public void testSHA256Checksum() throws Exception {
@@ -294,7 +300,7 @@ public class FileSystemResolverTest extends AbstractDependencyResolverTest {
/**
* Tests that <code>SHA-512</code> algorithm can be used for checksums on resolvers
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testSHA512Checksum() throws Exception {
@@ -887,7 +893,7 @@ public class FileSystemResolverTest extends AbstractDependencyResolverTest {
/**
* Publishing with transaction=true and an unsupported pattern must fail.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testUnsupportedTransaction() throws Exception {
@@ -916,7 +922,7 @@ public class FileSystemResolverTest extends AbstractDependencyResolverTest {
/**
* Publishing with transaction=true and an unsupported combination of patterns must fail.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testUnsupportedTransaction2() throws Exception {
@@ -948,7 +954,7 @@ public class FileSystemResolverTest extends AbstractDependencyResolverTest {
/**
* Publishing with transaction=true and overwrite mode must fail.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testUnsupportedTransaction3() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/resolver/IvyRepResolverTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/resolver/IvyRepResolverTest.java b/test/java/org/apache/ivy/plugins/resolver/IvyRepResolverTest.java
index 6091f5e..bce5415 100644
--- a/test/java/org/apache/ivy/plugins/resolver/IvyRepResolverTest.java
+++ b/test/java/org/apache/ivy/plugins/resolver/IvyRepResolverTest.java
@@ -95,9 +95,10 @@ public class IvyRepResolverTest extends AbstractDependencyResolverTest {
}
/**
- * IVY-625: should fail if no ivyroot specified.
+ * Test case for IVY-625. Must fail if no ivyroot specified.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-625">IVY-625</a>
*/
@Test
public void testMandatoryRoot() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/plugins/resolver/URLResolverTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/plugins/resolver/URLResolverTest.java b/test/java/org/apache/ivy/plugins/resolver/URLResolverTest.java
index f894598..7b11e4e 100644
--- a/test/java/org/apache/ivy/plugins/resolver/URLResolverTest.java
+++ b/test/java/org/apache/ivy/plugins/resolver/URLResolverTest.java
@@ -383,7 +383,7 @@ public class URLResolverTest extends AbstractDependencyResolverTest {
/**
* Tests that the timeout constraint set on the URL resolver is used correctly by the resolver
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testTimeoutConstraint() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/util/TestXmlHelper.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/util/TestXmlHelper.java b/test/java/org/apache/ivy/util/TestXmlHelper.java
index cea9e0b..440e35a 100644
--- a/test/java/org/apache/ivy/util/TestXmlHelper.java
+++ b/test/java/org/apache/ivy/util/TestXmlHelper.java
@@ -40,7 +40,7 @@ public class TestXmlHelper {
* @param xPathExpression The XPath expression to evaluate
* @param returnType The expected return type of the {@link XPathExpression#evaluate(Object, QName) evaluation}
* @return The result
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
public static Object evaluateXPathExpr(final File xmlFile, final XPathExpression xPathExpression, final QName returnType)
throws Exception {
@@ -67,7 +67,7 @@ public class TestXmlHelper {
* @param xpathExpression The XPath expression to evaluate
* @param returnType The expected return type of the {@link XPathExpression#evaluate(Object, QName) evaluation}
* @return The result
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
public static Object evaluateXPathExpr(final File xmlFile, final String xpathExpression, final QName returnType) throws Exception {
return evaluateXPathExpr(xmlFile, XPathFactory.newInstance().newXPath().compile(xpathExpression), returnType);
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/util/url/AbstractURLHandlerTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/util/url/AbstractURLHandlerTest.java b/test/java/org/apache/ivy/util/url/AbstractURLHandlerTest.java
index d30f1dd..e792c04 100644
--- a/test/java/org/apache/ivy/util/url/AbstractURLHandlerTest.java
+++ b/test/java/org/apache/ivy/util/url/AbstractURLHandlerTest.java
@@ -32,7 +32,10 @@ import static org.junit.Assert.assertEquals;
public class AbstractURLHandlerTest {
/**
- * JUnit test for IVY-923.
+ * Test case for IVY-923.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-923">IVY-923</a>
*/
@Test
public void testNormalizeToStringWithSpaceURL() throws Exception {
@@ -43,7 +46,10 @@ public class AbstractURLHandlerTest {
}
/**
- * JUnit test for IVY-923.
+ * Test case for IVY-923.
+ *
+ * @throws Exception if something goes wrong
+ * @see <a href="https://issues.apache.org/jira/browse/IVY-923">IVY-923</a>
*/
@Test
public void testNormalizeToStringWithPlusCharacter() throws Exception {
http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/c1f910cc/test/java/org/apache/ivy/util/url/ApacheURLListerTest.java
----------------------------------------------------------------------
diff --git a/test/java/org/apache/ivy/util/url/ApacheURLListerTest.java b/test/java/org/apache/ivy/util/url/ApacheURLListerTest.java
index 6ab2543..226a920 100644
--- a/test/java/org/apache/ivy/util/url/ApacheURLListerTest.java
+++ b/test/java/org/apache/ivy/util/url/ApacheURLListerTest.java
@@ -33,7 +33,7 @@ public class ApacheURLListerTest {
/**
* Tests {@link ApacheURLLister#retrieveListing(URL, boolean, boolean)}.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testRetrieveListing() throws Exception {
@@ -62,7 +62,7 @@ public class ApacheURLListerTest {
/**
* Tests {@link ApacheURLLister#retrieveListing(URL, boolean, boolean)}.
*
- * @throws Exception
+ * @throws Exception if something goes wrong
*/
@Test
public void testRetrieveListingWithSpaces() throws Exception {