Skip to content

Conversation

@tehut
Copy link

@tehut tehut commented May 10, 2017

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 played in Postman for a bit but once I got a basic url working I tried to write a bunch of queries from the console
Describe your API Wrapper. How did you decide on the methods you created? I created two instance methods find and all as well as a helper method that cleans up data from the response and saves it in exposed instance variables on the instance of the Edamam object. My .all instance method looks up the query and returns hits, the find method returns recipes and I use my clean method to prepare the uri & from data for the find call.
Describe an edge case or failure case test you wrote for your API Wrapper. I tested to make sure the program would raise a custom error if given bad data or data that returned no search reults
Explain how VCR aids in testing an API. VCR allows us to "replay" a single API call and run all of our tests against it which is important because most APIs have call limits. They tend to be high but if we're running our tests as often as we should we could burn through a lot of API calls
What is the Heroku URL of your deployed application? muncher.herokuapp.com
Do you have any recommendations on how we could improve this project for the next cohort? Nope, I thought this was a challenging project with lots of room to be creative!

@droberts-sea
Copy link

API Muncher

What We're Looking For

Feature Feedback
Core Requirements
Git hygiene yes
Comprehension questions yes
General
Rails fundamentals (RESTful routing, use of named paths) yes
Semantic HTML yes
Errors are reported to the user yes - good work
API Wrapper to handle the API requests yes - see inline comments
Controller testing no
Styling
Foundation Styling for responsive layout yes
Search View yes
List View yes
Show View which opens in a new tab yes
API Features
The App attributes Edaman yes
The VCR casettes do not contain the API key yes
Overall

Good work overall! The one thing that's missing is good controller testing. Being able to effectively test a controller is a culmination of all the Rails knowledge and understanding we've worked on so far, and it's a good learning experience. Make sure you practice it on Video Store.

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.

2 participants