Closed omochi closed 4 months ago
CommandTestHelper.swift に書かれたテスト用のユーティリティメソッドが、 XCTest のエクステンションとして定義されています。
CommandTestHelper.swift
XCTest
内容が特に XCTest のインスタンスと紐付ける必要がないもので、 設計として不適切なため脱エクステンションします。
将来的にこの辺りのプロセス制御系のコードを、 プロジェクト全体で整理したいので、 その際の変更を小さくする意図もあります。
なお、経緯を調べたところ、最初にこのエクステションを定義した時には、 self を参照してそこから Bundle を取得するロジックが書かれていたので、 メソッドとして定義するのは合理的です。 https://github.com/swiftwasm/carton/blob/5fe8c8edac087f1ed3de487c327a6621e622d1b1/Tests/CartonCLITests/CommandTestHelper.swift#L181 今はそのメンバはありません。
self
Bundle
CommandTestHelper.swift
に書かれたテスト用のユーティリティメソッドが、XCTest
のエクステンションとして定義されています。内容が特に
XCTest
のインスタンスと紐付ける必要がないもので、 設計として不適切なため脱エクステンションします。将来的にこの辺りのプロセス制御系のコードを、 プロジェクト全体で整理したいので、 その際の変更を小さくする意図もあります。
なお、経緯を調べたところ、最初にこのエクステションを定義した時には、
self
を参照してそこからBundle
を取得するロジックが書かれていたので、 メソッドとして定義するのは合理的です。 https://github.com/swiftwasm/carton/blob/5fe8c8edac087f1ed3de487c327a6621e622d1b1/Tests/CartonCLITests/CommandTestHelper.swift#L181 今はそのメンバはありません。