pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[IMP] sale_mrp: Remove test_sale_mrp_flow dependency to AccountTestCommon

Browse Source 
l10n runbot builds are all failing when running at least one test depending of AccountTestCommon because it:
- doesn't create a sandboxed testing environnement to manage the multi-currency, multi-company, the default company's currency, the exchange rates...
- doesn't setup a testing user then all tests are done using the superuser.
- doesn't provide a fully setup chart of accounts: exchange difference journal is not set, accounts have bad types, etc...
- is run sometimes at-install.

--task: 2301180
This commit is contained in:
Laurent Smet
2020-07-31 07:11:32 +00:00
parent 1824f69cad
commit b0216e3eb1
1 changed files with 35 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/sale_mrp/tests/test_sale_mrp_flow.py
+35 -32
View File
@@ -1,7 +1,7 @@
# -*- coding: utf-8 -*-
# Part of Odoo. See LICENSE file for full copyright and licensing details.
from odoo.addons.account.tests.common import AccountTestCommon
from odoo.addons.stock_account.tests.test_anglo_saxon_valuation_reconciliation_common import ValuationReconciliationTestCommon
from odoo.tests import common, Form
from odoo.exceptions import UserError
from odoo.tools import mute_logger, float_compare
@@ -9,11 +9,12 @@ from odoo.tools import mute_logger, float_compare
# these tests create accounting entries, and therefore need a chart of accounts
@common.tagged('post_install', '-at_install')
class TestSaleMrpFlow(AccountTestCommon):
class TestSaleMrpFlow(ValuationReconciliationTestCommon):
@classmethod
def setUpClass(cls):
super(TestSaleMrpFlow, cls).setUpClass()
def setUpClass(cls, chart_template_ref=None):
super().setUpClass(chart_template_ref=chart_template_ref)
# Useful models
cls.StockMove = cls.env['stock.move']
cls.UoM = cls.env['uom.uom']
@@ -24,8 +25,6 @@ class TestSaleMrpFlow(AccountTestCommon):
cls.categ_unit = cls.env.ref('uom.product_uom_categ_unit')
cls.categ_kgm = cls.env.ref('uom.product_uom_categ_kgm')
cls.stock_location = cls.env.ref('stock.stock_location_stock')
cls.warehouse = cls.env.ref('stock.warehouse0')
cls.uom_kg = cls.env['uom.uom'].search([('category_id', '=', cls.categ_kgm.id), ('uom_type', '=', 'reference')], limit=1)
cls.uom_kg.write({
@@ -232,8 +231,8 @@ class TestSaleMrpFlow(AccountTestCommon):
# Create product A, B, C, D.
# --------------------------
route_manufacture = self.warehouse.manufacture_pull_id.route_id
route_mto = self.warehouse.mto_pull_id.route_id
route_manufacture = self.company_data['default_warehouse'].manufacture_pull_id.route_id
route_mto = self.company_data['default_warehouse'].mto_pull_id.route_id
product_a = self._create_product('Product A', self.uom_unit, routes=[route_manufacture, route_mto])
product_c = self._create_product('Product C', self.uom_kg)
product_b = self._create_product('Product B', self.uom_dozen, routes=[route_manufacture, route_mto])
@@ -416,7 +415,7 @@ class TestSaleMrpFlow(AccountTestCommon):
'product_id': product_c.id,
'product_uom_id': self.uom_kg.id,
'product_qty': 20,
'location_id': self.stock_location.id})
'location_id': self.company_data['default_warehouse'].lot_stock_id.id})
inventory.action_validate()
# --------------------------------------------------
@@ -464,7 +463,7 @@ class TestSaleMrpFlow(AccountTestCommon):
'product_id': product_c.id,
'product_uom_id': self.uom_kg.id,
'product_qty': 27.5025,
'location_id': self.stock_location.id})
'location_id': self.company_data['default_warehouse'].lot_stock_id.id})
inventory.action_validate()
# Assign product to manufacturing order of product A.
@@ -500,7 +499,7 @@ class TestSaleMrpFlow(AccountTestCommon):
'categ_id': self.env.ref('product.product_category_all').id,
})
# Remove the MTO route as purchase is not installed and since the procurement removal the exception is directly raised
product.write({'route_ids': [(6, 0, [self.warehouse.manufacture_pull_id.route_id.id])]})
product.write({'route_ids': [(6, 0, [self.company_data['default_warehouse'].manufacture_pull_id.route_id.id])]})
product_wood_panel = self.env['product.product'].create({
'name': 'Wood Panel',
@@ -532,7 +531,8 @@ class TestSaleMrpFlow(AccountTestCommon):
# if `delivery` module is installed, a default property is set for the carrier to use
# However this will lead to an extra line on the SO (the delivery line), which will force
# the SO to have a different flow (and `invoice_state` value)
partner.property_delivery_carrier_id = False
if 'property_delivery_carrier_id' in partner:
partner.property_delivery_carrier_id = False
f = Form(self.env['sale.order'])
f.partner_id = partner
@@ -589,7 +589,7 @@ class TestSaleMrpFlow(AccountTestCommon):
'factor': 1,
'uom_type': 'bigger',
'rounding': 1.0})
self.company = self.env.ref('base.main_company')
self.company = self.company_data['company']
self.company.anglo_saxon_accounting = True
self.partner = self.env['res.partner'].create({'name': 'My Test Partner'})
self.category = self.env.ref('product.product_category_1').copy({'name': 'Test category','property_valuation': 'real_time', 'property_cost_method': 'fifo'})
@@ -625,14 +625,17 @@ class TestSaleMrpFlow(AccountTestCommon):
'uom_id': self.uom_unit.id,
'categ_id': self.category.id,
'standard_price': 10})
self.env['stock.quant'].create({
# Create quants with sudo to avoid:
# "You are not allowed to create 'Quants' (stock.quant) records. No group currently allows this operation."
self.env['stock.quant'].sudo().create({
'product_id': self.component1.id,
'location_id': self.env.ref('stock.stock_location_stock').id,
'location_id': self.company_data['default_warehouse'].lot_stock_id.id,
'quantity': 6.0,
})
self.env['stock.quant'].create({
self.env['stock.quant'].sudo().create({
'product_id': self.component2.id,
'location_id': self.env.ref('stock.stock_location_stock').id,
'location_id': self.company_data['default_warehouse'].lot_stock_id.id,
'quantity': 3.0,
})
self.bom = self.env['mrp.bom'].create({
@@ -704,7 +707,7 @@ class TestSaleMrpFlow(AccountTestCommon):
# Updating the quantities in stock to prevent
# a 'Not enough inventory' warning message.
stock_location = self.env.ref('stock.stock_location_stock')
stock_location = self.company_data['default_warehouse'].lot_stock_id
self.env['stock.quant']._update_available_quantity(self.component_a, stock_location, 20)
self.env['stock.quant']._update_available_quantity(self.component_b, stock_location, 10)
self.env['stock.quant']._update_available_quantity(self.component_c, stock_location, 30)
@@ -824,7 +827,7 @@ class TestSaleMrpFlow(AccountTestCommon):
# Updating the quantities in stock to prevent
# a 'Not enough inventory' warning message.
stock_location = self.env.ref('stock.stock_location_stock')
stock_location = self.company_data['default_warehouse'].lot_stock_id
self.env['stock.quant']._update_available_quantity(self.component_a, stock_location, 56)
self.env['stock.quant']._update_available_quantity(self.component_b, stock_location, 28)
self.env['stock.quant']._update_available_quantity(self.component_c, stock_location, 84)
@@ -1160,7 +1163,7 @@ class TestSaleMrpFlow(AccountTestCommon):
# Updating the quantities in stock to prevent
# a 'Not enough inventory' warning message.
stock_location = self.env.ref('stock.stock_location_stock')
stock_location = self.company_data['default_warehouse'].lot_stock_id
self.env['stock.quant']._update_available_quantity(component_uom_unit, stock_location, 240)
self.env['stock.quant']._update_available_quantity(component_uom_dozen, stock_location, 10)
self.env['stock.quant']._update_available_quantity(component_uom_kg, stock_location, 0.03)
@@ -1347,11 +1350,11 @@ class TestSaleMrpFlow(AccountTestCommon):
stock_location_components = self.env['stock.location'].create({
'name': 'Shelf 1',
'location_id': self.env.ref('stock.warehouse0').lot_stock_id.id,
'location_id': self.company_data['default_warehouse'].lot_stock_id.id,
})
stock_location_14 = self.env['stock.location'].create({
'name': 'Shelf 2',
'location_id': self.env.ref('stock.warehouse0').lot_stock_id.id,
'location_id': self.company_data['default_warehouse'].lot_stock_id.id,
})
kit_1 = self._create_product('Kit1', self.uom_unit)
@@ -1379,7 +1382,7 @@ class TestSaleMrpFlow(AccountTestCommon):
'rule_ids': [(0, 0, {
'name': 'Shelf1 -> Customer',
'action': 'pull',
'picking_type_id': self.ref('stock.picking_type_in'),
'picking_type_id': self.company_data['default_warehouse'].in_type_id.id,
'location_src_id': stock_location_components.id,
'location_id': self.ref('stock.stock_location_customers'),
})],
@@ -1391,7 +1394,7 @@ class TestSaleMrpFlow(AccountTestCommon):
'rule_ids': [(0, 0, {
'name': 'Shelf2 -> Customer',
'action': 'pull',
'picking_type_id': self.ref('stock.picking_type_in'),
'picking_type_id': self.company_data['default_warehouse'].in_type_id.id,
'location_src_id': stock_location_14.id,
'location_id': self.ref('stock.stock_location_customers'),
})],
@@ -1403,8 +1406,8 @@ class TestSaleMrpFlow(AccountTestCommon):
'route_ids': [(4, route_shelf2.id)]})
# Set enough quantities to make 1 kit_uom_in_kit in WH1
self.env['stock.quant']._update_available_quantity(component_shelf1, self.env.ref('stock.warehouse0').lot_stock_id, 15)
self.env['stock.quant']._update_available_quantity(component_shelf2, self.env.ref('stock.warehouse0').lot_stock_id, 10)
self.env['stock.quant']._update_available_quantity(component_shelf1, self.company_data['default_warehouse'].lot_stock_id, 15)
self.env['stock.quant']._update_available_quantity(component_shelf2, self.company_data['default_warehouse'].lot_stock_id, 10)
# Creating a sale order for 5 kits and confirming it
order_form = Form(self.env['sale.order'])
@@ -1487,8 +1490,8 @@ class TestSaleMrpFlow(AccountTestCommon):
self.assertEqual(move_component_kg.product_uom_qty, 0.583)
def test_product_type_service_1(self):
route_manufacture = self.warehouse.manufacture_pull_id.route_id.id
route_mto = self.warehouse.mto_pull_id.route_id.id
route_manufacture = self.company_data['default_warehouse'].manufacture_pull_id.route_id.id
route_mto = self.company_data['default_warehouse'].mto_pull_id.route_id.id
self.uom_unit = self.env.ref('uom.product_uom_unit')
# Create finished product
@@ -1536,8 +1539,8 @@ class TestSaleMrpFlow(AccountTestCommon):
Cancel the delivery and the production order. Then duplicate
the delivery. Another production order should be created."""
route_manufacture = self.warehouse.manufacture_pull_id.route_id.id
route_mto = self.warehouse.mto_pull_id.route_id.id
route_manufacture = self.company_data['default_warehouse'].manufacture_pull_id.route_id.id
route_mto = self.company_data['default_warehouse'].mto_pull_id.route_id.id
self.uom_unit = self.env.ref('uom.product_uom_unit')
# Create finished product
@@ -1590,8 +1593,8 @@ class TestSaleMrpFlow(AccountTestCommon):
Cancel the production order and the delivery. Then duplicate
the delivery. Another production order should be created."""
route_manufacture = self.warehouse.manufacture_pull_id.route_id.id
route_mto = self.warehouse.mto_pull_id.route_id.id
route_manufacture = self.company_data['default_warehouse'].manufacture_pull_id.route_id.id
route_mto = self.company_data['default_warehouse'].mto_pull_id.route_id.id
self.uom_unit = self.env.ref('uom.product_uom_unit')
# Create finished product