TrueSparrowSystems / AI-SalesSparrow-Android-App

MIT License
1 stars 0 forks source link

Updated test ids #88

Closed Abujdnyaneshwari closed 1 year ago

Abujdnyaneshwari commented 1 year ago

Summary:

Added test ids

Changelog:

Added test ids

Summary by CodeRabbit

coderabbitai[bot] commented 1 year ago

Walkthrough

The changes primarily focus on enhancing the testability of the application by ensuring unique identifiers for each element in a list. Additionally, minor adjustments have been made to improve code readability and user interface text.

Changes

File Path Summary
.../sales/common_components/NotesCard.kt Appended index to userAvatarTestId, testTag, and contentDescription properties for unique identification during testing.
.../sales/common_components/SearchNameSheetContainer.kt Changed "No results found" text to "No Result Found" and added a new line for better readability.
.../sales/common_components/SearchUserName.kt Set contentDescription property of a Modifier to the value of searchNameTestId.
.../sales/common_components/UserAvatar.kt Changed testTag property from a string literal to the variable userAvatarTestId.
.../sales/screens/AccountDetailsScreen.kt Moved index variable declaration outside the loop to improve code readability and eliminate unnecessary redeclaration.

🐇💻

In the land of code where the shadows lie,

We make our changes under a cloudless sky.

Unique IDs for all, no two the same,

In the grand scheme of tests, it's not just a game.

Readability improved, with a touch so light,

Our code now dances into the night. 🌙✨

Tips ### Chat with CodeRabbit Bot (`@coderabbitai`) - Mention `@coderabbitai` in any *review comment* for bot assistance. The bot stays silent unless called upon. - Note: Review comments are made on code diffs or files, not on the PR overview. ### Pause Incremental Reviews - Insert `@coderabbitai: ignore` in the PR description to halt the bot's ongoing reviews. Remove the line to resume.