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

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

ただいまの
回答率

90.11%

PHPunitのテストコードの書き方について

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,482

tomona

score 30

XAMPP環境でCakePHPを使いながらPHPunitの導入テストを行っております。
しかし、公式マニュアルを読んでもいまいちテストコードの書き方がわかりません。

例えば公式マニュアル「Example 2.1 PHPUnit での配列操作のテスト」にて下記の通りテストコードが書いてありますが、テスト元のクラス(この場合ですとStackクラス)のコードが書かれていないため何を検証しているのか不明です。
https://phpunit.readthedocs.io/ja/latest/writing-tests-for-phpunit.html#

<?php
use PHPUnit\Framework\TestCase;

class StackTest extends TestCase
{
    public function testPushAndPop()
    {
        $stack = [];
        $this->assertSame(0, count($stack));

        array_push($stack, 'foo');
        $this->assertSame('foo', $stack[count($stack)-1]);
        $this->assertSame(1, count($stack));

        $this->assertSame('foo', array_pop($stack));
        $this->assertSame(0, count($stack));
    }
}

そこでQiita等の記事を参考に自分で下記の通りサンプルコードを作成しテストを実行しようとしましたがうまくいかない状態です。
手順4のエラーメッセージによるとaftertaxが定義されていないとのことですが、テスト元Class + Testという命名規則でテスト元のClassの定義が引き継がれるものと思っておりましたが違うのでしょうか。
テストコードの書き方が誤っているのでしょうか。ご教示いただけますと幸いです。

【テストの手順】

  1. Bake allコマンドを用いてBooksというクラスでMVCファイル及びテストコードのスクリプトを作成
  2. Booksコントローラー(Class BooksController)内に下記テスト元コードの関数を追加し消費税込み金額を計算する関数を作成
  3. コマンドでtestsディレクトリに自動生成されたBooksControllerTestに下記テストコードを記入し、引数に100を入れると108が返ることをassertSameで検証
  4. phpunitを実行するも「Call to undefined function App\Test\TestCase\Controller\aftertax()」と表示される。

【テスト元コード】

<?php
namespace App\Controller;

use App\Controller\AppController;

/**
 * Books Controller
 *
 * @property \App\Model\Table\BooksTable $Books
 *
 * @method \App\Model\Entity\Book[]|\Cake\Datasource\ResultSetInterface paginate($object = null, array $settings = [])
 */
class BooksController extends AppController
{
//追加した関数
public function aftertax($f){
        return $f*1.08;
}
//以下略
}
?>

【テストコード】

<?php
namespace App\Test\TestCase\Controller;

use App\Controller\BooksController;
use Cake\TestSuite\IntegrationTestCase;

/**
 * App\Controller\BooksController Test Case
 */
class BooksControllerTest extends IntegrationTestCase
{

//追加したテストコード
    public function testAftertax(){

        $this->assertSame(108,aftertax(100));
    } 
//以下略
}
?>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

テスト元のクラス(この場合ですとStackクラス)のコードが書かれていないため何を検証しているのか不明です。

んーと、提示されたテストコードの場合、Stackクラスに関係するテストコードは一切無いので、Stackクラスについて知る必要はないです。

$stack = [];               ← 配列$stackを新規生成
$this->assertSame(0, count($stack));  ← $stackの要素数が0であることをテスト

array_push($stack, 'foo');        ← $stackに新要素'foo'を追加
$this->assertSame('foo', $stack[count($stack)-1]); ← $stackの最後の要素が'foo'であることをテスト
$this->assertSame(1, count($stack));  ← $stackの要素数が1個であることをテスト

$this->assertSame('foo', array_pop($stack)); ← $stackから要素'foo'を削除
$this->assertSame(0, count($stack));      ← $stackの要素数が0である事をテスト

こんな感じです。つまり、ここで行っているテストはただの普通のPHPの処理に対するテストです。

んーで、CakePHPはよく知りませんが、普通のPHPUnitの経験で答えると多分ですけど

<?php
namespace App\Test\TestCase\Controller;

use App\Controller\BooksController;
use Cake\TestSuite\IntegrationTestCase;

/**
 * App\Controller\BooksController Test Case
 */
class BooksControllerTest extends IntegrationTestCase
{

//追加したテストコード
    public function testAftertax(){

        $this->assertSame(108,aftertax(100)); ← ここ
    } 
//以下略
}

これ、aftertax() は、クラスのメンバメソッドの意味になって、つまりBooksControllerTestクラスのメンバを探しに行っちゃいます。でも、それは存在しないので見つかりませんと言われるわけです。

ここで呼び出したいのは BooksControllerクラス の方のメソッドですから、このインスタンスを生成してそれを呼び出す必要があります。

そこで、こんな感じにします。

<?php
namespace App\Test\TestCase\Controller;

use App\Controller\BooksController;
use Cake\TestSuite\IntegrationTestCase;

require "BooksControllerクラスファイルへのパス/BooksControllerクラスファイル";

/**
 * App\Controller\BooksController Test Case
 */
class BooksControllerTest extends IntegrationTestCase
{

    private $obj;

    public function __construct(){
        $this->obj = new App\Controller\BooksController();
    }

//追加したテストコード
    public function testAftertax(){

        $this->assertSame(108, $this->obj->aftertax(100));
    } 
//以下略
}
?>

ネームスペースとかでエラー出るかもしれませんけど、大体こんな感じです。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/28 13:16

    回答ありがとうございます!
    クラスをインスタンスする必要があったのですね。

    下記にて実行できました。
    public function testAftertax(){
    $obj = new BooksController();
    $this->assertSame(108.0, $obj->aftertax(100));
    }

    キャンセル

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

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