tests: Remove /settings/notifications from buggy endpoints.

This commit is contained in:
Vishnu KS 2019-10-07 13:48:53 +05:30 committed by Tim Abbott
parent 21ab8c542a
commit e7419f815a
2 changed files with 39 additions and 2 deletions

View File

@ -2180,6 +2180,12 @@ paths:
schema:
type: boolean
example: true
- name: notification_sound
in: query
description: Notification sound name.
schema:
type: string
example: ding
- name: enable_desktop_notifications
in: query
description: Enable visual desktop notifications for private messages
@ -2221,6 +2227,12 @@ paths:
schema:
type: boolean
example: true
- name: enable_login_emails
in: query
description: Enable email notifications for new logins to account.
schema:
type: boolean
example: true
- name: message_content_in_email_notifications
in: query
description: Include the message's content in missed messages email
@ -2228,6 +2240,33 @@ paths:
schema:
type: boolean
example: true
- name: pm_content_in_desktop_notifications
in: query
description: Include content of private messages in desktop notifications.
schema:
type: boolean
example: true
- name: desktop_icon_count_display
in: query
description: >
Unread count summary (appears in desktop sidebar and browser tab)
* 1 - All unreads
* 2 - Private messages and mentions
* 3 - None
schema:
type: integer
enum:
- 1
- 2
- 3
example: 1
- name: realm_name_in_notifications
in: query
description: Include organization name in subject of missed message emails.
schema:
type: boolean
example: true
security:
- basicAuth: []
responses:

View File

@ -237,8 +237,6 @@ class OpenAPIArgumentsTest(ZulipTestCase):
buggy_documentation_endpoints = set([
'/events',
'/users/me/subscriptions/muted_topics',
# List of flags is broader in actual code; fix is to just add them
'/settings/notifications',
# Docs need update for subject -> topic migration
'/messages/{message_id}',
])