Add CLI driver tests

.github/workflows/build.yml

@@ -0,0 +1,43 @@
+name: CI Build
+
+on: [push, pull_request]
+
+jobs:
+  build:
+    name: Build and Test
+    runs-on: ${{ matrix.os }}
+    # We want to run on external PRs, but not on internal ones as push automatically builds
+    # H/T: https://github.com/Dart-Code/Dart-Code/commit/612732d5879730608baa9622bf7f5e5b7b51ae65
+    if: github.event_name == 'push' || github.event.pull_request.head.repo.full_name != 'amzn/ion-cli'
+    strategy:
+      matrix:
+        # TODO: add windows after fixing cmake version error issues
+        # see for example Windows build workflow: https://github.com/amzn/ion-rust/blob/a4b154cc0a5b5b661a45ac14d3719a501573d8f2/.github/workflows/rust.yml#L13-L28
+        os: [ubuntu-latest, macos-latest]
+
+    steps:
+      - name: Git Checkout
+        uses: actions/checkout@v2
+        with:
+          submodules: recursive
+      - name: Rust Toolchain
+        uses: actions-rs/toolchain@v1
+        with:
+          profile: minimal
+          toolchain: stable
+          override: true
+      - name: Cargo Build
+        uses: actions-rs/cargo@v1
+        with:
+          command: build
+          args: --verbose --workspace
+      - name: Cargo Test
+        uses: actions-rs/cargo@v1
+        with:
+          command: test
+          args: --verbose --workspace -- --test-threads=1
+      - name: Rustfmt Check
+        uses: actions-rs/cargo@v1
+        with:
+          command: fmt
+          args: --verbose -- --check

Cargo.toml

@@ -24,7 +24,12 @@ ion-schema = "0.1.0"
 [build-dependencies]
 cmake = "0.1.44"
 
+[dev-dependencies]
+rstest = "~0.10.0"
+assert_cmd = "~1.0.5"
+tempfile = "~3.2.0"
+
 [[bin]]
 name = "ion"
-test = false
+test = true
 bench = false

build.rs

@@ -35,15 +35,24 @@ fn main() {
     // which libraries to link against and in which directories they can be found.
 
     // ion_events library
-    println!("cargo:rustc-link-search=native={}/build/tools/events", ion_c_release_dir);
+    println!(
+        "cargo:rustc-link-search=native={}/build/tools/events",
+        ion_c_release_dir
+    );
     println!("cargo:rustc-link-lib=static=ion_events_static");
 
     // ion_c library
-    println!("cargo:rustc-link-search=native={}/build/ionc", ion_c_release_dir);
+    println!(
+        "cargo:rustc-link-search=native={}/build/ionc",
+        ion_c_release_dir
+    );
     println!("cargo:rustc-link-lib=static=ionc_static");
 
     // decNumber library
-    println!("cargo:rustc-link-search=native={}/build/decNumber", ion_c_release_dir);
+    println!(
+        "cargo:rustc-link-search=native={}/build/decNumber",
+        ion_c_release_dir
+    );
     println!("cargo:rustc-link-lib=static=decNumber_static");
 
     // C++ library
@@ -63,7 +72,10 @@ fn main() {
     }
 
     // ion-c CLI library
-    println!("cargo:rustc-link-search=native={}/build/tools/cli/", ion_c_release_dir);
+    println!(
+        "cargo:rustc-link-search=native={}/build/tools/cli/",
+        ion_c_release_dir
+    );
     println!("cargo:rustc-link-lib=static=ion_cli_main");
 
     // Only rebuild ion-c if that submodule directory is updated

src/bin/ion/commands/beta/mod.rs

@@ -1,26 +1,23 @@
 pub mod inspect;
 pub mod schema;
 
+use crate::commands::{CommandConfig, CommandRunner};
 use anyhow::Result;
 use clap::{App, ArgMatches};
-use crate::commands::{CommandRunner, CommandConfig};
 
 // To add a beta subcommand, add your new command to the `beta_subcommands`
 // and `runner_for_beta_subcommands` functions.
 
 // Creates a Vec of CLI configurations for all of the available built-in commands
 pub fn beta_subcommands() -> Vec<CommandConfig> {
-    vec![
-        inspect::app(),
-        schema::app()
-    ]
+    vec![inspect::app(), schema::app()]
 }
 
 pub fn runner_for_beta_subcommand(command_name: &str) -> Option<CommandRunner> {
     let runner = match command_name {
         "inspect" => inspect::run,
         "schema" => schema::run,
-        _ => return None
+        _ => return None,
     };
     Some(runner)
 }
@@ -39,7 +36,7 @@ pub fn run(_command_name: &str, matches: &ArgMatches<'static>) -> Result<()> {
             "The requested beta command ('{}') is not supported and clap did not generate an error message.",
             command_name
         );
-        unreachable!(message);
+        unreachable!("{}", message);
     }
     Ok(())
 }