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

Mw wave06 #6

Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

Mw wave06 #6

wants to merge 31 commits into from

Conversation

MelodyW2022
Copy link

No description provided.

MelodyW2022 and others added 30 commits December 14, 2022 11:57
…_id> and POST /planets routes. refactored the planet_id validation function.
mass datatype converted to float. table for Planet is renamed "planets".
…eated a migration file for changing the mass datatype to string
…ction to separate param kwargs for SQLAlchemy query function
…_.py to accept dev and test db. added test_get_all_planets_with_empty_db_returns_empty_list
created 3 fixtures:app, client, two_saved_planets.updated app/__init_…
…essfully, get planet/<id> with an invalid id
added tests for get /planet/<id> with an empty db, update planet succ…
Added unit tests 3, 6, and 3 edge cases for get method with query kwa…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants