SC_Response::sendRedirect() transactionid= を画一的に追加せず、一定条件での継承のみとする #922

[seasoftjapan]

fixed #922

[codecov]

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 55.63%. Comparing base (3c70f3c) to head (146e122).

Files	Patch %	Lines
data/class/SC_Response.php	20.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #960      +/-   ##
==========================================
- Coverage   55.72%   55.63%   -0.10%     
==========================================
  Files          75       75              
  Lines        8913     8917       +4     
==========================================
- Hits         4967     4961       -6     
- Misses       3946     3956      +10     

Flag	Coverage Δ
tests	55.63% <20.00%> (-0.10%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[seasoftjapan]

@nanasess デバッグ中に何かの都合(詳細失念)でポート番号を変更していたのですが、改めて試すと :9999 とか、テキトーな番号でもローカルの phpunit は通るようでした。
tests/class/fixtures/server/common.php で putenv('HTTP_URL=http://127.0.0.1:8085/'); しているから、実際のポート番号は何でも良い感じでしょうか。
そうなると、8085 は避けて、元の 8053 にしておくのが適切でしょうか?

[nanasess]

ここで使用するポート番号はビルトインウェブサーバーが LISTEN する番号ですので、他と被らなければ何でもいいと思います。
8085 でも OK です

[seasoftjapan]

承知いたしました。とりあえず、現状で大丈夫そうですね。
パラレル実行するときに問題となりそうなので、一応 Issue 登録しました。#964

[nanasess]

LGTM

[nanasess]

ここで使用するポート番号はビルトインウェブサーバーが LISTEN する番号ですので、他と被らなければ何でもいいと思います。
8085 でも OK です