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

DBtest/findById #21

Merged
merged 9 commits into from
Dec 21, 2023
Merged

DBtest/findById #21

merged 9 commits into from
Dec 21, 2023

Conversation

sugao-2211
Copy link
Owner

@sugao-2211 sugao-2211 commented Dec 18, 2023

CRUD処理すべてを備えたREST APIの作成

概要

消耗品の在庫一覧についてAPIを作成しました。
今回は試験研究などで使用される試薬を題材にして在庫一覧表を作成しました。

今回はfindById()メソッドのDB単体テストになります。
コミット一覧

これまでの実施結果はこちら
https://github.com/sugao-2211/stockListProject/blob/main/README.md

プロジェクトの進捗を作成しました。(作業しながら作成中)
https://github.com/users/sugao-2211/projects/1
今回のタスク: #19 (comment)

今回実施箇所
findById()メソッドのDB単体テスト

  • findById()メソッドのDB単体テスト

    • idを指定したときに該当する在庫情報が取得できること
    • 存在しないidを指定したときに空のOptionalが返されること

    @Test
    @DataSet(value = "datasets/stockList.yml")
    @Transactional
    void idを指定したときに該当する在庫情報取得できること() {
    Optional<StockList> stockList = stockListMapper.findById(1);
    assertThat(stockList)
    .contains(new StockList(1, "メタノール", "HPLC用", 3, "L", LocalDate.of(2023, 5, 24)));
    }
    @Test
    @DataSet(value = "datasets/stockList.yml")
    @Transactional
    void 存在しないidを指定したときにのOptionalがされること() {
    Optional<StockList> stockList = stockListMapper.findById(99);
    assertThat(stockList).isEmpty();
    }

  • 実行結果

    • idを指定したときに該当する在庫情報が取得できること
      スクリーンショット 2023-12-18 22 22 07
    • 存在しないidを指定したときに空のOptionalが返されること
      スクリーンショット 2023-12-18 22 22 37

今回の実施結果を下記のREADMEにも反映しています。
("Read処理の実装"の中にあります。)
https://github.com/sugao-2211/stockListProject/blob/DBtest/findById/README.md

@DataSet(value = "datasets/stockList.yml")
@Transactional
void idを指定したときに該当する在庫情報が取得できること() {
Optional<StockList> stockList = stockListMapper.findById(1);

Choose a reason for hiding this comment

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

以前も話した気はしますが、stockListだとList型だと一瞬錯覚するのでよくないですね...

Copy link
Owner Author

Choose a reason for hiding this comment

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

以前は大元の変更になるので、そのままにさせて欲しいとお話ししてそのときはOKをいただきましたが
改めてコメントをいただいたのを受けて対応いたしました。

コミットが細くなってしまい申し訳ありません。
(コミットの内容はスクリーンショットをご確認ください。)
ご確認お願いいたします。

スクリーンショット 2023-12-20 22 05 19

@sugao-2211 sugao-2211 merged commit 5c95569 into main Dec 21, 2023
@sugao-2211 sugao-2211 mentioned this pull request Dec 21, 2023
14 tasks
@sugao-2211 sugao-2211 deleted the DBtest/findById branch December 21, 2023 00:56
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