Update unit tests for the logging system.

[Takym]

cocoa-mhlw#67 の単体テストのエラーを修正する為に実験しています。
この PR は差分を確認する為に使っています。

[Takym]

cocoa-mhlw#67 (comment) から

テスト失敗について

ログを見る限り下記の処理で失敗している様です。

cocoa/Covid19Radar/Tests/Covid19Radar.UnitTests/Services/Logs/LoggerServiceTests.cs

Line 44 in bc848ce

using (var sr = new StreamReader("~/.cocoa/logs/cocoa_log_20201101.csv"))

cocoa/Covid19Radar/Tests/Covid19Radar.UnitTests/Services/Logs/LoggerServiceTests.cs

Line 88 in bc848ce

Mock.Get(mockILogPathService).Verify(s => s.LogsDirPath, Times.Exactly(2));

cocoa/Covid19Radar/Tests/Covid19Radar.UnitTests/Services/Logs/LoggerServiceTests.cs

Line 107 in bc848ce

Mock.Get(mockILogPathService).Verify(s => s.LogsDirPath, Times.Once());

cocoa/Covid19Radar/Tests/Covid19Radar.UnitTests/Services/Logs/LoggerServiceTests.cs

Line 126 in bc848ce

Mock.Get(mockILogPathService).Verify(s => s.LogsDirPath, Times.Once());

[Takym]

cocoa-mhlw#67 (comment) から

テスト失敗について

自分の環境の Visual Studio で実験してみた所、「System.IO.IOException : The process cannot access the file '<ログファイル名>' because it is being used by another process.」というエラーが出て失敗しました。しかし、LogWriter.LogFile ではファイルの共有は読み取りと書き込みの両方を許可している筈です。

追記1

DeleteLogsDirIfExists を無効化して試しています。

追記2

LoggerServiceTests では文字列でファイル名を指定しています。そして、文字列でファイル名を渡した場合、StreamReader のコンストラクタ内部では FileShare.Read でストリームを作成している様です。これが原因の可能性もありそうです。

[Takym]

1fd2885 によりファイルの共有エラーは改善したが、別のエラーが発生した。

追記

原因は DeleteLogsDirIfExists を無効化しているためだった。

[Takym]

9cef515 により DeleteLogsDirIfExists を再び有効化した。殆どのエラーは消えたが、まだ動かないテストケースもある。

• Info_Success_Async
• StartMethod_Success
• StartMethod_Success_LogsDir_Exists_LogFile_Exists
• StartMethod_Success_LogsDir_Exists_LogFile_Not_Exists
• StartMethod_Success_LogsDir_Not_Exists
• OpenWebViewCommandTests

[Takym]

テストケースを少し修正したら更にエラーが改善された。
CSV は正規表現で解析していたが、読み難かったので解析処理を CocoaLogViewer から拝借してきた。

---

HomePage の OpenWebViewCommandTests は恐らく PC の言語（日本語）とテストケースの言語（英語）が一致しない為にエラーが発生している。

[Takym]

残りのエラーはマルチタスクが原因であると思われる。

[Takym]

StartMethod_Success は null 値が原因でエラーが発生していた（修正済み）。

[Takym]

排他制御に関する問題の原因は不明。

[Takym]

排他制御に関する問題は修正した。

残りのエラーは下記の4つ。

• StartMethod_Success_LogsDir_Exists_LogFile_Exists
• StartMethod_Success_LogsDir_Exists_LogFile_Not_Exists
• StartMethod_Success_LogsDir_Not_Exists
• OpenWebViewCommandTests (単体テストの言語依存を解消 (#269) cocoa-mhlw/cocoa#272 により解決)

[Takym]

下記三つの失敗の原因は LogFilePath の呼び出し回数によるものだった。

• StartMethod_Success_LogsDir_Exists_LogFile_Exists
• StartMethod_Success_LogsDir_Exists_LogFile_Not_Exists
• StartMethod_Success_LogsDir_Not_Exists