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

Opened 6 years ago

Last modified 6 years ago

テスト> SC_Utils_isInternalDomainTestの改修

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

Description

  1. 一致するケースのURLが決め打ちになっている( http://test.local.jp/)
    テストする環境に合わせて、URLを変更しないといけないので、設定情報のURLを利用するようにさせたい。
  1. 「testsfIsInternalDomain_ドメインが一致しない場合_falseが返る」のテストが間違っている
    テスト内容に反したexpectedが指定されている。

Change History

comment:1 Changed 6 years ago by shutta

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

r23524 にて修正。

comment:2 Changed 6 years ago by shutta

でも、テストとしてはURLは設定情報を利用せずに、厳密に通るはずのURLをわざわざ指定するようにした方が良いんですかね?

comment:3 Changed 6 years ago by kim

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

shutta 様

どちらも一長一短だとは思いますが、EC-CUBEのテストケースの利用用途を考えると

ご対応の内容で問題ないと思います。

クローズいたします。

Note: See TracTickets for help on using tickets.