テストコードを書いてみたのですが、理解があっているのかを確認したいです。

Question

laravel勉強中のものです。

調べながら初めてテストを書いたのですが、流れとしてこれで良いのか不安なので質問させて頂きます。

私が記載しているコメントアウトは処理の説明(自分の理解)を書いているのですが、あっていますでしょうか？

<?php

namespace Tests\Feature;

use Illuminate\Foundation\Testing\RefreshDatabase;
use Illuminate\Foundation\Testing\WithFaker;
use Tests\TestCase;
use App\User;

class LoginTest extends TestCase
{
    use RefreshDatabase;

    protected $user;

    public function setUp(): void
    {
        parent::setUp();

        // テストユーザ作成
        $this->user = factory(User::class)->create();
    }
    /**
     * A basic feature test example.
     *
     * @return void
     */
    public function test_正しいパスワードの場合()
    {
        $response = $this->get('/login');
        $response->assertStatus(200);

        // ログインする
        $response = $this->post('/login', ['email' => $this->user->email, 'password' => 'Test1234']);
        // リダイレクトでページ遷移してくるのでstatusは302
        $response->assertStatus(302);
        // リダイレクトで帰ってきた時のパス
        $response->assertRedirect('/');
        // このユーザーがログイン認証されているか
        $this->assertAuthenticatedAs($this->user);

    }

    public function test_間違ったパスワードの場合()
    {
        $response = $this->get('/login');
        $response->assertStatus(200);

        // パスワードが正しく無い状態でログイン
        $response = $this->post('/login', ['email' => $this->user->email, 'password' => 'Test123']);
        // リダイレクトで戻ってくる。
        $response->assertStatus(302);
        // 戻ってきた時はログインページにいる事
        $response->assertRedirect('/login');
        // 失敗しているので認証されていない事
        $this->assertGuest();
        
    }

    public function test_ログアウトが正しくできるか()
    {
        // ログイン状態の作成
        $response = $this->actingAs($this->user);
        $response = $this->get('/');
        $response->assertStatus(200);
        // ログアウト処理をする
        $this->post('logout');
        // ログアウト出来たら200番が帰ってきているか
        $response->assertStatus(200);
　　　　　// ログインページにいる事。ここは確認の仕方はこれで大丈夫でしょうか。
　　　　　$response = $this->get('/login');
　　　　　$response->assertStatus(200);
        // 認証されていないことを確認
        $this->assertGuest();
    }
}

<?php

/** @var \Illuminate\Database\Eloquent\Factory $factory */

use App\User;
use Faker\Generator as Faker;
use Illuminate\Support\Str;

/*
|--------------------------------------------------------------------------
| Model Factories
|--------------------------------------------------------------------------
|
| This directory should contain each of the model factory definitions for
| your application. Factories provide a convenient way to generate new
| model instances for testing / seeding your application's database.
|
*/

$factory->define(User::class, function (Faker $faker) {
    return [
        'name' => $faker->name,
        'email' => $faker->unique()->safeEmail,
        'email_verified_at' => now(),
        'password' => bcrypt('Test1234'),
        'remember_token' => Str::random(10),
    ];
});

以下確認したい点です。

・リダイレクト処理は302、ajaxなどapiリクエストはJsonを付けて書き、statusは200が返ってくる。
・現在そのページにいる事を確認したい場合はget(url)とassertStatus(200)で確認する。(他の確認方法はありますか？)

参考にしたサイト
参考1
参考2

laravel7.x

Comments

[m.ts10806]

どこまでの確度を意識して作るかにもよります。
ある程度のアプリケーションでということでしたら、「単体テスト」として項目を洗い出してテスト仕様書を作るところからになりますし、そのインプットとしては詳細設計書があります。

[tenlife]

コメントありがとうございます。
確かにそもそものどこまでの？と言う部分がぬけていました。
オリジナルアプリを作っているので、まずは今回のような単体テスト、httpアクセスのテストなどの基本的な部分をメインで進める。この認識で大丈夫でしょうか。

[m.ts10806]

>httpアクセスのテストなどの基本的な部分をメインで進める。この認識で大丈夫でしょうか。

結局は「決める」ことなので、自身で箇条書きレベルで良いので想定するテストケースを洗い出して「ここをテストする」と決めてその通りにテストを書いていくだけかと思います。
基本はINPUT-OUTPUTが合っているかを確認します。
「どういう値を与えたらどういう結果になるか」ですね。

[tenlife]

返信ありがとうございます。
承知しました。
まずはどこまでのテストを実装したいのか？を明確にしていく所から始めていきます。

[m.ts10806]

タグにあるような「テスト駆動開発」であれば、そもそも設計段階からテスト項目が決まっているものと思います。