[ADD] test file and resolved travis issues

product_sequence/README.rst

@@ -21,8 +21,6 @@ Otherwise the setting of the unique constraint will fail and the module will fai
 Usage
 =====
 
-#. Go to ...
-
 .. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
    :alt: Try me on Runbot
    :target: https://runbot.odoo-community.org/runbot/135/9.0

product_sequence/__init__.py

@@ -4,4 +4,4 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 from . import models
-from .models.product_product import update_null_and_slash_codes
+from .hooks import pre_init_hook

product_sequence/__openerp__.py

@@ -19,7 +19,7 @@
     'demo': [
         'demo/product_product.xml'
     ],
-    'pre_init_hook': 'update_null_and_slash_codes',
+    'pre_init_hook': 'pre_init_hook',
     'auto_install': False,
     'installable': True,
 }

product_sequence/demo/product_product.xml

@@ -5,6 +5,7 @@
     Copyright (C) 2016 Sodexis
     The licence is in the file __openerp__.py
 -->
+
 <odoo>
     <record id="product.product_product_0" model="product.product">
         <field name="default_code">SERVICE</field>
@@ -15,4 +16,4 @@
     <record id="product.product_product_2" model="product.product">
         <field name="default_code">OSA</field>
     </record>
-</odoo>
+</odoo>

product_sequence/hooks.py

@@ -0,0 +1,17 @@
+# -*- coding: utf-8 -*-
+# © 2004 Tiny SPRL
+# © 2016 Sodexis
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+
+def pre_init_hook(cr):
+    """
+    Updates existing codes matching the default '/' or
+    empty. Primarily this ensures installation does not
+    fail for demo data.
+    :param cr: database cursor
+    :return: void
+    """
+    cr.execute("UPDATE product_product "
+               "SET default_code = '!!mig!!' || id "
+               "WHERE default_code IS NULL OR default_code = '/';")

product_sequence/models/product_product.py

@@ -6,19 +6,6 @@
 from openerp import models, fields, api, _
 
 
-def update_null_and_slash_codes(cr):  # pragma: no cover
-    """
-    Updates existing codes matching the default '/' or
-    empty. Primarily this ensures installation does not
-    fail for demo data.
-    :param cr: database cursor
-    :return: void
-    """
-    cr.execute("UPDATE product_product "
-               "SET default_code = '!!mig!!' || id "
-               "WHERE default_code IS NULL OR default_code = '/';")
-
-
 class ProductProduct(models.Model):
     _inherit = 'product.product'
 
@@ -51,7 +38,7 @@ def write(self, vals):
             super(ProductProduct, product).write(vals)
         return True
 
-    @api.one
+    @api.multi
     def copy(self, default=None):
         if default is None:
             default = {}

product_sequence/tests/__init__.py

@@ -0,0 +1,5 @@
+# -*- coding: utf-8 -*-
+# © 2016 Sodexis
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from . import test_product_sequence

product_sequence/tests/test_product_sequence.py

@@ -0,0 +1,24 @@
+# -*- coding: utf-8 -*-
+# © 2016 Sodexis
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+
+from openerp.tests.common import TransactionCase
+
+
+class TestProductSequence(TransactionCase):
+    """Tests for creating product with and without Product Sequence"""
+
+    def setUp(self):
+        super(TestProductSequence, self).setUp()
+        self.product_product = self.env['product.product']
+
+    def test_product_create_with_default_code(self):
+        product_id = self.product_product.create(dict(
+            name="Apple",
+            default_code='PROD01'
+        ))
+        self.assertEqual(product_id.default_code, 'PROD01')
+
+    def test_product_create_without_default_code(self):
+        product_id_1 = self.product_product.create(dict(name="Orange"))
+        self.assertRegexpMatches(str(product_id_1.default_code), r'PR/*')