Skip to content

Commit

Permalink
[FIX] spreadsheet_dashboard: unskip expense dashboard test
Browse files Browse the repository at this point in the history
The test was skipped because the expense dashboard was broken when
the test was merged.
See odoo@17afe29

closes odoo#109017

Signed-off-by: Rémi Rahir (rar) <rar@odoo.com>
  • Loading branch information
LucasLefevre committed Jan 3, 2023
1 parent e5b60e6 commit 4d4c192
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 15 deletions.
7 changes: 0 additions & 7 deletions addons/spreadsheet_dashboard/tests/test_dashboard_data.py
Expand Up @@ -10,12 +10,5 @@ def test_validate_dashboard_data(self):
"""validate fields and models used in dashboards"""
dashboards = self.env["spreadsheet.dashboard"].search([])
for dashboard in dashboards:
# this dashboard is skipped because it's currently broken
# but I still want to merge this test right now to avoid other broken dashboards
if dashboard == self.env.ref(
"spreadsheet_dashboard_hr_expense.spreadsheet_dashboard_expense",
raise_if_not_found=False,
):
continue
with self.subTest(dashboard.name):
self.validate_spreadsheet_data(dashboard.raw, dashboard.name)
Expand Up @@ -1040,10 +1040,6 @@
"8": {
"field": "product_id",
"type": "many2one"
},
"9": {
"field": "product_id",
"type": "many2one"
}
},
"listFields": {
Expand Down Expand Up @@ -1092,10 +1088,6 @@
"8": {
"field": "employee_id",
"type": "many2one"
},
"9": {
"field": "employee_id",
"type": "many2one"
}
},
"listFields": {
Expand Down

0 comments on commit 4d4c192

Please sign in to comment.