Excel形式以外（YAML形式）のテストデータを読み込めるようにする

[kiyotis]

概要

NTF のテストデータ読み込み機構を、Excel 形式以外（YAML 形式）のテストデータでも動くようにする。あわせて、対応の過程で判明した本体側の3つの不整合を解消する。

いずれも Excel 形式の既存動作は変えない方針で実装し、既存テストの非破壊を負のコントロール（変異注入）で確認している。

変更内容

ソースコード（src/main 7ファイル、+265 / -42）

1. テストデータ解析のキャッシュ制御を Template Method で整理

TestDataParsingTemplate / DataFileParser / ListMapParser / TableDataParser

parse(directory, resource, id, saveCache) の saveCache = false は、従来は生テストデータのキャッシュ（TEST_DATA_CACHE）と PoiXlsReader のキャッシュだけを抑止しており、ListMapParser・TableDataParser・DataFileParser がそれぞれ持つ解析結果キャッシュには効いていなかった。

TestDataParsingTemplate#parse(String) をテンプレートメソッドにし、各サブクラスのキャッシュ処理を cacheEnabled() / tryLoadFromCache(String) / prepareResult() / storeToCache(String) のフックへ寄せた。saveCache = false、またはキャッシュを持たないサブクラスの場合は、構造解析の本体 doParse(String) を直接呼び、どのキャッシュも経由しない。

DataFileParser は「キャッシュにヒットしても値は再利用せず再構築する。ただし空の場合のみキャッシュ値を使う」という独自戦略を持つため、cachedParse(String) を override して従来の挙動を維持している。

担保: TestDataParsingTemplateTest#testSaveCacheControlsCaching — 同一リソースに対し 2 回 parse したとき、saveCache = true ではリーダの open 呼び出しが 1 回、saveCache = false では 2 回であることを表明する。

2. SendSyncSupport — YAML 形式でのテストデータ再読み込み

sendSyncTestData に fileExtensions を設定しない構成（YAML 形式相当）では、FilePathSetting#getFileIfExists が返すのはファイルではなくディレクトリである。従来のキャッシュ判定はその起点自身の lastModified() しか見ていなかったため、ディレクトリ配下のテストデータを更新しても再読み込みされなかった。

SendSyncSupport.createTestDataInfo の中で、起点を根とする Map<起点からの相対パス, lastModified> のスナップショット（getTimestampSnapshot(File)）を採取し、キャッシュ済みのスナップショットと equals で比較する方式に変更した。キーの区切りは環境非依存とするため / に固定し、ファイル・ディレクトリの双方をエントリに含める。スナップショットは getMessages より前に 1 度だけ採取し、比較にも記録にも同じ値を使う。

Excel 形式では getFileIfExists が返すのはファイルであり File#isDirectory() が false のため、スナップショットは起点自身の 1 エントリだけになり、実質的な判定は従来と変わらない。

修正は SendSyncSupport 内に閉じている（nablarch-core の FilePathSetting に変更なし）。追加したのは package private な getTimestampSnapshot(File)、private な collectTimestamp(File, String, Map)、および private static class TestDataInfo のフィールドのみで、公開 API の宣言差分は 0 件。

担保: SendSyncSupportTest に、ディレクトリ配下の更新（#testReloadTestDataInDirectory）、サブディレクトリ配下の更新（#testReloadWhenFileInSubDirectoryIsUpdated）、未来 mtime のファイルが同居する場合（#testReloadWhenFutureTimestampedFileExists）、読み込み中の書き換え（#testReloadWhenTestDataIsUpdatedWhileReading）、非ディレクトリ起点でのスナップショット内容（#testGetTimestampSnapshotForNotDirectory）を追加。fixture は @After で内容と mtime を復元する。

3. TableData#loadData() — 期待値のカラム名が 0 件でも DB を読む

loadData() は期待値のカラム名が 0 件のとき DB を読まずに 0 行を返していた。このため DB に行が残っていても Assertion.assertTableEquals が必ず PASS する偽陰性が生じていた。

カラム名の宣言は「検証対象カラムの絞り込み」であって「検証するかどうか」のスイッチではない。またYAMLの rows: [] はカラム名を書く場所が構造上存在しないため、YAML 形式では必ずこの経路に落ちる。

loadData() の中で、局所変数の colNames が 0 件のときに dbInfo.getColumns(tableName) を SELECT 対象カラムとするよう変更した。フィールド columnNames は書き換えない（getClone() が Object#clone() の浅いコピーで配列参照を期待値と共有するため）。getColumnNames()、準備データ投入、マスタデータ投入、変換ツール経路、表形式リーダ（src/main/java/nablarch/test/core/reader/ 配下）にはいずれも差分がない。現行動作へ戻すための設定項目・分岐フラグは追加していない。

担保: AssertionTest#testAssertTableEqualsWithEmptyColumnNames（DB に 1 行ある状態で失敗すること）と #testAssertTableEqualsWithEmptyColumnNamesOnEmptyTable（DB が空なら通ること）の 2 件で、分岐の両方向を固定する。

4. MasterDataSetUpper — Excel 形式以外のマスタデータファイルに対応

MasterDataSetUpper はマスタデータファイルの拡張子を見ずに POI へ渡し、シートを列挙していた。このため YAML 形式のマスタデータファイルを渡すと RuntimeException: test data file open failed.（cause は IllegalArgumentException: Your InputStream was neither an OLE2 stream, nor an OOXML stream）で失敗していた。

getAllTableData(File) の冒頭で Excel 形式かどうかを判定し、Excel 形式以外はシート列挙を経由せず TestDataParser#getSetupTableData(dir, 拡張子を除いたファイル名) を 1 回だけ呼ぶようにした。拡張子はリーダ／パーサ側が付与する責務であるため、リソース名には拡張子を含めない。

Excel 形式と判定する拡張子は 小文字の xls / xlsx のみとし、PoiXlsReader.isAcceptableFilename と同じ定義に揃えた。大小無視や xlsm の追加は行っていない。NTF が Excel テストデータとして認識する拡張子は既にこの定義であり、ここで別の定義を作ると NTF 内に Excel の定義が 2 つできるためである。

Excel 形式の経路は修正前と同一で、差分は getMasterFileNameWithoutSuffix の呼び出しをループ外へ引き上げた点のみ。追加したのは private な定数とメソッドだけで、公開 API の宣言差分は 0 件。

担保: MasterDataSetUpperTest#testSetUpMasterDataFromNonExcelFile — SETUP_TABLE ブロックを 2 つ持つ非 Excel 形式ファイルを MasterDataSetUpper.main に渡し、1 ファイルから 2 テーブル分（3 行・2 行）が投入されることを表明する。行数は既存 fixture と変えてあり、既存 fixture の混入では通らない。

利用プロジェクトへの影響（挙動変更）

1. TableData#loadData() は public であり、本変更は利用プロジェクトから見た挙動変更である。 期待値のカラム名が 0 件の TableData に対し、従来は DB を読まずに 0 行を返していたが、本変更後は DB を読む。本リポジトリの src/main では Assertion.java:81（Assertion.assertTableEquals）が唯一の呼び出し元である。nablarch-testing-yaml・nablarch-testing-converter・nablarch-testing-junit5・nablarch-testing-rest・nablarch-testing-integration も走査したが、製品コードからの呼び出しは無い。利用プロジェクトのコードは走査していない。

2. MasterDataSetUpper に .xlsm や大文字 .XLS のマスタデータファイルを渡していた場合、非 Excel 経路へ落ちる。 従来は拡張子を見ずに POI へ渡していたため、POI の内容判定により Excel として開けていた。本リポジトリでは該当 0 件（Git 追跡ファイルで xls 59 件・xlsx 2 件、xlsm・xlsb・大文字拡張子は 0 件。git ls-files | grep -iE '\.(xls|xlsx|xlsm|xlsb)$' で確認）だが、利用プロジェクトのテストデータは走査していない。

3. parse(..., saveCache = false) の経路では、解析結果キャッシュも参照・保存されなくなる。 従来この経路でもサブクラスの結果キャッシュは効いていた。saveCache = true（既定）の経路に変更はない。

テストコード（src/test 15 ファイル、+956）

SendSyncSupportTest（+635）、AssertionTest（+53）、TestDataParsingTemplateTest（+39）、MasterDataSetUpperTest（+25）、および Excel 形式以外のテストデータ読み込みを検証するためのテスト専用リーダ TsvTestDataReader（+87）と fixture。

ドキュメント（docs/pr75/）

• docs/testdata-converter-design.md — 変換ツール設計書
• docs/ntf-testdata-loading.md — テストデータ読み込み機構の解説
• docs/ntf-empty-table-assertion.md — 期待値 0 件テーブルの扱いに関する設計文書（上記 3 の正となる文書）
• docs/ntf-testdata-doc.md / docs/ntf-testdata-doc-examples-*.md / docs/ntf-doc-terms.md — NTF テストデータ解説書
• ntf-impl-spec-list.md、steering.md、checks/21.md・checks/22.md・checks/23.md — 実装方針と、各変更の完了根拠（実測値の全量）

テスト結果

mvn clean test（Java 17 + Maven 3.9.9）

Tests run: 854, Failures: 0, Errors: 0, Skipped: 7
BUILD SUCCESS

Skipped 7 件の内訳は HtmlConvertTest 2・RequestTestingMessagingClientTest 2・DbAccessTestSupportTest 2・FileUtilsTest 1 で、いずれも本変更前から存在する Assume によるもの。