Merge lp:~robru/gwibber/baseplugin into lp:~barry/gwibber/py3
Proposed by
Robert Bruce Park
Status: | Merged |
---|---|
Merged at revision: | 1403 |
Proposed branch: | lp:~robru/gwibber/baseplugin |
Merge into: | lp:~barry/gwibber/py3 |
Diff against target: |
74 lines (+44/-3) 2 files modified
gwibber/gwibber/plugins/base.py (+14/-0) gwibber/gwibber/tests/test_plugins.py (+30/-3) |
To merge this branch: | bzr merge lp:~robru/gwibber/baseplugin |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Barry Warsaw | Approve | ||
Review via email: mp+121492@code.launchpad.net |
Description of the change
Add BasePlugin._login method with test coverage. This method represents the common bits of the nearly verbatim Client._login method that existed in every plugin that we have.
To post a comment you must log in.
Approved and merged, with a few minor changes and a bunch of comments. Actually, I made one substantive change: I moved the return statement in BasePlugin._login() to inside the lock context manager. Putting it outside opens a small race condition I think, but it's too hard to test, so I just added a comment.