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

Rafa 15 unit test #17

Merged
merged 5 commits into from May 25, 2016
Merged

Rafa 15 unit test #17

merged 5 commits into from May 25, 2016

Conversation

rafaparadela
Copy link

@rafaparadela rafaparadela commented May 25, 2016

This PR fixes #15 . More details:

  • Adds a mock server for unit testing.
  • Adds a proper way to read config parameters
  • Add unit tests for Users APIs
  • Add unit tests for Repos APIs
  • Add unit tests for Auth APIs

@rafaparadela
Copy link
Author

@franciscodr Could you review please? Thank you

@franciscodr
Copy link
Member

LGTM
LGTM

@rafaparadela rafaparadela merged commit 63674ae into master May 25, 2016
@rafaparadela rafaparadela deleted the rafa-15-unit-test branch May 25, 2016 08:31
@47degdev
Copy link
Contributor

this is the comment

@47degdev 47degdev added bug and removed bug labels Feb 13, 2020
@bond15
Copy link
Contributor

bond15 commented Feb 13, 2020

this is the comment

@bond15 bond15 added bug and removed bug labels Feb 13, 2020
@47degdev
Copy link
Contributor

this is the comment

@47degdev 47degdev added bug and removed bug labels Feb 13, 2020
@47degdev
Copy link
Contributor

this is the comment

@47degdev 47degdev added bug and removed bug labels Feb 13, 2020
@rafaparadela
Copy link
Author

@bond15 Are we proving the API to add comments in PRs by using this one? If so... I need to unsubscribe from this thread :-)

@bond15
Copy link
Contributor

bond15 commented Feb 13, 2020

this is the comment

@bond15 bond15 added bug and removed bug labels Feb 13, 2020
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.

Unit testing
4 participants