Remove an unnecessary check when choosing the best candidate

Author: jcabot

src/main/java/com/xatkit/core/recognition/AbstractIntentRecognitionProvider.java

@@ -208,8 +208,9 @@ protected abstract RecognizedIntent getIntentInternal(@NonNull String input, @No
      */
     protected RecognizedIntent getBestCandidate(@NonNull Collection<RecognizedIntent> recognizedIntents,
                                                 @NonNull StateContext context) {
-        checkArgument(!recognizedIntents.isEmpty(), "Cannot get the best candidate from the provided collection: the "
-                + "collection is empty");
+        //This checkArgument seems unnecessary as the stream filter already includes and else that can cover this
+        // scenario
+        //checkArgument(!recognizedIntents.isEmpty(), "Cannot get the best candidate from the provided collection: the collection is empty");
         RecognizedIntent bestCandidate =
                 recognizedIntents.stream()
                         .sorted(Comparator.comparingDouble(RecognizedIntent::getRecognitionConfidence).reversed())