テストが落ちる問題の修正

[nobuhiko]

guzzleかmailcatcherの返り値が変わったぽい

[nanasess]

PHPStan のエラーは別PRで対応します

[nanasess]

@nobuhiko Guzzle が SecurityChecker にひっかかってるので、 curl とか別の実装にした方がよいかもですね
https://github.com/EC-CUBE/ec-cube2/runs/7570709045?check_suite_focus=true

[nobuhiko]

@nanasess phpunitのバージョンに依存してGuzzleの低いバージョンが入っちゃってる感じなんですかね。

[nobuhiko]

安全なバージョンだと6.5.8だけど php: >=5.5
https://packagist.org/packages/guzzlehttp/guzzle#6.5.8

[nanasess]

@nobuhiko 現状、 MailCatcher API に対して get と delete しかしてないので、 file_get_contents() と delete だけ curl とかでもよいかもしれないですね。
ちょっと時間みつけてやってみようと思います

[nobuhiko]

@nanasess file_get_contentsで全部対処しときました

[codecov-commenter]

Codecov Report

Merging #562 (30bb157) into master (4ee4912) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #562   +/-   ##
=======================================
  Coverage   54.63%   54.63%           
=======================================
  Files          76       76           
  Lines        9058     9058           
=======================================
  Hits         4949     4949           
  Misses       4109     4109           

Flag	Coverage Δ
tests	54.63% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4ee4912...30bb157. Read the comment docs.

[nanasess]

@nobuhiko すばらしい！ありがとうございます！

[chihiro-adachi]

@nobuhiko
ありがとうございます、マージしました