Skip to content

Refactor Broadcast class and add unit tests #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

scubaninja
Copy link
Owner

@scubaninja scubaninja commented Oct 10, 2024

This pull request includes significant updates to the Broadcast model and adds unit tests to ensure the correctness of the model's behavior. The most important changes include adding documentation comments in Polish, modifying the Broadcast class structure, and introducing unit tests for the Broadcast class.

Updates to Broadcast model:

  • Added detailed XML documentation comments in Polish to the properties and methods of the Broadcast class.
  • Removed methods FromMarkdownEmail, FromMarkdown, and ContactCount from the Broadcast class.
  • Made the constructor of the Broadcast class private to prevent direct instantiation from outside the class.

Addition of unit tests:

  • Created a new BroadcastTest class in server/Models/tests.cs to test the default values, property setting, and handling of null values in the Broadcast class.

@scubaninja scubaninja self-assigned this Oct 10, 2024
@scubaninja scubaninja closed this May 26, 2025
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