cf56239da2
## 概要 [Task3427: Pipeline上でdocker-composeを用いてMySQLを起動する方法を調査する](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3427) - MySQLでUnitTestを実施する仕組みを作成 - DevContainerにUnitTestからのみアクセスされるMySQLが動くコンテナ( `test_mysql_db` )を追加 - テストでMySQLを使用し、上記 `test_mysql_db` に対してアクセスするよう構成 - テストの前処理で `test_mysql_db` の全てのテーブルをTRUNCATEする処理を実行し、データをクリアする - 常にテスト用MySQLは起動しているが、テスト実行前にクリアされるので空っぽ前提の状態でテストを実施できる - 実際にMySQLを使用して実行されるテストを1つ追加 - `users.service.spec.ts | UsersService.createUser` が対象 - Pipeline上でDevContainerを起動し、DevContainer内で `npm ci` `(UnitTest用DBへの) migration` `npm run test` を行う仕組みを作成 - Pipelineを追加 - **StagingPipelineでテスト・ステップを切り出し** - Pipelineテスト用DevContainer（のdocker-compose.yml）を作成 - 構成は通常の物と同一だが、ネットワーク設定だけ外部のMySQLやRedisと通信しない前提に変更している - テストの実行環境を自己ホストではなく、一般で提供されているマシンに変更 - 自己ホストのマシンにdocker-composeが入ってない/ビルド後のクリア処理が大変等の理由 - テストで使用する環境変数を `.env.test` という別環境変数に切り出し - そうすることで、PipelineでKeyVaultへのアクセスをする必要がなくなる - **migrationの漏れを修正** - テストが通らないことで発覚したmigration漏れを修正 - **テストコードを修正** - **Date型のミリ秒単位の誤差を setMillseconds(0) を用いることで0にセットするコードを追加（ライセンス付近）** ## レビューポイント - **WIPレビュー以降の追加の修正で違和感のある箇所はないか** - コミット履歴等で追っていただいた方が楽かと思います - 直近の`Merge branch 'develop' into feature/3427/poc-mysql-test` 以降がWIP以降に変更されたコードです - **レビュー通過後、developのPipelineについては手動で同等のコードに変更→PRをdevelopにマージ予定だが進め方として問題はないか** - [※WIPでレビュー済み] テスト用DevContainerを別途作成したが、現行のDevContainerを使用するよう頑張った方がいいか？ - [※WIPでレビュー済み] CI/CDの実行速度面を考慮し、使用されないMySQLとRedisのサービスとネットワーク作成が同居しているdocker-compose.ymlの実行を避けたが、管理対象は増えているので議論の余地はある - [※WIPでレビュー済み] MySQLでUnitTestを実行する際に懸念事項はないか - [※WIPでレビュー済み] Dockerを起動する関係でCI/CDのスピードが数分遅くなると思われるが、許容可能か - [※WIPでレビュー済み] `.env.test` に置き換えて問題ないか ## 動作確認状況 - ローカル＆Pipelineで実行して確認済
6 lines
231 B
SQL
6 lines
231 B
SQL
-- +migrate Up
|
|
ALTER TABLE `users_archive` DROP COLUMN `license_alert`;
|
|
|
|
|
|
-- +migrate Down
|
|
ALTER TABLE `users_archive` ADD COLUMN `license_alert` BOOLEAN NOT NULL COMMENT 'ライセンスの期限切れ通知をするかどうか'; |