tests: Removed the need for test framework to depend on libtest_mimic

ruffle-rs · Nov 10, 2023 · eb72cfb · eb72cfb
1 parent 2b450af
commit eb72cfb
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 10 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/tests/framework/Cargo.toml b/tests/framework/Cargo.toml
@@ -25,7 +25,6 @@ tracing = { workspace = true}
 tracing-subscriber = { workspace = true }
 serde = "1.0.190"
 toml = "0.8.6"
-libtest-mimic = "0.6.1"
 anyhow = "1.0.75"
 async-channel = "1.9.0"
 

diff --git a/tests/framework/src/test.rs b/tests/framework/src/test.rs
@@ -52,7 +52,7 @@ impl Test {
         mut before_start: impl FnMut(Arc<Mutex<Player>>) -> Result<()>,
         mut before_end: impl FnMut(Arc<Mutex<Player>>) -> Result<()>,
         environment: &impl Environment,
-    ) -> std::result::Result<(), libtest_mimic::Failed> {
+    ) -> Result<()> {
         set_logger();
         let movie =
             SwfMovie::from_path(&self.swf_path, None).map_err(|e| anyhow!(e.to_string()))?;
@@ -93,15 +93,15 @@ impl Test {
         Ok(())
     }
 
-    fn socket_events(&self) -> Result<Option<Vec<SocketEvent>>, libtest_mimic::Failed> {
+    fn socket_events(&self) -> Result<Option<Vec<SocketEvent>>> {
         Ok(if self.socket_path.is_file() {
             Some(SocketEvent::from_file(&self.socket_path)?)
         } else {
             None
         })
     }
 
-    fn input_injector(&self) -> Result<InputInjector, libtest_mimic::Failed> {
+    fn input_injector(&self) -> Result<InputInjector> {
         Ok(if self.input_path.is_file() {
             InputInjector::from_file(&self.input_path)?
         } else {

diff --git a/tests/tests/external_interface/tests.rs b/tests/tests/external_interface/tests.rs
@@ -11,7 +11,7 @@ pub fn external_interface_avm1(
     environment: &impl Environment,
 ) -> Result<(), libtest_mimic::Failed> {
     set_logger();
-    Test::from_options(
+    Ok(Test::from_options(
         TestOptions {
             num_frames: Some(1),
             ..Default::default()
@@ -63,14 +63,14 @@ pub fn external_interface_avm1(
             Ok(())
         },
         environment,
-    )
+    )?)
 }
 
 pub fn external_interface_avm2(
     environment: &impl Environment,
 ) -> Result<(), libtest_mimic::Failed> {
     set_logger();
-    Test::from_options(
+    Ok(Test::from_options(
         TestOptions {
             num_frames: Some(1),
             ..Default::default()
@@ -113,5 +113,5 @@ pub fn external_interface_avm2(
             Ok(())
         },
         environment,
-    )
+    )?)
 }
diff --git a/tests/tests/regression_tests.rs b/tests/tests/regression_tests.rs
@@ -69,7 +69,7 @@ fn main() {
                         }
                     } else {
                         match unwind_result {
-                            Ok(r) => r,
+                            Ok(r) => Ok(r?),
                             Err(e) => resume_unwind(e),
                         }
                     }