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

[REF] stock: tests common

Browse Source 
This class referenced demo data but even if they weren't installed, the
tests were passing. Remove the definition and usages.

closes odoo/odoo#35343

Signed-off-by: Simon Lejeune (sle) <sle@openerp.com>
This commit is contained in:
Simon Lejeune
2019-07-31 13:20:38 +00:00
parent 13a535a091
commit 5c8342f9b5
5 changed files with 0 additions and 66 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/product_expiry/tests/test_stock_production_lot.py
-3
View File
@@ -28,7 +28,6 @@ class TestStockProductionLot(TestStockCommon):
})
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location
@@ -125,7 +124,6 @@ class TestStockProductionLot(TestStockCommon):
})
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -174,7 +172,6 @@ class TestStockProductionLot(TestStockCommon):
self.lot1_productCCC = self.LotObj.create({'name': 'Lot 1 ProductCCC', 'product_id': self.productCCC.id})
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
addons/stock/tests/common.py
-3
View File
@@ -4,7 +4,6 @@ from odoo.tests import common
class TestStockCommon(common.SavepointCase):
@classmethod
def setUpClass(cls):
super(TestStockCommon, cls).setUpClass()
@@ -22,8 +21,6 @@ class TestStockCommon(common.SavepointCase):
cls.LotObj = cls.env['stock.production.lot']
# Model Data
cls.partner_agrolite_id = cls.ModelDataObj.xmlid_to_res_id('base.res_partner_2')
cls.partner_delta_id = cls.ModelDataObj.xmlid_to_res_id('base.res_partner_4')
cls.picking_type_in = cls.ModelDataObj.xmlid_to_res_id('stock.picking_type_in')
cls.picking_type_out = cls.ModelDataObj.xmlid_to_res_id('stock.picking_type_out')
cls.supplier_location = cls.ModelDataObj.xmlid_to_res_id('stock.stock_location_suppliers')
addons/stock/tests/test_move2.py
-34
View File
@@ -13,7 +13,6 @@ class TestPickShip(TestStockCommon):
picking_client = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
@@ -32,7 +31,6 @@ class TestPickShip(TestStockCommon):
picking_pick = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.pack_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
@@ -53,7 +51,6 @@ class TestPickShip(TestStockCommon):
picking_ship = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
@@ -70,7 +67,6 @@ class TestPickShip(TestStockCommon):
picking_pack = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.pack_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
@@ -88,7 +84,6 @@ class TestPickShip(TestStockCommon):
picking_pick = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.pack_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
@@ -383,7 +378,6 @@ class TestPickShip(TestStockCommon):
picking_client = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
dest = self.MoveObj.create({
@@ -400,7 +394,6 @@ class TestPickShip(TestStockCommon):
picking_pick = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.pack_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
@@ -747,7 +740,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -785,7 +777,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -824,7 +815,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -869,7 +859,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -919,7 +908,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
move1 = self.MoveObj.create({
@@ -965,7 +953,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
move1 = self.MoveObj.create({
@@ -1010,7 +997,6 @@ class TestSinglePicking(TestStockCommon):
receipt = self.env['stock.picking'].create({
'location_id': self.supplier_location,
'location_dest_id': self.stock_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
})
move1 = self.MoveObj.create({
@@ -1049,7 +1035,6 @@ class TestSinglePicking(TestStockCommon):
delivery = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -1094,7 +1079,6 @@ class TestSinglePicking(TestStockCommon):
delivery = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
product = self.kgB
@@ -1128,7 +1112,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
move1 = self.MoveObj.create({
@@ -1184,7 +1167,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
move1 = self.MoveObj.create({
@@ -1243,7 +1225,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
move1 = self.MoveObj.create({
@@ -1308,7 +1289,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
move1 = self.MoveObj.create({
@@ -1362,7 +1342,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -1415,7 +1394,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -1449,7 +1427,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -1490,7 +1467,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -1531,7 +1507,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.pack_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -1574,7 +1549,6 @@ class TestSinglePicking(TestStockCommon):
receipt = self.env['stock.picking'].create({
'location_id': self.supplier_location,
'location_dest_id': self.stock_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
})
self.MoveObj.create({
@@ -1622,7 +1596,6 @@ class TestSinglePicking(TestStockCommon):
receipt = self.env['stock.picking'].create({
'location_id': self.supplier_location,
'location_dest_id': self.stock_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
})
self.MoveObj.create({
@@ -1668,7 +1641,6 @@ class TestSinglePicking(TestStockCommon):
receipt = self.env['stock.picking'].create({
'location_id': self.supplier_location,
'location_dest_id': self.stock_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
})
move_1 = self.MoveObj.create({
@@ -1711,7 +1683,6 @@ class TestSinglePicking(TestStockCommon):
receipt1 = self.env['stock.picking'].create({
'location_id': self.supplier_location,
'location_dest_id': warehouse.wh_input_stock_loc_id.id,
'partner_id': self.partner_delta_id,
'picking_type_id': warehouse.in_type_id.id,
})
move_receipt_1 = self.MoveObj.create({
@@ -1726,7 +1697,6 @@ class TestSinglePicking(TestStockCommon):
receipt2 = self.env['stock.picking'].create({
'location_id': self.supplier_location,
'location_dest_id': warehouse.wh_input_stock_loc_id.id,
'partner_id': self.partner_delta_id,
'picking_type_id': warehouse.in_type_id.id,
})
move_receipt_2 = self.MoveObj.create({
@@ -1767,7 +1737,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
self.MoveObj.create({
@@ -1802,7 +1771,6 @@ class TestSinglePicking(TestStockCommon):
delivery_order = self.env['stock.picking'].create({
'location_id': self.stock_location,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
move_a = self.MoveObj.create({
@@ -1867,7 +1835,6 @@ class TestStockUOM(TestStockCommon):
})
picking_in = self.env['stock.picking'].create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location
@@ -2347,7 +2314,6 @@ class TestRoutes(TestStockCommon):
picking_ship = self.env['stock.picking'].create({
'location_id': ship_location.id,
'location_dest_id': self.customer_location,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
addons/stock/tests/test_stock_flow.py
-23
View File
@@ -31,7 +31,6 @@ class TestStockFlow(TestStockCommon):
# ----------------------------------------------------------------------
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -168,7 +167,6 @@ class TestStockFlow(TestStockCommon):
# ======================================================================
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -449,7 +447,6 @@ class TestStockFlow(TestStockCommon):
# ----------------------------------------------------------------------
picking_in_A = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -548,7 +545,6 @@ class TestStockFlow(TestStockCommon):
# ----------------------------------------------------------------------
picking_in_B = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -741,7 +737,6 @@ class TestStockFlow(TestStockCommon):
before_out_quantity = self.kgB.qty_available
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -783,7 +778,6 @@ class TestStockFlow(TestStockCommon):
# ======================================================================
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -904,7 +898,6 @@ class TestStockFlow(TestStockCommon):
productKG = self.ProductObj.create({'name': 'Product KG', 'uom_id': self.uom_kg.id, 'uom_po_id': self.uom_kg.id, 'type': 'product'})
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -953,7 +946,6 @@ class TestStockFlow(TestStockCommon):
self.assertEqual(move.product_uom.id, self.uom_tone.id, 'Wrong unit of measure in done move.')
self.assertEqual(productKG.qty_available, 1000, 'Wrong quantity available of product (%s found instead of 1000)' % (productKG.qty_available))
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -1211,7 +1203,6 @@ class TestStockFlow(TestStockCommon):
picking_in.use_create_lots = False
self.productA.tracking = 'lot'
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -1238,7 +1229,6 @@ class TestStockFlow(TestStockCommon):
wizard = self.env[(res_dict.get('res_model'))].browse(res_dict.get('res_id'))
wizard.process()
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'name': 'testpicking',
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
@@ -1269,7 +1259,6 @@ class TestStockFlow(TestStockCommon):
def test_40_pack_in_pack(self):
""" Put a pack in pack"""
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.pack_location,
'location_dest_id': self.customer_location})
@@ -1282,7 +1271,6 @@ class TestStockFlow(TestStockCommon):
'location_id': self.pack_location,
'location_dest_id': self.customer_location})
picking_pack = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.pack_location})
@@ -1296,7 +1284,6 @@ class TestStockFlow(TestStockCommon):
'location_dest_id': self.pack_location,
'move_dest_ids': [(4, move_out.id, 0)]})
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -1390,7 +1377,6 @@ class TestStockFlow(TestStockCommon):
def test_50_create_in_out_with_product_pack_lines(self):
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -1418,7 +1404,6 @@ class TestStockFlow(TestStockCommon):
self.assertEqual(sum(x.quantity for x in pack1.quant_ids), 4.0, 'Pack 1 should have 4 pieces')
self.assertEqual(sum(x.quantity for x in pack2.quant_ids), 6.0, 'Pack 2 should have 6 pieces')
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -1449,7 +1434,6 @@ class TestStockFlow(TestStockCommon):
def test_60_create_in_out_with_product_pack_lines(self):
picking_in = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
@@ -1478,7 +1462,6 @@ class TestStockFlow(TestStockCommon):
self.assertEqual(sum(x.quantity for x in pack1.quant_ids), 120, 'Pack 1 should have 120 pieces')
self.assertEqual(sum(x.quantity for x in pack2.quant_ids), 80, 'Pack 2 should have 80 pieces')
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -1541,7 +1524,6 @@ class TestStockFlow(TestStockCommon):
# create a "all at once" delivery order for two products
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -1589,7 +1571,6 @@ class TestStockFlow(TestStockCommon):
# -----------------------------------------------------------
# create a "all at once" delivery order for two products
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -1633,7 +1614,6 @@ class TestStockFlow(TestStockCommon):
# create a "partial" delivery order for two products
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -1680,7 +1660,6 @@ class TestStockFlow(TestStockCommon):
# "partial" and "force assign" scenario
# -----------------------------------------------------------
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -1705,7 +1684,6 @@ class TestStockFlow(TestStockCommon):
changes to 'confirmed'.
"""
picking_out = self.PickingObj.create({
'partner_id': self.partner_agrolite_id,
'picking_type_id': self.picking_type_out,
'location_id': self.stock_location,
'location_dest_id': self.customer_location})
@@ -1765,7 +1743,6 @@ class TestStockFlow(TestStockCommon):
"""
picking = self.PickingObj.create({
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_in,
'location_id': self.supplier_location,
'location_dest_id': self.stock_location})
addons/stock_picking_batch/tests/test_batch_picking.py
-3
View File
@@ -8,7 +8,6 @@ class TestBatchPicking(TransactionCase):
super(TestBatchPicking, self).setUp()
self.stock_location = self.env.ref('stock.stock_location_stock')
self.customer_location = self.env.ref('stock.stock_location_customers')
self.partner_delta_id = self.env['ir.model.data'].xmlid_to_res_id('base.res_partner_4')
self.picking_type_out = self.env['ir.model.data'].xmlid_to_res_id('stock.picking_type_out')
self.productA = self.env['product.product'].create({
'name': 'Product A',
@@ -24,7 +23,6 @@ class TestBatchPicking(TransactionCase):
self.picking_client_1 = self.env['stock.picking'].create({
'location_id': self.stock_location.id,
'location_dest_id': self.customer_location.id,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})
@@ -41,7 +39,6 @@ class TestBatchPicking(TransactionCase):
self.picking_client_2 = self.env['stock.picking'].create({
'location_id': self.stock_location.id,
'location_dest_id': self.customer_location.id,
'partner_id': self.partner_delta_id,
'picking_type_id': self.picking_type_out,
})