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

test_parallel_get_campaigns error due to no self.mixins() #38

Closed
tpetr opened this issue May 21, 2012 · 4 comments · Fixed by #39
Closed

test_parallel_get_campaigns error due to no self.mixins() #38

tpetr opened this issue May 21, 2012 · 4 comments · Fixed by #39

Comments

@tpetr
Copy link
Contributor

tpetr commented May 21, 2012

test_parallel_get_campaigns has been failing for a long time... BaseClient used to have a mixins() method (i assme for injecting mixins), but now it's part of the constructor:

def __init__(self, api_key=None, timeout=10, mixins=[], access_token=None, refresh_token=None, client_id=None, **extra_options):

we need to update the test to pass PyCurlMixin to the constructor instead of using self.mixins()

@tpetr
Copy link
Contributor Author

tpetr commented May 21, 2012

i tap @jpetr to fix this...

@jpetr jpetr mentioned this issue May 22, 2012
jpetr added a commit that referenced this issue May 23, 2012
@dcancel
Copy link

dcancel commented May 31, 2012

Is this still open?

@tpetr
Copy link
Contributor Author

tpetr commented May 31, 2012

#39 fixed this but the pull request didn't auto-close it. closing manually...

@tpetr tpetr closed this as completed May 31, 2012
@dcancel
Copy link

dcancel commented May 31, 2012

Thank you kindly

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 a pull request may close this issue.

2 participants