From 74b09730456bd11492d969c8cf00cfde3b02bba3 Mon Sep 17 00:00:00 2001
From: Laurent Mignon <laurent.mignon@acsone.eu>
Date: Thu, 6 Oct 2016 10:46:28 +0200
Subject: [PATCH] Add more tests and fixes issues found by tests

---
 report_py3o/models/ir_actions_report_xml.py |  2 +-
 report_py3o/tests/test_report_py3o.py       | 87 ++++++++++++++++++++-
 2 files changed, 86 insertions(+), 3 deletions(-)

diff --git a/report_py3o/models/ir_actions_report_xml.py b/report_py3o/models/ir_actions_report_xml.py
index a4cc3e5d..bf262914 100644
--- a/report_py3o/models/ir_actions_report_xml.py
+++ b/report_py3o/models/ir_actions_report_xml.py
@@ -36,7 +36,7 @@ class IrActionsReportXml(models.Model):
     @api.constrains("py3o_is_local_fusion", "py3o_server_id",
                     "py3o_filetype")
     def _check_py3o_server_id(self):
-        is_native = Formats().get_format(self.py3o_filetype)
+        is_native = Formats().get_format(self.py3o_filetype).native
         if ((not is_native or not self.py3o_is_local_fusion) and
                 not self.py3o_server_id):
             raise ValidationError(_(
diff --git a/report_py3o/tests/test_report_py3o.py b/report_py3o/tests/test_report_py3o.py
index b2ed2f11..ac58f59c 100644
--- a/report_py3o/tests/test_report_py3o.py
+++ b/report_py3o/tests/test_report_py3o.py
@@ -2,15 +2,59 @@
 # Copyright 2016 ACSONE SA/NV
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).).
 
-
 import mock
+import os
+import pkg_resources
+
+from py3o.formats import Formats
 
 from openerp.tests.common import TransactionCase
-import openerp.tests
+from openerp.exceptions import ValidationError
+
+from ..py3o_parser import TemplateNotFound
+from base64 import b64encode
 
 
 class TestReportPy3o(TransactionCase):
 
+    def test_no_local_fusion_without_fusion_server(self):
+        report = self.env.ref("report_py3o.res_users_report_py3o")
+        self.assertTrue(report.py3o_is_local_fusion)
+        with self.assertRaises(ValidationError) as e:
+            report.py3o_is_local_fusion = False
+        self.assertEqual(
+            e.exception.name,
+            "Can not use not native format in local fusion. "
+            "Please specify a Fusion Server")
+
+    def test_no_native_format_without_fusion_server(self):
+        report = self.env.ref("report_py3o.res_users_report_py3o")
+        formats = Formats()
+        is_native = formats.get_format(report.py3o_filetype).native
+        self.assertTrue(is_native)
+        new_format = None
+        for name in formats.get_known_format_names():
+            format = formats.get_format(name)
+            if not format.native:
+                new_format = name
+                break
+        self.assertTrue(new_format)
+        with self.assertRaises(ValidationError) as e:
+            report.py3o_filetype = new_format
+        self.assertEqual(
+            e.exception.name,
+            "Can not use not native format in local fusion. "
+            "Please specify a Fusion Server")
+
+    def test_required_py3_filetype(self):
+        report = self.env.ref("report_py3o.res_users_report_py3o")
+        self.assertEqual(report.report_type, "py3o")
+        with self.assertRaises(ValidationError) as e:
+            report.py3o_filetype = False
+        self.assertEqual(
+            e.exception.name,
+            "Field 'Output Format' is required for Py3O report")
+
     def test_reports(self):
         report = self.env.ref("report_py3o.res_users_report_py3o")
         with mock.patch('openerp.addons.report_py3o.py3o_parser.'
@@ -35,3 +79,42 @@ class TestReportPy3o(TransactionCase):
             res = report.render_report(
                 self.env.user.ids, report.report_name, {})
             self.assertEqual(('test result', '.pdf'), res)
+
+    def test_report_template_configs(self):
+        report = self.env.ref("report_py3o.res_users_report_py3o")
+        # the demo template is specified with a relative path in in the module
+        # path
+        tmpl_name = report.py3o_template_fallback
+        flbk_filename = pkg_resources.resource_filename(
+            "openerp.addons.%s" % report.module,
+            tmpl_name)
+        self.assertTrue(os.path.exists(flbk_filename))
+        res = report.render_report(
+            self.env.user.ids, report.report_name, {})
+        self.assertTrue(res)
+        # The generation fails if the tempalte is not found
+        report.module = False
+        with self.assertRaises(TemplateNotFound), self.env.cr.savepoint():
+            report.render_report(
+                self.env.user.ids, report.report_name, {})
+
+        # the template can also be provivided as an abspaath
+        report.py3o_template_fallback = flbk_filename
+        res = report.render_report(
+            self.env.user.ids, report.report_name, {})
+        self.assertTrue(res)
+
+        # the tempalte can also be provided as a binay field
+        report.py3o_template_fallback = False
+
+        with open(flbk_filename) as tmpl_file:
+            tmpl_data = b64encode(tmpl_file.read())
+        py3o_template = self.env['py3o.template'].create({
+            'name': 'test_template',
+            'py3o_template_data': tmpl_data,
+            'filetype': 'odt'})
+        report.py3o_template_id = py3o_template
+        report.py3o_template_fallback = flbk_filename
+        res = report.render_report(
+            self.env.user.ids, report.report_name, {})
+        self.assertTrue(res)