diff --git a/zerver/lib/actions.py b/zerver/lib/actions.py index c9f3f3744a..b89b94e309 100644 --- a/zerver/lib/actions.py +++ b/zerver/lib/actions.py @@ -2832,7 +2832,7 @@ def do_change_bot_owner(user_profile: UserProfile, bot_owner: UserProfile, user_profile.save() # Can't use update_fields because of how the foreign key works. event_time = timezone_now() RealmAuditLog.objects.create(realm=user_profile.realm, acting_user=acting_user, - modified_user=user_profile, event_type='bot_owner_changed', + modified_user=user_profile, event_type=RealmAuditLog.BOT_OWNER_CHANGED, event_time=event_time) update_users = bot_owner_user_ids(user_profile) diff --git a/zerver/models.py b/zerver/models.py index b49c2e9b72..e956c5d424 100644 --- a/zerver/models.py +++ b/zerver/models.py @@ -1976,6 +1976,8 @@ class RealmAuditLog(models.Model): REALM_DEACTIVATED = 'realm_deactivated' REALM_REACTIVATED = 'realm_reactivated' + BOT_OWNER_CHANGED = 'bot_owner_changed' + event_type = models.CharField(max_length=40) # type: str def __str__(self) -> str: diff --git a/zerver/tests/test_audit_log.py b/zerver/tests/test_audit_log.py index 6f5b6d89fe..4f6c363e0e 100644 --- a/zerver/tests/test_audit_log.py +++ b/zerver/tests/test_audit_log.py @@ -88,7 +88,7 @@ class TestRealmAuditLog(ZulipTestCase): bot = self.notification_bot() bot_owner = self.example_user('hamlet') do_change_bot_owner(bot, bot_owner, admin) - self.assertEqual(RealmAuditLog.objects.filter(event_type='bot_owner_changed', + self.assertEqual(RealmAuditLog.objects.filter(event_type=RealmAuditLog.BOT_OWNER_CHANGED, event_time__gte=now).count(), 1) self.assertEqual(bot_owner, bot.bot_owner)