OpenPNE 3

このチケットの 3.4.x 系へのバックポートチケット #2413 で以下のように示されているので、本チケットでも言及しておきます。

動作確認が困難なチケットですので、レビューのみでOKとします。

これまでの「携帯電話IPアドレス帯域リストの更新」では、その内容の性質上、動作テストは行なっていないようです（詳細は後述）。対応時のレビューおよび、定期的な確認（IPリストに誤りがないか、キャリアが新しい情報を公開していないか）が重要ということです。

ところで、IPアドレス帯域リストの更新作業は定期的に必要な作業であると思われるにも拘らず、過去のチケットを検索してみても、そのチケットタイトルやチケットへの記述が十分でなく統一もされていないようです。

このチケットは現在テスト待ちですが、これまで同様おそらく動作テストは行わないと思われます。しかしながら、テスト待ち状態から完了へステータスを変更するのは基本的にテスターに任せることにします（テストしないことと、テストしないでよいと判断することは意味が異なるため）。

動作テストを不要としている背景¶

IPアドレス帯域リストの更新に伴って行うべき動作テストについて、次の目的のテストが考えられます。

• (t1) IPアドレス帯域リストで記述しているアクセスを許可すべき IP アドレスについて、「その全ての IP アドレスから」アクセスできるか
• (t2) IPアドレス帯域リストのファイルが読み込まれ、それにより IP 制限が機能するか。記述されている「いくつかの IP アドレスから」はアクセスでき、記述していない IP からはアクセスできない動作となっているか

IP アドレス帯域リストを用意している目的は次のものです。

• (p1) 携帯版ページは、携帯からアクセスされることを想定しており、 PC などから携帯ページにアクセスできないようにしたいためにこれを使って IP 制限を行う（未確認だが、おそらくもともとの目的はこれ）
• (p2) 個体識別番号によるかんたんログインを用いている場合に、なりすましログインを防ぐというセキュリティ上の理由で IP 制限を行う（これについては http://www.openpne.jp/pne-downloads/mobile_ip_list/ から情報を辿れる）

後者 (p2) は、スマートフォンや JavaScript を使用できるフィーチャーフォンの登場により、 OpenPNE としては 2010 年初頭に顕在化した問題です。

一般的な動作テストの目的は、ソースコードが正しそうな（つまり適切なレビューが済んでいる）場合にも、本当に動作上に問題がないかを確認するためであり、手間を考えなければ、本来は (p1) のために (t1) を、 (p2) のために (t2) を行なうべきです。

しかし次の理由で、最近ではIPアドレス帯域リストの更新時に動作テストは行なわれていません。

• (t1) を行なっていない理由
  • 記述している特定の IP アドレスについて、その IP からアクセスするというテストが容易ではない（携帯端末からのアクセスに対する IP アドレスは、各キャリアのゲートウェイ通過時に付与されるため、携帯端末に対して固定の IP が使われるわけではなく、またユーザが使われる IP を指定することができない）
• (t2) を行なっていない理由
  • 記述しているいくつかの IP についてアクセスが許可され、記述していない IP についてアクセスが拒否されるかについては、「更新前には動作していた」ことを前提にしている
  • IPアドレス帯域リストが読み込まれるかは、更新時に毎回動作確認する必要がないと判断している（詳細は最下部の段落を参照）
    • これはファイルに文法エラーなどが無いことを前提としており、その前提はレビューによって保証している（つまりテスト段階では IP 制限自体が動作しない可能性を考えていない）。
  • 追加あるいは削除した IP について、その IP からのアクセスが想定通りの結果に変わっているかどうかについては、 (t1) を行わない理由と同様の理由で行なっていない

(t1) について想定と異なる不具合があった場合は、「携帯ページにアクセスできるはずの携帯端末なのに、携帯ページにアクセスできない」という問題が生じることになり、 (t2) について想定と異なる不具合があった場合は、「個体識別番号を用いたかんたんログイン方式の悪用により、なりすましログインがされてしまう」という問題が生じることになります（なりすましが可能な端末で使われる IP を許可してしまっているということ）。

(t2) の動作テストが失敗する可能性については、実装者テスト（テスターテストとは異なるフェーズ）とレビュー、および「これを動作テスト項目としなくても OpenPNE を動作させる時点で問題があれば気付く可能性が高いこと」によって防いでいるのではないかと考えます。