806 adding data testid attribute

[memisrose]

Description

Testing team noticed issues grabbing certain elements during automated testing.

Changes Made

Adding data-id's to elements that were hard to pinpoint during testing. Also adding Testing section to ReadMe.

How to Test

data-testid is now available on certain elements while inspecting.

[github-actions]

@CodiumAI-Agent /describe

[QodoAI-Agent]

Title

806 adding data testid attribute

---

User description

Description

Testing team noticed issues grabbing certain elements during automated testing.

Changes Made

Adding data-id's to elements that were hard to pinpoint during testing. Also adding Testing section to ReadMe.

How to Test

data-testid is now available on certain elements while inspecting.

---

PR Type

• Tests

---

Description

• Add data-testid to new app name input

• Add dynamic data-testid to copy buttons

---

Changes walkthrough 📝

Relevant files

Tests

NewAppModal.tsx Add data-testid to new app modal input                                      packages/client/src/components/app/NewAppModal.tsx Inserted inputProps with data-testid "new-app-name" Enhances test element identification +3/-0      EngineUsagePage.tsx Add dynamic data-testid to copy button                                      packages/client/src/pages/engine/EngineUsagePage.tsx Added data-testid attribute on copy button Utilizes label for dynamic id creation +1/-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

[QodoAI-Agent]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Consistency Check Verify that the addition of the testing identifier in inputProps aligns with the overall testing strategy and naming conventions used throughout similar components. inputProps={{ 'data-testid': 'new-app-name', }} Value Validation Confirm that the concatenated value for data-testid using the label variable reliably produces a valid identifier, especially when label might have unexpected content. data-testid={label + '-copy'}

[github-actions]

@CodiumAI-Agent /improve

[QodoAI-Agent]

PR Code Suggestions ✨

Category	Suggestion	Impact
Possible issue	Ensure fallback for label Verify that label is always defined to avoid unintended "undefined" values in the test-id; consider adding a fallback value. packages/client/src/pages/engine/EngineUsagePage.tsx [95] -+ data-testid={label + '-copy'} ++ data-testid={(label || 'default') + '-copy'} Suggestion importance[1-10]: 5 __ Why: The suggestion proposes a fallback for the 'label' variable to prevent potential "undefined" values in the test-id, which is a useful quality improvement though not critical.	Low

[github-actions]

@CodiumAI-Agent /update_changelog

[QodoAI-Agent]

Changelog updates: 🔄

2025-04-24 [*][https://github.com//pull/822]

Added

• data-testids across various components for improved automated testing
• “Testing” section in README with guidelines for test identifiers

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