Skip to content
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

Add test case for pill sheet group #551

Merged
merged 8 commits into from
Mar 30, 2022
Merged

Add test case for pill sheet group #551

merged 8 commits into from
Mar 30, 2022

Conversation

bannzai
Copy link
Owner

@bannzai bannzai commented Mar 28, 2022

Abstract

entity/PillSheetGroup の sequentialTodayPillNumber と sequentialLastTakenPillNumber のテストケースを追加

Why

この部分に変更を加えるため先にテストケースを追加した

Links

ref: #548

Checked

  • Analyticsのログを入れたか
  • 境界値に対してのUnitTestを書いた
  • パターン分岐が発生するWidgetに対してWidgetTestを書いた
  • リリースノートを追加した
    ; -x---------------------- >8 ------------------------
    ; Do not modify or remove the line above.
    ; Everything below it will be ignored.

Requesting a pull to bannzai:main from bannzai:add/test/pill_sheet_group

Write a message for this pull request. The first block
of text is the title and the rest is the description.

Changes:

168689a (bannzai, 81 seconds ago)
Remove unnecessary code

f31de9f (bannzai, 2 minutes ago)
Add test case

977ce4d (bannzai, 7 minutes ago)
Rename test case

9799bb3 (bannzai, 8 minutes ago)
to two pill sheet

e416ccc (bannzai, 8 minutes ago)
Add multiple pill sheet test case

f1e2eba (bannzai, 13 minutes ago)
Add test case

a31ff5e (bannzai, 17 minutes ago)
Rename to lastTakenPillNumber

9f6059e (bannzai, 19 minutes ago)
Add test case

@bannzai bannzai marked this pull request as ready for review March 30, 2022 07:17
@bannzai bannzai merged commit dca4930 into main Mar 30, 2022
@bannzai bannzai deleted the add/test/pill_sheet_group branch March 30, 2022 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant