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.

Sign up for GitHub

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

Jump to bottom

test: write unit tests for all routes #282

Open
77 tasks
insertish opened this issue Sep 5, 2023 · 2 comments
Open
77 tasks

test: write unit tests for all routes #282

insertish opened this issue Sep 5, 2023 · 2 comments
Labels
help wanted Extra attention is needed

Comments

@insertish
Copy link
Member

Routes missing unit tests:

  • channel/ack
  • channel/delete
  • channel/edit
  • group/add_member
  • group/remove_member
  • invite/create
  • members/fetch
  • messages/bulk_delete
  • messages/clear_reactions
  • messages/delete
  • messages/edit
  • messages/fetch
  • messages/query
  • messages/react
  • messages/search
  • messages/send
  • messages/unreact
  • channel/permissions/set_default
  • channel/permissions/set_role
  • channel/webhook/create
  • channel/webhook/fetch_all
  • emoji/create
  • emoji/delete
  • emoji/fetch
  • invite/delete
  • invite/fetch
  • invite/join
  • onboard/complete
  • onboard/hello
  • push/sub
  • push/unsub
  • safety/report_content
  • server/ban_create
  • server/ban_list
  • server/ban_remove
  • server/channel_create
  • server/emoji_list
  • server/invites_list
  • server/member_edit
  • server/all_members
  • server/fetch_member
  • server/kick_member
  • server/permissions/set_default
  • server/permissions/set_role
  • server/roles/create
  • server/roles/delete
  • server/roles/edit
  • server/ack
  • server/create
  • server/delete
  • server/edit
  • server/fetch
  • sync/get_settings
  • sync/get_unreads
  • sync/set_settings
  • users/add_friend
  • users/block_user
  • users/change_username
  • users/edit_user
  • users/fetch_dms
  • users/fetch_profile
  • users/fetch_self
  • users/fetch_user_flags
  • users/fetch_user
  • users/find_mutual
  • users/get_default_avatar
  • users/open_dm
  • users/remove_friend
  • users/send_friend_request
  • users/unblock_user
  • webhook/delete_by_token
  • webhook/delete
  • webhook/edit_by_token
  • webhook/edit
  • webhook/execute
  • webhook/fetch_by_token
  • webhook/fetch
@insertish insertish mentioned this issue Sep 5, 2023
Closed
@insertish insertish pinned this issue Sep 5, 2023
@insertish insertish added the help wanted Extra attention is needed label Oct 14, 2023
@toastxc
Copy link
Contributor

toastxc commented Dec 27, 2023

I'd love to help with testing, but how would they work?
Should the tests be executed or ignored by Github?
If Github is to be used, how could it be done without uploading bot tokens?

Some more information on how to help would be greatly appreciated.

@insertish
Copy link
Member Author

I'd love to help with testing, but how would they work? Should the tests be executed or ignored by Github? If Github is to be used, how could it be done without uploading bot tokens?

Some more information on how to help would be greatly appreciated.

Tests spin up their own environment in which fixtures are loaded.
Events can also be tested within this environment.

@insertish insertish moved this to 🏗 In Progress in Revolt Project Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
Revolt Project
Status: 🕒 Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@insertish @toastxc