[maven-resolver] branch MRSOLVER-151 updated: Fix

[michaelo]

This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch MRSOLVER-151
in repository https://gitbox.apache.org/repos/asf/maven-resolver.git


The following commit(s) were added to refs/heads/MRSOLVER-151 by this push:
     new 2b9bdc6  Fix
2b9bdc6 is described below

commit 2b9bdc6cd1acdd1e6b7c6569364b1e28fcf4aa66
Author: Michael Osipov <micha...@apache.org>
AuthorDate: Wed Dec 23 23:15:22 2020 +0100

    Fix
---
 .../internal/impl/DefaultChecksumPolicyProviderTest.java   | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/DefaultChecksumPolicyProviderTest.java
 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/DefaultChecksumPolicyProviderTest.java
index 9a483fd..dbfd82e 100644
--- 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/DefaultChecksumPolicyProviderTest.java
+++ 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/DefaultChecksumPolicyProviderTest.java
@@ -27,6 +27,8 @@ import org.eclipse.aether.repository.RemoteRepository;
 import org.eclipse.aether.repository.RepositoryPolicy;
 import org.eclipse.aether.spi.connector.checksum.ChecksumPolicy;
 import org.eclipse.aether.transfer.TransferResource;
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.core.Is.is;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
@@ -124,7 +126,7 @@ public class DefaultChecksumPolicyProviderTest
         }
     }
 
-    @Test( expected = IllegalArgumentException.class )
+    @Test
     public void testGetEffectiveChecksumPolicy_UnknownPolicies()
     {
         String[][] testCases =
@@ -133,10 +135,12 @@ public class DefaultChecksumPolicyProviderTest
                 { RepositoryPolicy.CHECKSUM_POLICY_IGNORE, 
RepositoryPolicy.CHECKSUM_POLICY_IGNORE } };
         for ( String[] testCase : testCases )
         {
-            assertEquals( "unknown vs " + testCase[1], testCase[0],
-                          provider.getEffectiveChecksumPolicy( session, 
CHECKSUM_POLICY_UNKNOWN, testCase[1] ) );
-            assertEquals( "unknown vs " + testCase[1], testCase[0],
-                          provider.getEffectiveChecksumPolicy( session, 
testCase[1], CHECKSUM_POLICY_UNKNOWN ) );
+            IllegalArgumentException e = assertThrows( 
IllegalArgumentException.class,
+                    () -> provider.getEffectiveChecksumPolicy( session, 
CHECKSUM_POLICY_UNKNOWN, testCase[1] ) );
+            assertThat( e.getMessage(), is("Unsupported policy: unknown") );
+            e = assertThrows( IllegalArgumentException.class,
+                    () -> provider.getEffectiveChecksumPolicy( session, 
testCase[1], CHECKSUM_POLICY_UNKNOWN ) );
+            assertThat( e.getMessage(), is("Unsupported policy: unknown") );
         }
     }