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

Add dummy API tests #11

Open
MalteT opened this issue Oct 24, 2021 · 1 comment
Open

Add dummy API tests #11

MalteT opened this issue Oct 24, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@MalteT
Copy link
Owner

MalteT commented Oct 24, 2021

There should be tests running the app against a dummy API to prevent me from actually breaking the JSON parsing or similar.
@MalteT MalteT added the enhancement New feature or request label Oct 24, 2021
MalteT added a commit that referenced this issue Oct 25, 2021
@MalteT
Add Api trait
b21f921 
This should abstract away from the actual implementation and allow
for reproducible, local tests with a DummyApi.
Tests have been rewritten to use the DummyApi.
Prep for #11.
@MalteT
Copy link
Owner Author

MalteT commented Oct 27, 2021 •
edited
Loading

  • /canteens
    • /canteens with position and radius
    • /canteens/[id]
      • /canteens/[id]/days
        • /canteens/[id]/days/[day]
          • /canteens/[id]/days/[day]/meals
            • /canteens/[id]/days/[day]/meals/[mid]

MalteT added a commit that referenced this issue Oct 27, 2021
@MalteT
Abstract over Cache
321037a 
Add `Cache` trait and implement it for the `Cacache` and `DummyCache`.
Fix tests to use the DummyCache, dropping old dependencies.
See #11.
MalteT added a commit that referenced this issue Oct 27, 2021
@MalteT
Fix race between tests, add DummyCache
6960ae6 
Reworked DummyCache indexing, removed broken top-level tests.
Added a simple test deserializing a canteen.
Added DummyCache for tests.
See #11.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@MalteT