mod_wasm icon indicating copy to clipboard operation
mod_wasm copied to clipboard

Complete `wasm_runtime.so` unit tests

Open gzurl opened this issue 2 years ago • 0 comments

Is your feature request related to a problem? Please describe.

The wasm_runtime.so Rust library is at the core of mod_wasm. https://github.com/vmware-labs/mod_wasm/tree/main/wasm_runtime/src

Thus, it requires minimal unit tests to harden its stability.

Describe the solution you'd like

Originally, minimal unit tests were included for the low-level const_c_char_to_str() at ffi_utils.rs. The unit tests coverage for this library should be increased in all files.

  • [ ] ffi_utils.rs
  • [ ] c_api.rs
  • [ ] config.rs
  • [x] module.rs
  • [ ] wasi_ctx.rs
  • [ ] wasm_engine.rs
  • [ ] execution_ctx.rs

Describe alternatives you've considered

No response

Additional context

No response

gzurl avatar Jul 25 '23 08:07 gzurl