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

Steffany Ampers #39

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Steffany Ampers #39

wants to merge 12 commits into from

Conversation

steffnay
Copy link

@steffnay steffnay commented May 7, 2018

API Muncher

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
How did you go about exploring the Edamam API, how did you try querying the API? I used the docs and Postman. There was a useful example API query in the documentation so I used that and started exploring the response information that way
Describe your API Wrapper. How did you decide on the methods you created? I created a wrapper that makes two different API calls. One is in the EdamamApiWrapper#recipe_search method, where an array of Recipe instances are returned. The other is EdamamApiWrapper#get_recipe, which uses the q value in the query, then creates a new instance of Recipe and returns it.
Describe an edge case or failure case test you wrote for your API Wrapper. I wasn't able to write any tests, but I thought about cases where the Recipe variables are nil
Explain how VCR aids in testing an API. It's important to minimize the number of API calls, so VCR saves the API's responses as cassettes so new calls don't need to be made when running tests
What is the Heroku URL of your deployed application? https://yum-muncher.herokuapp.com/
Provide a link to the Trello board you used

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.

1 participant