105

Unit testing conn() using mock:

app.py

import mysql.connector
import os, urlparse


def conn():
    if "DATABASE_URL" in os.environ:
        url = urlparse(os.environ["DATABASE_URL"])
        g.db = mysql.connector.connect(
            user=url.username,
            password=url.password,
            host=url.hostname,
            database=url.path[1:],
        )
    else:
        return "Error"

test.py

def test_conn(self):
    with patch(app.mysql.connector) as mock_mysql:
        with patch(app.os.environ) as mock_environ:
            con()
            mock_mysql.connect.assert_callled_with("credentials")

Error: Assertion mock_mysql.connect.assert_called_with is not called.

which I believe it is because 'Database_url' is not in my patched os.environ and because of that test call is not made to mysql_mock.connect.

Questions:

  1. What changes do I need to make this test code work?

  2. Do I also have to patch urlparse?

funnydman
  • 9,083
  • 4
  • 40
  • 55
immrsteel
  • 1,333
  • 4
  • 13
  • 18

7 Answers7

145

You can try unittest.mock.patch.dict solution. Just call conn with a dummy argument:

import mysql.connector
import os, urlparse
from unittest import mock


@mock.patch.dict(os.environ, {"DATABASE_URL": "mytemp"}, clear=True)  # why need clear=True explained here https://stackoverflow.com/a/67477901/248616
# If clear is true then the dictionary will be cleared before the new values are set.
def conn(mock_A):
    print os.environ["mytemp"]
    if "DATABASE_URL" in os.environ:
        url = urlparse(os.environ["DATABASE_URL"])
        g.db = mysql.connector.connect(
            user=url.username,
            password=url.password,
            host=url.hostname,
            database=url.path[1:],
        )
    else:
        return "Error"

Or if you don't want to modify your original function try this solution:

import os
from unittest import mock

def func():
    print os.environ["mytemp"]


def test_func():
    k = mock.patch.dict(os.environ, {"mytemp": "mytemp"})
    k.start()
    func()
    k.stop()


test_func()
vks
  • 67,027
  • 10
  • 91
  • 124
  • As it doesn't make a difference for the question / answer, I removed the wrong Python code from both :-) – Martin Thoma Jun 30 '20 at 15:15
  • I just want to highlight the `patch.dict()` option `clear=True`! I intuitively expected this to be the default behavior, plus the `unittest` docs examples are technically correct but give the wrong impression, and don't highlight the `clear` option. – Mike B Sep 29 '22 at 20:37
51

For this, I find that pytest's monkeypatch fixture leads to better code when you need to set environment variables:

def test_conn(monkeypatch):
    monkeypatch.setenv('DATABASE_URL', '<URL WITH CREDENTIAL PARAMETERS>')
    with patch(app.mysql.connector) as mock_mysql:
        conn()
    mock_mysql.connect.assert_called_with(<CREDENTIAL PARAMETERS>)
Toote
  • 3,323
  • 2
  • 19
  • 25
17

The accepted answer is correct. Here's a decorator @mockenv to do the same.

def mockenv(**envvars):
    return mock.patch.dict(os.environ, envvars)


@mockenv(DATABASE_URL="foo", EMAIL="bar@gmail.com")
def test_something():
    assert os.getenv("DATABASE_URL") == "foo"

Adam Hughes
  • 14,601
  • 12
  • 83
  • 122
13

In my use case, I was trying to mock having NO environmental variable set. To do that, make sure you add clear=True to your patch.

with patch.dict(os.environ, {}, clear=True):
    func()
Caleb Robinson
  • 1,010
  • 13
  • 21
10

At the head of your file mock environ before importing your module:

with patch.dict(os.environ, {'key': 'mock-value'}):
    import your.module
0script0
  • 471
  • 5
  • 11
  • 1
    This helped me out because the key and value I needed were required at import time rather than at the time of the function call – Yirmi Aug 19 '22 at 19:22
3

You can also use something like the modified_environ context manager describe in this question to set/restore the environment variables.

with modified_environ(DATABASE_URL='mytemp'):
    func()
Community
  • 1
  • 1
Laurent LAPORTE
  • 21,958
  • 6
  • 58
  • 103
1

Little improvement to answer here

@mock.patch.dict(os.environ, {"DATABASE_URL": "foo", "EMAIL": "bar@gmail.com"})
def test_something():
    assert os.getenv("DATABASE_URL") == "foo"