Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

[FEAT] Test authorization for every route #155

Open
stijndcl opened this issue Mar 24, 2022 · 1 comment
Open

[FEAT] Test authorization for every route #155

stijndcl opened this issue Mar 24, 2022 · 1 comment
Labels
backend Issue related to the backend bonus Bonus feature, not required, low priority enhancement New feature or request
Milestone

Comments

@stijndcl
Copy link
Contributor

stijndcl commented Mar 24, 2022

Every route should be tested four to five times:

  • Admin
  • Coach (in this edition, if applicable)
  • Coach but not in this edition (not applicable to all of the routes, not all of them are linked to editions)
  • Invalid token
  • No authorization at all
@stijndcl stijndcl added enhancement New feature or request backend Issue related to the backend labels Mar 24, 2022
@stijndcl stijndcl added this to the Milestone 2 milestone Mar 24, 2022
@stijndcl stijndcl self-assigned this Mar 24, 2022
@stijndcl stijndcl modified the milestones: Milestone 2, Milestone 3 Apr 4, 2022
@stijndcl stijndcl added the bonus Bonus feature, not required, low priority label Apr 4, 2022
@stijndcl stijndcl removed their assignment Apr 4, 2022
@stijndcl
Copy link
Contributor Author

stijndcl commented Apr 4, 2022

This would literally take weeks to test as we need hundreds upon hundreds of extra tests. The dependencies work for the routes where they've been tested, I don't want to waste precious development time on this right now. Might write some more of I have time in the end, but for now I'm putting it on the backburner.

@stijndcl stijndcl modified the milestones: Milestone 3, Milestone 4 May 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backend Issue related to the backend bonus Bonus feature, not required, low priority enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant