Fix test: Check React element props instead of component calls

React.createElement doesn't call the component immediately - it returns an element.
Updated tests to check result.type and result.props instead of expecting
component function to be called.

Author: Erik Rasmussen

src/renderComponent.test.js

@@ -74,7 +74,7 @@ describe("renderComponent", () => {
     // This test reproduces issue #1055
     // When lazyProps has getter-only properties (like 'active'), and props contains
     // a property with the same name, it should not attempt to overwrite the getter
-    const Component = jest.fn(() => null);
+    const Component = () => null;
     const props = {
       component: Component,
       active: "value-from-props", // This would cause "Cannot set property active" error
@@ -89,27 +89,30 @@ describe("renderComponent", () => {
     const name = "TestComponent";
 
     // Should not throw "Cannot set property active"
-    expect(() => renderComponent(props, lazyProps, name)).not.toThrow();
-
-    expect(Component).toHaveBeenCalled();
-    const componentProps = Component.mock.calls[0][0];
+    let result;
+    expect(() => {
+      result = renderComponent(props, lazyProps, name);
+    }).not.toThrow();
 
+    // Check the React element was created with correct props
+    expect(result.type).toBe(Component);
+    
     // The getter-only property should remain and use the getter value
-    expect(componentProps.active).toBe("value-from-getter");
+    expect(result.props.active).toBe("value-from-getter");
 
     // Custom props should still be passed through
-    expect(componentProps.customProp).toBe("custom");
+    expect(result.props.customProp).toBe("custom");
   });
 
   it("should handle getter-only properties in all render paths", () => {
-    // Test with render prop
-    const render = jest.fn();
     const lazyProps = {};
     Object.defineProperty(lazyProps, "active", {
       get: () => "getter-value",
       enumerable: true,
     });
 
+    // Test with render prop
+    const render = jest.fn();
     renderComponent(
       { render, active: "prop-value" },
       lazyProps,
@@ -127,5 +130,15 @@ describe("renderComponent", () => {
     );
     expect(children).toHaveBeenCalled();
     expect(children.mock.calls[0][0].active).toBe("getter-value");
+
+    // Test with component prop
+    const Component = () => null;
+    const result = renderComponent(
+      { component: Component, active: "prop-value" },
+      lazyProps,
+      "TestComponent",
+    );
+    expect(result.type).toBe(Component);
+    expect(result.props.active).toBe("getter-value");
   });
 });