Skip to content

feat: Add optional Icon field to Goal model and implement GetForUser test - #348

Open
SaurabhForge wants to merge 1 commit into
fencer-so:mainfrom
SaurabhForge:feat/add-goal-icon
Open

feat: Add optional Icon field to Goal model and implement GetForUser test#348
SaurabhForge wants to merge 1 commit into
fencer-so:mainfrom
SaurabhForge:feat/add-goal-icon

Conversation

@SaurabhForge

Copy link
Copy Markdown

Summary

This pull request adds an optional icon property to the Goal model and implements test coverage for GetForUser.

Changes

  • Goal.cs: Added public string? Icon { get; set; } to the Goal model.
  • GoalControllerTests.cs: Implemented GetForUser() test verifying non-null return, proper assignment to Goal, and user ID matching.
  • FakeCollections.cs: Added UserId = "1" to fake goal entries to support testing user-specific goal retrieval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant