test(renderer): add iframe block tests

[stelbailey]

Description

added tests for the iframe block

Changes Made

How to Test

1. Steps to reproduce/test the behavior
2. Expected outcomes

Notes

[github-actions]

@CodiumAI-Agent /describe

[CodiumAI-Agent]

Title

test(renderer): add iframe block tests

---

User description

Description

added tests for the iframe block

Changes Made

How to Test

1. Steps to reproduce/test the behavior
2. Expected outcomes

Notes

---

PR Type

Tests

---

Description

• Add tests for IframeBlock rendering and attributes

• Verify pointer event toggling behavior

• Ensure visibility toggles on show flag

• Check custom and default style application

---

Changes walkthrough 📝

	Relevant files
Tests	IframeBlock.spec.tsx` IframeBlock rendering and styling tests                                    libs/renderer/src/testing/block-defaults/IframeBlock.spec.tsx Introduce tests for default rendering attributes Test disabling pointer interactions Validate show/hide block behavior Assess custom and fallback styling logic
Additional files	IframeBlock.spec.tsx +118/-0

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

@CodiumAI-Agent /review

[CodiumAI-Agent]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Fragile Selector The test "applies default styles when data.style is not provided" selects the wrapper by a generic "span" tag, which can be brittle. Consider querying by the component's data attribute for more reliable selection. const wrapper = container.querySelector("span"); expect(wrapper).toHaveStyle({ width: "100%", height: "400px" }); }); Type Mismatch The show property is handled as a string ("true"/"false") in the test data. Verify that the component expects string values and not booleans to avoid potential flaky behavior. show: "true", },

[github-actions]

@CodiumAI-Agent /improve

[CodiumAI-Agent]

PR Code Suggestions ✨

Category	Suggestion	Impact
General	Use boolean for show flag Replace the string values for show with booleans to match expected API usage. Update both "true" and "false" cases in the tests to use true/false. This aligns the test setup with how the component likely interprets the show flag. libs/renderer/src/testing/block-defaults/IframeBlock.spec.tsx [14] -show: "true", +show: true, Suggestion importance[1-10]: 7 __ Why: Changing show to a boolean aligns the test setup with expected component props and prevents the test from passing incorrect string values.	Medium
	Use consistent wrapper selector Select the wrapper consistently by its data-block attribute instead of a tag name. This makes the test more resilient to markup changes and matches how other tests locate the block root. libs/renderer/src/testing/block-defaults/IframeBlock.spec.tsx [115] -const wrapper = container.querySelector("span"); +const wrapper = container.querySelector("[data-block='iframe']"); Suggestion importance[1-10]: 5 __ Why: Selecting the wrapper by [data-block='iframe'] makes the test more robust against markup changes and matches other selectors.	Low
	Use string style assertion Use a style string in toHaveStyle for consistency with other tests. This makes the assertion clearer and avoids potential object-to-string matching issues. libs/renderer/src/testing/block-defaults/IframeBlock.spec.tsx [98] -expect(iframe).toHaveStyle({ width: "100%", height: "100%" }); +expect(iframe).toHaveStyle("width: 100%; height: 100%;"); Suggestion importance[1-10]: 4 __ Why: Using a style string for toHaveStyle matches other tests and avoids potential mismatches with object syntax.	Low
Possible issue	Assert iframe absence when hidden Add an assertion to confirm that no <iframe> is rendered when show is false. This directly verifies the component hides the frame rather than just clearing the wrapper content. libs/renderer/src/testing/block-defaults/IframeBlock.spec.tsx [76] expect(block).toBeEmptyDOMElement(); +expect(container.querySelector("iframe")).toBeNull(); Suggestion importance[1-10]: 5 __ Why: Adding a check that no <iframe> is in the DOM directly verifies the component hides the iframe, improving test coverage.	Low

[github-actions]

@CodiumAI-Agent /update_changelog

[CodiumAI-Agent]

Changelog updates: 🔄

2025-05-30 *

Added

• Tests for the iframe block in the renderer suite

to commit the new content to the CHANGELOG.md file, please type:
'/update_changelog --pr_update_changelog.push_changelog_changes=true'