Browse Source

[FIX] mis_builder test back to green

pull/189/head
Stéphane Bidoul 8 years ago
parent
commit
d054ee6354
  1. 2
      mis_builder/models/mis_builder.py
  2. 4
      mis_builder/tests/mis.report.instance.period.csv
  3. 22
      mis_builder/tests/test_mis_builder.py

2
mis_builder/models/mis_builder.py

@ -320,7 +320,7 @@ class MisReportKpi(models.Model):
def render(self, lang_id, value):
""" render a KPI value as a unicode string, ready for display """
assert len(self) == 1
if value is None or value == AccountingNone:
if value is None or value is AccountingNone:
return ''
elif self.type == 'num':
return self._render_num(lang_id, value, self.divider,

4
mis_builder/tests/mis.report.instance.period.csv

@ -1,2 +1,2 @@
"id","duration","name","offset","type","sequence"
"mis_report_instance_period_test","1","today","","Day",""
"id","duration","name","offset","type","sequence","mode"
"mis_report_instance_period_test","1","today","","Day","","relative"

22
mis_builder/tests/test_mis_builder.py

@ -39,15 +39,17 @@ class TestMisBuilder(common.TransactionCase):
self.assertDictContainsSubset(
{'content':
[{'kpi_name': u'total test',
'default_style': False,
'default_style': '',
'default_xlsx_style': {},
'cols': [{'period_id': self.ref('mis_builder.'
'mis_report_instance_'
'period_test'),
'style': None,
'style': '',
'xlsx_style': {},
'prefix': False,
'suffix': False,
'expr': 'len(test)',
'val_c': 'total_test = len(test)',
'expr': u'len(test)',
'val_c': u'total_test = len(test)',
'val': 0,
'val_r': u'\u202f0\xa0',
'is_percentage': False,
@ -57,6 +59,14 @@ class TestMisBuilder(common.TransactionCase):
'header':
[{'kpi_name': '',
'cols': [{'date': '07/31/2014',
'name': u'today'}]
}],
'colspan': 1,
'name': u'today',
}],
},
{'kpi_name': '',
'cols': [{'colspan': 1,
'name': '',
}],
},
],
}, data)
Loading…
Cancel
Save