Skip to content

Start adding unit tests #34

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
Open
abdullahselek wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Start adding unit tests #34

abdullahselek wants to merge 5 commits into from

Conversation

@abdullahselek
Copy link
Contributor

@abdullahselek abdullahselek commented Jan 5, 2018

  • With this commit there is a new folder named tests which is created to contain test classes. For now there is only one, which is for PKIBuilder. Coverage for PKIBuilder is now 100%. It's the first shape of the fix for Unit testing support for all endpoints and cases #33.
  • To install the dependencies for unit tests just run pip install -r requirements.testing.txt
  • For running the tests you can use py.test -s -v command.

@abdullahselek
Add dependency list for unit tests
8c92d22 
* pip install -r requirements.testing.txt
@abdullahselek
Add unit tests for pki_builder
85a8fe7 
* As I see IyzipayResource is not public so I started adding
tests for pki_builder.

* To run the tests `py.test -s -v`
@abdullahselek
Add tests for append_price
daa4978
@abdullahselek
Add tests for append_array
dfd7748
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@yyenigun yyenigun yyenigun approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@abdullahselek @yyenigun