2
100%
master: 100%

Ran 01 Dec 2015 02:05AM UTC

Files 41

Run time 1s

Badge

Embed ▾

Committed 30 Nov 2015 11:55PM UTC coverage: 93.713% (-0.08%) from 93.788%

Job # TOX_ENV=py27

Build Type

Pull #347

travis-ci

Committed by

jonparrott

Commit Message

[DO NOT MERGE] Re-write Storage

This is a rough-cut to sanity check the concept. Docstrings and full tests still missing.

New:
* `client.Storage` is an abstract base class.
* `locked_storage.LockedStorage` provides locking based on context managers and defaults to `threading.Lock`.

Changed:
* All `Storage` subclasses have been updated to use the new base class.
* `file.Storage` renamed to `file_storage.FileStorage`.
* `keyring_storage.Storage` renamed to `keyring_storage.KeyringStorage`.
* `django_orm.Storage` renamed to `django_orm.DjangoOrmStorage`.
* `KeyringStorage` and `FileStorage` are no longer threadsafe by default.

**Questionable changes**

As mentioned in [a comment on #344], `Credentials` has the unfortunate behavior of using the Storage's lock to prevent concurrently refreshing the credentials. IMO, Refresh locking should be handled as a separate concern. For now, `Storage` implements a no-op `lock()`.

Pull Request Pull Request #347: [DO NOT MERGE] Re-write Storage

Coverage Stats

5187 of 5535 relevant lines covered (93.71%)

0.94 hits per line

google / oauth2client / 777 / 2
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 777.2 (TOX_ENV=py27)

google / oauth2client / 777 / 2 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 777.2 (TOX_ENV=py27)

google / oauth2client / 777 / 2
100%
master: 100%

README BADGES
x