Skip to content
This repository has been archived by the owner on Jan 26, 2021. It is now read-only.

Added database checks #380

Merged
merged 29 commits into from
Aug 24, 2016
Merged

Added database checks #380

merged 29 commits into from
Aug 24, 2016

Conversation

smarshy
Copy link
Contributor

@smarshy smarshy commented Aug 23, 2016

Added database checks to tests corresponding to CRUD operations in following cases and number of tests-

  1. Sign up Admin/volunteer with invalid/valid credentials (16 tests )
  2. Create/edit event with valid/invalid details (9 tests)
  3. Create/edit job with valid/invalid details (9 tests)
  4. Create/edit shift with valid/invalid details (9 tests)
  5. Create/edit/delete organization (5 tests )
  6. Delete event, job, shift (6 tests)
  7. Shift registration, cancel shift registration, edit volunteer profile (5 tests)
  8. Add/edit/cancel hours by volunteer/admin (5 tests)

This PR contains the commits from PR #375 as the changes for database checks have been made after the tests were redesigned to follow POM. This was done as otherwise there would be too many merge conflicts to fix.

The actual changes can be seen here once PR #375 is merged or can be currently seen here

@coveralls
Copy link

Coverage Status

Coverage increased (+4.5%) to 88.229% when pulling ed3cd29 on smarshy:add-db into fd85eb3 on systers:develop.

@tapaswenipathak
Copy link
Contributor

Thank you. ✨

@tapaswenipathak tapaswenipathak merged commit 69d1da1 into anitab-org:develop Aug 24, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants