testing host_bindgen changes

Signed-off-by: Shailesh Vashishth <shavashishth@gmail.com>

Author: vshailesh

src/hyperlight_component_macro/src/lib.rs

@@ -146,7 +146,13 @@ impl Parse for BindgenInputParams {
                         path = Some(std::path::PathBuf::from(value.value()));
                     }
                     _ => {
-                        return Err(syn::Error::new(key.span(), format!("unknown parameter '{}'; expected 'path' or 'world_name'", key)));
+                        return Err(syn::Error::new(
+                            key.span(),
+                            format!(
+                                "unknown parameter '{}'; expected 'path' or 'world_name'",
+                                key
+                            ),
+                        ));
                     }
                 }
                 // Parse optional comma
@@ -163,8 +169,8 @@ impl Parse for BindgenInputParams {
         if world_name.is_some() && path.is_none() {
             return Err(syn::Error::new(
                 proc_macro2::Span::call_site(),
-                "If `world_name` is provided in the brace syntax, `path` must also be provided."
-            )); 
+                "If `world_name` is provided in the brace syntax, `path` must also be provided.",
+            ));
         }
         Ok(Self { world_name, path })
     }

src/hyperlight_component_util/src/component.rs

@@ -118,20 +118,10 @@ pub fn read_component_single_exported_type<'a>(
                                 // picks the world index if world_name is passed in the proc_macro
                                 // else picks the index of last type, exported by core module
                                 if let Some(world) = world_name.as_ref() {
-
                                     let name = ce.name.0;
                                     if name.eq_ignore_ascii_case(&world) {
                                         selected_type_idx = Some(ctx.types.len() - 1);
-                                    } 
-                                    
-                                    // match ce.name {
-                                    //     wasmparser::ComponentExportName(name) => {
-                                    //         if name.eq_ignore_ascii_case(&world) {
-                                    //             selected_type_idx =
-                                    //                 Some(ctx.types.len() - 1);
-                                    //         }
-                                    //     }
-                                    // }
+                                    }
                                 } else {
                                     selected_type_idx = Some(ctx.types.len() - 1);
                                 }
@@ -176,9 +166,9 @@ pub fn read_component_single_exported_type<'a>(
             Some(Defined::Component(c)) => c,
             _ => panic!("final export is not component"),
         },
-        None => match &world_name{
+        None => match &world_name {
             Some(name) => panic!("world '{}' not found in component", name),
-            None => panic!("no exported type")
-        }
+            None => panic!("no exported type"),
+        },
     }
 }

src/hyperlight_component_util/src/util.rs

@@ -54,13 +54,7 @@ pub fn read_wit_type_from_file<R, F: FnMut(String, &etypes::Component) -> R>(
 
 /// Deal with `$HYPERLIGHT_COMPONENT_MACRO_DEBUG`: if it is present,
 /// save the given token stream (representing the result of
-/// macroexpansion) to the debug file and include that file instead of
-/// directly returning the given token stream.
-/// 
-/// Why do so?
-/// we can return the token stream as well.
-/// it would be better wouldn't it?
-/// Additionally this is causing a race condition.
+/// macroexpansion) to the debug file and then return the token stream
 pub fn emit_decls(decls: proc_macro2::TokenStream) -> proc_macro2::TokenStream {
     if let Ok(dbg_out) = std::env::var("HYPERLIGHT_COMPONENT_MACRO_DEBUG") {
         if let Ok(file) = syn::parse2(decls.clone()) {

src/hyperlight_host/tests/wit_test.rs

@@ -449,11 +449,4 @@ mod pick_world_binding_test {
         assert_eq!(first_import.r#key, "dummyKey");
         assert_eq!(first_import.r#value, "dummyValue");
     }
-
-    #[should_panic] 
-    #[test]
-    fn test_second_world_import_panic() { 
-        let second_export = SecondworldResources::new();
-        assert!(second_export)
-    }
 }