fix: address code review

Signed-off-by: Tomasz Andrzejak <andreiltd@gmail.com>

Author: andreiltd

src/hyperlight_component_macro/src/lib.rs

@@ -119,7 +119,8 @@ struct BindgenInputParams {
 fn source_or_env(source: Option<util::WitSource>) -> util::WitSource {
     source.unwrap_or_else(|| {
         util::WitSource::Wasm(std::path::PathBuf::from(
-            std::env::var_os("WIT_WORLD").unwrap(),
+            std::env::var_os("WIT_WORLD")
+                .expect("WIT_WORLD must be set when bindgen input is omitted"),
         ))
     })
 }

src/hyperlight_component_util/tests/wasmtime_guest_codegen.rs

@@ -77,16 +77,19 @@ fn wasmtime_guest_codegen_emits_wasmtime_flags_macro() {
 fn read_wit_type_accepts_wasm_encoded_wit() {
     let wasm_path =
         encode_wit_fixture_to_wasm(&fixture_path("../tests/rust_guests/witguest/guest.wit"));
-    let kebab_name = util::read_wit_type(
+    let (kebab_name, has_import_or_export) = util::read_wit_type(
         util::WitSource::Wasm(wasm_path.clone()),
         None,
         |kebab_name, ct| {
-            assert!(!ct.imports.is_empty() || !ct.instance.unqualified.exports.is_empty());
-            kebab_name
+            (
+                kebab_name,
+                !ct.imports.is_empty() || !ct.instance.unqualified.exports.is_empty(),
+            )
         },
     );
-    std::fs::remove_file(wasm_path).expect("temporary wasm fixture should be removed");
+    std::fs::remove_file(&wasm_path).expect("temporary wasm fixture should be removed");
 
+    assert!(has_import_or_export);
     assert_eq!(kebab_name, "test:wit/test");
 }