Add support for running YAML tests against a server built from local branch (#3161)

In order to increase coverage - specifically, running the current
version on multiple JVMs - this change adds the ability to build and run
multi-server tests against the current local version.
The test run will treat the locally-built server the same as any other
downloaded server. That means that 4 test runs will be performed:
- embedded then remote (with and without continuation forcing)
- remote then embedded (with and without continuation forcing)

This effectively adds one more downloaded server to the list.
An option was considered (and can be quite easily implemented) to reduce
coverage by treating this server differently (and only run a subset of
the configs) but given the large number of servers we already plan to
download anyway, this didn't seem material.

Author: ohadzeliger

yaml-tests/src/main/java/com/apple/foundationdb/relational/yamltests/YamlExecutionContext.java

@@ -521,6 +521,11 @@ public ContextOptions mergeFrom(ContextOptions other) {
         public <T> T getOrDefault(ContextOption<T> prop, T defaultValue) {
             return (T)map.getOrDefault(prop, defaultValue);
         }
+
+        @Override
+        public String toString() {
+            return map.toString();
+        }
     }
 
     public static class ContextOption<T> {
@@ -546,5 +551,10 @@ public boolean equals(final Object o) {
         public int hashCode() {
             return Objects.hashCode(name);
         }
+
+        @Override
+        public String toString() {
+            return name;
+        }
     }
 }

yaml-tests/src/main/java/com/apple/foundationdb/relational/yamltests/YamlTestExtension.java

@@ -121,12 +121,14 @@ public void afterAll(final ExtensionContext context) throws Exception {
                                 return e;
                             }
                         }).filter(Objects::nonNull).findFirst();
-        for (ExternalServer server : servers) {
-            try {
-                server.stop();
-            } catch (Exception ex) {
-                if (logger.isWarnEnabled()) {
-                    logger.warn("Failed to stop server " + server.getVersion() + " on " + server.getPort());
+        if (servers != null) {
+            for (ExternalServer server : servers) {
+                try {
+                    server.stop();
+                } catch (Exception ex) {
+                    if (logger.isWarnEnabled()) {
+                        logger.warn("Failed to stop server " + server.getVersion() + " on " + server.getPort());
+                    }
                 }
             }
         }

yaml-tests/yaml-tests.gradle

@@ -122,14 +122,17 @@ task cleanExternalServerDirectory(type: Delete) {
     delete project.layout.buildDirectory.dir('externalServer')
 }
 
-task downloadExternalServer(type: Copy) {
-    dependsOn "cleanExternalServerDirectory"
-    from resolveOtherServer(new HashSet<String>())
+task serverJars(type: Copy) {
+    dependsOn ":fdb-relational-server:package", "cleanExternalServerDirectory"
     into project.layout.buildDirectory.dir('externalServer')
+    from resolveOtherServer(new HashSet<String>())
+    from (rootProject.project("fdb-relational-server").layout.files('.dist')) {
+        include("*-SNAPSHOT-all.jar")
+    }
 }
 
 test {
-    dependsOn "downloadExternalServer"
+    dependsOn "serverJars"
     systemProperty("yaml_testing_external_server", project.layout.buildDirectory.dir('externalServer').get().asFile)
     // These are the system properties that are looked at by the YamlTesting framework to initialize the execution context
     var isNightly = System.getenv("COM_APPLE_FOUNDATIONDB_RELATIONAL_YAMLTESTS_NIGHTLY")