Skip to content

Commit

Permalink
[22.0.x] Fix test annotations from recent merge (#9406)
Browse files Browse the repository at this point in the history
* Fix test annotations from recent merge

Fixing CI for an upcoming security release

* Ignore new tests on miri
  • Loading branch information
alexcrichton authored Oct 9, 2024
1 parent 22bfd21 commit 0efa267
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions tests/all/module.rs
Original file line number Diff line number Diff line change
Expand Up @@ -337,6 +337,7 @@ fn tail_call_defaults() -> Result<()> {
/// Smoke test for registering and unregistering modules (and their rec group
/// entries) concurrently.
#[test]
#[cfg_attr(miri, ignore)]
fn concurrent_type_registry_modifications() -> Result<()> {
let _ = env_logger::try_init();

Expand Down Expand Up @@ -482,12 +483,16 @@ fn concurrent_type_registry_modifications() -> Result<()> {
Ok(())
}

#[wasmtime_test(wasm_features(function_references))]
fn concurrent_type_modifications_and_checks(config: &mut Config) -> Result<()> {
#[test]
#[cfg_attr(miri, ignore)]
fn concurrent_type_modifications_and_checks() -> Result<()> {
const THREADS_CHECKING: usize = 4;

let _ = env_logger::try_init();

let mut config = Config::new();
config.wasm_function_references(true);

let engine = Engine::new(&config)?;

let mut threads = Vec::new();
Expand Down

0 comments on commit 0efa267

Please sign in to comment.