From 20a2dfce5a9cfe68d48adf6c88c8c8b80f13c7df Mon Sep 17 00:00:00 2001 From: Stefan Rijnhart Date: Wed, 3 Feb 2021 20:22:01 +0100 Subject: [PATCH] [IMP] Actually generate report in test Plus other tests to make it to the codecov target --- .../tests/test_account_asset_management.py | 34 +++++++++++++++++++ .../tests/test_asset_management_xls.py | 11 +++++- 2 files changed, 44 insertions(+), 1 deletion(-) diff --git a/account_asset_management/tests/test_account_asset_management.py b/account_asset_management/tests/test_account_asset_management.py index 46de4edda..3cf7647fd 100644 --- a/account_asset_management/tests/test_account_asset_management.py +++ b/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')) diff --git a/account_asset_management/tests/test_asset_management_xls.py b/account_asset_management/tests/test_asset_management_xls.py index ac4cb0f4b..a05673b21 100644 --- a/account_asset_management/tests/test_asset_management_xls.py +++ b/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'])