Reviewed: https://review.opendev.org/650171
Committed:
https://git.openstack.org/cgit/openstack/nova/commit/?id=87b7f0c2aff153eecd8bdf3cb5503287d0e77a24
Submitter: Zuul
Branch: master
commit 87b7f0c2aff153eecd8bdf3cb5503287d0e77a24
Author: melanie witt <melwi...@gmail.com>
Date: Fri Apr 5 01:24:23 2019 +0000
Fix SynchronousThreadPoolExecutorFixture mock of Future
The fixture is not mocking the Future.result() method correctly and an
attempt to access it results in the following error:
AttributeError: Mock object has no attribute 'result'
Instead of mocking the submit() method, we can just swap out the
GreenThreadPoolExecutor for the SynchronousExecutor to make the
executor synchronous for testing.
A few changes were also needed in unit tests that are running with the
thread pool executor. Because the thread pool executor is created in
the compute manager's __init__ method, we need to patch the synchronous
executor in before calling the base test class's setUp method. Next, a
couple of unit tests were asserting that an exception would be raised
from the thread pool executor's submit() method but that will not
happen because if an exception is raised during a submitted task, that
exception will not be raised until Future.result() is called. And
Future.result() is never called in the compute manager.
Closes-Bug: #1823278
Change-Id: I719c222a5fa3d654e7397f126a1338e653b7bbcc
** Changed in: nova
Status: In Progress => Fix Released
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1823278
Title:
SynchronousThreadPoolExecutorFixture mock spec does not work
Status in OpenStack Compute (nova):
Fix Released
Bug description:
The fixture's fake submit() method returns a mocked future that is
intended to be a Mock with spec=futurist.Future but the speccing
doesn't work because the spec is passed a string when it needs to be a
class or an instance. Because of the bug, code like this:
self.useFixture(fixtures.SynchronousThreadPoolExecutorFixture())
executor = futurist.GreenThreadPoolExecutor()
future = executor.submit(...)
result = future.result()
raises the following error:
AttributeError: Mock object has no attribute 'result'
The mock spec just needs to be changed from a string to a class or
instance.
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1823278/+subscriptions
--
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : yahoo-eng-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help : https://help.launchpad.net/ListHelp
