Skip to content

feat: nablarch-testing-integration モジュールの初期構築 - #1

Open
kiyotis wants to merge 40 commits into
developfrom
feature/migrate-integration-test
Open

feat: nablarch-testing-integration モジュールの初期構築#1
kiyotis wants to merge 40 commits into
developfrom
feature/migrate-integration-test

Conversation

@kiyotis

@kiyotis kiyotis commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Why

  • Excel/YAML ダブルテストは nablarch-testing-converternablarch-testing-yaml の両方に依存します。
  • nablarch-testing 本体に置くと、本体が converteryaml に依存することになり、converteryaml がすでに本体に依存しているため循環参照になります。
  • そのため、独立したモジュール nablarch-testing-integration を新規作成してここに置きます。

What

  • NTF本体の既存テストコードを流用し、Excel形式のテストデータをYAMLに変換したうえで、同じテストロジックが両形式で同一の結果をもたらすかを確認する結合テストを構築します。
  • 各テスト機能について XxxTest(Excelベース、本体から流用)と XxxYamlTest(YAMLサブ、getTestConfig() オーバーライドのみ追加)が対になっています。
  • ロジック・テスト期待値・アサーションの変更はありません。

Testing

  • Java 17 / Maven 3.9.9 環境で mvn install を実行し、YAML サブクラス20本すべてが BUILD SUCCESS であることを確認しました。
  • Tests run: 546, Failures: 0, Errors: 0, Skipped: 18

References

kiyotis added 30 commits June 23, 2026 13:24
kiyotis and others added 8 commits June 23, 2026 14:21
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…format/

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- src/test/java/nablarch/test/core/reader/testdata.txt を追加
- src/test/java/nablarch/test/core/reader/BasicTestDataParserTest.xls を追加
- src/test/java/nablarch/test/core/reader/YamlTestDataParserTest/ 配下16件を追加
- pom.xml: surefire reuseForks=false / forkCount=1 を追加(H2インメモリDB汚染対策)
- README.md: 実行方法の説明を更新

結果: Tests run: 546, Failures: 0, Errors: 0, Skipped: 18 / BUILD SUCCESS

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…I と Files.list() リークを修正

- pom.xml: json-schema-validator 3.0.2 → 1.5.9(yaml 側と統一、3.x は tools.jackson 依存で互換なし)
- YamlSchemaValidationTest: SchemaRegistry/SpecificationVersion → JsonSchemaFactory/SpecVersion API に書き換え
- YamlSchemaValidationTest: Files.list() を try-with-resources で囲いディレクトリハンドルリークを修正

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kiyotis kiyotis changed the title feat: migrate integration tests from nablarch-testing feat: nablarch-testing-integration モジュールの初期構築 Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant