openstack/mistral
Code Issues Proposed changes

Merge "Address a TODO comment in the legacy action provider test"

Browse Source
This commit is contained in:
Zuul 2020-10-06 07:44:30 +00:00 committed by Gerrit Code Review
commit a8049a2469
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
mistral/tests/unit/actions/test_legacy_action_provider.py
View File

@ -120,6 +120,11 @@ class LegacyActionProviderTest(base.BaseTest):
) )
self.assertEqual('output, delay=0', action_desc.params_spec) self.assertEqual('output, delay=0', action_desc.params_spec)
@mock.patch.object(
legacy.LegacyActionProvider,
'_get_action_generators',
mock.MagicMock(return_value=[TestActionGenerator])
)
def test_only_action_plugins(self): def test_only_action_plugins(self):
self.override_config( self.override_config(
'load_action_generators', 'load_action_generators',
@ -129,9 +134,6 @@ class LegacyActionProviderTest(base.BaseTest):
provider = legacy.LegacyActionProvider() provider = legacy.LegacyActionProvider()
# TODO(rakhmerov): Implement loading actions from generators
# and test with a generator.
action_descs = provider.find_all() action_descs = provider.find_all()
prefix = 'mistral.actions.std_actions' prefix = 'mistral.actions.std_actions'