JUnitの単体テストの結果がうまく理解できません

Question

現在「スッキリわかるJava入門実践編」でJavaの学習を進めているものです。
単体テストをJUnitを使って行っているのですが、以下のgetDateFuture()の部分が失敗となってしまいます。

package test.java;

import static org.junit.jupiter.api.Assertions.*;
import org.junit.jupiter.api.*;
import java.time.*;
import java.time.format.*;
import main.java.Movie;

public class MovieTest {

	@Test
	public void movie1(){
		Movie m = new Movie();
	}
	
	@Test
	public void movie2(){
		Movie m = new Movie("まきの");
		
	}
	@Test
	public void getTitle(){
		Movie m = new Movie();
		m.setTitle("ゴジラ");
		assertEquals("ゴジラ", m.getTitle());
	}
	
	@Test
	public void getDate(){
		Movie m = new Movie();
		DateTimeFormatter fmt = 
			DateTimeFormatter.ofPattern("yyyy/MM/dd");
		LocalDate ldate = LocalDate.parse("2020/06/11", fmt);
		m.setReleaseDate(ldate);
		assertEquals(ldate, m.getReleaseDate());
	}
	@Test
	public void getTitleNull(){
		Movie m = new Movie();
		assertThrows(IllegalArgumentException.class, () 
				-> m.setTitle(null));
	}
	@Test
	public void getDateNull() {
		Movie m = new Movie();
		assertThrows(IllegalArgumentException.class, ()
				-> m.setReleaseDate(null));
	}
	@Test
	public void getDateFuture(){
		Movie m = new Movie();
		DateTimeFormatter fmt = 
			DateTimeFormatter.ofPattern("yyyy/MM/dd");
		LocalDate ldate = LocalDate.parse("2200/06/11", fmt);
		try {
			m.setReleaseDate(ldate);
		}catch(Exception e) {
			return;
		}
		fail();
	}
}

そして、このテストを実施しているのが以下のファイルです。

package main.java;
import java.time.*;

public class Movie{
	String name;
	String title;
	LocalDate ldate;
	
	public Movie() {
		
	}
	public Movie(String name) {
		this.name = name;
	}
	
	public String getTitle() {
		return this.title;
	}
	
	public void setTitle(String title) {
		if(title == null) {
			throw new IllegalArgumentException("タイトルが未入力です");
		}
		this.title = title;
	}
	
	public LocalDate getReleaseDate() {
		return this.ldate;
	}
	
	public void setReleaseDate(LocalDate ldate) {
		LocalDate l = LocalDate.now();
		
		if(ldate == null ) {
			throw new IllegalArgumentException("公開日が未入力です");
		}
		if(l.compareTo(ldate) == -1) {
			throw new IllegalArgumentException("公開日が未来です");
		}
		
		this.ldate = ldate;
	}
}

テストした後の画面が以下のものです。

試しに失敗している箇所のfail()をコメントアウトして再度実施したところ、失敗は発生せず正常に終了しました。

なぜfail()を記述していると良くないのでしょうか？またfail()の代わりとなるものがあれば押してていただきたいと思います。よろしくお願いします。

Answer 1

なぜfail()を記述していると良くないのでしょうか？

「良くない」のではなく、 fail() が実行される状況になったら「テストを失敗したとみなす」と記述してあるだけです。
正常に実行されていればここは呼び出させるはずがない、という場所に記述すると間違っていることが検知できるので、使いみちはあるのです。

https://junit.org/junit5/docs/5.7.2/api/org.junit.jupiter.api/org/junit/jupiter/api/Assertions.html#fail()

Comments

[hirono_suke]

私自身がJavaの知識が浅く、似たような記述があった例題をそのまま書き写した結果failメソッドを欠いたに過ぎなかったので、matherさんのおかげで今回の疑問もfailメソッドの本来の使い方も理解することができました。ありがとうございました。

Answer 2

テストをするのはバグを見つけるためです。
ですから、ついでにそのままデバッグしては如何でしょう。
…テストコードが何をテストしているのかを理解していなくてはなりませんが。

Answer 3

なぜfail()を記述していると良くないのでしょうか？またfail()の代わりとなるものがあれば押してていただきたいと思います。

逆に質問したいのですが・・・

なぜ、failメソッドを記述しようとしていますか？
fail()メソッドの意味を理解していますか？

テストを失敗させたいからfail()を書いているんじゃないんですか？

現在「スッキリわかるJava入門実践編」は読んでないのですが、本当に
そのようなテストコードを書いているのでしょうか？
もし書いてあるなら、エラッタで修正情報がでているかもしれません。