Fix wrong method name assert_called_once
There isn't method assert_called_once, and it doesn't really check anything. We'd better check mock.called directly. Change-Id: Iffbaa176c3efd41084a655adcbbb0821d8ad496d
This commit is contained in:
@@ -106,7 +106,7 @@ def test_clean_up_error_debug():
|
|||||||
else:
|
else:
|
||||||
assert False, 'Should have had an exception'
|
assert False, 'Should have had an exception'
|
||||||
|
|
||||||
app.clean_up.assert_called_once()
|
assert app.clean_up.called
|
||||||
call_args = app.clean_up.call_args_list[0]
|
call_args = app.clean_up.call_args_list[0]
|
||||||
assert call_args == mock.call(mock.ANY, 1, mock.ANY)
|
assert call_args == mock.call(mock.ANY, 1, mock.ANY)
|
||||||
args, kwargs = call_args
|
args, kwargs = call_args
|
||||||
@@ -123,7 +123,7 @@ def test_error_handling_clean_up_raises_exception():
|
|||||||
)
|
)
|
||||||
app.run(['error'])
|
app.run(['error'])
|
||||||
|
|
||||||
app.clean_up.assert_called_once()
|
assert app.clean_up.called
|
||||||
call_args = app.clean_up.call_args_list[0]
|
call_args = app.clean_up.call_args_list[0]
|
||||||
assert call_args == mock.call(mock.ANY, 1, mock.ANY)
|
assert call_args == mock.call(mock.ANY, 1, mock.ANY)
|
||||||
args, kwargs = call_args
|
args, kwargs = call_args
|
||||||
@@ -150,7 +150,7 @@ def test_error_handling_clean_up_raises_exception_debug():
|
|||||||
else:
|
else:
|
||||||
assert False, 'Should have had an exception'
|
assert False, 'Should have had an exception'
|
||||||
|
|
||||||
app.clean_up.assert_called_once()
|
assert app.clean_up.called
|
||||||
call_args = app.clean_up.call_args_list[0]
|
call_args = app.clean_up.call_args_list[0]
|
||||||
assert call_args == mock.call(mock.ANY, 1, mock.ANY)
|
assert call_args == mock.call(mock.ANY, 1, mock.ANY)
|
||||||
args, kwargs = call_args
|
args, kwargs = call_args
|
||||||
@@ -167,7 +167,7 @@ def test_normal_clean_up_raises_exception():
|
|||||||
)
|
)
|
||||||
app.run(['mock'])
|
app.run(['mock'])
|
||||||
|
|
||||||
app.clean_up.assert_called_once()
|
assert app.clean_up.called
|
||||||
call_args = app.clean_up.call_args_list[0]
|
call_args = app.clean_up.call_args_list[0]
|
||||||
assert call_args == mock.call(mock.ANY, 0, None)
|
assert call_args == mock.call(mock.ANY, 0, None)
|
||||||
|
|
||||||
@@ -181,7 +181,7 @@ def test_normal_clean_up_raises_exception_debug():
|
|||||||
)
|
)
|
||||||
app.run(['--debug', 'mock'])
|
app.run(['--debug', 'mock'])
|
||||||
|
|
||||||
app.clean_up.assert_called_once()
|
assert app.clean_up.called
|
||||||
call_args = app.clean_up.call_args_list[0]
|
call_args = app.clean_up.call_args_list[0]
|
||||||
assert call_args == mock.call(mock.ANY, 0, None)
|
assert call_args == mock.call(mock.ANY, 0, None)
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user