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

[WIP] Improvements to unit tests. #3574

Merged
merged 12 commits into from
Jun 13, 2017
Merged

Conversation

dmeltzer
Copy link
Contributor

Getting this out for review... Don't merge yet, it's still a WIP and probably breaks some current tests. Here's where I'm going with it though.

  • Break up modelfactory into multiple files, populate many states.
  • Begin testing validation at the unit test level, test relationships.
  • Add tests for Asset::availableForCheckout.
  • Model factories now generate all needed relationships on demand,
    which allows us to unit test with a empty database.
  • To faciliate the empty database, we move to using sqlite in memory as
    the unit testing database.

@dmeltzer dmeltzer changed the title [WIP] Improvemenets to unit tests. [WIP] Improvements to unit tests. May 14, 2017
* Break up modelfactory into multiple files, populate many states.
* Begin testing validation at the unit test level, test relationships.
* Add tests for Asset::availableForCheckout.
* Model factories now generate all needed relationships on demand,
  which allows us to unit test with a empty database.
* To faciliate the empty database, we move to using sqlite in memory as
  the unit testing database.
…users associated with location. Still need some work to show assets.
… rather than via web requests. Also test delete permissions.
@snipe snipe merged commit 5d4920c into snipe:develop Jun 13, 2017
@dmeltzer dmeltzer deleted the test-improvements branch September 18, 2017 23:43
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