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

implement new AddPermissionWithContext func within TestSNSAPI #94

Closed
wants to merge 7 commits into from

Conversation

usererror
Copy link

This will add unimplemented functions for the TestSNSAPI interface to make tests pass. If this build passes should I also add the other context funcs?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 40.909% when pulling 16a7e6b on scotts-nyt:master into 12c3a2a on NYTimes:master.

@coveralls
Copy link

coveralls commented Mar 31, 2017

Coverage Status

Coverage remained the same at 40.59% when pulling 3990588 on scotts-nyt:master into 12c3a2a on NYTimes:master.

@coveralls
Copy link

coveralls commented Mar 31, 2017

Coverage Status

Coverage remained the same at 40.59% when pulling f341875 on scotts-nyt:master into 12c3a2a on NYTimes:master.

@coveralls
Copy link

coveralls commented Mar 31, 2017

Coverage Status

Coverage remained the same at 40.59% when pulling 1fac6f5 on scotts-nyt:master into 12c3a2a on NYTimes:master.

@coveralls
Copy link

coveralls commented Mar 31, 2017

Coverage Status

Coverage remained the same at 40.59% when pulling d5e9463 on scotts-nyt:master into 12c3a2a on NYTimes:master.

@coveralls
Copy link

coveralls commented Mar 31, 2017

Coverage Status

Coverage remained the same at 40.59% when pulling 876119a on scotts-nyt:master into 12c3a2a on NYTimes:master.

@coveralls
Copy link

coveralls commented Mar 31, 2017

Coverage Status

Coverage remained the same at 40.59% when pulling 9e8d78a on scotts-nyt:master into 12c3a2a on NYTimes:master.

@oliver-nyt
Copy link
Contributor

falling

@jprobinson
Copy link
Contributor

We pulled this in with PR #95

@jprobinson jprobinson closed this Apr 4, 2017
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

4 participants