Use wasmtime_test instead of test

This allows a couple of things:
* Pre-configuring `Winch` as the strategy
* Handling the target architetures supported by Winch

Author: saulecabrera

tests/all/epoch_interruption.rs

@@ -5,6 +5,7 @@ use anyhow::anyhow;
 use std::sync::atomic::{AtomicBool, Ordering};
 use std::sync::Arc;
 use wasmtime::*;
+use wasmtime_test_macros::wasmtime_test;
 
 fn build_engine() -> Arc<Engine> {
     let mut config = Config::new();
@@ -449,12 +450,10 @@ async fn drop_future_on_epoch_yield() {
     assert_eq!(true, alive_flag.load(Ordering::Acquire));
 }
 
-#[test]
+#[wasmtime_test(strategies(not(Cranelift)))]
 #[cfg_attr(miri, ignore)]
-fn ensure_compatibility_between_winch_and_epoch() -> Result<()> {
-    let mut config = Config::new();
+fn ensure_compatibility_between_winch_and_epoch(config: &mut Config) -> Result<()> {
     config.epoch_interruption(true);
-    config.strategy(Strategy::Winch);
     let result = Engine::new(&config);
     match result {
         Ok(_) => anyhow::bail!("Expected incompatibility between epoch interruption and Winch"),

tests/all/fuel.rs

@@ -1,4 +1,5 @@
 use wasmtime::*;
+use wasmtime_test_macros::wasmtime_test;
 use wast::parser::{self, Parse, ParseBuffer, Parser};
 use wast::token::Span;
 
@@ -257,12 +258,10 @@ fn get_fuel_clamps_at_zero() -> Result<()> {
     Ok(())
 }
 
-#[test]
+#[wasmtime_test(strategies(not(Cranelift)))]
 #[cfg_attr(miri, ignore)]
-fn ensure_compatibility_between_winch_and_fuel() -> Result<()> {
-    let mut config = Config::new();
+fn ensure_compatibility_between_winch_and_fuel(config: &mut Config) -> Result<()> {
     config.consume_fuel(true);
-    config.strategy(Strategy::Winch);
     let result = Engine::new(&config);
     match result {
         Ok(_) => anyhow::bail!("Expected incompatibility between fuel and Winch"),

tests/all/winch_engine_features.rs

@@ -1,11 +1,10 @@
 use wasmtime::*;
+use wasmtime_test_macros::wasmtime_test;
 
-#[test]
+#[wasmtime_test(strategies(not(Cranelift)))]
 #[cfg_attr(miri, ignore)]
-fn ensure_compatibility_between_winch_and_table_lazy_init() -> Result<()> {
-    let mut config = Config::new();
+fn ensure_compatibility_between_winch_and_table_lazy_init(config: &mut Config) -> Result<()> {
     config.table_lazy_init(false);
-    config.strategy(Strategy::Winch);
     let result = Engine::new(&config);
     match result {
         Ok(_) => {
@@ -22,12 +21,10 @@ fn ensure_compatibility_between_winch_and_table_lazy_init() -> Result<()> {
     Ok(())
 }
 
-#[test]
+#[wasmtime_test(strategies(not(Cranelift)))]
 #[cfg_attr(miri, ignore)]
-fn ensure_compatibility_between_winch_and_signals_based_traps() -> Result<()> {
-    let mut config = Config::new();
+fn ensure_compatibility_between_winch_and_signals_based_traps(config: &mut Config) -> Result<()> {
     config.signals_based_traps(false);
-    config.strategy(Strategy::Winch);
     let result = Engine::new(&config);
     match result {
         Ok(_) => {
@@ -46,12 +43,12 @@ fn ensure_compatibility_between_winch_and_signals_based_traps() -> Result<()> {
     Ok(())
 }
 
-#[test]
+#[wasmtime_test(strategies(not(Cranelift)))]
 #[cfg_attr(miri, ignore)]
-fn ensure_compatibility_between_winch_and_generate_native_debuginfo() -> Result<()> {
-    let mut config = Config::new();
+fn ensure_compatibility_between_winch_and_generate_native_debuginfo(
+    config: &mut Config,
+) -> Result<()> {
     config.debug_info(true);
-    config.strategy(Strategy::Winch);
     let result = Engine::new(&config);
     match result {
         Ok(_) => {