It was using a non-existing method that did nothing and
that masked other problems with the tests that used it.
Changed to use assert_has_calls() and fixed the problems
that fell out.
(cherry picked from commit b85cfa96118d73ad87b150e488f295cbf9a2c140)
Reviewed: https:/ /review. openstack. org/200654 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=5eff0019dc9 28b9f6d132d1893 18c757457b7e8b
Committed: https:/
Submitter: Jenkins
Branch: stable/kilo
commit 5eff0019dc928b9 f6d132d189318c7 57457b7e8b
Author: Kevin Benton <email address hidden>
Date: Thu Jul 9 16:54:23 2015 -0700
Fix failures introduced by the new version of mock
===
'assert_ has_calls' that have changed behavior in the new version
(no longer accepts single calls).
Calls to non-existent assert methods that did not exist and were
silently passing.
Use of autospec on a class with decorated functions.
Conflicts:
neutron/ tests/base. py
(cherry picked from commit 3a871a73b88f8b7 5d64c88271b6fc6 cdd4b1cfc0)
===
Fix dhcp _test_sync_ state_helper asserting calls wrong
It was using a non-existing method that did nothing and
that masked other problems with the tests that used it.
Changed to use assert_has_calls() and fixed the problems
that fell out.
(cherry picked from commit b85cfa96118d73a d87b150e488f295 cbf9a2c140)
===
Closes-Bug: #1473369 0f0229ec3c5071f 7a470445c98
Change-Id: I164a9af2a7f9ac