Skip to content

Conversation

@shahad-97
Copy link
Contributor

@shahad-97 shahad-97 commented Jun 27, 2025

🛠️ Summary

Briefly explain what this pull request does.


📌 Description

Add test for CategoryTaskViewModel


✅️Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style (formatting, new lines, etc.)
  • Refactoring (no functional changes, renaming)
  • Other (describe the changes):

✅ Checklist:

  • The PR is to the develop branch
  • My code follows the style and architecture of the project
  • I have performed a self-review of my code
  • I tested the application on my computer and does everything as expected

🔍 Testing

Please check what has been verified before merging:

  • Code compiles successfully
  • Unit tests (if any) pass
  • Feature behaves as expected
  • Manually tested the affected logic


📎 Related Issues / Tasks

Link to any related issue or task:

  • Resolves: #issue_number (if applicable)

👥 Reviewers

Optional: Tag team members you want to review the changes
Example: @teammate1 @leadDev

raouf-kamel
raouf-kamel previously approved these changes Jun 27, 2025
Copy link
Contributor

@raouf-kamel raouf-kamel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice test coverage! Clean and focused implementation 👏

Copy link
Collaborator

@AimanYosofiUlt AimanYosofiUlt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice Job, Just see my comments

raouf-kamel
raouf-kamel previously approved these changes Jun 27, 2025
Copy link
Contributor

@raouf-kamel raouf-kamel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice test coverage! Clean and focused implementation 👏

AimanYosofiUlt
AimanYosofiUlt previously approved these changes Jul 3, 2025
@shahad-97 shahad-97 merged commit 3a7f3e2 into dev Jul 3, 2025
@AimanYosofiUlt AimanYosofiUlt deleted the test/category-task-viewmodel-test branch July 4, 2025 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants