Skip to content

feature/add-unittest-code-for-gugu-service 서비스 테스트 코드 추가 및 코드정리#3

Open
jungjinhwan0101 wants to merge 10 commits intomainfrom
feature/add-unittest-code-for-gugu-service
Open

feature/add-unittest-code-for-gugu-service 서비스 테스트 코드 추가 및 코드정리#3
jungjinhwan0101 wants to merge 10 commits intomainfrom
feature/add-unittest-code-for-gugu-service

Conversation

@jungjinhwan0101
Copy link
Collaborator

No description provided.

.gitignore Outdated
# PyPI configuration file
.pypirc

.idea No newline at end of file

Choose a reason for hiding this comment

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

End of line에 newline 추가 부탁드립니다.

requirements.txt Outdated

isort
ruff
pytest-cov No newline at end of file

Choose a reason for hiding this comment

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

End of line에 newline 추가 부탁드립니다.

README.md Outdated
Comment on lines +74 to +75


Choose a reason for hiding this comment

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

1줄만 뗘주시면 좋으 ㄹ것 같습니다.

### Coverage

```bash
PYTHONPATH=$(pwd)/src pytest --cov --cov-report=term --cov-report=html tests/

Choose a reason for hiding this comment

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

브랜치 커버리지 측정으로 업데이트 해주세요

README.md Outdated

```bash
PYTHONPATH=$(pwd)/src pytest --cov --cov-report=term --cov-report=html tests/
``` No newline at end of file

Choose a reason for hiding this comment

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

이 뒤에 report 읽는 방법도 적어주셔야겠는데요?

{"n": 2},
],
)
def test_invalid_non_integer_input(invalid_input):

Choose a reason for hiding this comment

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

type 적용 부탁드립니다.

@sonarqubecloud
Copy link

@FlabKenneth FlabKenneth changed the title feture/Gugu 서비스 테스트 코드 추가 및 코드정리 feature/add-unittest-code-for-gugu-service 서비스 테스트 코드 추가 및 코드정리 Jun 24, 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.

3 participants