Skip to content
This repository has been archived by the owner on Nov 5, 2019. It is now read-only.

Fixing code that causes test_oauth2client to fail. #86

Merged
merged 2 commits into from
Dec 6, 2014

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Dec 6, 2014

In particular:

  • Removing executable bit from "test_oauth2client".
  • Temporarily disabling slow (network bound) tests.
  • Making client.SETTINGS a class so it can be mutable.
  • Making GoogleCredentials._get_implicit_credentials a classmethod
    so it can access cls.
  • Changing use of urlparse.parse_qs to urllib.parse.parse_qs (did
    not get converted during Py3K switch).

Partially addresses #85. Not closing out until slow tests are
re-enabled and other test failures of py33openssl14 and
py34openssl14 pass.

In particular:
- Removing executable bit from "test_oauth2client".
- Temporarily disabling slow (network bound) tests.
- Making client.SETTINGS a class so it can be mutable.
- Making GoogleCredentials._get_implicit_credentials a classmethod
  so it can access `cls`.
- Changing use of `urlparse.parse_qs` to `urllib.parse.parse_qs` (did
  not get converted during Py3K switch).

Partially addresses googleapis#85. Not closing out until slow tests are
re-enabled and other test failures of `py33openssl14` and
`py34openssl14` pass.
@dhermes
Copy link
Contributor Author

dhermes commented Dec 6, 2014

@craigcitro This will fail when Travis runs because of the failures in py33openssl14 and py34openssl14. I can disable them so that they pass here and then we re-enable and fix in another PR.

WDYT?

craigcitro added a commit that referenced this pull request Dec 6, 2014
Fixing code that causes test_oauth2client to fail.
@craigcitro craigcitro merged commit b20d3d6 into googleapis:master Dec 6, 2014
@dhermes dhermes deleted the fix-immutable-settings branch December 6, 2014 02:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants