Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

e2e tests for sse (server sent events) #10839

Closed
15 of 19 tasks
ScharfViktor opened this issue Apr 26, 2024 · 2 comments
Closed
15 of 19 tasks

e2e tests for sse (server sent events) #10839

ScharfViktor opened this issue Apr 26, 2024 · 2 comments
Assignees
Labels

Comments

@ScharfViktor
Copy link
Contributor

ScharfViktor commented Apr 26, 2024

related tasks and PRs:

e2e tests for SSE:

spaces resources:

creating:

  • upload file/folder
  • create file/folder (.txt .odt)
  • restore file/folder from trash
  • move file/folder
  • copy file/folder
  • lock/unlock file

updating:

  • rename file/folder
  • change file content
  • restore file version

space:

  • disable
  • enable
  • delete
  • change description/space image/space icon
  • space-member-added
  • space-member-removed
  • space-share-updated

share:

  • share created
  • share updated
  • share deleted

event in the eventStream in the GET api/v1/notifications/sse request:

NOTIFICATION = 'userlog-notification',
POSTPROCESSING_FINISHED = 'postprocessing-finished',
FILE_LOCKED = 'file-locked',
FILE_UNLOCKED = 'file-unlocked',
FILE_TOUCHED = 'file-touched',
ITEM_RENAMED = 'item-renamed',
ITEM_TRASHED = 'item-trashed',
ITEM_RESTORED = 'item-restored',
ITEM_MOVED = 'item-moved',
FOLDER_CREATED = 'folder-created',
SPACE_MEMBER_ADDED = 'space-member-added',
SPACE_MEMBER_REMOVED = 'space-member-removed',
SPACE_SHARE_UPDATED = 'space-share-updated',
SHARE_CREATED = 'share-created',
SHARE_REMOVED = 'share-removed',
SHARE_UPDATED = 'share-updated',
LINK_CREATED = 'link-created',
LINK_REMOVED = 'link-removed',
LINK_UPDATED = 'link-updated'

Besides the appearance/disappearance of resources on the web page, we usually check the api responses. but checking notifications/sse seems tricky. We need to somehow listen GET api/v1/notifications/sse during the whole test, which is triggered only once after login and gets 200 with an empty response.

@ScharfViktor
Copy link
Contributor Author

The sse is not yet fully implemented. Therefore, only this test suite is available for now

@saw-jan
Copy link
Member

saw-jan commented May 24, 2024

The sse is not yet fully implemented. Therefore, only this test suite is available for now

So we are skipping these for now. 👍

space:

  • disable
  • enable
  • delete
  • change description/space image/space icon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants