Remove skipped tests

Author: Ostap-Zherebetskyi

api_tests/base/test_utils.py

@@ -5,7 +5,6 @@
 from django.conf import settings as django_conf_settings
 
 from rest_framework import fields
-from rest_framework.exceptions import ValidationError
 
 from api.base import utils as api_utils
 from osf.models.base import coerce_guid, Guid, GuidMixin, OptionalGuidMixin, VersionedGuidMixin, InvalidGuid
@@ -63,49 +62,6 @@ def test_push_status_message_no_response(self):
             except BaseException:
                 assert False, f'Exception from push_status_message via API v2 with type "{status}".'
 
-    @unittest.skip('[ANG-959] Make push_status_message NOOP (#11350)')
-    def test_push_status_message_expected_error(self):
-        status_message = 'This is a message'
-        try:
-            push_status_message(status_message, kind='error')
-            assert False, 'push_status_message() should have generated a ValidationError exception.'
-
-        except ValidationError as e:
-            assert (
-                e.detail[0] == status_message
-            ), 'push_status_message() should have passed along the message with the Exception.'
-
-        except RuntimeError:
-            assert False, 'push_status_message() should have caught the runtime error and replaced it.'
-
-        except BaseException:
-            assert False, 'Exception from push_status_message when called from the v2 API with type "error"'
-
-    @unittest.skip('[ANG-959] Make push_status_message NOOP (#11350)')
-    @mock.patch('framework.status.get_session')
-    def test_push_status_message_unexpected_error(self, mock_get_session):
-        status_message = 'This is a message'
-        exception_message = 'this is some very unexpected problem'
-        mock_session = mock.Mock()
-        mock_session.attach_mock(mock.Mock(side_effect=RuntimeError(exception_message)), 'get')
-        mock_get_session.return_value = mock_session
-        try:
-            push_status_message(status_message, kind='error')
-            assert False, 'push_status_message() should have generated a RuntimeError exception.'
-        except ValidationError:
-            assert False, 'push_status_message() should have re-raised the RuntimeError not gotten ValidationError.'
-        except RuntimeError as e:
-            assert str(e) == exception_message, (
-                'push_status_message() should have re-raised the '
-                'original RuntimeError with the original message.'
-            )
-
-        except BaseException:
-            assert False, (
-                'Unexpected Exception from push_status_message when called '
-                'from the v2 API with type "error"'
-            )
-
 
 @pytest.mark.django_db
 class TestCoerceGuid: