diff --git a/watcher/objects/audit.py b/watcher/objects/audit.py index 53d4d6146..fc044ab21 100644 --- a/watcher/objects/audit.py +++ b/watcher/objects/audit.py @@ -61,7 +61,6 @@ from watcher.objects import fields as wfields class State(object): ONGOING = 'ONGOING' SUCCEEDED = 'SUCCEEDED' - SUBMITTED = 'SUBMITTED' FAILED = 'FAILED' CANCELLED = 'CANCELLED' DELETED = 'DELETED' diff --git a/watcher/tests/api/v1/test_actions.py b/watcher/tests/api/v1/test_actions.py index b86259c40..10be8bc45 100644 --- a/watcher/tests/api/v1/test_actions.py +++ b/watcher/tests/api/v1/test_actions.py @@ -444,7 +444,7 @@ class TestPatch(api_base.FunctionalTest): def test_patch_not_allowed(self, mock_utcnow): test_time = datetime.datetime(2000, 1, 1, 0, 0) mock_utcnow.return_value = test_time - new_state = objects.audit.State.SUBMITTED + new_state = objects.audit.State.SUCCEEDED response = self.get_json('/actions/%s' % self.action.uuid) self.assertNotEqual(new_state, response['state']) diff --git a/watcher/tests/api/v1/test_audit_templates.py b/watcher/tests/api/v1/test_audit_templates.py index e17443db4..651b06b14 100644 --- a/watcher/tests/api/v1/test_audit_templates.py +++ b/watcher/tests/api/v1/test_audit_templates.py @@ -697,7 +697,7 @@ class TestAuditTemplatePolicyEnforcement(api_base.FunctionalTest): self._common_policy_check( "audit_template:update", self.patch_json, '/audit_templates/%s' % audit_template.uuid, - [{'path': '/state', 'value': objects.audit.State.SUBMITTED, + [{'path': '/state', 'value': objects.audit.State.SUCCEEDED, 'op': 'replace'}], expect_errors=True) def test_policy_disallow_create(self): diff --git a/watcher/tests/api/v1/test_audits.py b/watcher/tests/api/v1/test_audits.py index e523cfddc..6853a8224 100644 --- a/watcher/tests/api/v1/test_audits.py +++ b/watcher/tests/api/v1/test_audits.py @@ -267,7 +267,7 @@ class TestPatch(api_base.FunctionalTest): test_time = datetime.datetime(2000, 1, 1, 0, 0) mock_utcnow.return_value = test_time - new_state = objects.audit.State.SUBMITTED + new_state = objects.audit.State.SUCCEEDED response = self.get_json('/audits/%s' % self.audit.uuid) self.assertNotEqual(new_state, response['state']) @@ -287,7 +287,7 @@ class TestPatch(api_base.FunctionalTest): def test_replace_non_existent_audit(self): response = self.patch_json( '/audits/%s' % utils.generate_uuid(), - [{'path': '/state', 'value': objects.audit.State.SUBMITTED, + [{'path': '/state', 'value': objects.audit.State.SUCCEEDED, 'op': 'replace'}], expect_errors=True) self.assertEqual(404, response.status_int) self.assertEqual('application/json', response.content_type) @@ -722,7 +722,7 @@ class TestAuditPolicyEnforcement(api_base.FunctionalTest): self._common_policy_check( "audit:update", self.patch_json, '/audits/%s' % audit.uuid, - [{'path': '/state', 'value': objects.audit.State.SUBMITTED, + [{'path': '/state', 'value': objects.audit.State.SUCCEEDED, 'op': 'replace'}], expect_errors=True) def test_policy_disallow_create(self): diff --git a/watcher_tempest_plugin/tests/api/admin/test_audit.py b/watcher_tempest_plugin/tests/api/admin/test_audit.py index 10c5ad025..b175966c7 100644 --- a/watcher_tempest_plugin/tests/api/admin/test_audit.py +++ b/watcher_tempest_plugin/tests/api/admin/test_audit.py @@ -27,7 +27,7 @@ from watcher_tempest_plugin.tests.api.admin import base class TestCreateUpdateDeleteAudit(base.BaseInfraOptimTest): """Tests for audit.""" - audit_states = ['ONGOING', 'SUCCEEDED', 'SUBMITTED', 'FAILED', + audit_states = ['ONGOING', 'SUCCEEDED', 'FAILED', 'CANCELLED', 'DELETED', 'PENDING'] def assert_expected(self, expected, actual, @@ -152,7 +152,7 @@ class TestCreateUpdateDeleteAudit(base.BaseInfraOptimTest): class TestShowListAudit(base.BaseInfraOptimTest): """Tests for audit.""" - audit_states = ['ONGOING', 'SUCCEEDED', 'SUBMITTED', 'FAILED', + audit_states = ['ONGOING', 'SUCCEEDED', 'FAILED', 'CANCELLED', 'DELETED', 'PENDING'] @classmethod