Ticket #2572 (closed バグ指摘: 修正済)

Opened 6 years ago

Last modified 6 years ago

テスト> 税率5%でしか考慮されていない部分がある

Reported by: shutta Owned by: shutta
Priority: Milestone: EC-CUBE2.13.3
Component: その他 Version: 2.13.1
Keywords: Cc:
修正済み: yes

Description

税率が5%でしか考慮されていない部分があるので、5%以外に設定しているとテストにこける。

Change History

comment:1 Changed 6 years ago by shutta

  • Owner changed from somebody to shutta
  • 修正済み set
  • Status changed from new to assigned

r23526 にて修正。

ただ、他の箇所に倣ってSC_Helper_TaxRule_Ex::sfCalcIncTaxを使用するようにしたが、そもそもテストとしては厳密に税率計算した結果の数値をダイレクトに指定した方が良いのでしょうか?
その場合は、SC_Helper_TaxRule_Ex::sfCalcIncTaxを使用している部分は、現行の税率8%で計算した数値に書き換え、税率が改正された際は都度最新の税率の数値へ更新していくような感じですかね。

comment:2 Changed 6 years ago by kim

  • Status changed from assigned to closed
  • Resolution set to 修正済

shutta 様

sfCalcIncTax のテストは別途、手計算した内容で試験されているという前提で、本対応で問題ないと思います。

こちらでクローズさせて頂ます。

Note: See TracTickets for help on using tickets.