Techsystech
/
server-tools
mirror of https://github.com/OCA/server-tools.git
3
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[PEP-8]

pull/43/head
Laurent Mignon (aka lmi) 2014-12-08 11:15:15 +01:00
parent c2fec40d13
commit e3e191b455
2 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
auth_from_http_remote_user/__openerp__.py
View File

@ -40,8 +40,8 @@ Use case.
--------- ---------
The module allows integration with external security systems [#]_ that can pass 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, along authentication of a user via Remote_User HTTP header field. In many
this is achieved via server like Apache HTTPD or nginx proxying Odoo. 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:: When proxying your Odoo server with Apache or nginx, It's
important to filter out the Remote_User HTTP header field before your important to filter out the Remote_User HTTP header field before your

12
auth_from_http_remote_user/tests/test_res_users.py
View File

@ -69,9 +69,9 @@ class test_res_users(common.TransactionCase):
@unittest.skipIf(os.environ.get('TRAVIS'), @unittest.skipIf(os.environ.get('TRAVIS'),
'When run by travis, tests runs on a database with all ' 'When run by travis, tests runs on a database with all '
'required addons from server-tools and their dependencies' 'required addons from server-tools and their dependencies'
'installed. Even if `auth_from_http_remote_user` does not ' ' installed. Even if `auth_from_http_remote_user` does '
'require the `mail` module, The previous installation of ' 'not require the `mail` module, The previous installation'
'the mail module has created the column ' ' of the mail module has created the column '
'`notification_email_send` as REQUIRED into the table ' '`notification_email_send` as REQUIRED into the table '
'res_partner. BTW, it\'s no more possible to copy a ' 'res_partner. BTW, it\'s no more possible to copy a '
'res_user without an intefirty error') 'res_user without an intefirty error')
@ -81,8 +81,10 @@ class test_res_users(common.TransactionCase):
res_users_obj = self.registry('res.users') res_users_obj = self.registry('res.users')
vals = {'sso_key': '123'} vals = {'sso_key': '123'}
res_users_obj.write(self.cr, self.uid, self.uid, vals) 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) self.assertDictContainsSubset(vals, read_vals)
copy = res_users_obj.copy(self.cr, self.uid, self.uid) 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')) self.assertFalse(read_vals.get('sso_key'))