From 456268af61206a14e492c5983d7f045fc7254531 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Arnaud=20He=CC=81ritier?= Date: Tue, 11 Jun 2013 22:28:48 +0200 Subject: [PATCH] =?UTF-8?q?Code=20cleanup=20-=20=C2=A0Remove=20Redundant?= =?UTF-8?q?=20Cast(s)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../artifact/versioning/VersionRangeTest.java | 102 +++++++++--------- .../DefaultLegacyArtifactCollector.java | 6 +- .../metadata/MetadataResolutionResult.java | 4 +- .../transform/TransformationManagerTest.java | 2 +- .../apache/maven/project/ModelUtilsTest.java | 2 +- .../t00/ProjectInheritanceTest.java | 2 +- .../t02/ProjectInheritanceTest.java | 2 +- .../DefaultArtifactCollectorTest.java | 2 +- .../CanonicalProjectBuilderTest.java | 4 +- 9 files changed, 63 insertions(+), 63 deletions(-) diff --git a/maven-artifact/src/test/java/org/apache/maven/artifact/versioning/VersionRangeTest.java b/maven-artifact/src/test/java/org/apache/maven/artifact/versioning/VersionRangeTest.java index 5845f2a5cd..e155941e4f 100644 --- a/maven-artifact/src/test/java/org/apache/maven/artifact/versioning/VersionRangeTest.java +++ b/maven-artifact/src/test/java/org/apache/maven/artifact/versioning/VersionRangeTest.java @@ -57,7 +57,7 @@ public void testRange() VersionRange range = VersionRange.createFromVersionSpec( "(,1.0]" ); List restrictions = range.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - Restriction restriction = (Restriction) restrictions.get( 0 ); + Restriction restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.0", restriction.getUpperBound().toString() ); @@ -70,7 +70,7 @@ public void testRange() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.0", range.getRecommendedVersion().toString() ); restrictions = range.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -81,7 +81,7 @@ public void testRange() range = VersionRange.createFromVersionSpec( "[1.0]" ); restrictions = range.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.0", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.0", restriction.getUpperBound().toString() ); @@ -93,7 +93,7 @@ public void testRange() range = VersionRange.createFromVersionSpec( "[1.2,1.3]" ); restrictions = range.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.3", restriction.getUpperBound().toString() ); @@ -105,7 +105,7 @@ public void testRange() range = VersionRange.createFromVersionSpec( "[1.0,2.0)" ); restrictions = range.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.0", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "2.0", restriction.getUpperBound().toString() ); @@ -117,7 +117,7 @@ public void testRange() range = VersionRange.createFromVersionSpec( "[1.5,)" ); restrictions = range.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.5", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -129,13 +129,13 @@ public void testRange() range = VersionRange.createFromVersionSpec( "(,1.0],[1.2,)" ); restrictions = range.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.0", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); assertNull( CHECK_VERSION_RECOMMENDATION, range.getRecommendedVersion() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -183,7 +183,7 @@ public void testIntersections() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.0", mergedRange.getRecommendedVersion().toString() ); List restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - Restriction restriction = (Restriction) restrictions.get( 0 ); + Restriction restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -193,7 +193,7 @@ public void testIntersections() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.1", mergedRange.getRecommendedVersion().toString() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -206,7 +206,7 @@ public void testIntersections() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.1", mergedRange.getRecommendedVersion().toString() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.0", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -218,7 +218,7 @@ public void testIntersections() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.1", mergedRange.getRecommendedVersion().toString() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -230,7 +230,7 @@ public void testIntersections() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.1", mergedRange.getRecommendedVersion().toString() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.1", restriction.getLowerBound().toString() ); @@ -242,7 +242,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -254,7 +254,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -266,7 +266,7 @@ public void testIntersections() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.1", mergedRange.getRecommendedVersion().toString() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.2", restriction.getUpperBound().toString() ); @@ -278,7 +278,7 @@ public void testIntersections() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.1", mergedRange.getRecommendedVersion().toString() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.1", restriction.getUpperBound().toString() ); @@ -290,7 +290,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.1", restriction.getUpperBound().toString() ); @@ -302,7 +302,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.0", restriction.getUpperBound().toString() ); @@ -314,12 +314,12 @@ public void testIntersections() assertEquals( CHECK_VERSION_RECOMMENDATION, "1.2", mergedRange.getRecommendedVersion().toString() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.0", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -331,12 +331,12 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.0", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -348,12 +348,12 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertNull( CHECK_LOWER_BOUND, restriction.getLowerBound() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.1", restriction.getUpperBound().toString() ); assertFalse( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertNull( CHECK_UPPER_BOUND, restriction.getUpperBound() ); @@ -365,7 +365,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.3", restriction.getUpperBound().toString() ); @@ -377,7 +377,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.3", restriction.getUpperBound().toString() ); @@ -389,7 +389,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.3", restriction.getUpperBound().toString() ); @@ -401,7 +401,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.3", restriction.getUpperBound().toString() ); @@ -413,7 +413,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.3", restriction.getUpperBound().toString() ); @@ -425,7 +425,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.3", restriction.getUpperBound().toString() ); @@ -437,7 +437,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.2", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.3", restriction.getUpperBound().toString() ); @@ -449,7 +449,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.1", restriction.getUpperBound().toString() ); @@ -468,7 +468,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.1", restriction.getUpperBound().toString() ); @@ -480,7 +480,7 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 1, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.4", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.4", restriction.getUpperBound().toString() ); @@ -492,12 +492,12 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.2", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.3", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.4", restriction.getUpperBound().toString() ); @@ -509,12 +509,12 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.2", restriction.getUpperBound().toString() ); assertFalse( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.3", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.4", restriction.getUpperBound().toString() ); @@ -526,12 +526,12 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.2", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.3", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.4", restriction.getUpperBound().toString() ); @@ -543,12 +543,12 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.2", restriction.getUpperBound().toString() ); assertFalse( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.3", restriction.getLowerBound().toString() ); assertFalse( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.4", restriction.getUpperBound().toString() ); @@ -581,12 +581,12 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 2, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.2", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.3", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.4", restriction.getUpperBound().toString() ); @@ -598,17 +598,17 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 3, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.2", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.3", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.4", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 2 ); + restriction = restrictions.get( 2 ); assertEquals( CHECK_LOWER_BOUND, "1.5", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.5", restriction.getUpperBound().toString() ); @@ -620,17 +620,17 @@ public void testIntersections() assertNull( CHECK_VERSION_RECOMMENDATION, mergedRange.getRecommendedVersion() ); restrictions = mergedRange.getRestrictions(); assertEquals( CHECK_NUM_RESTRICTIONS, 3, restrictions.size() ); - restriction = (Restriction) restrictions.get( 0 ); + restriction = restrictions.get( 0 ); assertEquals( CHECK_LOWER_BOUND, "1.1", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.2", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 1 ); + restriction = restrictions.get( 1 ); assertEquals( CHECK_LOWER_BOUND, "1.3", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.4", restriction.getUpperBound().toString() ); assertTrue( CHECK_UPPER_BOUND_INCLUSIVE, restriction.isUpperBoundInclusive() ); - restriction = (Restriction) restrictions.get( 2 ); + restriction = restrictions.get( 2 ); assertEquals( CHECK_LOWER_BOUND, "1.5", restriction.getLowerBound().toString() ); assertTrue( CHECK_LOWER_BOUND_INCLUSIVE, restriction.isLowerBoundInclusive() ); assertEquals( CHECK_UPPER_BOUND, "1.6", restriction.getUpperBound().toString() ); diff --git a/maven-compat/src/main/java/org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java b/maven-compat/src/main/java/org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java index d69e8125f8..f734416b24 100644 --- a/maven-compat/src/main/java/org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java +++ b/maven-compat/src/main/java/org/apache/maven/repository/legacy/resolver/DefaultLegacyArtifactCollector.java @@ -219,7 +219,7 @@ private ManagedVersionMap getManagedVersionsMap( Artifact originatingArtifact, M } /* remove the originating artifact if it is also in managed versions to avoid being modified during resolution */ - Artifact managedOriginatingArtifact = (Artifact) versionMap.get( originatingArtifact.getDependencyConflictId() ); + Artifact managedOriginatingArtifact = versionMap.get( originatingArtifact.getDependencyConflictId() ); if ( managedOriginatingArtifact != null ) { @@ -465,7 +465,7 @@ private void recurse( ArtifactResolutionResult result, ResolutionNode node, // added to the artifact before we retrive the metadata // for the artifact; otherwise we may end up with unwanted // dependencies. - Artifact ma = (Artifact) managedVersions.get( childKey ); + Artifact ma = managedVersions.get( childKey ); ArtifactFilter managedExclusionFilter = ma.getDependencyFilter(); if ( null != managedExclusionFilter ) { @@ -602,7 +602,7 @@ private void recurse( ArtifactResolutionResult result, ResolutionNode node, private void manageArtifact( ResolutionNode node, ManagedVersionMap managedVersions, List listeners ) { - Artifact artifact = (Artifact) managedVersions.get( node.getKey() ); + Artifact artifact = managedVersions.get( node.getKey() ); // Before we update the version of the artifact, we need to know // whether we are working on a transitive dependency or not. This diff --git a/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataResolutionResult.java b/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataResolutionResult.java index 263f90681a..1d9e9c46c0 100644 --- a/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataResolutionResult.java +++ b/maven-compat/src/main/java/org/apache/maven/repository/metadata/MetadataResolutionResult.java @@ -63,8 +63,8 @@ public void setTree( MetadataTreeNode root ) public void initTreeProcessing( PlexusContainer plexus ) throws ComponentLookupException { - classpathTransformation = (ClasspathTransformation) plexus.lookup( ClasspathTransformation.class ); - conflictResolver = (GraphConflictResolver) plexus.lookup( GraphConflictResolver.class ); + classpathTransformation = plexus.lookup( ClasspathTransformation.class ); + conflictResolver = plexus.lookup( GraphConflictResolver.class ); } //---------------------------------------------------------------------------- public MetadataGraph getGraph() diff --git a/maven-compat/src/test/java/org/apache/maven/artifact/transform/TransformationManagerTest.java b/maven-compat/src/test/java/org/apache/maven/artifact/transform/TransformationManagerTest.java index 93b466c972..38450929f3 100644 --- a/maven-compat/src/test/java/org/apache/maven/artifact/transform/TransformationManagerTest.java +++ b/maven-compat/src/test/java/org/apache/maven/artifact/transform/TransformationManagerTest.java @@ -30,7 +30,7 @@ public class TransformationManagerTest public void testTransformationManager() throws Exception { - ArtifactTransformationManager tm = (ArtifactTransformationManager) lookup( ArtifactTransformationManager.class ); + ArtifactTransformationManager tm = lookup( ArtifactTransformationManager.class ); List tms = tm.getArtifactTransformations(); diff --git a/maven-compat/src/test/java/org/apache/maven/project/ModelUtilsTest.java b/maven-compat/src/test/java/org/apache/maven/project/ModelUtilsTest.java index 06b2e1d149..9c0cbd18c5 100644 --- a/maven-compat/src/test/java/org/apache/maven/project/ModelUtilsTest.java +++ b/maven-compat/src/test/java/org/apache/maven/project/ModelUtilsTest.java @@ -54,7 +54,7 @@ public void testShouldUseMainPluginDependencyVersionOverManagedDepVersion() ModelUtils.mergePluginDefinitions( plugin, mgtPlugin, false ); - assertEquals( dep.getVersion(), ((Dependency) plugin.getDependencies().get( 0 ) ).getVersion() ); + assertEquals( dep.getVersion(), plugin.getDependencies().get( 0 ).getVersion() ); } private Dependency createDependency( String gid, diff --git a/maven-compat/src/test/java/org/apache/maven/project/inheritance/t00/ProjectInheritanceTest.java b/maven-compat/src/test/java/org/apache/maven/project/inheritance/t00/ProjectInheritanceTest.java index 4b98a23ba2..0ee7980766 100644 --- a/maven-compat/src/test/java/org/apache/maven/project/inheritance/t00/ProjectInheritanceTest.java +++ b/maven-compat/src/test/java/org/apache/maven/project/inheritance/t00/ProjectInheritanceTest.java @@ -68,7 +68,7 @@ public void testProjectInheritance() // Value taken from p2 // ---------------------------------------------------------------------- - assertEquals( "mailing-list", ( (MailingList) p4.getMailingLists().get( 0 ) ).getName() ); + assertEquals( "mailing-list", p4.getMailingLists().get( 0 ).getName() ); // ---------------------------------------------------------------------- // Value taken from p1 diff --git a/maven-compat/src/test/java/org/apache/maven/project/inheritance/t02/ProjectInheritanceTest.java b/maven-compat/src/test/java/org/apache/maven/project/inheritance/t02/ProjectInheritanceTest.java index 43ba71dd41..2bcf7b57c5 100644 --- a/maven-compat/src/test/java/org/apache/maven/project/inheritance/t02/ProjectInheritanceTest.java +++ b/maven-compat/src/test/java/org/apache/maven/project/inheritance/t02/ProjectInheritanceTest.java @@ -95,7 +95,7 @@ public void testProjectInheritance() // Value taken from p2 // ---------------------------------------------------------------------- - assertEquals( "mailing-list", ( (MailingList) project4.getMailingLists().get( 0 ) ).getName() ); + assertEquals( "mailing-list", project4.getMailingLists().get( 0 ).getName() ); // ---------------------------------------------------------------------- // Value taken from p1 diff --git a/maven-compat/src/test/java/org/apache/maven/repository/legacy/resolver/DefaultArtifactCollectorTest.java b/maven-compat/src/test/java/org/apache/maven/repository/legacy/resolver/DefaultArtifactCollectorTest.java index f8fe1a3e11..58dd3fa213 100644 --- a/maven-compat/src/test/java/org/apache/maven/repository/legacy/resolver/DefaultArtifactCollectorTest.java +++ b/maven-compat/src/test/java/org/apache/maven/repository/legacy/resolver/DefaultArtifactCollectorTest.java @@ -76,7 +76,7 @@ protected void setUp() source = new Source(); artifactFactory = (ArtifactFactory) lookup( ArtifactFactory.ROLE ); - artifactCollector = (LegacyArtifactCollector) lookup( LegacyArtifactCollector.class ); + artifactCollector = lookup( LegacyArtifactCollector.class ); projectArtifact = createArtifactSpec( "project", "1.0", null ); } diff --git a/maven-core/src/test/java/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.java b/maven-core/src/test/java/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.java index 5421f94f47..8b6454da11 100644 --- a/maven-core/src/test/java/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.java +++ b/maven-core/src/test/java/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.java @@ -86,9 +86,9 @@ public void testProjectBuilder() List executions = plugin.getExecutions(); - PluginExecution execution = (PluginExecution) executions.get( 0 ); + PluginExecution execution = executions.get( 0 ); - String g0 = (String) execution.getGoals().get( 0 ); + String g0 = execution.getGoals().get( 0 ); assertEquals( "plexus:runtime", g0 );