test: amountToHangul 테스트 추가 #135

fe-dudu · 2024-06-25T12:23:17Z

Overview

amountToHangul 함수의 error case test를 추가하여 Uncovered Line을 제거합니다.

PR Checklist

I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

changeset-bot · 2024-06-25T12:23:20Z

⚠️ No Changeset found

Latest commit: 0db609b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

vercel · 2024-06-25T12:23:20Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
es-hangul	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 3, 2024 6:27am

src/amountToHangul.spec.ts

okinawaa · 2024-06-29T12:00:11Z

src/amountToHangul.spec.ts

+
+  it('숫자로 된 금액이 80글자를 넘을 시 에러 발생', () => {
+    const longNumberString = '1'.repeat(81);
+    assert.throws(() => amountToHangul(longNumberString), Error, `convert range exceeded : ${longNumberString}`);


이 프로젝트에서 사용하는 vitest가 제공해주는 toThrow 메서드가 아닌, assert를 사용하면 어떠한 이점이 있나요?
제가 생각한건 다음과 같아요.

expect(() => amountToHangul(longNumberString)).toThrow(`convert range exceeded : ${longNumberString}`); });

특별한 이점은 없고 combineHangulCharacter.spec.ts, hangul.spec.ts에서 node 기본 모듈인 assert의 throws 메서드를 사용하다보니 동일하게 사용하려고 했습니다. vitest에서 제공해주는 toThrow 메서드를 사용하는 것이 일관성이 있을 것 같아요 !

combineHangulCharacter.spec.ts, hangul.spec.ts에서도 assert.throws대신 toThrow 메서드를 사용하도록 변경되면 좋을 것 같은데 추가로 PR 올려도 괜찮을까요 ?

@okinawaa 님 이건 좀 다른 주제이긴 한데요. vitest에서 제공하는 toThrow는 toThrowError의 별칭인데요. 현재 테스트 코드 대다수에서 toThrowError를 사용하고 있는데 하나로 통일하여 일관성을 지키는게 좋지 않을까 싶은데 어떠신가요?

vitest-dev/vitest#2570

두 메서드는 동일하지만 error를 테스트할때 toThrowError를 사용하는 것이 직관적이겠네요 !

okinawaa

감사합니다

okinawaa · 2024-07-03T06:08:39Z

conflic해결만 부탁드립니다 @fe-dudu

* test: amountToHangul 테스트 추가 * test: amountToHangul 테스트 코드 수정 * test: amountToHangul 테스트 코드 스타일 변경 * test: amountToHangul test 개선 * fix: assert.throws대신 vitest toThrow로 메서드 변경 * fix: vitest toThrowError 메서드로 변경 --------- Co-authored-by: kinndohyun <dohyun.kim@carbonsaurus.com> Co-authored-by: 박찬혁 <pgg6713@gmail.com>

* fix. 겹모음과 관련된 이상 현상들 수정 * fix. fix index * fix. 오타 수정 Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix. 오타 수정 Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix. 업데이트 로직 반영 * test: internal 테스트 코드를 작성합니다 (#137) * test: internal 테스트 코드를 작성합니다 * test: vitest에서 제공해주는 toThrowError로 변경 --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * docs: josa에 관련된 문서에 잘못된 부분을 수정합니다 - '이에/에' 삭제 Co-authored-by: 박찬혁 <pgg6713@gmail.com> * chore: version packages (#143) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * feat: 한글 문자열에 대한 검증, assert, parsing 함수를 구현합니다. (#136) * feat: 한글 문자열인지 boolean 반환 함수 * feat: 한글 문자열인지 assert 함수 * feat: stringify * test: isHangulString, assertHangulString 테스트 코드 * feat: parse 함수 * test: parse 함수 테스트 * refactor: naming 통일 * feat : 문장의 각 단어 중 첫 문자만 뽑는 함수추가 ( #128 이슈에 대한 ) (#133) * feat : 문장의 각 단어 중 첫 문자만 뽑는 함수추가 * test 및 함수 추가 * add : 한글 문장인지 여부 판별 함수 추가 * fix : 한글 문장인지 여부 판별 기저 및 오류 추가 / arg 이름 변경 test 추가 * fix: src/_internal/hangul.ts Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix : rename function getFirstHangulLetters -> getHangulAcronym * fix : rename function export function isHangulOnly로 변경 * fix : lint error * fix : index에 추가 * chore : doc 추가 * fix : 문서화 한글 영어 바뀐거 바로 변경 * fix : isHangul로 대체 #136 으로 * chore : doc수정 * Update docs/src/pages/docs/api/getHangulAcronym.en.mdx Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix : Update hangul.ts 안쓰는 메소드 삭제 * Update getHangulAcronym.ko.mdx * Update getHangulAcronym.en.mdx * Update src/getHangulAcronym.ts Co-authored-by: 박찬혁 <pgg6713@gmail.com> * Update src/getHangulAcronym.ts Co-authored-by: 박찬혁 <pgg6713@gmail.com> * Update getHangulAcronym.ts * Update getHangulAcronym.spec.ts * Create fair-brooms-drive.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * chore: version packages (#145) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * feat: 문자열에서 한글만 반환하는 extractHangul을 구현합니다. (#130) * feat: parseHangul * fix: parseHangul의 이름을 extractHangul로 수정 및 테스트 코드 보완 * test: 일관된 테스트 코드 작성이 될 수 있도록 describe 설명 수정 * test: 테스트 코드 수정 * docs: extractHangul의 문서 작성 * test: 테스트 문구 수정 * fix: index.ts에 export 추가 * Create fresh-students-sit.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * chore: version packages (#146) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * fix : getHangulacronym함수를 acronymizeHangul 메서드로 대체합니다. (#148) * Update and rename getHangulAcronym.ts to acronymizeHangul.ts * Update and rename getHangulAcronym.spec.ts to acronymizeHangul.spec.ts * Update index.ts * Update and rename getHangulAcronym.en.mdx to acronymizeHangul.en.mdx * Update and rename getHangulAcronym.ko.mdx to acronymizeHangul.ko.mdx * Create odd-squids-sin.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * chore: version packages (#150) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 (#144) * fix: amountToHangul이 소수점, 숫자도 대응할 수 있도록 수정 * docs: amountToHangul의 영어 문서 작성 * Create famous-cheetahs-sneeze.md * Update famous-cheetahs-sneeze.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix: packlint ESLint error (#149) * test: utils 기능들의 테스트 케이스를 개선합니다 (#151) * test: hasBatchim 함수의 테스트 케이스를 추가합니다 * test: hasSingleBatchim의 테스트 케이스를 추가합니다 * fix: nextra theme asset 개선 (#152) * chore: version packages (#153) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * fix: 패키지가 노출하는 인터페이스를 명확히 하기 위해 index.ts를 named export로 수정합니다 (#157) * fix: exposing public apis strictly * Create wild-cows-juggle.md * chore: version packages (#160) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * docs: CONTRIBUTING.md 파일에 메서드 컨벤션을 작성합니다. (#132) * Update CONTRIBUTING.md * fix * write contribution.md Co-authored-by: Jonghyeon Ko <61593290+manudeli@users.noreply.github.com> Co-authored-by: Dongkyu Kim <55759551+po4tion@users.noreply.github.com> * Update .github/CONTRIBUTING.md Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> * Update .github/CONTRIBUTING.md Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> * Update .github/CONTRIBUTING.md Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> --------- Co-authored-by: Jonghyeon Ko <61593290+manudeli@users.noreply.github.com> Co-authored-by: Dongkyu Kim <55759551+po4tion@users.noreply.github.com> Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> * test: amountToHangul 테스트 추가 (#135) * test: amountToHangul 테스트 추가 * test: amountToHangul 테스트 코드 수정 * test: amountToHangul 테스트 코드 스타일 변경 * test: amountToHangul test 개선 * fix: assert.throws대신 vitest toThrow로 메서드 변경 * fix: vitest toThrowError 메서드로 변경 --------- Co-authored-by: kinndohyun <dohyun.kim@carbonsaurus.com> Co-authored-by: 박찬혁 <pgg6713@gmail.com> * fix: 소수점 추가 후 발생한 '영'읽기 버그 수정 (#159) * fix: 소수점 추가 후 발생한 '영'읽기 버그 수정 * Create big-cups-call.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> * docs: acronymizeHangul 리턴 타입 표기 수정 (#161) * chore: version packages (#163) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * fix: 일관된 toThrowError 메서드 사용 (#166) Co-authored-by: kinndohyun <dohyun.kim@carbonsaurus.com> * chore(eslint): 의도치 않은 naming을 제거하기 위해 cspell 추가 (#164) * chore(eslint): add cspell to check bad naming * chore: update * Create grumpy-singers-love.md --------- Co-authored-by: 박찬혁 <pgg6713@gmail.com> Co-authored-by: Dongkyu Kim <po4tion0429@gmail.com> Co-authored-by: Jaemin Kim <jmkim326@gmail.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: seunghee <sa02045@naver.com> Co-authored-by: taehyun <126179088+KNU-K@users.noreply.github.com> Co-authored-by: SeongMin Kim <86355699+Collection50@users.noreply.github.com> Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com> Co-authored-by: Jonghyeon Ko <61593290+manudeli@users.noreply.github.com> Co-authored-by: Dongkyu Kim <55759551+po4tion@users.noreply.github.com> Co-authored-by: 김도현 <101170386+fe-dudu@users.noreply.github.com> Co-authored-by: kinndohyun <dohyun.kim@carbonsaurus.com> Co-authored-by: Song Hyo Jin <shj@xenosi.de> Co-authored-by: wnhlee <40269597+2wheeh@users.noreply.github.com>

kinndohyun added 3 commits June 25, 2024 20:55

test: amountToHangul 테스트 추가

f501224

test: amountToHangul 테스트 코드 수정

8f72144

test: amountToHangul 테스트 코드 스타일 변경

3675cd1

fe-dudu requested review from okinawaa and raon0211 as code owners June 25, 2024 12:23

vercel bot had a problem deploying to Preview June 25, 2024 12:24 Failure

po4tion reviewed Jun 27, 2024

View reviewed changes

src/amountToHangul.spec.ts Outdated Show resolved Hide resolved

test: amountToHangul test 개선

4f7a721

vercel bot had a problem deploying to Preview June 28, 2024 11:53 Failure

okinawaa reviewed Jun 29, 2024

View reviewed changes

fix: assert.throws대신 vitest toThrow로 메서드 변경

5ba6490

vercel bot had a problem deploying to Preview June 30, 2024 04:29 Failure

fix: vitest toThrowError 메서드로 변경

60bc00f

vercel bot had a problem deploying to Preview June 30, 2024 08:45 Failure

okinawaa previously approved these changes Jul 2, 2024

View reviewed changes

Merge branch 'main' into main

0db609b

okinawaa dismissed their stale review via 0db609b July 3, 2024 06:25

vercel bot deployed to Preview July 3, 2024 06:27 View deployment

okinawaa approved these changes Jul 3, 2024

View reviewed changes

okinawaa merged commit 9b256b9 into toss:main Jul 3, 2024
9 checks passed

fe-dudu mentioned this pull request Jul 3, 2024

Fix: 일관된 toThrowError 메서드 사용 #166

Merged

1 task

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test: amountToHangul 테스트 추가 #135

test: amountToHangul 테스트 추가 #135

fe-dudu commented Jun 25, 2024 •

edited

Loading

changeset-bot bot commented Jun 25, 2024 •

edited

Loading

vercel bot commented Jun 25, 2024 •

edited

Loading

okinawaa Jun 29, 2024

fe-dudu Jun 30, 2024

po4tion Jun 30, 2024 •

edited

Loading

fe-dudu Jun 30, 2024

okinawaa left a comment

okinawaa commented Jul 3, 2024

test: amountToHangul 테스트 추가 #135

test: amountToHangul 테스트 추가 #135

Conversation

fe-dudu commented Jun 25, 2024 • edited Loading

Overview

PR Checklist

changeset-bot bot commented Jun 25, 2024 • edited Loading

⚠️ No Changeset found

vercel bot commented Jun 25, 2024 • edited Loading

okinawaa Jun 29, 2024

Choose a reason for hiding this comment

fe-dudu Jun 30, 2024

Choose a reason for hiding this comment

po4tion Jun 30, 2024 • edited Loading

Choose a reason for hiding this comment

fe-dudu Jun 30, 2024

Choose a reason for hiding this comment

okinawaa left a comment

Choose a reason for hiding this comment

okinawaa commented Jul 3, 2024

fe-dudu commented Jun 25, 2024 •

edited

Loading

changeset-bot bot commented Jun 25, 2024 •

edited

Loading

vercel bot commented Jun 25, 2024 •

edited

Loading

po4tion Jun 30, 2024 •

edited

Loading