Use one process for all tests.

Instead of building the test suites inherited from BoringSSL as
seperate executables, link them all together into one executable,
giving all their `main` functions unique names.

This allows all the tests to be run, even on platforms that don't have
traditional process spawning, and avoids the need to keep track of
directory names even on platforms that do support process spawning.

This also makes it easier to integrate new BoringSSL test suites on
Windows, because we don't need to create a new `vcxproj` file for each
one.

Having one test executable may also make code coverage easier.

Makefile

@@ -20,16 +20,10 @@ include mk/ring.mk
 
 OBJS += \
   $(RING_OBJS) \
-  $(RING_TEST_LIB_OBJS) \
-  $(RING_TEST_MAIN_OBJS) \
   $(NULL)
 
 LIBS += \
-  $(RING_LIB) \
+  $(RING_LIBS) \
   $(NULL)
 
-EXES += $(RING_TEST_EXES)
-
-check:: check-ring
-
 include mk/bottom_of_makefile.mk

build.rs

@@ -107,7 +107,6 @@ fn build_c_code(out_dir: &str) -> Result<(), std::env::VarError> {
 
     let command_name;
     let args;
-    let lib_path;
     if !use_msbuild {
         command_name = "make";
         // Environment variables |CC|, |CXX|, etc. will be inherited from this
@@ -117,7 +116,6 @@ fn build_c_code(out_dir: &str) -> Result<(), std::env::VarError> {
         } else {
             "RELWITHDEBINFO"
         };
-        lib_path = Path::new(out_dir).join("lib");
         args = vec![
             format!("-j{}", num_jobs),
             format!("TARGET={}", target_str),
@@ -145,7 +143,6 @@ fn build_c_code(out_dir: &str) -> Result<(), std::env::VarError> {
             format!("/p:OutRootDir={}/", out_dir),
             format!("/p:GENERATED_CODE_DIR={}", out_dir),
         ];
-        lib_path = Path::new(&out_dir).join("lib");
     }
 
     if !std::process::Command::new(command_name)
@@ -157,7 +154,16 @@ fn build_c_code(out_dir: &str) -> Result<(), std::env::VarError> {
         panic!("{} execution failed", command_name);
     }
 
+    let lib_path = Path::new(out_dir).join("lib");
     println!("cargo:rustc-link-search=native={}", lib_path.to_str().unwrap());
     println!("cargo:rustc-link-lib=static={}-core", LIB_NAME);
+
+    // XXX: Ideally, this would only happen for `cargo test`, but we don't know
+    // how to do that yet.
+    println!("cargo:rustc-link-lib=static={}-test", LIB_NAME);
+    if !use_msbuild {
+        println!("cargo:rustc-flags=-l dylib=stdc++");
+    }
+
     Ok(())
 }

crypto/aes/aes_test.cc

@@ -19,6 +19,8 @@
 #include <openssl/crypto.h>
 
 
+extern "C" int bssl_aes_test_main();
+
 static bool TestAES(const uint8_t *key, size_t key_len,
                     const uint8_t plaintext[AES_BLOCK_SIZE],
                     const uint8_t ciphertext[AES_BLOCK_SIZE]) {
@@ -66,7 +68,7 @@ static bool TestAES(const uint8_t *key, size_t key_len,
   return true;
 }
 
-int main() {
+extern "C" int bssl_aes_test_main() {
   CRYPTO_library_init();
 
   // Test vectors from FIPS-197, Appendix C.