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

Kirsten and Victoria - Ampers - Video Store API #18

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

Conversation

Lasiorhine
Copy link

Video Store API

Congratulations! You're submitting your assignment!
If you didn't get to the functionality the question is asking about, reply with what you would have done if you had completed it.

Comprehension Questions

Question Answer
Explain how you came up with the design of your ERD, based on the seed data. We went through the seeds and examined the relationships they implied. We then refined that a few times during the coding process.
Describe a set of positive and negative test cases you implemented for a model. For the movies_checked_out_count method in the Customer model, we wrote tests for situations in which customers had (1) No previous, completed rentals and no current open rentals; (2) No previous, completed rentals, but one or more open rentals; (3) Previous, completed rentals and no current, open rentals; (4) .both previous, completed rentals and current, open rentals.
Describe a set of positive and negative test cases you implemented for a controller. For the movie show method, we created tests for both situations where the client provides good data about a movie, and for when the client provides bad data.
How does your API respond when bad data is sent to it? It returns customized error messages.
Describe one of your custom model methods and why you chose to wrap that functionality into a method. We created a custom model method for in movie for available_inventory. We approached it that way because it involved a calculation, and that kind of operation really doesn't belong in the controller, so the model was the only reasonable place to put it.
Do you have any recommendations on how we could improve this project for the next cohort? QC the Postman tests a little more, maybe?
Link to Trello Join the board with this link: https://trello.com/invite/b/VHz5pYkG/4516c3f8f7967efcfc119c182a665006/videostoreapi
Link to ERD https://www.lucidchart.com/invitations/accept/b20739b5-9d6b-48aa-ac9b-e93ff6fd6a18

Lasiorhine and others added 30 commits May 7, 2018 14:06
…ke more fails before completely approving these, but they're good enough to push. Fail for date-object test still present because the test is still in draft form.
Lasiorhine and others added 28 commits May 8, 2018 15:54
… a merge conflict. It has zero effects on the functionality of anything.
…of movie relationships and the new testes that the available_inventory method will soon need.
…omer's movies_checked_out method are written functionaly, and about halfway to green.
@CheezItMan
Copy link

Video Store

What We're Looking For

Feature Feedback
Core Requirements
Git hygiene Good number of commits, good commit messages, both partner contributing
Comprehension questions Check
General
Business Logic in Models Check
All required endpoints return expected JSON data Check
Requests respond with appropriate HTTP Status Code Check
Errors are reported Check
Testing
Passes all Smoke Tests . Check
Model Tests - all relations, validations, and custom functions test positive & negative cases Check
Controller Tests - URI parameters and data in the request body have positive & negative cases Missing RentalsController testing,
Overall Overall well done, you hit the major learning goals. Nice work! You are now finished with Rails!


def cannot_update
if !is_checked_out?
puts "hiiiiiii"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hellooooo, :)

require "test_helper"

describe RentalsController do
# it "must be a real test" do

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tests...

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