Skip to content
This repository has been archived by the owner on Jul 24, 2020. It is now read-only.

Write tests for ReservationsController #403

Closed
austinczarnecki opened this issue Jun 27, 2013 · 4 comments
Closed

Write tests for ReservationsController #403

austinczarnecki opened this issue Jun 27, 2013 · 4 comments
Assignees
Milestone

Comments

@austinczarnecki
Copy link
Contributor

This has its own issue because it's huge and needs to be done before we can start refactoring.

@shippy
Copy link
Contributor

shippy commented Jun 19, 2014

Since this is some of our core functionality, adding this to milestone 3.3.0. (I thought that @ebmaher wrote some of these, but I can't find the branch, so we'll just start anew.)

@shippy
Copy link
Contributor

shippy commented Jun 24, 2014

Plan:

  • Write super-basic happy/unhappy path tests (response is success, assigns a variable, renders template v. response is redirect, sets the flash, doesn't render template) for all routes
    • GET index
    • GET show
    • GET new
    • POST create
    • GET edit
    • PUT update
    • DELETE destroy
    • GET manage
    • GET current
    • PUT checkout
    • PUT checkin
    • GET checkout_email
    • GET checkin_email
    • PUT renew
  • Write more detailed functionality tests for each route

@shippy
Copy link
Contributor

shippy commented Jul 2, 2014

Pull request with coverage of vast majority of methods in #606.

@shippy
Copy link
Contributor

shippy commented Jul 2, 2014

#606 is merged, closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants