質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.51%

  • Python

    11683questions

    Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

  • Python 3.x

    9776questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

Python3のmock.patchがurlopenでうまく働かないのはなぜでしょうか?

解決済

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 1,213

uni8inu

score 117

前提・実現したいこと

PythonのUnitテストを作成しております。
urlopen()を使用している部分の異常時ケースのテストを作成したく思います。

mock.patchを使ってurlopen()をダミー関数に置き換えるようにしました。
しかし、import文の書き方によって、うまくmockに置き換わらない場合がありました。
これはmockの仕様なのでしょうか?それとも自分の記述方法が良くないためでしょうか?
お詳しい方、ご教示いただけますでしょうか?

テスト失敗時

テスト対象

# terateil_q/my_urlopen.py
from urllib.request import urlopen # this is test NG

def my_urlopen_1():

    url = "https://www.google.co.jp/"
    response = urlopen(url)

    if response.getcode() == 200:
        return True
    else:
        return False

テストケース

#terateil_q/test_my_urlopen.py
from unittest import mock
from unittest import TestCase
from terateil_q.my_urlopen import *

class MyUrlopenTests(TestCase):
    def test_my_urlopen_normal(self):
        retval = my_urlopen_1()
        self.assertTrue(retval)  # test pass

    def test_my_urlopen_codeerror2(self):
        def dummy_urlopen(*args, **kwargs):
            """ urlopen mock
            :return HTTPRespnse duck typing
            """
            class Dummy_response():
                def __init__(self):
                    self.code = 404

                def getcode(self):
                    return self.code

            return Dummy_response()

        with mock.patch("urllib.request.urlopen", side_effect=dummy_urlopen) as m:
            retval = my_urlopen_1()
            self.assertFalse(retval)  # test NG / AssertionError: True is not false

pdbでトレースした所、urlopenがmockに置き換わっておりませんでした。
そのため普通にurlopenが動いて、retvalがTrueとなり、結果として、テストがNGとなりました。

うまくいったパターン1

インポートタイミングを変えるとtest passしました。動的インポートなので実行速度的に気分悪いです。

# terateil_q/my_urlopen.py
def my_urlopen_1():
    from urllib.request import urlopen # change import timeng / this is test OK

    url = "https://www.google.co.jp/"
    response = urlopen(url)

    if response.getcode() == 200:
        return True
    else:
        return False

うまくいったパターン2

import宣言を変えてもtest passしました。importにトップレベルパッケージを指定しました。
関数呼び出し部分が非常に残念になってしまいました・・・。

# terateil_q/my_urlopen.py
import urllib # change this

def my_urlopen_1():

    url = "https://www.google.co.jp/"
    response = urllib.request.urlopen(url) # kanasii call

    if response.getcode() == 200:
        return True
    else:
        return False


この方法を毎回使わないといけないでしょうか?
テスト対象をテスト用に変更しなければならないというのは、なんか残念な感じがしております。

補足情報(言語/FW/ツール等のバージョンなど)

Python3
PyCharm

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

mock.patch は名前空間内での名前が参照しているオブジェクトを差し替えます。
このため、対象オブジェクトが別の名前空間にimport済みの場合、元の名前空間で差し替えても、別の名前空間では効果がありません。

今回の例でいうと、 urllib.requests.urlopen の差し替えを urllib.requests モジュールの名前空間内で行っていますが、そのオブジェクト(urlopen)はすでに my_urlopen 名前空間にimport済みです。
importした方のオブジェクトをmock.patchで差し替えてください。

with mock.patch("urllib.request.urlopen", side_effect=dummy_urlopen) as m:


これを以下のようにしてください

with mock.patch("my_urlopen.urlopen", side_effect=dummy_urlopen) as m:

詳しくは http://docs.python.jp/3/library/unittest.mock.html#where-to-patch を参照してください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/11/24 16:09

    なるほど!よくわかりました。指摘いただいたコードで期待する動作になりました。
    ご教示いただいたpageも参考いたします。ありがとうございました。

    キャンセル

同じタグがついた質問を見る

  • Python

    11683questions

    Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

  • Python 3.x

    9776questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。