From 78fa67cbeb73b459f82b9b0728e71da5359adf78 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Tue, 27 Jan 2015 18:12:47 +0100
Subject: [PATCH] Add check for default model access

When group access exists, a default model access should be defined otherwise
all access are given to non group users.

issue4485
review5941002
---
 asset.xml                   | 14 ++++++++++++++
 tests/test_account_asset.py | 20 +++-----------------
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/asset.xml b/asset.xml
index e39ef470af..f9dfcd0a69 100644
--- a/asset.xml
+++ b/asset.xml
@@ -102,6 +102,13 @@
             <field name="rule_group" ref="rule_group_asset"/>
         </record>
 
+        <record model="ir.model.access" id="access_asset">
+            <field name="model" search="[('model', '=', 'account.asset')]"/>
+            <field name="perm_read" eval="False"/>
+            <field name="perm_write" eval="False"/>
+            <field name="perm_create" eval="False"/>
+            <field name="perm_delete" eval="False"/>
+        </record>
         <record model="ir.model.access" id="access_asset_account_admin">
             <field name="model" search="[('model', '=', 'account.asset')]"/>
             <field name="group" ref="account.group_account_admin"/>
@@ -130,6 +137,13 @@
             <field name="name">asset_line_tree</field>
         </record>
 
+        <record model="ir.model.access" id="access_asset_line">
+            <field name="model" search="[('model', '=', 'account.asset.line')]"/>
+            <field name="perm_read" eval="False"/>
+            <field name="perm_write" eval="False"/>
+            <field name="perm_create" eval="False"/>
+            <field name="perm_delete" eval="False"/>
+        </record>
         <record model="ir.model.access" id="access_asset_line_account_admin">
             <field name="model" search="[('model', '=', 'account.asset.line')]"/>
             <field name="group" ref="account.group_account_admin"/>
diff --git a/tests/test_account_asset.py b/tests/test_account_asset.py
index 6fdde4dada..7c929d7618 100644
--- a/tests/test_account_asset.py
+++ b/tests/test_account_asset.py
@@ -3,27 +3,13 @@
 import unittest
 import doctest
 import trytond.tests.test_tryton
-from trytond.tests.test_tryton import test_view, test_depends, test_menu_action
+from trytond.tests.test_tryton import ModuleTestCase
 from trytond.tests.test_tryton import doctest_setup, doctest_teardown
 
 
-class AccountAssetTestCase(unittest.TestCase):
+class AccountAssetTestCase(ModuleTestCase):
     'Test AccountAsset module'
-
-    def setUp(self):
-        trytond.tests.test_tryton.install_module('account_asset')
-
-    def test0005views(self):
-        'Test views'
-        test_view('account_asset')
-
-    def test0006depends(self):
-        'Test depends'
-        test_depends()
-
-    def test0007menu_actions(self):
-        'Test menu actions'
-        test_menu_action('account_asset')
+    module = 'account_asset'
 
 
 def suite():