[IMP] Actually generate report in test

Plus other tests to make it to the codecov target

account_asset_management/tests/test_account_asset_management.py

@@ -598,3 +598,37 @@ class TestAssetManagement(SavepointCase):
         # deviations if that is necessary.
         self.assertAlmostEqual(
             asset.depreciation_line_ids[12].amount, 166.63, places=2)
+
+    def test_15_account_asset_group(self):
+        """ Group's name_get behaves differently depending on code and context
+        """
+        group_fa = self.env.ref(
+            'account_asset_management.account_asset_group_fa')
+        group_tfa = self.env.ref(
+            'account_asset_management.account_asset_group_tfa')
+        # Groups are displayed by code (if any) plus name
+        self.assertEqual(
+            self.env['account.asset.group']._name_search('FA'),
+            [(group_fa.id, 'FA Fixed Assets')])
+        # Groups with code are shown by code in list views
+        self.assertEqual(
+            self.env['account.asset.group'].with_context(
+                params={'view_type': 'list'})._name_search('FA'),
+            [(group_fa.id, 'FA')])
+        self.assertEqual(
+            self.env['account.asset.group']._name_search('TFA'),
+            [(group_tfa.id, 'TFA Tangible Fixed Assets')])
+        group_tfa.code = False
+        group_fa.code = False
+        self.assertEqual(
+            group_fa.name_get(),
+            [(group_fa.id, 'Fixed Assets')])
+        # Groups without code are shown by truncated name in lists
+        self.assertEqual(
+            group_tfa.name_get(),
+            [(group_tfa.id, 'Tangible Fixed Assets')])
+        self.assertEqual(
+            group_tfa.with_context(params={'view_type': 'list'}).name_get(),
+            [(group_tfa.id, 'Tangible Fixed A...')])
+        self.assertFalse(
+            self.env['account.asset.group']._name_search('stessA dexiF'))

account_asset_management/tests/test_asset_management_xls.py

@@ -22,6 +22,9 @@ class TestAssetManagementXls(SavepointCase):
         cls._load('account', 'test', 'account_minimal_test.xml')
         cls._load('account_asset_management', 'tests',
                   'account_asset_test_data.xml')
+        # Ensure we have something to report on
+        cls.env.ref('account_asset_management.'
+                    'account_asset_asset_ict0').validate()
         module = __name__.split('addons.')[1].split('.')[0]
         cls.xls_report_name = '{}.asset_report_xls'.format(module)
         cls.wiz_model = cls.env['wiz.account.asset.report']
@@ -39,9 +42,15 @@ class TestAssetManagementXls(SavepointCase):
         cls.report_action = cls.xls_report.xls_export()
 
     def test_01_action_xls(self):
-        """ Check report XLS action """
+        """ Check report XLS action and generate report """
         self.assertDictContainsSubset(
             {'type': 'ir.actions.report',
              'report_type': 'xlsx',
              'report_name': self.xls_report_name},
             self.report_action)
+        model = self.env[
+            'report.%s' % self.report_action['report_name']].with_context(
+                active_model=self.xls_report._name,
+                **self.report_action['context'])
+        model.create_xlsx_report(
+            self.xls_report.ids, data=self.report_action['data'])