Skip to content
This repository was archived by the owner on Nov 17, 2020. It is now read-only.

Increase tests coverage #4

Merged
merged 25 commits into from
Jul 2, 2017
Merged

Increase tests coverage #4

merged 25 commits into from
Jul 2, 2017

Conversation

@menecio
Copy link
Owner

@menecio menecio commented Jul 2, 2017

Closes #2

@menecio menecio added this to the 0.2 Release milestone Jul 2, 2017
@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 63.303% when pulling 427fbbe on menecio-increase-coverage into fcfabb3 on master.

@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 63.303% when pulling 3609e43 on menecio-increase-coverage into fcfabb3 on master.

@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+4.1%) to 67.431% when pulling 14a4b14 on menecio-increase-coverage into fcfabb3 on master.

@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+4.6%) to 67.89% when pulling f13bc6a on menecio-increase-coverage into fcfabb3 on master.

menecio added 2 commits July 2, 2017 13:21
@menecio
Change allowed methods for add_plugin
d4b5ace 
Before I was allowing PUT/PATCH to add a plugin to an api, inspired
by Kong, now it allows POST to add a plugin to an api.

Fixed a bug that was crashing adding a plugin to an api but allowed
to modify it.
@menecio
Add plugins test cases
cca73f9
@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+16.5%) to 79.817% when pulling cca73f9 on menecio-increase-coverage into fcfabb3 on master.

@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+25.6%) to 88.889% when pulling cd7b163 on menecio-increase-coverage into fcfabb3 on master.

@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+31.6%) to 94.907% when pulling aaeb16d on menecio-increase-coverage into fcfabb3 on master.

@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+31.6%) to 94.907% when pulling df4fdbe on menecio-increase-coverage into fcfabb3 on master.

@coveralls
Copy link

coveralls commented Jul 2, 2017

Coverage Status

Coverage increased (+31.6%) to 94.907% when pulling 309610c on menecio-increase-coverage into fcfabb3 on master.

1 similar comment
@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+31.6%) to 94.907% when pulling 309610c on menecio-increase-coverage into fcfabb3 on master.

@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+28.7%) to 91.964% when pulling d03fd4a on menecio-increase-coverage into fcfabb3 on master.

menecio added 2 commits July 2, 2017 18:15
@menecio
Fix consumer_key verification, improve code.
c0d91ce 
Returning the consumer_key instance or None, this to allow removal
of credentials if plugin was configured to hide_credentials.
@menecio
Fix settings and add test case when plugin is set to key_in_body
10d293c
@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+31.4%) to 94.667% when pulling 10d293c on menecio-increase-coverage into fcfabb3 on master.

1 similar comment
@coveralls
Copy link

coveralls commented Jul 2, 2017 •
edited
Loading

Coverage Status

Coverage increased (+31.4%) to 94.667% when pulling 10d293c on menecio-increase-coverage into fcfabb3 on master.

@menecio menecio merged commit ef8d407 into master Jul 2, 2017
@menecio menecio deleted the menecio-increase-coverage branch July 2, 2017 16:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

0.2 Release

Development

Successfully merging this pull request may close these issues.

3 participants

@menecio @coveralls