Refactor filterTestDefinitions method

Updated the method to use streams for filtering test definitions. Included sorting of results by test name before collecting into a list. This improves readability and potential efficiency of the function.

Author: eomasters-repos

src/main/java/org/eomasters/davalien/Davalien.java

@@ -289,12 +289,13 @@ private void runGptTests(List<TestInst> activeTests) {
   static List<TestDefinition> filterTestDefinitions(List<TestDefinition> allTestDefinitions, List<String> testNames,
       List<String> tags) {
 
-    if ((testNames == null || testNames.isEmpty()) && (tags == null || tags.isEmpty())) {
-      return allTestDefinitions;
+    Stream<TestDefinition> activeTestDefinitions = allTestDefinitions.stream();
+    if ((testNames != null && !testNames.isEmpty()) || (tags != null && !tags.isEmpty())) {
+      activeTestDefinitions = activeTestDefinitions
+          .filter(testDefinition -> isFiltered(testDefinition, testNames, tags));
     }
+    return activeTestDefinitions.sorted(Comparator.comparing(TestDefinition::getTestName)).collect(Collectors.toList());
 
-    return allTestDefinitions.stream().filter(testDefinition -> isFiltered(testDefinition, testNames, tags)).collect(
-        Collectors.toList());
   }
 
   private static boolean isFiltered(TestDefinition testDefinition, List<String> testNames, List<String> tags) {