keinos / tmc-tests

✅ TMC-Data format regulation check commands.

Profile of KEINOS

• Name: KEINOS
• Taxonomy: Japonicus Chilangosis Mexicanus
• Details: Look deep inside KEINOS
• Open Sources: https://github.com/KEINOS @ GitHub
• Articles: https://qiita.com/KEINOS @ Qiita

Qiita Status

GitHub Status

tmc-tests's Issues

検証データの複数受付。REG-002, REG-003 がプロトコルに準拠していない

確認スクリプトのプロトコルの「入力」要件によると、複数の検証データの同時受付は禁止されていますが、REG-002 および REG-003 は複数受付も処理しています。

確認コマンドによってデータの渡し方が異なることになるので、他の確認コマンドの実装前に統一した方がいいと思われます。

REG-002

TMC-TESTS/REG-002

Lines 37 to 46 in 11c8b02

	if ($is_stdin) {
	while ($name_file = fgets(STDIN)) {
	$result = (is_uniformed_filename($name_file) && $result);
	}
	} else {
	$name_files = explode(PHP_EOL, $name_files);
	foreach ($name_files as $name_file) {
	$result = (is_uniformed_filename($name_file) && $result);
	}
	}

REG-003

TMC-TESTS/REG-003

Lines 27 to 33 in 11c8b02

	if ($is_stdin) {
	while ($input_line = fgets(STDIN)) {
	$input_lines[] = $input_line;
	}
	} else {
	$input_lines = explode(PHP_EOL, $argument);
	}

REG-002: 無効な（存在しない）日付でもパスする

2月30日などの存在しない無効な日付でもパスします。

$ ./REG-002 TMC-20190230.json
$ echo $?
0

有効な放送日は検知できなくても、少なくとも実在する日付であるかのチェックは必要かと思われます。

提案

function validateDate($date, $format = 'Ymd')
{
    $d = DateTime::createFromFormat($format, $date);
    return $d && $d->format($format) == $date;
}

• オンラインで動作確認する @ paiza.IO
• 出典：「checkdate | User Contributed Notes」@ PHP Manural
• 参考文献：「【PHP】DateTimeクラスでcheckdate()より汎用性のある日付チェックを行う」@ Qiita

ヘルプが実装されていません。REG-001〜003

引数なしの確認コマンド呼び出しの場合は、ヘルプを表示しないといけないのですが、REG-001, REG-002, REG-003 には実装されていません。

エラーのみで、必要な引数（入力）情報がわかりません。

$ ./REG-001 || echo $?
Argument missing.
1
$ ./REG-002 || echo $?
Argument missing.
2
$ ./REG-003 || echo $?
Argument missing.
3

REG-000: 標準入力受付しない

サンプル（テンプレート）の REG-000 が標準入力で検証データを受け付けません。

$ ./REG-000 'Hello World!'
$ echo $?
0
$ echo 'Hello World!' | ./REG-000 -
NG (Bad argument. Must be \'Hello World!\'. Given: \'-\')
$ echo $?
1

テストも検証していません。（引数渡しのみテストしています）

TMC-TESTS/tests/REG-000_Test.php

Lines 6 to 23 in 6ae3b62

	public function testReg000()
	{
	$path_dir_reg = '..';
	$name_file_reg = 'REG-000';
	$path_file_reg = $path_dir_reg . '/' . $name_file_reg;
	$message = 'Hello World!';
	$lastline = exec("${path_file_reg} '${message}' 2>&1", $output, $return_var);
	$this->assertTrue($return_var === 0, $lastline);
	$message = 'Hello World! ';
	$lastline = exec("${path_file_reg} '${message}' 2>&1", $output, $return_var);
	$this->assertTrue($return_var === 1, $lastline);
	$message = 'Hell World!';
	$lastline = exec("${path_file_reg} '${message}' 2>&1", $output, $return_var);
	$this->assertTrue($return_var === 1, $lastline);
	}