CMemory::AppendRawDataのテストを追加

[beru]

PR の目的

CMemory::AppendRawData のテストを追加する事で不具合が検出されやすいようにします。

カテゴリ

• リファクタリング

PR の背景

#1638 で不具合の修正が行われたので、今後同様の不具合が入る事を防止する為にテストを追加しました。

PR のメリット

今後 CMemory::AppendRawData の実装が再度変更された際に同様の不具合が入る事を防ぎやすくなるのではないかと思います。

PR のデメリット (トレードオフとかあれば)

テストの実行時間が少し増えます。

仕様・動作説明

テスト追加なので本体の動作には影響はありません。

PR の影響範囲

テスト追加なので本体の動作には影響はありません。

テスト内容

• Test Explorer で追加したテスト CMemory, AppendRawData を実行して緑の Passed になる事を確認しました。
• CMemory::AppendRawData()でAllocBuffer条件ミスを修正 #1638 の修正を revert してから sakura プロジェクトをビルド後に、Test Explorer で追加したテスト CMemory, AppendRawData を実行すると赤の Failed になる事を確認しました。

関連 issue, PR

#1638

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[AppVeyorBot]

✅ Build sakura 1.0.3703 completed (commit aa462973bb by @beru)

[berryzplus]

CMemoryは将来的に、完全に廃止したいと考えています。

廃止する理由「テストコードのないレガシーコードなので」を考えていましたが、そういえばこないだの変更でテストコード付けたのを思い出し、「特に問題ない」の結論に至りました。

テスト内容自体は問題ないと思います。

[beru]

レビューありがとうございます。Mergeします。