VB.NETの連携テストが破綻する原因とテストデータの密結合を防ぐリファクタ

VB.NETの連携テストが破綻する原因とテストデータの密結合を防ぐリファクタ手法を解説する記事のアイキャッチ バックエンド

VB.NETの連携テスト、気づけばテストケースの追加に伴いメンテナンスコストが爆発していませんか。
特にデータベースアクセスを伴うバッチ処理や帳票出力では、テストデータの準備だけで工数の大半を消費し、実装変更のたびに既存テストが赤く染まる――そんな経験は開発現場で一度は目にする光景です。

この問題の本質は、テストコードと特定のテストデータセットとの過度な結合にあります。
特に、事前に用意したExcelやCSVのマスタデータをそのまま使い回すアプローチは、一見効率的に見えて破綻への最短経路です。
カラム追加や制約変更が生じるたびに全テストデータの再生成が求められ、結果としてテスト自体が「変更に弱い重たい資産」と化します。

では、どう対処するか。
鍵はテストデータの生成ロジックをテストコード内に閉じ込め、必要最小限の属性だけを持つエンティティをオンデマンドで構築するリファクタです。
具体的には以下の3ステップを提案します。

  • テスト基底クラスに「デフォルト値を持つエンティティ生成メソッド」を実装し、各テストケースでは差分だけをオーバーライドするビルダーパターンを採用する
  • 外部ファイルに依存するテストデータを段階的に廃止し、代わりにメモリ内SQLiteやEF CoreのInMemoryプロバイダを活用してスキーマ変更の影響を局所化する
  • テスト間で共有する「共通フィクスチャ」を極力減らし、各テストメソッド内で独立したトランザクションスコープを使いテスト終了時に自動ロールバックさせる

例えば、従来の「テスト用マスタ.csv」を読み込む処理を、以下のようなビルダークラスに置き換えることで、カラム追加時にも影響範囲が当該テストメソッドのみに限定されます。

Public Class OrderBuilder
    Private _id As Integer = 1
    Private _amount As Decimal = 1000
    Public Function WithAmount(value As Decimal) As OrderBuilder
        _amount = value : Return Me
    End Function
    Public Function Build() As Order
        Return New Order With {.Id = _id, .Amount = _amount}
    End Function
End Class

さらに、テストデータの密結合を可視化するために、以下の観点で既存テストを棚卸しすることをお勧めします。

評価軸 良好な状態 要注意な状態 破綻寸前の状態
データ準備コード量 テスト1件あたり5行以内 10〜20行の共通関数に依存 外部ファイルの読み込み+変換ロジックが30行超
スキーマ変更時の修正箇所 ビルダークラスのみ 複数テストクラスで分散 全テストデータファイルの手動編集
テスト実行順序依存性 なし(並列実行可能) 一部のテストで順序が必要 全テストが固定のINSERT順に依存

このリファクタは決して「美しさ」のためではありません。
変更に対する耐性とデバッグ時の原因特定のしやすさを向上させる実践的な投資です。
最初の数週間は工数が増えるものの、3回目のスキーマ変更以降は明らかに差分が現れるはずです。

では、あなたのプロジェクトで最初に手をつけるべきテストクラスはどれでしょうか。
最も頻繁に変更されるテーブルに依存するテストから着手するのがセオリーです。
そのテストだけでもビルダーパターンに置き換えれば、明日からの結合デバッグ時間が確実に短縮されるでしょう。

  1. なぜVB.NETの連携テストは時間経過とともに破綻するのか
  2. 連携テストが抱える3つの根本的課題
    1. 課題1:テストデータの静的依存と外部ファイルへの過信
    2. 課題2:スキーマ変更に対する無防備な結合
    3. 課題3:テスト間の状態汚染と実行順序への暗黙の依存
  3. テストデータの密結合が生む負のスパイラル
    1. スパイラルの起点:スキーマ変更とテストデータ修正の連鎖
    2. スパイラルの加速:テスト修正が新たなバグを埋め込む
    3. スパイラルの頂点:テストスイートの信頼性崩壊
    4. スパイラルを断ち切るための視点転換
  4. リファクタリング前夜:既存テストの棚卸しと優先順位付け
    1. ステップ1:テストケースの分類と影響度マッピング
    2. ステップ2:デッドテストの発見と削除
    3. ステップ3:テストデータの依存関係グラフを作成する
    4. ステップ4:リファクタリングの順序を決定する
    5. ステップ5:測定基準と成功条件を定義する
    6. いよいよ実践へ
  5. ビルダーパターンでテストデータ生成を局所化する
    1. なぜビルダーパターンがテストに有効なのか
    2. VB.NETにおける典型的なビルダー実装
    3. ビルダー導入時の注意点と拡張テクニック
    4. ビルダーと既存テストの置き換え手順
  6. インメモリデータベースを活用したスキーマ変更耐性の向上
    1. なぜインメモリDBがスキーマ変更に強いのか
    2. SQLiteインメモリとEF Core InMemoryの使い分け
    3. VB.NETでの具体的な実装例
    4. インメモリDB導入時の注意点
    5. 既存テストへの段階的な導入戦略
  7. トランザクションスコープでテスト独立性を確保する実践手法
    1. トランザクションスコープがもたらす2つの効果
    2. 具体的な実装パターン(VB.NET + SQLite)
    3. 注意すべき3つの落とし穴
    4. トランザクションスコープとインメモリDB以外の選択肢
    5. テスト並列実行への道筋
    6. ビルダー+インメモリDB+トランザクションの三種の神器
  8. リファクタ後の運用ルールとCIパイプラインへの組み込み
    1. 運用ルール1:新規テストは必ずビルダー経由でデータ生成する
    2. 運用ルール2:インメモリDBと実DBの使い分け基準を明文化する
    3. 運用ルール3:トランザクションスコープの誤用を防止する静的解析
    4. CIパイプラインへの組み込み戦略
    5. リファクタ効果の定量的評価指標
    6. 最後に:文化としてのテスト品質
  9. まとめ:密結合を断ち切り、変更に強いテスト資産を手に入れる

なぜVB.NETの連携テストは時間経過とともに破綻するのか

複雑に絡み合ったテストコードと赤いエラーメッセージが表示されるIDEのスクリーンショット

VB.NETによるエンタープライズアプリケーション開発において、連携テストは品質保証の要です。
しかし、プロジェクトが半年、一年と経過するにつれ、そのテストスイートはほとんどメンテナンス不可能なほどの重い負債へと変貌を遂げることが少なくありません。
なぜでしょうか。
単に「コードが汚い」という話ではなく、連携テスト特有の構造的脆弱性が時間の経過とともに顕在化するからです。

まず、連携テストは単体テストと異なり、データベース、ファイルシステム、外部APIといった現実のリソースに依存します。
この依存性自体は正しい動作検証のために不可欠ですが、問題はその依存が「暗黙的」かつ「固定的」である点にあります。
初期フェーズでは少数のテストケースと静的なテストデータで回っていたものが、機能追加に伴いテストケースが増え、データのバリエーションも拡張されていきます。
このとき、多くの開発者は「とりあえず既存のテストデータを流用しよう」と考えます。
これが破綻の第一歩です。

次に、VB.NETの特性として、ADO.NETやEntity Frameworkを介したデータアクセスが多く、テスト時に実際のDBスキーマに依存するケースが大半です。
スキーマ変更(カラム追加、型変更、制約追加)が発生するたびに、既存のテストデータセットは「変更前のスキーマ」で作られているため、テスト実行時に外部キー違反やNULL制約エラーが頻発します。
その都度、テストデータファイル(ExcelやCSV)を手動で修正する運用が続けば、テスト維持コストは指数関数的に増大します。

さらに看過できないのは、テスト間の実行順序依存です。
テストデータを共有する運用では、あるテストがINSERTしたレコードを別のテストがUPDATEし、さらに別のテストがDELETEする――といった暗黙の連鎖が生まれます。
この状態でテストを並列実行しようものなら、結果はカオスです。
CIパイプラインでランダムに失敗するテストが発生したら、まずこの順序依存を疑うべきでしょう。

加えて、VB.NETのレガシーコードベースでは、テストプロジェクト自体が古い.NET Frameworkに依存し、最新のテストフレームワーク(xUnit.netやNUnit)の機能を十分に活用できていないケースもあります。
その結果、[TestInitialize]や[TestCleanup]といったライフサイクルメソッドすら適切に設計されず、テストクラス間で静的フィールドを介した状態共有が行われている――そんな光景を私は何度も目にしてきました。

ここで重要な問いが浮かびます。
本当に連携テストが悪いのでしょうか。
いえ、違います。
悪いのは「テストデータの設計思想」です。
連携テストは本来、実環境に近い状態でシステム全体の振る舞いを検証するための強力なツールです。
しかし、それを静的なマスタデータと手動メンテナンスに頼るアプローチで運用し続ける限り、テストは「変更を恐れる重石」に変わります。

あるプロジェクトでは、たった1つのテーブルにカラムを追加するだけで、関連する50を超えるテストケースのデータ準備スクリプトを再生成するのに丸2日を要した事例を私は知っています。
その間、開発者は本来の機能実装から遠ざかり、テストの修正に追われました。
これではアジャイルもスクラムも絵に描いた餅です。

では、どうすればこの負のスパイラルから脱却できるのか。
その答えは、テストデータを「固定資産」ではなく「生成可能なリソース」として捉え直すことにあります。
具体的には、テストコード内で必要なエンティティをその都度構築するビルダーパターンや、インメモリデータベースを用いたスキーマ変更に強いアーキテクチャ、そしてテスト終了時に自動で状態をリセットするトランザクション管理です。

これらは決して新しい技法ではありません。
しかし、VB.NETの現場では「とりあえず動いているから」と先送りにされがちです。
その先送りが、最終的にテストスイート全体の廃棄や、手動テストへの回帰という最悪のシナリオを招くのです。

幸い、リファクタリングは決して遅くありません。
最初の一歩として、今日からでもテストデータ生成ロジックを共通化する小さなヘルパークラスを導入することから始められます。
そのインパクトは想像以上に大きく、次章以降で具体的な手法を段階的に解説していきます。

連携テストが抱える3つの根本的課題

テストデータ依存、スキーマ変更、実行順序依存の3つを図示した模式図

連携テストが破綻するメカニズムを理解するには、その構造的な問題を個別の要素に分解するのが近道です。
私がこれまで現場で診断してきた数多くのVB.NETプロジェクトを振り返ると、ほぼ例外なく以下の3つの根本的課題が絡み合っていることが分かります。
それぞれ単独でも十分に厄介ですが、これらが相互に増幅し合うことで、テストスイートは手に負えない状態へと陥ります。

課題1:テストデータの静的依存と外部ファイルへの過信

最初の課題は、テストデータを外部ファイル(Excel、CSV、JSON)で管理し、それをテスト実行時にインポートするという古典的かつ広く普及したアプローチにあります。
この方法の最大の魅力は「非エンジニアでもデータを編集できる」という点ですが、それは同時に最大の落とし穴でもあります。

  • スキーマ変更のたびに全ファイルのカラム構成を見直す必要が生じる
  • ファイル間で同一エンティティの定義が重複し、変更漏れが頻発する
  • バイナリファイル(Excel)の場合、差分検出が困難でレビューが事実上不可能になる
  • 大量のファイルをプロジェクトに含めることで、リポジトリサイズが肥大化する

特にVB.NETのWindows FormsやASP.NET Web Formsのレガシープロジェクトでは、この「テストデータ=Excelファイル」という暗黙のルールが染み付いており、新しいメンバーがそれを疑うことすらありません。
しかし、この慣習はテストを「データ管理業務」に変質させ、本来の検証ロジックから開発者の注意をそらしてしまうのです。

課題2:スキーマ変更に対する無防備な結合

2つ目の課題は、テストコードがデータベースの物理スキーマに過度に密結合している点です。
これは単にテーブル名やカラム名をハードコードするという次元の話ではありません。
テスト内でINSERT文を直書きし、そのカラム順序やデフォルト値の有無を暗黙に仮定しているケースが非常に多いのです。

例えば、以下のようなコードを目にしたことはないでしょうか。

Dim cmd As New SqlCommand("INSERT INTO Orders (OrderDate, CustomerId, Total) VALUES ('2026-01-01', 100, 5000)", connection)
cmd.ExecuteNonQuery()

このテストが成立する条件は、OrdersテーブルにOrderDate、CustomerId、Totalという3つのカラムがその順序で存在し、かつ他のカラムが全てNULL許容またはデフォルト値を持つことです。
ここに新しい必須カラムStatusが追加された瞬間、このテストは実行時エラーを吐きます。
しかも、そのエラーは「カラム数が一致しない」というSQLレベルでの例外であり、テストの意図とはまったく無関係な場所で発生します。
原因特定に数十分を要することは珍しくありません。

さらに深刻なのは、Entity FrameworkのCode Firstでマイグレーションを運用している場合です。
マイグレーションが適用されるたびに、既存のテストデータ用SQLスクリプトが陳腐化します。
その都度、全スクリプトをマイグレーション後のスキーマに合わせて書き直すのは、現実的な運用とは言えません。

課題3:テスト間の状態汚染と実行順序への暗黙の依存

3つ目の課題は、テストケース同士がデータベースの共有状態を介して密かに依存し合うという構造的欠陥です。
単体テストであれば、各テストメソッドは独立したインスタンスで動作し、状態を共有することはほぼありません。
しかし連携テストでは、同じデータベースに対して複数のテストが読み書きを行うため、避けようとしても状態の交差が発生します。

典型的な悪いパターンとして、以下のような運用があります。

  • テストAが「マスタデータを挿入」し、そのIDを静的な変数に保存する
  • テストBがそのIDを使って関連レコードを更新する
  • テストCが更新後の値を検証する

この一連の流れは、単一の巨大なテストメソッドとして書かれていればまだマシです。
問題は、これらが別々のテストメソッドとして定義され、しかもクラス内の実行順序がアルファベット順や宣言順に依存している場合です。
テストフレームワークによって実行順序が変われば、突然テストが失敗し始める――そんな「フレーキーテスト」に悩まされた経験は多くの開発者に共通するでしょう。

また、テストクラス間で静的フィールドを介してデータベース接続やトランザクションを共有する設計も同様に危険です。
あるテストが例外を発生させてトランザクションをロールバックし忘れると、後続の全テストが不正な状態で実行され、結果として無関係なテストが赤くなるという連鎖障害が起きます。

これらの3つの課題は、いずれもテストデータのライフサイクル管理に共通する設計判断の誤りに起因しています。
つまり、データを「永続的な共有資産」として扱うか、「一時的な生成物」として扱うかという根本的な思想の違いです。
多くのプロジェクトは、楽観的に前者を選び、後でその代償を支払うことになるのです。

では、これらの課題をどう解決するか。
次章では、まず現状のテストスイートをどう評価し、どの優先順位でリファクタリングを始めるかという実践的な戦略について論じます。
いきなり全てを書き換えるのではなく、影響範囲とリスクを最小化しながら進めるための勘所を押さえていきましょう。

テストデータの密結合が生む負のスパイラル

外部ファイルとテストコードが密に結びつき修正コストが増大する様子のフローチャート

前章では連携テストの3つの根本的課題を整理しました。
これらの課題は、それぞれ独立して存在するのではなく、相互に強化し合うフィードバックループを形成します。
このループこそが「負のスパイラル」の正体であり、一度回り始めるとプロジェクトの生産性を著しく損ないます。
このメカニズムを可視化することが、リファクタリングの第一歩です。

スパイラルの起点:スキーマ変更とテストデータ修正の連鎖

この負のスパイラルは、データベーススキーマの変更というごく普通の開発作業から始まります。
例えば、要件変更でOrdersテーブルにPriority列を追加するとしましょう。
単体テストであれば、影響を受けるコードは数箇所で済みます。
しかし密結合した連携テストでは、以下のような連鎖反応が生じます。

  • 外部テストデータファイル(Excel/CSV)が新しいスキーマに未対応のため、インポートスクリプトが失敗する
  • エンジニアは各ファイルを手動で開き、列を追加してデフォルト値を埋める作業を強いられる
  • この作業が数十ファイルに及ぶ場合、数時間から数日を要する
  • 作業中に一部のファイルで列順序やデータ型を間違え、テスト実行時に予期せぬエラーが発生する
  • エラーの原因がデータファイルにあると特定するまで、さらにデバッグ時間が消費される

この時点で、開発者は「たった1列の追加」に想像以上の工数を費やしています。
そしてこの体験が、スキーマ変更そのものへの心理的抵抗を生みます。
結果として、本来なら早期に行うべきリファクタリングや機能改善が先送りにされ、コードベースの劣化が加速するのです。

スパイラルの加速:テスト修正が新たなバグを埋め込む

さらに深刻なのは、緊急対応として行うテスト修正が、別のテストを壊すという二次被害です。
先の例で、エンジニアが慌てて全テストデータファイルを修正したとします。
しかし、ファイルAではPriorityのデフォルト値を"Low"とし、ファイルBでは"High"とした場合、テスト間で期待する振る舞いが暗黙に変わってしまいます。

ここで発生するのが「テストの不安定性」です。
CI環境で実行すると、ファイルの読み込み順序やデータベースの初期化タイミングによって、同じソースコードでもテストの成否が変わるというフレーキーな状態に陥ります。
この状態を放置すると、開発者は「テストがたまたま失敗しただけ」と軽視し、本当のバグを見逃すリスクが高まります。

また、テストデータ修正の過程で、不要なデータが混入したり、外部キー制約を満たすために無関係なマスタレコードを追加したりすることもあります。
これらの「よけいなデータ」は、後々に別のテストで想定外の結合結果を生み、デバッグの難易度を指数関数的に引き上げます。
私があるプロジェクトで実際に遭遇したケースでは、テスト用に追加したダミーユーザーが本番環境に誤ってデプロイされかけたこともありました。
テストデータと本番データの境界が曖昧になることは、セキュリティ上も決して看過できません。

スパイラルの頂点:テストスイートの信頼性崩壊

負のスパイラルが最終的に到達するのは、テストスイート全体への信頼喪失です。
以下のような症状が顕著に現れ始めます。

  • テスト実行時間が著しく増大し(初期の5倍以上)、デプロイメントパイプラインがボトルネック化する
  • 失敗したテストの原因が「テストデータの問題」なのか「実装のバグ」なのか区別がつかなくなる
  • 開発者が「緑のテスト」よりも「赤いテスト」を無視する習慣を身につける(エラーサイレンシング)
  • 新機能追加時にテストを書くよりも、既存テストをコメントアウトする方が早いと判断される
  • 最終的に、テスターによる手動テストに頼る割合が増え、自動化の意義が失われる

この状態を「テストスイートの腐敗」と呼びます。
腐敗したテストは、もはや品質保証の道具ではなく、開発効率を阻害する重荷です。
そして悲しいことに、このスパイラルは一度回り出すと自走し続けます。
なぜなら、各開発者は自分のタスクに集中するあまり、テスト全体の健全性にまで気を配る余裕がないからです。

スパイラルを断ち切るための視点転換

ここで重要なのは、このスパイラルは「テストコードの質が悪い」という個人のスキル問題ではなく、システム設計上の構造的欠陥であるという認識です。
つまり、優れたエンジニアでも、密結合を前提としたアーキテクチャの上では同じスパイラルに巻き込まれます。

では、どうすればこのループから脱出できるのか。
その答えは、「テストデータを共有固定資産から、各テストが自己完結的に生成・破棄する一時オブジェクトへとパラダイムシフトする」ことに尽きます。
これにはビルダーパターンやインメモリデータベースといった具体的な技法が有効ですが、それ以前にチーム全体で「テストデータはテストコードの一部である」という認識を共有することが成功の鍵です。

次の章では、このスパイラルを実際に断ち切るための最初のアクションとして、既存テストの棚卸しと優先順位付けという現実的なアプローチを提示します。
全てを一気に変えようとせず、影響の大きい箇所から段階的に改善していく戦略が、長続きするリファクタリングの秘訣です。

リファクタリング前夜:既存テストの棚卸しと優先順位付け

プロジェクト内のテストクラスを一覧表示し優先度を色分けしたエクスプローラー画面

負のスパイラルを断ち切ると決意したなら、まずは現状のテストスイートを冷静に評価することから始めましょう。
いきなりビルダーパターンやインメモリDBを導入しても、対象が不明確では効果は半減します。
ここでは、リファクタリングの効果を最大化するための「棚卸し」と「優先順位付け」の具体的な手順を解説します。

ステップ1:テストケースの分類と影響度マッピング

最初に行うべきは、全テストケースを以下の軸で分類することです。

  • 重要度:ビジネスクリティカルな機能を検証しているか(決済処理、在庫更新など)
  • 変更頻度:過去3ヶ月間で関連する実装コードが何回変更されたか
  • テスト実行時間:単体で実行した際の所要時間(特に重いDBアクセスを含むもの)
  • データ依存度:外部ファイルや固定IDに依存している度合い(高い順に優先対応)

この分類は、チーム内の開発者数名でのホワイトボードセッションでも構いません。
重要なのは「すべてのテストを平等に扱わない」という姿勢です。
変更頻度が高く、かつ重要度も高いテストこそ、最初のリファクタリング対象として最適です。
なぜなら、その改善が最も早く開発現場のボトルネック解消に寄与するからです。

ステップ2:デッドテストの発見と削除

棚卸しの過程で、実行されていない、あるいは無意味なテストが少なからず存在することに気づくはずです。
例えば、以下のようなパターンです。

  • コメントアウトされたまま放置されているテストメソッド
  • 常に成功するが、何も検証していないアサーションのないテスト
  • 別のテストと完全に重複しているテスト
  • 既に廃止された機能に対するテスト

これらのデッドテストは、メンテナンスコストを増やすだけでなく、テストスイート全体の信頼性を損ないます。
思い切って削除する勇気もリファクタリングの一部です。
Gitの履歴があれば、後で必要になったときに復元できるという安心感もあります。

ステップ3:テストデータの依存関係グラフを作成する

ここが本丸です。
各テストがどのテーブル、どのカラム、どの外部キー制約に依存しているかを可視化します。
手作業は大変ですが、以下のような簡易的な表を用意すると効果的です。

テストクラス名 依存テーブル 依存カラム 外部ファイル有無 実行順序依存の有無
OrderServiceTest Orders, Customers OrderDate, CustomerId あり(Orders.csv) あり(テストBの後に実行必須)
InventoryTest Products, Stock ProductId, Quantity なし なし
ShipmentTest Orders, Shipments OrderId, Status あり(Shipments.json) あり(テストAの後に実行必須)

この表を作成するだけで、どのテストが最も多くの他テストと結合しているかが一目瞭然です。
結合の多いテストほど、リファクタリングによる波及効果が大きいため、優先度を上げるべきです。

ステップ4:リファクタリングの順序を決定する

上記の情報を基に、以下の優先順位付けルールを適用します。

  • 最優先(フェーズ1):変更頻度が高く、かつ依存テーブル数が3つ以上で、外部ファイルを使用しているテスト
  • 次優先(フェーズ2):実行時間が5秒以上かかり、かつ実行順序依存があるテスト
  • 最終(フェーズ3):上記に該当しないが、重要度が高いビジネスロジックのテスト
  • 対象外:デッドテストまたは単体テストで十分代替可能なもの

このフェーズ分けにより、チーム全体でリファクタリングの進捗を共有しやすくなります。
また、フェーズ1の改善が完了した時点で、すでにCIの安定性やデバッグ効率に顕著な変化が現れるはずです。

ステップ5:測定基準と成功条件を定義する

リファクタリングは「やったつもり」で終わらせてはいけません。
以下の定量的な指標を事前に設定し、改善効果を測定しましょう。

  • テスト実行時間の平均値(全テストスイート)
  • テスト失敗時の原因特定にかかる平均時間(開発者アンケート)
  • スキーマ変更に伴うテスト修正工数(人日)
  • フレーキーテスト(ランダムに失敗するテスト)の発生件数

これらの数値をリファクタリング開始前に記録しておくことで、投資対効果をチームやマネジメントに説明する材料にもなります。

いよいよ実践へ

棚卸しが完了したら、次章から実際のリファクタリング手法に入ります。
最初に取り組むのは、ビルダーパターンによるテストデータ生成の局所化です。
この手法は導入コストが比較的低く、かつ即効性があるため、フェーズ1のテストに最適です。
まずは1つのテストクラスを選び、そのデータ生成部分だけをビルダーに置き換えてみてください。
その小さな成功体験が、その後の大きな改善への原動力となります。

ビルダーパターンでテストデータ生成を局所化する

VB.NETのビルダークラスでデフォルト値と差分指定のみでエンティティを生成するコード例

前章までの棚卸しで、どのテストから手をつけるべきか明確になったはずです。
では実際のリファクタリング手法に入ります。
最初に取り上げるのはビルダーパターンです。
これはGoFのデザインパターンの一つですが、テストデータ生成の文脈では「テストデータビルダー」と呼ぶのが適切でしょう。
その目的は、複雑なエンティティやDTOを柔軟かつ可読性高く生成するための専用クラスを提供することにあります。

なぜビルダーパターンがテストに有効なのか

従来のテストコードでは、テストメソッド内でNew Orderを直接呼び出し、そのプロパティを一つずつ設定するのが一般的でした。
しかし、エンティティに含まれるプロパティが10個、20個と増えるにつれ、以下のような問題が顕著になります。

  • 各テストで同じような初期化コードが繰り返され、冗長になる
  • 必須プロパティが追加された場合、全テストメソッドを修正する必要が生じる
  • テストの意図(どのプロパティが重要か)が埋もれてしまう

ビルダーパターンはこれらの問題をデフォルト値の集中管理と差分指定による構築という2つのメカニズムで解決します。
具体的には、ビルダークラスが全てのプロパティに妥当なデフォルト値を保持し、テストメソッドは関心のあるプロパティだけを上書きするのです。

VB.NETにおける典型的なビルダー実装

ここで、前章でも触れたOrderエンティティを例に、実際のVB.NETコードを示します。

Public Class OrderBuilder
    Private _id As Integer = 0
    Private _orderDate As Date = Today
    Private _customerId As Integer = 100
    Private _total As Decimal = 1000
    Private _status As String = "Pending"
    Private _priority As String = "Medium"

    Public Function WithId(value As Integer) As OrderBuilder
        _id = value
        Return Me
    End Function

    Public Function WithOrderDate(value As Date) As OrderBuilder
        _orderDate = value
        Return Me
    End Function

    Public Function WithCustomerId(value As Integer) As OrderBuilder
        _customerId = value
        Return Me
    End Function

    Public Function WithTotal(value As Decimal) As OrderBuilder
        _total = value
        Return Me
    End Function

    Public Function WithStatus(value As String) As OrderBuilder
        _status = value
        Return Me
    End Function

    Public Function WithPriority(value As String) As OrderBuilder
        _priority = value
        Return Me
    End Function

    Public Function Build() As Order
        Return New Order With {
            .Id = _id,
            .OrderDate = _orderDate,
            .CustomerId = _customerId,
            .Total = _total,
            .Status = _status,
            .Priority = _priority
        }
    End Function
End Class

このビルダーを使うと、テストメソッドは以下のように簡潔かつ意図が明確になります。

<TestMethod>
Public Sub 高優先度の注文は配送料が無料になる()
    Dim order = New OrderBuilder() _
        .WithPriority("High") _
        .WithTotal(5000) _
        .Build()
    ' 検証ロジック...
End Sub

重要なのは、デフォルト値がビルダー内で一括管理されているため、仮にOrderに新しい必須プロパティDeliveryDateが追加されても、ビルダーのBuildメソッドとデフォルト値設定箇所だけを修正すれば、全てのテストが影響を受けないという点です。
各テストメソッドに手を入れる必要は一切ありません。

ビルダー導入時の注意点と拡張テクニック

ビルダーパターンは強力ですが、いくつかの落とし穴もあります。
まず、ビルダークラスが巨大化しすぎないように注意してください。
エンティティに30以上のプロパティがある場合、ビルダーも30のメソッドを持つことになり、それ自体が保守対象になります。
この場合は、関連するプロパティごとにサブビルダーを分割するか、またはファクトリメソッドと組み合わせてデフォルトのバリエーション(例:DefaultOrder()、HighPriorityOrder())を用意すると良いでしょう。

また、ビルダーはテストコード内でのみ使用することを徹底してください。
プロダクションコードにビルダーを持ち込むと、ドメインロジックとテスト支援コードが混在し、設計が曖昧になります。
テストプロジェクトにのみビルダークラスを配置するのが正しい分離です。

ビルダーと既存テストの置き換え手順

実際のリファクタリングでは、以下のステップで進めることをお勧めします。

  1. 対象エンティティのビルダークラスを新規作成し、全てのプロパティにデフォルト値を設定する
  2. 既存のテストメソッドから、直接のプロパティ代入をビルダー呼び出しに置き換える(このとき、テストの意図に関係ないプロパティはビルダーのデフォルトに委ねる)
  3. 置き換え後もテストが同じ結果を返すことを確認する(レッドグリーンリファクタリングの原則)
  4. 全てのテストがビルダーに移行できたら、古いテストデータ用の外部ファイルや定数定義を削除する

この置き換えにより、テストコードの行数は平均で30%から50%削減されるというデータもあります。
何より、新しい開発者がテストを読んだときに「何を検証したいのか」が明確になるという副次効果は計り知れません。

ビルダーパターンは、本章で紹介した以外にも連鎖メソッドの戻り値に自分自身を返すというシンプルな仕組みだけで成り立っているため、VB.NETに限らずあらゆるオブジェクト指向言語で応用可能です。
次の章では、さらに踏み込んでインメモリデータベースを用いたスキーマ変更耐性の向上について解説します。
ビルダーが「データの中身」を柔軟にするのに対し、インメモリDBは「データの格納先」そのものを変更に強くするアプローチです。
この2つを組み合わせることで、連携テストの堅牢性は飛躍的に高まるでしょう。

インメモリデータベースを活用したスキーマ変更耐性の向上

SQLiteインメモリとEF Core InMemoryの切り替えスイッチを表すアイコン図

ビルダーパターンでテストデータの「中身」を柔軟にしたなら、次はデータの「格納先」そのものを見直します。
連携テストの最大の痛みであるスキーマ変更への耐性を飛躍的に高める手法として、インメモリデータベースの活用は非常に有効です。
ここでは、VB.NET環境での具体的な実装戦略と、導入時のトレードオフについて論じます。

なぜインメモリDBがスキーマ変更に強いのか

従来の連携テストは、開発用の実データベース(SQL ServerやPostgreSQLなど)に対してテストを実行していました。
このアプローチでは、スキーマ定義がテストコードと暗黙的に結合しているため、マイグレーションのたびにテスト環境の再構築やデータの再投入が必須でした。

これに対し、インメモリデータベース(SQLiteのインメモリモードやEF CoreのInMemoryプロバイダ)は、テスト実行時にスキーマをコードベースから動的に生成します。
具体的には、Entity FrameworkのCode Firstマイグレーションを利用していれば、テストの初期化処理でDbContextのDatabase.EnsureCreated()を呼び出すだけで、最新のスキーマがその場で構築されるのです。
これにより、スキーマ変更の影響をテストコードが受けにくくなり、開発者はマイグレーションスクリプトの作成に集中できます。

SQLiteインメモリとEF Core InMemoryの使い分け

VB.NETのエコシステムで主に選択肢となるのは、SQLiteのインメモリモードとEF CoreのInMemoryプロバイダの2つです。
それぞれに特性があるため、プロジェクトの要件に応じて使い分ける必要があります。

特徴 SQLiteインメモリ EF Core InMemory
リレーショナル機能 完全サポート(外部キー、トランザクション、インデックス) 制限あり(参照整合性の一部は擬似的)
実環境との近接性 高い(本番がSQL ServerでもSQLiteの方言に注意) 低い(LINQ動作は近いが、SQL固有の関数は非対応)
セットアップ速度 やや遅い(接続オープン時にスキーマ構築) 非常に高速(メモリ上のコレクション操作)
テストの独立性 各テストで新しい接続文字列を使えば分離可能 デフォルトで共有インスタンスに注意が必要

基本的な推奨は、本番環境がリレーショナルDBであり、かつ複雑なJOINやストアドプロシージャを使わない場合はSQLiteインメモリを選択します。
一方、単純なCRUD中心で、とにかくテストの高速化を優先したい場合はEF Core InMemoryが適しています。

VB.NETでの具体的な実装例

ここでは、SQLiteインメモリを用いたテスト基底クラスの実装例を示します。

Imports Microsoft.Data.Sqlite
Imports Microsoft.EntityFrameworkCore

Public MustInherit Class IntegrationTestBase
    Private _connection As SqliteConnection
    Protected _context As MyDbContext

    <TestInitialize>
    Public Sub Setup()
        _connection = New SqliteConnection("DataSource=:memory:")
        _connection.Open()

        Dim options = New DbContextOptionsBuilder(Of MyDbContext) _
            .UseSqlite(_connection) _
            .Options

        _context = New MyDbContext(options)
        _context.Database.EnsureCreated()  ' スキーマを最新に構築
    End Sub

    <TestCleanup>
    Public Sub Cleanup()
        _context?.Dispose()
        _connection?.Close()
        _connection?.Dispose()
    End Sub
End Class

この基底クラスを継承することで、各テストメソッドは完全に独立したデータベースインスタンスを利用できます。
EnsureCreatedはDbContextのモデル定義に基づいてテーブルを作成するため、スキーマ変更があってもテストコード側の修正は原則不要です。

インメモリDB導入時の注意点

しかし、この手法にはいくつかの落とし穴も存在します。
まず、インメモリDBは実DBと完全に互換ではないという事実です。
特に、SQL Server固有の関数(GETDATE()やNEWID()など)を使用している場合、SQLiteでは異なる構文(CURRENT_TIMESTAMPやRANDOM())を要求されます。
この問題を回避するには、テスト用にリポジトリ層をモック化するか、またはSQLの方言を抽象化する設計を事前に導入しておく必要があります。

また、大量のテストデータを生成する場合、インメモリDBのパフォーマンスが低下することがあります。
これはメモリ上のデータ構造が大きくなるほどGCの負荷が増すためです。
対策としては、各テストクラス単位でDBインスタンスを共有し、トランザクションでテスト間の分離を図る方法もありますが、その場合は前章で指摘した順序依存の問題が再発するリスクを伴います。

既存テストへの段階的な導入戦略

インメモリDBへの移行は、全テストを一度に切り替えるのではなく、影響の大きいテストから段階的に進めることをお勧めします。
具体的な手順は以下の通りです。

  1. まずは新規作成するテストからインメモリDBを採用し、運用ノウハウを蓄積する
  2. 次に、変更頻度が高い既存テスト(フェーズ1対象)を、インメモリDBベースのテスト基底クラスに移行する
  3. 移行後、テスト実行時間とスキーマ変更時の修正工数を測定し、効果を可視化する
  4. チーム内で成果が共有できたら、残りのテストも順次移行する

この段階的アプローチにより、移行リスクを最小化しながら、組織全体のテスト文化を徐々に変えていくことが可能です。

インメモリDBはビルダーパターンと非常に相性が良いことも強調しておきます。
ビルダーで生成したエンティティをそのままインメモリDBに保存し、検証する――この一連の流れは、テストコードの可読性と保守性を極限まで高めます。
次の章では、この2つの手法をさらに補完するトランザクションスコープを用いたテスト独立性の確保について掘り下げます。
データ生成と格納先が整ったら、あとは実行環境の分離を徹底するだけです。

トランザクションスコープでテスト独立性を確保する実践手法

各テストメソッドが独立したトランザクションで実行され自動ロールバックするシーケンス図

ビルダーパターンでデータ生成を局所化し、インメモリデータベースでスキーマ変更耐性を高めたとしても、最後に残る課題がテスト間の状態汚染です。
どれだけ優れたデータ生成ロジックを持っていても、あるテストが書き込んだレコードが別のテストに影響を与える限り、フレーキーテストは撲滅できません。
この問題を解決するのが、トランザクションスコープを用いたテスト実行単位の分離です。
ここでは、VB.NETにおける実践的な実装パターンと、その落とし穴を解説します。

トランザクションスコープがもたらす2つの効果

トランザクションスコープ(.NETのSystem.Transactions.TransactionScopeクラス)をテストに導入することで、主に以下の2つの効果が得られます。

  • テスト終了時の自動ロールバック:各テストメソッドの終了時にトランザクションをコミットせず破棄することで、データベースの状態をテスト開始前に完全に戻せる
  • ネストされたトランザクションの分離:テスト内でさらにトランザクションを張るプロダクションコードがあっても、テスト全体のトランザクションと適切に協調できる

これにより、テスト実行順序に依存しない安定したテストスイートが実現します。
並列実行も理論上は可能になり、CIパイプラインのスループット向上にも貢献します。

具体的な実装パターン(VB.NET + SQLite)

先述のSQLiteインメモリDBと組み合わせる場合、以下のようなテスト基底クラスが典型的なパターンです。

Imports System.Transactions
Imports Microsoft.Data.Sqlite
Imports Microsoft.EntityFrameworkCore

Public MustInherit Class TransactionalIntegrationTestBase
    Private _connection As SqliteConnection
    Private _scope As TransactionScope
    Protected _context As MyDbContext

    <TestInitialize>
    Public Sub Setup()
        _connection = New SqliteConnection("DataSource=:memory:")
        _connection.Open()

        Dim options = New DbContextOptionsBuilder(Of MyDbContext) _
            .UseSqlite(_connection) _
            .Options

        _context = New MyDbContext(options)
        _context.Database.EnsureCreated()

        ' トランザクションスコープを開始(分離レベルはReadCommittedが無難)
        _scope = New TransactionScope(TransactionScopeOption.Required, 
                                       New TransactionOptions With {
                                           .IsolationLevel = IsolationLevel.ReadCommitted
                                       }, 
                                       TransactionScopeAsyncFlowOption.Enabled)
    End Sub

    <TestCleanup>
    Public Sub Cleanup()
        _scope?.Dispose()  ' ここで暗黙的にロールバックされる
        _context?.Dispose()
        _connection?.Close()
        _connection?.Dispose()
    End Sub
End Class

この実装の要点は、TransactionScopeをTestInitializeで生成し、TestCleanupでDisposeすることで、明示的なロールバック処理を一切書かずにテスト間の分離を実現している点です。
TransactionScopeはデフォルトでDispose時にコミットされていないトランザクションをロールバックするため、この動作を利用します。

注意すべき3つの落とし穴

このパターンは非常に強力ですが、実運用では以下の点に注意が必要です。

  • トランザクションの昇格問題:SQLiteインメモリは軽量ですが、TransactionScopeが分散トランザクションに昇格すると、MSDTC(Microsoft Distributed Transaction Coordinator)への依存が発生し、テスト環境が複雑化します。これを防ぐには、TransactionScopeのコンストラクタでTransactionScopeOption.RequiresNewを避け、常にRequiredを指定することが重要です
  • 非同期コードとの併用:VB.NETでもAsync/Awaitが一般的になりました。TransactionScopeを非同期メソッドで使う場合は、TransactionScopeAsyncFlowOption.Enabledを必ず指定しないと、非同期コンテキストをまたいでトランザクションが引き継がれません。上記のコード例では既に指定済みです
  • テスト内での明示的コミット禁止:テストメソッド内でscope.Complete()を呼び出すと、テスト終了時にコミットが発生し、他のテストに影響を与えます。原則として、テストコード内でComplete()を絶対に呼ばないという規約をチームで徹底してください

トランザクションスコープとインメモリDB以外の選択肢

もしプロジェクトがSQL Serverなどの実データベースを使い続ける場合でも、トランザクションスコープは有効です。
その場合は、テスト開始前にデータベースのスナップショットを作成し、テスト終了時にリストアする代わりに、トランザクションでラップしてロールバックする方が圧倒的に高速です。
ただし、実DBではスキーマ変更の影響をインメモリほど吸収できないため、ビルダーパターンと組み合わせてテストデータの変更箇所を最小化する工夫が別途必要になります。

テスト並列実行への道筋

テスト独立性が確保されると、次に検討したくなるのが並列実行です。
幸い、xUnit.netやNUnitの最新バージョンでは、アセンブリ単位やクラス単位での並列実行がサポートされています。
トランザクションスコープ+インメモリDBの構成では、各テストクラスが別々のメモリDBインスタンスを持つため、理論上は完全な並列化が可能です。

ただし、実際には静的フィールドや共有リソース(ファイルシステム、環境変数)がボトルネックになることが多いため、まずはクラス単位での並列化から始め、徐々に並列度を上げていくアプローチが現実的です。
並列実行時のデバッグは難易度が高いため、最初の段階では直列実行のままでも、テストの順序依存が解消されただけでも大きな前進です。

ビルダー+インメモリDB+トランザクションの三種の神器

ここまでで、テストデータ生成(ビルダー)、データ格納先(インメモリDB)、実行分離(トランザクションスコープ)という3つのレイヤーが揃いました。
この3つを組み合わせることで、連携テストは外部ファイルに依存せず、スキーマ変更に強く、順序非依存で並列実行可能な、理想的な状態に近づきます。

次の章では、これらのリファクタリングを継続的に維持するための運用ルールと、CIパイプラインへの組み込み方について論じます。
せっかく改善したテストスイートも、運用ルールが曖昧だと再び腐敗が始まります。
長期的な成功のためには、仕組みだけでなくチームの習慣を変えることが不可欠です。

リファクタ後の運用ルールとCIパイプラインへの組み込み

GitHub Actions上でVB.NETテストが並列実行され成功しているダッシュボード画面

ここまでで、ビルダーパターン、インメモリデータベース、トランザクションスコープという3つの技術的施策を紹介しました。
しかし、これらの導入はゴールではなくスタートラインに過ぎません。
本当に重要なのは、リファクタリング後のテストスイートを持続可能な状態に保つ運用ルールを確立し、CIパイプラインに組み込んで自動化することです。
ここでは、現場で即実践できるルールセットと、CI統合のベストプラクティスを解説します。

運用ルール1:新規テストは必ずビルダー経由でデータ生成する

最も基本かつ効果的なルールは、「新しく書くテストは、ビルダーパターン以外の方法でテストデータを生成してはならない」というものです。
これにより、デフォルト値の一貫性が保たれ、将来のスキーマ変更時の修正範囲がビルダークラスに限定されます。

  • コードレビューでビルダー未使用のテストを指摘するチェックリストを用意する
  • ビルダーにないプロパティが必要な場合、そのプロパティをビルダーに追加してから使うようチームで合意する
  • 既存のレガシーテストを修正する際も、ついでにビルダー化する「ボーイスカウトルール」を推奨する

運用ルール2:インメモリDBと実DBの使い分け基準を明文化する

すべてのテストをインメモリDBに置き換える必要はありません。
ストアドプロシージャやビューに依存するテスト、あるいは大量データの集計性能を検証するテストは、実DBのテスト用インスタンスを使う方が適切です。
そこで、以下のような判断基準をドキュメント化します。

  • インメモリDB推奨:単純なCRUD、LINQベースのクエリ、スキーマ変更の影響を受けやすいモジュール
  • 実DB推奨:複雑なJOINを含むレポート、ストアドプロシージャの実行結果、インデックスチューニングが絡む性能検証
  • 併用可:リポジトリ層のテストはインメモリ、サービス層の結合テストは実DB、といったレイヤー別の使い分け

この基準をCIスクリプト内で分岐させることで、テスト実行時間と信頼性のトレードオフを調整できます。

運用ルール3:トランザクションスコープの誤用を防止する静的解析

TransactionScopeは便利ですが、開発者がうっかりComplete()を呼び出すと、テスト分離が破綻します。
これを防ぐために、カスタムのコード解析ルールを導入することを強くお勧めします。
VB.NETのプロジェクトでは、SonarQubeやRoslynアナライザをカスタマイズして、テストメソッド内でのComplete()呼び出しを警告するルールを追加できます。

また、TransactionScopeの生成をテスト基底クラスのみに制限し、各テストメソッドでは直接生成させない設計も有効です。
基底クラスのProtectedメソッドとしてトランザクション制御を隠蔽すれば、誤用の余地が格段に減ります。

CIパイプラインへの組み込み戦略

リファクタリング後のテストスイートは、CIパイプラインの中心的な品質ゲートとして位置づけるべきです。
具体的には、以下のような段階的パイプラインを設計します。

  • ステージ1(高速フィードバック):変更されたコードに関連するテストだけを抽出して実行(NUnitのTestCase属性やカテゴリ分割を活用)
  • ステージ2(フルスイート):全テストを直列で実行。ただし、インメモリDBテストと実DBテストは並列ジョブで分離し、全体の所要時間を短縮
  • ステージ3(レポート生成):テスト実行時間の傾向、失敗率、フレーキーテストの発生件数を可視化するダッシュボードを生成

特にフレーキーテストの自動検出は重要です。
CI上で同一テストを3回連続実行し、結果が変動するテストを自動的にレポートする仕組みを導入すれば、不安定なテストを早期に発見できます。

リファクタ効果の定量的評価指標

運用ルールとCIを整えたら、その効果を数値で測定しましょう。
以下の指標を週次でモニタリングすることを提案します。

指標 リファクタ前の値(例) 目標値 測定頻度
全テスト実行時間 12分 5分以内 毎ビルド
スキーマ変更時のテスト修正工数 2.5人日 0.5人日以内 マイグレーション毎
フレーキーテスト発生数(週間) 7件 0件 週次
新規テストのビルダー採用率 40% 100% 月次

これらの指標をチームのダッシュボードで可視化し、改善が止まったらすぐに対策を講じる体制を整えてください。
指標が悪化し始めたら、それは運用ルールが形骸化しているサインです。

最後に:文化としてのテスト品質

技術的施策と運用ルールは整いました。
しかし、最も重要なのはテストコードを「第一級の資産」として扱うチーム文化です。
プロダクションコードと同じレビュー基準、同じリファクタリングの習慣、同じ品質意識をテストコードにも適用する。
それが実現できたとき、あなたのチームの連携テストは真に持続可能なものとなるでしょう。

次章では、これまでの全内容を総括し、リファクタリング全体を通じて得られる長期的なビジネス価値について改めて整理します。
技術の細部に埋もれがちな「なぜこのリファクタリングが必要だったのか」という本質を、最後にしっかりと見つめ直しましょう。

まとめ:密結合を断ち切り、変更に強いテスト資産を手に入れる

赤いエラーから緑の成功マークに変わり、時計のアイコンが短縮された改善効果のインフォグラフィック

ここまで、VB.NETの連携テストが破綻する根本原因から、具体的なリファクタリング手法、そして運用ルールとCI統合に至るまでを体系的に解説してきました。
最後に、この一連の取り組みがもたらす本質的な価値と、それを持続させるためのマインドセットを整理しておきましょう。

このリファクタリングの目的は、決して「テストコードを美しくする」ことではありません。
ビジネス上の価値に直結する以下の3つの成果を手に入れることにあります。

  • 変更への応答速度の向上:スキーマ変更や要件追加のたびにテスト修正に数日を費やすのではなく、数時間、場合によっては数分で対応できるようになる。これは市場競争力に直結します
  • デバッグコストの劇的な削減:テスト失敗の原因が「データの問題」なのか「実装のバグ」なのかを即座に判断できるようになり、障害対応時間が短縮されます
  • 開発者体験(DX)の改善:テストが信頼できる品質ゲートとして機能することで、開発者は安心してリファクタリングや新機能追加に専念できます。これは結果的に優秀な人材の定着にも寄与します

これらを実現するために、私たちは3つの技術的レイヤーを積み上げました。
ビルダーパターンはデータ生成の柔軟性と保守性を提供し、インメモリデータベースはスキーマ変更に対する物理的耐性を高め、トランザクションスコープはテスト実行の独立性を保証します。
この三層構造は、それぞれが単独でも効果を発揮しますが、組み合わせることで相乗効果を生みます。

ただし、技術の導入だけでは十分ではありません。
運用ルールを明文化し、CIパイプラインに自動化の仕組みを組み込み、指標によるモニタリングを徹底することで、リファクタリング後の状態を「初期状態」として定着させる必要があります。
特に、新規テストに対するビルダー強制や、トランザクションスコープの誤用検出といった静的解析ルールは、チームの規模が大きくなるほど重要な役割を果たします。

ここで、ある誤解を解いておきたいと思います。
このリファクタリングは「完璧なテスト」を目指すものではありません。
むしろ、「変更に弱いテスト」から「変更に強いテスト」へと性質を変えることがゴールです。
完璧を求めてリファクタリングを永遠に続けるのではなく、現状のボトルネックを解消し、次の改善サイクルに移行する準備が整ったら、そこで一旦区切りをつける勇気も必要です。

また、このアプローチはVB.NETに限定されません。
JavaのJUnit、C#のxUnit、Pythonのpytestなど、どんな言語・フレームワークでも同様の考え方が適用できます。
本質は「テストデータを外部リソースに依存させない」という設計原則であり、それはプログラミング言語を超えた普遍的な教訓です。

最後に、このリファクタリングを始めるにあたって、小さな一歩を提案します。
明日からでも、最も頻繁に変更されるテーブルに依存するテストクラスを1つ選び、そのデータ生成部分だけをビルダーに置き換えてみてください。
たったそれだけの行為が、チーム内に「変えられる」という手応えを生み、その手応えが次のアクションを後押しします。

密結合を断ち切ることは、単にテストコードの改善にとどまりません。
それは、変化を恐れない開発文化の基盤を築くことでもあります。
テストがあなたの味方になり、変更が楽しいものになるとき、ソフトウェア開発の本質的な面白さが再び見えてくるはずです。
今日から、その第一歩を踏み出してください。

コメント

タイトルとURLをコピーしました