From e3e191b4550f8682b119f1e6acf2fceb6f6f135f Mon Sep 17 00:00:00 2001 From: "Laurent Mignon (aka lmi)" Date: Mon, 8 Dec 2014 11:15:15 +0100 Subject: [PATCH] [PEP-8] --- auth_from_http_remote_user/__openerp__.py | 4 ++-- auth_from_http_remote_user/tests/test_res_users.py | 14 ++++++++------ 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/auth_from_http_remote_user/__openerp__.py b/auth_from_http_remote_user/__openerp__.py index 5266d4d12..d4e9731c2 100644 --- a/auth_from_http_remote_user/__openerp__.py +++ b/auth_from_http_remote_user/__openerp__.py @@ -40,8 +40,8 @@ Use case. --------- The module allows integration with external security systems [#]_ that can pass -along authentication of a user via Remote_User HTTP header field. In many cases, -this is achieved via server like Apache HTTPD or nginx proxying Odoo. +along authentication of a user via Remote_User HTTP header field. In many +cases, this is achieved via server like Apache HTTPD or nginx proxying Odoo. .. important:: When proxying your Odoo server with Apache or nginx, It's important to filter out the Remote_User HTTP header field before your diff --git a/auth_from_http_remote_user/tests/test_res_users.py b/auth_from_http_remote_user/tests/test_res_users.py index 486531bc7..43d5481c6 100644 --- a/auth_from_http_remote_user/tests/test_res_users.py +++ b/auth_from_http_remote_user/tests/test_res_users.py @@ -68,10 +68,10 @@ class test_res_users(common.TransactionCase): @unittest.skipIf(os.environ.get('TRAVIS'), 'When run by travis, tests runs on a database with all ' - 'required addons from server-tools and their dependencies ' - 'installed. Even if `auth_from_http_remote_user` does not ' - 'require the `mail` module, The previous installation of ' - 'the mail module has created the column ' + 'required addons from server-tools and their dependencies' + ' installed. Even if `auth_from_http_remote_user` does ' + 'not require the `mail` module, The previous installation' + ' of the mail module has created the column ' '`notification_email_send` as REQUIRED into the table ' 'res_partner. BTW, it\'s no more possible to copy a ' 'res_user without an intefirty error') @@ -81,8 +81,10 @@ class test_res_users(common.TransactionCase): res_users_obj = self.registry('res.users') vals = {'sso_key': '123'} res_users_obj.write(self.cr, self.uid, self.uid, vals) - read_vals = res_users_obj.read(self.cr, self.uid, self.uid, ['sso_key']) + read_vals = res_users_obj.read( + self.cr, self.uid, self.uid, ['sso_key']) self.assertDictContainsSubset(vals, read_vals) copy = res_users_obj.copy(self.cr, self.uid, self.uid) - read_vals = res_users_obj.read(self.cr, self.uid, copy, ['sso_key']) + read_vals = res_users_obj.read( + self.cr, self.uid, copy, ['sso_key']) self.assertFalse(read_vals.get('sso_key'))