質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.61%

  • PHP

    19904questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • テスト駆動開発

    64questions

    テスト駆動開発は、 プログラム開発手法の一種で、 プログラムに必要な各機能をテストとして書き、 そのテストが動作する必要最低限な実装を行い コードを洗練させる、といったサイクルを繰り返す手法の事です。

1テストメソッドに複数のアサーションを入れるのは分からずらいと思うのですが、どうでしょうか?

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 387

mitsuru793

score 149

言語: PHP
テストFW: Codeception

<?php
// tests/unit/MyClassCest.php

class MyClassCest
{
  public funciton myMethodName(UnitTester $I)
  {
    // test 1
    // Doing
    $I->assertSame(true);

    // test 2
    // Doing
    $I->assertSame(true);
  }
}

上記のようにクラス単位でテストファイルを作り、メソッド単位で、テストメソッドを追加すると、
どのメソッドにテストが追加されているか分かりやすくなります。
しかし、アサーションが失敗するとどこで落ちているか分かりづらいです。
アサーションの最後の引数に$descriptionが指定出来るのですが、コードが読みづらくなる気がします。

1テストメソッド、1アサーションにした場合は、どのメソッドがテストされているかコード上分かりづらいと思います。
そこで下記のファイル構成を考えてみました。

<?php
// tests/unit/MyClass/myMethodNameCest.php

namespace MyClass;

class meyMethodName
{
  public funciton test1(UnitTester $I)
  {
    // Doing
    $I->assertSame(true);
  }

  public function test2(UnitTester $I)
  {
    // Doing
    $I->assertSame(true);
  }
}

この方法なら使用するテストFWに依存もしないと思います。
このようにファイル構成は珍しいでしょうか?
1テストメソッドに複数のアサーションを入れるのは、問題ないとされているのか他の方の意見をお聞きしたいです。
よろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

1テストメソッドに複数アサーションが普通だと思います。なぜなら、1つのテストメソッドで確認すべき項目が複数あることあることはよくありますし、アサーションを1つに限定するのは難しいと思います。

また、テストメソッドの単位ですが、「メソッド単位で、テストメソッドを追加」というよりは、同じメソッドでもテストの観点によって分けた方がいいと思います。具体的には、メソッドの正常系と異常系でテストメソッドを分けたり、境界値テストだけを分けたりなどです。

自分は、普段CakePHPで開発しており、CakePHPで用意されているFWに従ってテストを書いていますが、アサーションが正しくなかった行数と値がわかれば、そんなにわかりにくいとは思いません。

後は、GitHubなどで、普段使っているFWやライブラリのテストを参考にするといいかもしれません。CakePHPの場合。ここにありますが、複数アサーションしていますね。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/26 22:11

    ご回答ありがとうございます。返信が遅れてしまいました🙏
    Cakeだとクラス単位でテストファイルをつくるのですね。

    > 同じメソッドでもテストの観点によって分けた方がいいと思います。
    もしhogeというメソットがあれば、testHogeWhenGivenNull, testHogeReturnsTrue,などの複数のテストメソッドに分割するということですよね。

    hogeに関してのテストメソッドを立て続けに書いてもらえれば良いのですが、順番バラバラに書かれた場合、hogeに関するどんなテストがあるのか見にくいなと思いました。
    それとテストファイルが500行を超えた時も大変です。

    > 1テストメソッドに複数アサーションが普通だと思います。なぜなら、1つのテストメソッドで確認すべき項目が複数あることあることはよくありますし、アサーションを1つに限定するのは難しいと思います。
    > 普段使っているFWやライブラリのテストを参考にするといいかもしれません。

    OSSをよく参考にしますが、1クラス、1テストファイルにしていますね。凄い長いテストファイルを見ることもあります。
    しかし、IDEなどで連携させるならこの方式が良いと思いました。

    icchiiさんの意見を参考に再度考えてみたところ、テストがでかくなりすぎるならそれはクラスを分割するタイミングかなと思えるようになり、1クラス1テストファイルにしたいと思います。
    laravelのCollectionクラスだと、配列のラッパーなのでかなりテストファイルが大きいですが、それは仕方がないことかなと思いました。

    意見を頂き、ありがとうございました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • PHP

    19904questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • テスト駆動開発

    64questions

    テスト駆動開発は、 プログラム開発手法の一種で、 プログラムに必要な各機能をテストとして書き、 そのテストが動作する必要最低限な実装を行い コードを洗練させる、といったサイクルを繰り返す手法の事です。

  • トップ
  • PHPに関する質問
  • 1テストメソッドに複数のアサーションを入れるのは分からずらいと思うのですが、どうでしょうか?