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

Refactor API folder structure #237

Merged
merged 2 commits into from
Mar 12, 2023
Merged

Refactor API folder structure #237

merged 2 commits into from
Mar 12, 2023

Conversation

josecelano
Copy link
Member

@josecelano josecelano commented Mar 10, 2023 •
edited
Loading

Reorganize API in contexts:

Production code:

  • Extract API context auth_key.
  • Extract API context stats.
  • Extract API context torrent.
  • Extract API context whitelist.

Integration tests:

  • Extract test for context auth_key.
  • Extract test for context stats.
  • Extract test for context torrent.
  • Extract test for context whitelist.

@josecelano josecelano linked an issue Mar 10, 2023 that may be closed by this pull request
API overhaul: refactor API folder structure #157
Closed
@josecelano josecelano force-pushed the issue-157-reorganize-api-folder-structure branch from 40c13fd to 996dffd Compare March 10, 2023 18:31
@josecelano josecelano force-pushed the issue-157-reorganize-api-folder-structure branch from 996dffd to 19d33b4 Compare March 10, 2023 18:52
@josecelano
Copy link
Member Author

I've finished with the production code, now I will reorganize the test mods to mirror the production code.

@josecelano
refactor(api): reorganize api tests in contexts
8fe52c3 
After changing production code we follow the API contexts structure in
tests too.
@josecelano josecelano marked this pull request as ready for review March 12, 2023 09:45
@josecelano josecelano merged commit 2a60771 into torrust:develop Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@da2ce7 da2ce7 Awaiting requested review from da2ce7

@mickvandijke mickvandijke Awaiting requested review from mickvandijke

Assignees
No one assigned
Labels
None yet
Projects
Torrust Solution
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

API overhaul: refactor API folder structure
1 participant
@josecelano