Rework SqlLineTest so that it actually interacts with the database (#3273)

The previous code called `SqlLine.main` which would call `System.exit`
once SqlLine finished processing, which would mean that JUnit wouldn't
get any result back, and the test would be marked as ignored.
This doesn't seem like great design from the gradle perspective. During
the rework, I also discovered that:
1. If you create the `SqlLine` object, you can change the `OutputStream`
it uses (and error), so you don't need to change system out, or err.
2. The driver provided was wrong (it is `JDBCRelationalDriver`, not
`JDBCEmbedDriver`)
3. the url was wrong (it's `jdbc:relational` not `jdbc:embed`, and you
need three / before the database, not 1)
4. it missed the `-` before the `-e` in the argument list

Note: I thought about adding an interactive test, where it runs multiple
statements, but that started to get complicated, and didn't seem
worthwhile.

Author: ScottDugas

fdb-relational-cli/src/test/java/com/apple/foundationdb/relational/cli/SQLLineTest.java

@@ -20,17 +20,17 @@
 
 package com.apple.foundationdb.relational.cli;
 
-import com.apple.foundationdb.relational.recordlayer.RelationalKeyspaceProvider;
-
+import com.google.common.base.Charsets;
 import org.junit.jupiter.api.Test;
 import sqlline.SqlLine;
 
-import java.io.BufferedOutputStream;
+import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
-import java.io.PrintStream;
+import java.nio.charset.StandardCharsets;
 
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 /**
  * SQLline doesn't do the intellij terminal very well.
@@ -45,32 +45,33 @@ public class SQLLineTest {
      */
     @Test
     public void runSqlline() throws IOException {
-        PrintStream oldOut = System.out;
-        try (PrintStream ps = new PrintStream(new BufferedOutputStream(new ByteArrayOutputStream()));) {
-            System.setOut(ps);
-            SqlLine.main(new String[]{
+        try (ByteArrayOutputStream out = new ByteArrayOutputStream();
+                ByteArrayOutputStream err = new ByteArrayOutputStream();
+                ByteArrayInputStream in = new ByteArrayInputStream(new byte[0])) {
+            final SqlLine sqlLine = new SqlLine();
+            sqlLine.setOutputStream(out);
+            sqlLine.setErrorStream(err);
+            final SqlLine.Status status = sqlLine.begin(new String[] {
                     "-ac", "com.apple.foundationdb.relational.cli.sqlline.Customize",
                     // We don't need the below because we add our command over in our
                     // sqlline Customize class.
                     // "-ch", "sqlline.PlannerDebuggerCommandHandler",
-                    "-u", "jdbc:embed:/__SYS?schema=CATALOG",
-                    "-d", "com.apple.foundationdb.relational.jdbc.JDBCEmbedDriver",
+                    "-u", "jdbc:relational:///__SYS?schema=CATALOG",
+                    "-d", "com.apple.foundationdb.relational.jdbc.JDBCRelationalDriver",
                     "--maxWidth=257",
-                    "e", "select * from databases;"
-            });
-            ps.flush();
-            // Output should contain something like:
-            // +-------------+
-            // | DATABASE_ID |
-            // +-------------+
-            // | /__SYS      |
-            // +-------------+
-            // Do primitive check tht output string has above.
-            assertTrue(ps.toString().contains("DATABASE_ID"));
-            assertTrue(ps.toString().contains(RelationalKeyspaceProvider.SYS));
-        } finally {
-            // Restore old sysout.
-            System.setOut(oldOut);
+                    "-e", "select * from databases;"
+            }, in, false);
+            try {
+                assertEquals(status, SqlLine.Status.OK);
+
+                final String output = out.toString(Charsets.UTF_8);
+                assertThat(output).contains("DATABASE_ID")
+                        .contains("/__SYS");
+            } catch (AssertionError e) {
+                System.out.println(err.toString(StandardCharsets.UTF_8));
+                System.out.println(out.toString(StandardCharsets.UTF_8));
+                throw e;
+            }
         }
     }
 }