fix: logic for stubbing outgoing http and fetch-event

This commit fixes one bug and refactors the code to be more granular
in the case of differing feature sets.

The reality is more nuanced -- as we must stub wasi:http/types
differently depending on whether fetch-event is separately enabled.

Author: vados-cosmonic

crates/spidermonkey-embedding-splicer/src/stub_wasi.rs

@@ -139,8 +139,22 @@ pub fn stub_wasi(
         stub_stdio(&mut module)?;
     }
 
-    if !features.contains(&Features::Http) && !features.contains(&Features::FetchEvent) {
-        stub_http(&mut module)?;
+    match (
+        features.contains(&Features::Http),
+        features.contains(&Features::FetchEvent),
+    ) {
+        // If both are disabled, then disable all HTTP related imports
+        (false, false) => {
+            stub_http_types(&mut module)?;
+            stub_http_outgoing(&mut module)?;
+        }
+        // If HTTP is disabled but fetch-event is enabled we want to stub only the outgoing-handler import
+        // as wasi:http/types will be used by the fetch-event export.
+        (false, true) => {
+            stub_http_outgoing(&mut module)?;
+        }
+        // For all other cases we can avoid stubbing
+        _ => {}
     }
 
     let has_io = features.contains(&Features::Clocks)
@@ -380,7 +394,17 @@ fn stub_stdio(module: &mut Module) -> Result<()> {
     Ok(())
 }
 
-fn stub_http(module: &mut Module) -> Result<()> {
+fn stub_http_outgoing(module: &mut Module) -> Result<()> {
+    stub_wasi_imports(
+        module,
+        "wasi:http/outgoing-handler",
+        "handle",
+        unreachable_stub,
+    )?;
+    Ok(())
+}
+
+fn stub_http_types(module: &mut Module) -> Result<()> {
     stub_wasi_imports(
         module,
         "wasi:http/types",
@@ -753,12 +777,6 @@ fn stub_http(module: &mut Module) -> Result<()> {
         "[method]future-incoming-response.get",
         unreachable_stub,
     )?;
-    stub_wasi_imports(
-        module,
-        "wasi:http/outgoing-handler",
-        "handle",
-        unreachable_stub,
-    )?;
     Ok(())
 }