Skip to content

test: add test for NewSubscriber struct#185

Merged
reddevilmidzy merged 1 commit intomainfrom
test
Jul 16, 2025
Merged

test: add test for NewSubscriber struct#185
reddevilmidzy merged 1 commit intomainfrom
test

Conversation

@reddevilmidzy
Copy link
Owner

♟️ What’s this PR about?

NewSubscriber에 대한 간단한 테스트 추가하였습니다 (커버리지 확인용)

🔗 Related Issues / PRs

Copilot AI review requested due to automatic review settings July 16, 2025 15:02
@github-actions github-actions bot added this to the 5차 MVP 구현 milestone Jul 16, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a unit test for the NewSubscriber struct to verify its constructor and getter methods.

  • Adds a test for creating a NewSubscriber and checking its email, repository_url, and branch getters.
  • Uses valid sample values to assert equality of returned fields.
Comments suppressed due to low confidence (1)

rook/src/domain/new_subscriber.rs:51

  • Consider adding a test case where branch is None to ensure branch() correctly returns None when no branch is provided.
        assert_eq!(subscriber.branch(), branch.as_ref());

@coveralls-official
Copy link

Pull Request Test Coverage Report for Build 16323105496

Details

  • 9 of 9 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 74.414%

Totals Coverage Status
Change from base Build 16322081308: 0.4%
Covered Lines: 1588
Relevant Lines: 2134

💛 - Coveralls

@reddevilmidzy
Copy link
Owner Author

할렐루야

@reddevilmidzy reddevilmidzy merged commit a5c9369 into main Jul 16, 2025
6 checks passed
@reddevilmidzy reddevilmidzy deleted the test branch July 16, 2025 15:10
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.

2 participants