test_update_email_handler.py 3.0 KB

12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879
  1. from uuid import uuid4
  2. import pytest
  3. from pluggy import HookimplMarker
  4. from flaskbb.core.exceptions import PersistenceError, StopValidation, ValidationError
  5. from flaskbb.core.changesets import ChangeSetPostProcessor, ChangeSetValidator
  6. from flaskbb.core.user.update import EmailUpdate
  7. from flaskbb.user.models import User
  8. from flaskbb.user.services.update import DefaultEmailUpdateHandler
  9. def random_email():
  10. return "{}@not.real.at.all".format(str(uuid4()))
  11. class TestDefaultEmailUpdateHandler(object):
  12. def test_raises_stop_validation_if_errors_occur(
  13. self, mock, user, database, plugin_manager
  14. ):
  15. validator = mock.Mock(spec=ChangeSetValidator)
  16. validator.validate.side_effect = ValidationError(
  17. "new_email", "That's not even valid"
  18. )
  19. hook_impl = mock.Mock(spec=ChangeSetPostProcessor)
  20. plugin_manager.register(self.impl(hook_impl))
  21. email_change = EmailUpdate(user.email, random_email())
  22. handler = DefaultEmailUpdateHandler(
  23. db=database, validators=[validator], plugin_manager=plugin_manager
  24. )
  25. with pytest.raises(StopValidation) as excinfo:
  26. handler.apply_changeset(user, email_change)
  27. assert excinfo.value.reasons == [("new_email", "That's not even valid")]
  28. hook_impl.post_process_changeset.assert_not_called()
  29. def test_raises_persistence_error_if_save_fails(self, mock, user, plugin_manager):
  30. email_change = EmailUpdate(user.email, random_email())
  31. db = mock.Mock()
  32. db.session.commit.side_effect = Exception("no")
  33. hook_impl = mock.Mock(spec=ChangeSetPostProcessor)
  34. plugin_manager.register(self.impl(hook_impl))
  35. handler = DefaultEmailUpdateHandler(
  36. db=db, validators=[], plugin_manager=plugin_manager
  37. )
  38. with pytest.raises(PersistenceError) as excinfo:
  39. handler.apply_changeset(user, email_change)
  40. assert "Could not update email" in str(excinfo.value)
  41. hook_impl.post_process_changeset.assert_not_called()
  42. def test_actually_updates_email(self, user, database, mock, plugin_manager):
  43. new_email = random_email()
  44. email_change = EmailUpdate("test", new_email)
  45. hook_impl = mock.Mock(spec=ChangeSetPostProcessor)
  46. plugin_manager.register(self.impl(hook_impl))
  47. handler = DefaultEmailUpdateHandler(
  48. db=database, validators=[], plugin_manager=plugin_manager
  49. )
  50. handler.apply_changeset(user, email_change)
  51. same_user = User.query.get(user.id)
  52. assert same_user.email == new_email
  53. hook_impl.post_process_changeset.assert_called_once_with(
  54. user=user, email_update=email_change
  55. )
  56. @staticmethod
  57. def impl(post_processor):
  58. class Impl:
  59. @HookimplMarker("flaskbb")
  60. def flaskbb_email_updated(self, user, email_update):
  61. post_processor.post_process_changeset(
  62. user=user, email_update=email_update
  63. )
  64. return Impl()