Skip to content

fix: member_test.tar.gz・PostgreSQL member UPSERT・テスト改善#26

Merged
nobuhiko merged 1 commit into4.2from
feature/fix-member-migration
Mar 3, 2026
Merged

fix: member_test.tar.gz・PostgreSQL member UPSERT・テスト改善#26
nobuhiko merged 1 commit into4.2from
feature/fix-member-migration

Conversation

@nobuhiko
Copy link
Owner

@nobuhiko nobuhiko commented Mar 3, 2026

Summary

  • member_test.tar.gz: ./ プレフィックスを除去。PharData ドライバーが ./ ディレクトリエントリを / として解釈し RuntimeException を投げていた
  • PostgreSQL UPSERT: 2.x系CSVカラムマッピング追加 (member_idid, workwork_id, authorityauthority_id, ranksort_no)。既存の4.0系マッピングはDBカラム名が間違っておりデッドコードだった
  • PostgreSQL UPSERT: convertDataTypesForPostgreSQL() / setIdSeq() / creator_id デフォルト値 / del_flg==1 時の email 処理を追加(saveToC と同等の処理)
  • テスト: メンバー件数の完全一致を以上比較に変更し、移行固有データ (id=99, login_id=testadmin) の検証を追加

Test plan

  • CI の PostgreSQL ジョブで member_test データプロバイダが通ること
  • CI の MySQL ジョブで既存テストが引き続き通ること
  • testバックアップファイルをアップロードできるかテスト の全 data provider が通ること

🤖 Generated with Claude Code

…ピング追加・テスト改善

- member_test.tar.gz: PharDataドライバーがRuntimeExceptionを投げる原因の
  `./`プレフィックスを除去(他フィクスチャと統一)
- PostgreSQL UPSERT: 2.x系CSVカラム名マッピングを追加
  (member_id→id, work→work_id, authority→authority_id, rank→sort_no)
  既存の4.0系マッピングはDBカラム名が間違っていたためデッドコードだった
- PostgreSQL UPSERT: convertDataTypesForPostgreSQL()呼び出し追加
- PostgreSQL UPSERT: creator_idデフォルト値(1)、del_flg==1時のemail処理追加
- PostgreSQL UPSERT: setIdSeq()呼び出し追加(シーケンスリセット)
- テスト: メンバー件数の完全一致→以上比較に変更し、
  移行固有データ(id=99, login_id=testadmin)の検証を追加

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nobuhiko nobuhiko merged commit 74249a6 into 4.2 Mar 3, 2026
24 checks passed
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