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.

Sign up for GitHub

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

Jump to bottom

Tests need a minor cleanup #18

Closed
dnl-blkv opened this issue Aug 22, 2017 · 0 comments
Closed

Tests need a minor cleanup #18

dnl-blkv opened this issue Aug 22, 2017 · 0 comments
Assignees
@dnl-blkv
Milestone
v.0.10.0

Comments

@dnl-blkv
Copy link
Contributor

How to Reproduce

  1. Look at the tests structure

What Should Happen

  1. Test structure mimics the code structure
  2. Test naming is consistent (test classes all start with word Test)

What Happens

  1. TestApiContext is located in Model/Generated
  2. Test for Avatar is called AvatarTest
@dnl-blkv dnl-blkv added this to the v.0.10.0 milestone Aug 22, 2017
@dnl-blkv dnl-blkv self-assigned this Aug 22, 2017
dnl-blkv added a commit that referenced this issue Aug 22, 2017
@dnl-blkv
cleanup tests [#18]
d52ae38
@dnl-blkv dnl-blkv mentioned this issue Aug 22, 2017
Merged
OGKevin added a commit that referenced this issue Aug 22, 2017
@OGKevin
Merge pull request #19 from bunq/18-tests-cleanup
d99b44e 
cleanup tests [#18]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dnl-blkv dnl-blkv

Labels
None yet
Projects
None yet
Milestone
v.0.10.0
Development

No branches or pull requests
1 participant
@dnl-blkv