Age | Commit message (Collapse) | Author |
|
|
|
Remove __author__ lines and add contributors.md.
|
|
Resolves #626
|
|
|
|
No longer needed since Python 2.6 support has been dropped.
|
|
A new file, `_helpers.py`, was created without realizing that
`utils.py` existed for the same purpose.
Moving all to `_helpers.py`.
|
|
Also cleaned up some nested attribute access.
|
|
* Use context manager for assertRaises, fixes #536.
* Update usage of unittest to unittest2.
* Remove unneeded `if __name__ == '__main__':` clauses in test files.
|
|
This is so we get 100% line coverage in our tests. Also
adding unittest.main() lines to the files missing it:
test__helpers.py
test__pycrypto_crypt.py
test_crypt.py
test_devshell.py
test_gce.py
test_keyring_storage.py
test_service_account.py
test_util.py
|
|
This is all errors except E402: module level import not at top of file.
This is because in most (all?) files the __author__ global comes
before imports begin.
Ref: http://stackoverflow.com/a/24859703/1068170
|
|
Simply ran
pep8ify -w oauth2client/
pep8ify -w tests/
|
|
|
|
Also making sure the _urlsafe_b64encode returns bytes
and updating dependent code with the change in
return type.
|
|
Until now, code that depended on PyCrypto or OpenSSL was
defined conditionally (e.g. indented) in `crypt.py`. Rather than
grouping all these together, we factor out the library specific
behavior into standalone modules (but make the modules
private / protected).
In addition, added a `_helpers.py` module with common behavior
that was previously defined in multiple places.
Finally, beefed up some test cases so that the three newly added
modules had 100% test coverage.
Towards #212.
|