Add. hint message for missing @param value

- Added a helpful hint in the exception message when @param is missing a value and parameter name can't be resolved.
- Added a unit test to cover this case.

Files affected:
- Contract.java
- DefaultContractTest.java

Fixes OpenFeign#2337

Author: rojae

core/src/main/java/feign/Contract.java

@@ -327,7 +327,12 @@ public Default() {
               name = annotationName;
             }
             checkState(
-                emptyToNull(name) != null, "Param annotation was empty on param %s.", paramIndex);
+                emptyToNull(name) != null,
+                "Param annotation was empty on param %s.\nHint: %s",
+                paramIndex,
+                "Prefer using @Param(value=\"name\"), or compile your code with the -parameters flag.\n"
+                    + "If the value is missing, Feign attempts to retrieve the parameter name from bytecode, "
+                    + "which only works if the class was compiled with the -parameters flag.");
             nameParam(data, name, paramIndex);
             final Class<? extends Param.Expander> expander = paramAnnotation.expander();
             if (expander != Param.ToStringExpander.class) {

core/src/test/java/feign/DefaultContractTest.java

@@ -901,4 +901,20 @@ public Instant instant() {
       return Instant.ofEpochMilli(millis);
     }
   }
+
+  interface NoValueParamsInterface {
+    @RequestLine("GET /getSomething?id={id}")
+    String getSomething(@Param String id);
+  }
+
+  @Test
+  void errorMessageOnMissingParamNames() {
+    Throwable exception =
+        assertThrows(
+            IllegalStateException.class,
+            () -> contract.parseAndValidateMetadata(NoValueParamsInterface.class));
+    assertThat(exception.getMessage())
+        .contains("Param annotation was empty on param 0")
+        .contains("Hint");
+  }
 }