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

REST API tests enhancements #156

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rclations
Copy link
Contributor

This PR is a first-pass at #155

I'm not sure how you'd want to have a more comprehensive basic test setup, so I added a basic example in a block comment for now. Ideally I'd like to get your feedback on the approach you'd want to take here.

@rclations rclations force-pushed the rest-api-tests-enhancements branch from feb059a to f578185 Compare June 23, 2017 19:23
@@ -31,6 +42,70 @@ function test_class_access() {
* @since <%= version %>
*/
function test_sample() {
/*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for such a lengthy delay on looking at this PR.

Can you please remove the commented code and I'll get this merged?

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.

None yet

2 participants