[IMP] product_secondary_unit: black, isort

product_secondary_unit/__manifest__.py

@@ -1,21 +1,16 @@
 # Copyright 2018 Tecnativa - Sergio Teruel
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 {
-    'name': 'Product Secondary Unit',
-    'summary': 'Set a secondary unit per product',
-    'version': '12.0.1.0.0',
-    'development_status': 'Beta',
-    'category': 'Product',
-    'website': 'https://github.com/OCA/product-attribute',
-    'author': 'Tecnativa, Odoo Community Association (OCA)',
-    'license': 'AGPL-3',
-    'application': False,
-    'installable': True,
-    'depends': [
-        'product',
-    ],
-    'data': [
-        'security/ir.model.access.csv',
-        'views/product_views.xml',
-    ],
+    "name": "Product Secondary Unit",
+    "summary": "Set a secondary unit per product",
+    "version": "13.0.1.0.0",
+    "development_status": "Beta",
+    "category": "Product",
+    "website": "https://github.com/OCA/product-attribute",
+    "author": "Tecnativa, Odoo Community Association (OCA)",
+    "license": "AGPL-3",
+    "application": False,
+    "installable": True,
+    "depends": ["product"],
+    "data": ["security/ir.model.access.csv", "views/product_views.xml"],
 }

product_secondary_unit/models/product_second_unit.py

@@ -4,48 +4,45 @@
 
 
 class ProductSecondaryUnit(models.Model):
-    _name = 'product.secondary.unit'
-    _description = 'Product Secondary Unit'
+    _name = "product.secondary.unit"
+    _description = "Product Secondary Unit"
 
-    name = fields.Char(
-        required=True,
-        translate=True,
-    )
+    name = fields.Char(required=True, translate=True)
     code = fields.Char()
     product_tmpl_id = fields.Many2one(
-        comodel_name='product.template',
-        string='Product Template',
-        required=True,
+        comodel_name="product.template", string="Product Template", required=True
     )
     uom_id = fields.Many2one(
-        comodel_name='uom.uom',
-        string='Secondary Unit of Measure',
+        comodel_name="uom.uom",
+        string="Secondary Unit of Measure",
         required=True,
         help="Default Secondary Unit of Measure.",
     )
     factor = fields.Float(
-        string='Secondary Unit Factor',
-        default=1.0,
-        digits=0,
-        required=True,
+        string="Secondary Unit Factor", default=1.0, digits=0, required=True
     )
 
     @api.multi
     def name_get(self):
         result = []
         for unit in self:
-            result.append((unit.id, "{unit_name}-{factor}".format(
-                unit_name=unit.name,
-                factor=unit.factor))
+            result.append(
+                (
+                    unit.id,
+                    "{unit_name}-{factor}".format(
+                        unit_name=unit.name, factor=unit.factor
+                    ),
+                )
             )
         return result
 
     @api.model
-    def name_search(self, name='', args=None, operator='ilike', limit=100):
+    def name_search(self, name="", args=None, operator="ilike", limit=100):
         if args is None:
             args = []
-        units = self.search([('code', '=', name)] + args, limit=1)
+        units = self.search([("code", "=", name)] + args, limit=1)
         if not units:
             return super(ProductSecondaryUnit, self).name_search(
-                name=name, args=args, operator=operator, limit=limit)
+                name=name, args=args, operator=operator, limit=limit
+            )
         return units.name_get()

product_secondary_unit/models/product_template.py

@@ -4,11 +4,11 @@
 
 
 class ProductTemplate(models.Model):
-    _inherit = 'product.template'
+    _inherit = "product.template"
 
     secondary_uom_ids = fields.One2many(
-        comodel_name='product.secondary.unit',
-        inverse_name='product_tmpl_id',
-        string='Secondary Unit of Measure',
-        help='Default Secondary Unit of Measure.',
+        comodel_name="product.secondary.unit",
+        inverse_name="product_tmpl_id",
+        string="Secondary Unit of Measure",
+        help="Default Secondary Unit of Measure.",
     )

product_secondary_unit/tests/test_product_second_unit.py

@@ -10,40 +10,53 @@ class TestProductSecondaryUnit(SavepointCase):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
-        cls.product_uom_kg = cls.env.ref('uom.product_uom_kgm')
-        cls.product_uom_unit = cls.env.ref('uom.product_uom_unit')
-        cls.product = cls.env['product.template'].create({
-            'name': 'test',
-            'uom_id': cls.product_uom_kg.id,
-            'uom_po_id': cls.product_uom_kg.id,
-            'secondary_uom_ids': [
-                (0, 0, {
-                    'code': 'A',
-                    'name': 'unit-700',
-                    'uom_id': cls.product_uom_unit.id,
-                    'factor': 0.7,
-                }),
-                (0, 0, {
-                    'code': 'B',
-                    'name': 'unit-900',
-                    'uom_id': cls.product_uom_unit.id,
-                    'factor': 0.9,
-                }),
-            ],
-        })
-        cls.secondary_unit = cls.env['product.secondary.unit'].search([
-            ('product_tmpl_id', '=', cls.product.id),
-        ], limit=1)
+        cls.product_uom_kg = cls.env.ref("uom.product_uom_kgm")
+        cls.product_uom_unit = cls.env.ref("uom.product_uom_unit")
+        cls.product = cls.env["product.template"].create(
+            {
+                "name": "test",
+                "uom_id": cls.product_uom_kg.id,
+                "uom_po_id": cls.product_uom_kg.id,
+                "secondary_uom_ids": [
+                    (
+                        0,
+                        0,
+                        {
+                            "code": "A",
+                            "name": "unit-700",
+                            "uom_id": cls.product_uom_unit.id,
+                            "factor": 0.7,
+                        },
+                    ),
+                    (
+                        0,
+                        0,
+                        {
+                            "code": "B",
+                            "name": "unit-900",
+                            "uom_id": cls.product_uom_unit.id,
+                            "factor": 0.9,
+                        },
+                    ),
+                ],
+            }
+        )
+        cls.secondary_unit = cls.env["product.secondary.unit"].search(
+            [("product_tmpl_id", "=", cls.product.id)], limit=1
+        )
 
     def test_product_secondary_unit_name(self):
-        self.assertEqual(self.secondary_unit.name_get()[0][1], 'unit-700-0.7')
+        self.assertEqual(self.secondary_unit.name_get()[0][1], "unit-700-0.7")
 
     def test_product_secondary_unit_search(self):
-        args = [('product_tmpl_id.product_variant_ids', 'in',
-                 self.product.product_variant_ids.ids)]
-        name_get = self.env['product.secondary.unit'].name_search(
-            name='A', args=args)
+        args = [
+            (
+                "product_tmpl_id.product_variant_ids",
+                "in",
+                self.product.product_variant_ids.ids,
+            )
+        ]
+        name_get = self.env["product.secondary.unit"].name_search(name="A", args=args)
         self.assertEqual(len(name_get), 1)
-        name_get = self.env['product.secondary.unit'].name_search(
-            name='X', args=args)
+        name_get = self.env["product.secondary.unit"].name_search(name="X", args=args)
         self.assertEqual(len(name_get), 0)