992 lines
47 KiB
Python
992 lines
47 KiB
Python
|
# -*- coding: utf-8 -*-
|
||
|
|
# Part of Odoo. See LICENSE file for full copyright and licensing details.
|
||
|
|
|
||
|
|
from contextlib import closing
|
||
|
|
from datetime import datetime, timedelta
|
||
|
|
from unittest.mock import patch
|
||
|
|
|
||
|
|
from odoo import fields
|
||
|
|
from odoo.addons.mail.tests.common import mail_new_test_user
|
||
|
|
from odoo.exceptions import ValidationError
|
||
|
|
from odoo.tests.common import Form, TransactionCase
|
||
|
|
from odoo.exceptions import AccessError, RedirectWarning, UserError
|
||
|
|
|
||
|
|
|
||
|
|
class StockQuant(TransactionCase):
|
||
|
|
@classmethod
|
||
|
|
def setUpClass(cls):
|
||
|
|
super(StockQuant, cls).setUpClass()
|
||
|
|
cls.demo_user = mail_new_test_user(
|
||
|
|
cls.env,
|
||
|
|
name='Pauline Poivraisselle',
|
||
|
|
login='pauline',
|
||
|
|
email='p.p@example.com',
|
||
|
|
notification_type='inbox',
|
||
|
|
groups='base.group_user',
|
||
|
|
)
|
||
|
|
cls.stock_user = mail_new_test_user(
|
||
|
|
cls.env,
|
||
|
|
name='Pauline Poivraisselle',
|
||
|
|
login='pauline2',
|
||
|
|
email='p.p@example.com',
|
||
|
|
notification_type='inbox',
|
||
|
|
groups='stock.group_stock_user',
|
||
|
|
)
|
||
|
|
|
||
|
|
cls.product = cls.env['product.product'].create({
|
||
|
|
'name': 'Product A',
|
||
|
|
'type': 'product',
|
||
|
|
})
|
||
|
|
cls.product_lot = cls.env['product.product'].create({
|
||
|
|
'name': 'Product A',
|
||
|
|
'type': 'product',
|
||
|
|
'tracking': 'lot',
|
||
|
|
})
|
||
|
|
cls.product_consu = cls.env['product.product'].create({
|
||
|
|
'name': 'Product A',
|
||
|
|
'type': 'consu',
|
||
|
|
})
|
||
|
|
cls.product_serial = cls.env['product.product'].create({
|
||
|
|
'name': 'Product A',
|
||
|
|
'type': 'product',
|
||
|
|
'tracking': 'serial',
|
||
|
|
})
|
||
|
|
cls.stock_location = cls.env['stock.location'].create({
|
||
|
|
'name': 'stock_location',
|
||
|
|
'usage': 'internal',
|
||
|
|
})
|
||
|
|
cls.stock_subloc3 = cls.env['stock.location'].create({
|
||
|
|
'name': 'subloc3',
|
||
|
|
'usage': 'internal',
|
||
|
|
'location_id': cls.stock_location.id
|
||
|
|
})
|
||
|
|
cls.stock_subloc2 = cls.env['stock.location'].create({
|
||
|
|
'name': 'subloc2',
|
||
|
|
'usage': 'internal',
|
||
|
|
'location_id': cls.stock_location.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
def gather_relevant(self, product_id, location_id, lot_id=None, package_id=None, owner_id=None, strict=False):
|
||
|
|
quants = self.env['stock.quant']._gather(product_id, location_id, lot_id=lot_id, package_id=package_id, owner_id=owner_id, strict=strict)
|
||
|
|
return quants.filtered(lambda q: not (q.quantity == 0 and q.reserved_quantity == 0))
|
||
|
|
|
||
|
|
def test_get_available_quantity_1(self):
|
||
|
|
""" Quantity availability with only one quant in a location.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 1.0)
|
||
|
|
|
||
|
|
def test_get_available_quantity_2(self):
|
||
|
|
""" Quantity availability with multiple quants in a location.
|
||
|
|
"""
|
||
|
|
for i in range(3):
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 3.0)
|
||
|
|
|
||
|
|
def test_get_available_quantity_3(self):
|
||
|
|
""" Quantity availability with multiple quants (including negatives ones) in a location.
|
||
|
|
"""
|
||
|
|
for i in range(3):
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': -3.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
|
||
|
|
def test_get_available_quantity_4(self):
|
||
|
|
""" Quantity availability with no quants in a location.
|
||
|
|
"""
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
|
||
|
|
def test_get_available_quantity_5(self):
|
||
|
|
""" Quantity availability with multiple partially reserved quants in a location.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 10.0,
|
||
|
|
'reserved_quantity': 9.0,
|
||
|
|
})
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
'reserved_quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 1.0)
|
||
|
|
|
||
|
|
def test_get_available_quantity_6(self):
|
||
|
|
""" Quantity availability with multiple partially reserved quants in a location.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 10.0,
|
||
|
|
'reserved_quantity': 20.0,
|
||
|
|
})
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 5.0,
|
||
|
|
'reserved_quantity': 0.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location, allow_negative=True), -5.0)
|
||
|
|
|
||
|
|
def test_get_available_quantity_7(self):
|
||
|
|
""" Quantity availability with only one tracked quant in a location.
|
||
|
|
"""
|
||
|
|
lot1 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot1',
|
||
|
|
'product_id': self.product_lot.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product_lot.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 10.0,
|
||
|
|
'reserved_quantity': 20.0,
|
||
|
|
'lot_id': lot1.id,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_lot, self.stock_location, lot_id=lot1), 0.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_lot, self.stock_location, lot_id=lot1, allow_negative=True), -10.0)
|
||
|
|
|
||
|
|
def test_get_available_quantity_8(self):
|
||
|
|
""" Quantity availability with a consumable product.
|
||
|
|
"""
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_consu, self.stock_location), 0.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product_consu, self.stock_location)), 0)
|
||
|
|
with self.assertRaises(ValidationError):
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_consu, self.stock_location, 1.0)
|
||
|
|
|
||
|
|
def test_get_available_quantity_9(self):
|
||
|
|
""" Quantity availability by a demo user with access rights/rules.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.env = self.env(user=self.demo_user)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 1.0)
|
||
|
|
|
||
|
|
def test_increase_available_quantity_1(self):
|
||
|
|
""" Increase the available quantity when no quants are already in a location.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 1.0)
|
||
|
|
|
||
|
|
def test_increase_available_quantity_2(self):
|
||
|
|
""" Increase the available quantity when multiple quants are already in a location.
|
||
|
|
"""
|
||
|
|
for i in range(2):
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 2.0)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 3.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 2)
|
||
|
|
|
||
|
|
def test_increase_available_quantity_3(self):
|
||
|
|
""" Increase the available quantity when a concurrent transaction is already increasing
|
||
|
|
the reserved quanntity for the same product.
|
||
|
|
"""
|
||
|
|
quant = self.env['stock.quant'].search([('location_id', '=', self.stock_location.id)], limit=1)
|
||
|
|
if not quant:
|
||
|
|
self.skipTest('Cannot test concurrent transactions without demo data.')
|
||
|
|
product = quant.product_id
|
||
|
|
available_quantity = self.env['stock.quant']._get_available_quantity(product, self.stock_location, allow_negative=True)
|
||
|
|
# opens a new cursor and SELECT FOR UPDATE the quant, to simulate another concurrent reserved
|
||
|
|
# quantity increase
|
||
|
|
with closing(self.registry.cursor()) as cr:
|
||
|
|
cr.execute("SELECT id FROM stock_quant WHERE product_id=%s AND location_id=%s", (product.id, self.stock_location.id))
|
||
|
|
quant_id = cr.fetchone()
|
||
|
|
cr.execute("SELECT 1 FROM stock_quant WHERE id=%s FOR UPDATE", quant_id)
|
||
|
|
self.env['stock.quant']._update_available_quantity(product, self.stock_location, 1.0)
|
||
|
|
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(product, self.stock_location, allow_negative=True), available_quantity + 1)
|
||
|
|
self.assertEqual(len(self.gather_relevant(product, self.stock_location, strict=True)), 2)
|
||
|
|
|
||
|
|
def test_increase_available_quantity_4(self):
|
||
|
|
""" Increase the available quantity when no quants are already in a location with a user without access right.
|
||
|
|
"""
|
||
|
|
self.env = self.env(user=self.demo_user)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
|
||
|
|
def test_increase_available_quantity_5(self):
|
||
|
|
""" Increase the available quantity when no quants are already in stock.
|
||
|
|
Increase a subLocation and check that quants are in this location. Also test inverse.
|
||
|
|
"""
|
||
|
|
stock_sub_location = self.stock_location.child_ids[0]
|
||
|
|
product2 = self.env['product.product'].create({
|
||
|
|
'name': 'Product B',
|
||
|
|
'type': 'product',
|
||
|
|
})
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, stock_sub_location, 1.0)
|
||
|
|
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, stock_sub_location), 1.0)
|
||
|
|
|
||
|
|
self.env['stock.quant']._update_available_quantity(product2, stock_sub_location, 1.0)
|
||
|
|
self.env['stock.quant']._update_available_quantity(product2, self.stock_location, 1.0)
|
||
|
|
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(product2, self.stock_location), 2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(product2, stock_sub_location), 1.0)
|
||
|
|
|
||
|
|
def test_increase_available_quantity_6(self):
|
||
|
|
""" Increasing the available quantity in a view location should be forbidden.
|
||
|
|
"""
|
||
|
|
location1 = self.env['stock.location'].create({
|
||
|
|
'name': 'viewloc1',
|
||
|
|
'usage': 'view',
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
})
|
||
|
|
with self.assertRaises(ValidationError):
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, location1, 1.0)
|
||
|
|
|
||
|
|
def test_increase_available_quantity_7(self):
|
||
|
|
""" Setting a location's usage as "view" should be forbidden if it already
|
||
|
|
contains quant.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
self.assertTrue(len(self.stock_location.quant_ids.ids) > 0)
|
||
|
|
with self.assertRaises(UserError):
|
||
|
|
self.stock_location.usage = 'view'
|
||
|
|
|
||
|
|
def test_decrease_available_quantity_1(self):
|
||
|
|
""" Decrease the available quantity when no quants are already in a location.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, -1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location, allow_negative=True), -1.0)
|
||
|
|
|
||
|
|
def test_decrease_available_quantity_2(self):
|
||
|
|
""" Decrease the available quantity when multiple quants are already in a location.
|
||
|
|
"""
|
||
|
|
for i in range(2):
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 2.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 2)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, -1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 1.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 1)
|
||
|
|
|
||
|
|
def test_decrease_available_quantity_3(self):
|
||
|
|
""" Decrease the available quantity when a concurrent transaction is already increasing
|
||
|
|
the reserved quanntity for the same product.
|
||
|
|
"""
|
||
|
|
quant = self.env['stock.quant'].search([('location_id', '=', self.stock_location.id)], limit=1)
|
||
|
|
if not quant:
|
||
|
|
self.skipTest('Cannot test concurrent transactions without demo data.')
|
||
|
|
product = quant.product_id
|
||
|
|
available_quantity = self.env['stock.quant']._get_available_quantity(product, self.stock_location, allow_negative=True)
|
||
|
|
|
||
|
|
# opens a new cursor and SELECT FOR UPDATE the quant, to simulate another concurrent reserved
|
||
|
|
# quantity increase
|
||
|
|
with closing(self.registry.cursor()) as cr:
|
||
|
|
cr.execute("SELECT 1 FROM stock_quant WHERE id = %s FOR UPDATE", quant.ids)
|
||
|
|
self.env['stock.quant']._update_available_quantity(product, self.stock_location, -1.0)
|
||
|
|
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(product, self.stock_location, allow_negative=True), available_quantity - 1)
|
||
|
|
self.assertEqual(len(self.gather_relevant(product, self.stock_location, strict=True)), 2)
|
||
|
|
|
||
|
|
def test_decrease_available_quantity_4(self):
|
||
|
|
""" Decrease the available quantity that delete the quant. The active user should have
|
||
|
|
read,write and unlink rights
|
||
|
|
"""
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.env = self.env(user=self.demo_user)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, -1.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 0)
|
||
|
|
|
||
|
|
def test_increase_reserved_quantity_1(self):
|
||
|
|
""" Increase the reserved quantity of quantity x when there's a single quant in a given
|
||
|
|
location which has an available quantity of x.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 10.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 10.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 1)
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, 10.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 1)
|
||
|
|
|
||
|
|
def test_increase_reserved_quantity_2(self):
|
||
|
|
""" Increase the reserved quantity of quantity x when there's two quants in a given
|
||
|
|
location which have an available quantity of x together.
|
||
|
|
"""
|
||
|
|
for i in range(2):
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 5.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 10.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 2)
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, 10.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 2)
|
||
|
|
|
||
|
|
def test_increase_reserved_quantity_3(self):
|
||
|
|
""" Increase the reserved quantity of quantity x when there's multiple quants in a given
|
||
|
|
location which have an available quantity of x together.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 5.0,
|
||
|
|
'reserved_quantity': 2.0,
|
||
|
|
})
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 10.0,
|
||
|
|
'reserved_quantity': 12.0,
|
||
|
|
})
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 8.0,
|
||
|
|
'reserved_quantity': 3.0,
|
||
|
|
})
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 35.0,
|
||
|
|
'reserved_quantity': 12.0,
|
||
|
|
})
|
||
|
|
# total quantity: 58
|
||
|
|
# total reserved quantity: 29
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 29.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 4)
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, 10.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 19.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 4)
|
||
|
|
|
||
|
|
def test_increase_reserved_quantity_4(self):
|
||
|
|
""" Increase the reserved quantity of quantity x when there's multiple quants in a given
|
||
|
|
location which have an available quantity of x together.
|
||
|
|
"""
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 5.0,
|
||
|
|
'reserved_quantity': 7.0,
|
||
|
|
})
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 12.0,
|
||
|
|
'reserved_quantity': 10.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 2)
|
||
|
|
with self.assertRaises(UserError):
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, 10.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
|
||
|
|
def test_increase_reserved_quantity_5(self):
|
||
|
|
""" Decrease the available quantity when no quant are in a location.
|
||
|
|
"""
|
||
|
|
with self.assertRaises(UserError):
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
|
||
|
|
def test_decrease_reserved_quantity_1(self):
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 10.0,
|
||
|
|
'reserved_quantity': 10.0,
|
||
|
|
})
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 1)
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, -10.0, strict=True)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 10.0)
|
||
|
|
self.assertEqual(len(self.gather_relevant(self.product, self.stock_location)), 1)
|
||
|
|
|
||
|
|
def test_increase_decrease_reserved_quantity_1(self):
|
||
|
|
""" Decrease then increase reserved quantity when no quant are in a location.
|
||
|
|
"""
|
||
|
|
with self.assertRaises(UserError):
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
with self.assertRaises(RedirectWarning):
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, -1.0, strict=True)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
|
||
|
|
def test_action_done_1(self):
|
||
|
|
pack_location = self.env.ref('stock.location_pack_zone')
|
||
|
|
pack_location.active = True
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 2.0)
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, 2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, -2.0, strict=True)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 2.0)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, -2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, self.stock_location), 0.0)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, pack_location, 2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product, pack_location), 2.0)
|
||
|
|
|
||
|
|
def test_mix_tracked_untracked_1(self):
|
||
|
|
lot1 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot1',
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
# add one tracked, one untracked
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_location, 1.0)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_location, 1.0, lot_id=lot1)
|
||
|
|
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location), 2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location, strict=True), 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location, lot_id=lot1), 2.0)
|
||
|
|
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product_serial, self.stock_location, 1.0, lot_id=lot1, strict=True)
|
||
|
|
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location), 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location, strict=True), 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location, lot_id=lot1), 1.0)
|
||
|
|
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product_serial, self.stock_location, -1.0, lot_id=lot1, strict=True)
|
||
|
|
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location), 2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location, strict=True), 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location, lot_id=lot1), 2.0)
|
||
|
|
|
||
|
|
with self.assertRaises(RedirectWarning):
|
||
|
|
self.env['stock.quant']._update_reserved_quantity(self.product_serial, self.stock_location, -1.0, strict=True)
|
||
|
|
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location), 2.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location, strict=True), 1.0)
|
||
|
|
self.assertEqual(self.env['stock.quant']._get_available_quantity(self.product_serial, self.stock_location, lot_id=lot1), 2.0)
|
||
|
|
|
||
|
|
def test_access_rights_1(self):
|
||
|
|
""" Directly update the quant with a user with or without stock access rights should not raise
|
||
|
|
an AccessError only deletion will.
|
||
|
|
"""
|
||
|
|
quant = self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
self.env = self.env(user=self.demo_user)
|
||
|
|
with self.assertRaises(AccessError):
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
with self.assertRaises(AccessError):
|
||
|
|
quant.with_user(self.demo_user).write({'quantity': 2.0})
|
||
|
|
with self.assertRaises(UserError):
|
||
|
|
quant.with_user(self.demo_user).unlink()
|
||
|
|
|
||
|
|
self.env = self.env(user=self.stock_user)
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
quant.with_user(self.stock_user).with_context(inventory_mode=True).write({'inventory_quantity': 3.0})
|
||
|
|
with self.assertRaises(AccessError):
|
||
|
|
quant.with_user(self.stock_user).unlink()
|
||
|
|
|
||
|
|
def test_in_date_1(self):
|
||
|
|
""" Check that no incoming date is set when updating the quantity of an untracked quant.
|
||
|
|
"""
|
||
|
|
quantity, in_date = self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
self.assertEqual(quantity, 1)
|
||
|
|
self.assertNotEqual(in_date, None)
|
||
|
|
|
||
|
|
def test_in_date_1b(self):
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'quantity': 1.0,
|
||
|
|
})
|
||
|
|
quantity, in_date = self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 2.0)
|
||
|
|
self.assertEqual(quantity, 3)
|
||
|
|
self.assertNotEqual(in_date, None)
|
||
|
|
|
||
|
|
def test_in_date_2(self):
|
||
|
|
""" Check that an incoming date is correctly set when updating the quantity of a tracked
|
||
|
|
quant.
|
||
|
|
"""
|
||
|
|
lot1 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot1',
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
quantity, in_date = self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_location, 1.0, lot_id=lot1)
|
||
|
|
self.assertEqual(quantity, 1)
|
||
|
|
self.assertNotEqual(in_date, None)
|
||
|
|
|
||
|
|
def test_in_date_3(self):
|
||
|
|
""" Check that the FIFO strategies correctly applies when you have multiple lot received
|
||
|
|
at different times for a tracked product.
|
||
|
|
"""
|
||
|
|
lot1 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot1',
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
lot2 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot2',
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
in_date_lot1 = datetime.now()
|
||
|
|
in_date_lot2 = datetime.now() - timedelta(days=5)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_location, 1.0, lot_id=lot1, in_date=in_date_lot1)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_location, 1.0, lot_id=lot2, in_date=in_date_lot2)
|
||
|
|
|
||
|
|
quants = self.env['stock.quant']._update_reserved_quantity(self.product_serial, self.stock_location, 1)
|
||
|
|
|
||
|
|
# Default removal strategy is FIFO, so lot2 should be received as it was received earlier.
|
||
|
|
self.assertEqual(quants[0][0].lot_id.id, lot2.id)
|
||
|
|
|
||
|
|
def test_in_date_4(self):
|
||
|
|
""" Check that the LIFO strategies correctly applies when you have multiple lot received
|
||
|
|
at different times for a tracked product.
|
||
|
|
"""
|
||
|
|
lifo_strategy = self.env['product.removal'].search([('method', '=', 'lifo')])
|
||
|
|
self.stock_location.removal_strategy_id = lifo_strategy
|
||
|
|
lot1 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot1',
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
lot2 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot2',
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
in_date_lot1 = datetime.now()
|
||
|
|
in_date_lot2 = datetime.now() - timedelta(days=5)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_location, 1.0, lot_id=lot1, in_date=in_date_lot1)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_location, 1.0, lot_id=lot2, in_date=in_date_lot2)
|
||
|
|
|
||
|
|
quants = self.env['stock.quant']._update_reserved_quantity(self.product_serial, self.stock_location, 1)
|
||
|
|
|
||
|
|
# Removal strategy is LIFO, so lot1 should be received as it was received later.
|
||
|
|
self.assertEqual(quants[0][0].lot_id.id, lot1.id)
|
||
|
|
|
||
|
|
def test_in_date_5(self):
|
||
|
|
""" Receive the same lot at different times, once they're in the same location, the quants
|
||
|
|
are merged and only the earliest incoming date is kept.
|
||
|
|
"""
|
||
|
|
lot1 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot1',
|
||
|
|
'product_id': self.product_lot.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
|
||
|
|
from odoo.fields import Datetime
|
||
|
|
in_date1 = Datetime.now()
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_lot, self.stock_location, 1.0, lot_id=lot1, in_date=in_date1)
|
||
|
|
|
||
|
|
quant = self.env['stock.quant'].search([
|
||
|
|
('product_id', '=', self.product_lot.id),
|
||
|
|
('location_id', '=', self.stock_location.id),
|
||
|
|
])
|
||
|
|
self.assertEqual(len(quant), 1)
|
||
|
|
self.assertEqual(quant.quantity, 1)
|
||
|
|
self.assertEqual(quant.lot_id.id, lot1.id)
|
||
|
|
self.assertEqual(quant.in_date, in_date1)
|
||
|
|
|
||
|
|
in_date2 = Datetime.now() - timedelta(days=5)
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_lot, self.stock_location, 1.0, lot_id=lot1, in_date=in_date2)
|
||
|
|
|
||
|
|
quant = self.env['stock.quant'].search([
|
||
|
|
('product_id', '=', self.product_lot.id),
|
||
|
|
('location_id', '=', self.stock_location.id),
|
||
|
|
])
|
||
|
|
self.assertEqual(len(quant), 1)
|
||
|
|
self.assertEqual(quant.quantity, 2)
|
||
|
|
self.assertEqual(quant.lot_id.id, lot1.id)
|
||
|
|
self.assertEqual(quant.in_date, in_date2)
|
||
|
|
|
||
|
|
def test_closest_removal_strategy_tracked(self):
|
||
|
|
""" Check that the Closest location strategy correctly applies when you have multiple lot received
|
||
|
|
at different locations for a tracked product.
|
||
|
|
"""
|
||
|
|
closest_strategy = self.env['product.removal'].search([('method', '=', 'closest')])
|
||
|
|
self.stock_location.removal_strategy_id = closest_strategy
|
||
|
|
lot1 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot1',
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
lot2 = self.env['stock.lot'].create({
|
||
|
|
'name': 'lot2',
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'company_id': self.env.company.id,
|
||
|
|
})
|
||
|
|
in_date = datetime.now()
|
||
|
|
# Add a product from lot1 in stock_location/subloc2
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_subloc2, 1.0, lot_id=lot1, in_date=in_date)
|
||
|
|
# Add a product from lot2 in stock_location/subloc3
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product_serial, self.stock_subloc3, 1.0, lot_id=lot2, in_date=in_date)
|
||
|
|
# Require one unit of the product
|
||
|
|
quants = self.env['stock.quant']._update_reserved_quantity(self.product_serial, self.stock_location, 1)
|
||
|
|
|
||
|
|
# Default removal strategy is 'Closest location', so lot1 should be received as it was put in a closer location. (stock_location/subloc2 < stock_location/subloc3)
|
||
|
|
self.assertEqual(quants[0][0].lot_id.id, lot1.id)
|
||
|
|
|
||
|
|
def test_closest_removal_strategy_untracked(self):
|
||
|
|
""" Check that the Closest location strategy correctly applies when you have multiple products received
|
||
|
|
at different locations for untracked products."""
|
||
|
|
closest_strategy = self.env['product.removal'].search([('method', '=', 'closest')])
|
||
|
|
self.stock_location.removal_strategy_id = closest_strategy
|
||
|
|
# Add 2 units of product into stock_location/subloc2
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_subloc2.id,
|
||
|
|
'quantity': 2.0,
|
||
|
|
})
|
||
|
|
# Add 3 units of product into stock_location/subloc3
|
||
|
|
self.env['stock.quant'].create({
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': self.stock_subloc3.id,
|
||
|
|
'quantity': 3.0
|
||
|
|
})
|
||
|
|
# Request 3 units of product, with 'Closest location' as removal strategy
|
||
|
|
quants = self.env['stock.quant']._update_reserved_quantity(self.product, self.stock_location, 3)
|
||
|
|
|
||
|
|
# The 2 in stock_location/subloc2 should be taken first, as the location name is smaller alphabetically
|
||
|
|
self.assertEqual(quants[0][0].reserved_quantity, 2)
|
||
|
|
# The last one should then be taken in stock_location/subloc3 since the first location doesn't have enough products
|
||
|
|
self.assertEqual(quants[1][0].reserved_quantity, 1)
|
||
|
|
|
||
|
|
def test_in_date_6(self):
|
||
|
|
"""
|
||
|
|
One P in stock, P is delivered. Later on, a stock adjustement adds one P. This test checks
|
||
|
|
the date value of the related quant
|
||
|
|
"""
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0)
|
||
|
|
|
||
|
|
move = self.env['stock.move'].create({
|
||
|
|
'name': 'OUT 1 product',
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'product_uom_qty': 1,
|
||
|
|
'product_uom': self.product.uom_id.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'location_dest_id': self.ref('stock.stock_location_customers'),
|
||
|
|
})
|
||
|
|
move._action_confirm()
|
||
|
|
move._action_assign()
|
||
|
|
move.quantity_done = 1
|
||
|
|
move._action_done()
|
||
|
|
|
||
|
|
|
||
|
|
tomorrow = fields.Datetime.now() + timedelta(days=1)
|
||
|
|
with patch.object(fields.Datetime, 'now', lambda: tomorrow):
|
||
|
|
move = self.env['stock.move'].create({
|
||
|
|
'name': 'IN 1 product',
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'product_uom_qty': 1,
|
||
|
|
'product_uom': self.product.uom_id.id,
|
||
|
|
'location_id': self.ref('stock.stock_location_suppliers'),
|
||
|
|
'location_dest_id': self.stock_location.id,
|
||
|
|
})
|
||
|
|
move._action_confirm()
|
||
|
|
move._action_assign()
|
||
|
|
move.quantity_done = 1
|
||
|
|
move._action_done()
|
||
|
|
|
||
|
|
quant = self.env['stock.quant'].search([('product_id', '=', self.product.id), ('location_id', '=', self.stock_location.id), ('quantity', '>', 0)])
|
||
|
|
self.assertEqual(quant.in_date, tomorrow)
|
||
|
|
|
||
|
|
def test_quant_creation(self):
|
||
|
|
"""
|
||
|
|
This test ensures that, after an internal transfer, the values of the created quand are correct
|
||
|
|
"""
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 10.0)
|
||
|
|
|
||
|
|
move = self.env['stock.move'].create({
|
||
|
|
'name': 'Move 1 product',
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'product_uom_qty': 1,
|
||
|
|
'product_uom': self.product.uom_id.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'location_dest_id': self.stock_subloc2.id,
|
||
|
|
})
|
||
|
|
move._action_confirm()
|
||
|
|
move._action_assign()
|
||
|
|
move.quantity_done = 1
|
||
|
|
move._action_done()
|
||
|
|
|
||
|
|
quant = self.gather_relevant(self.product, self.stock_subloc2)
|
||
|
|
self.assertFalse(quant.inventory_quantity_set)
|
||
|
|
|
||
|
|
def test_unpack_and_quants_merging(self):
|
||
|
|
"""
|
||
|
|
When unpacking a package, if there are already some quantities of the
|
||
|
|
packed product in the stock, the quant of the on hand quantity and the
|
||
|
|
one of the package should be merged
|
||
|
|
"""
|
||
|
|
stock_location = self.env['stock.warehouse'].search([], limit=1).lot_stock_id
|
||
|
|
supplier_location = self.env.ref('stock.stock_location_suppliers')
|
||
|
|
picking_type_in = self.env.ref('stock.picking_type_in')
|
||
|
|
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, stock_location, 1.0)
|
||
|
|
|
||
|
|
picking = self.env['stock.picking'].create({
|
||
|
|
'picking_type_id': picking_type_in.id,
|
||
|
|
'location_id': supplier_location.id,
|
||
|
|
'location_dest_id': stock_location.id,
|
||
|
|
'move_ids': [(0, 0, {
|
||
|
|
'name': 'In 10 x %s' % self.product.name,
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'location_id': supplier_location.id,
|
||
|
|
'location_dest_id': stock_location.id,
|
||
|
|
'product_uom_qty': 10,
|
||
|
|
'product_uom': self.product.uom_id.id,
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
picking.action_confirm()
|
||
|
|
|
||
|
|
package = self.env['stock.quant.package'].create({
|
||
|
|
'name': 'Super Package',
|
||
|
|
})
|
||
|
|
picking.move_ids.move_line_ids.write({
|
||
|
|
'qty_done': 10,
|
||
|
|
'result_package_id': package.id,
|
||
|
|
})
|
||
|
|
picking.button_validate()
|
||
|
|
|
||
|
|
package.unpack()
|
||
|
|
|
||
|
|
quant = self.env['stock.quant'].search([('product_id', '=', self.product.id), ('on_hand', '=', True)])
|
||
|
|
self.assertEqual(len(quant), 1)
|
||
|
|
# The quants merging is processed thanks to a SQL query (see StockQuant._merge_quants).
|
||
|
|
# At that point, the ORM is not aware of the new value. So we need to invalidate the
|
||
|
|
# cache to ensure that the value will be the newest
|
||
|
|
quant.invalidate_recordset(['quantity'])
|
||
|
|
self.assertEqual(quant.quantity, 11)
|
||
|
|
|
||
|
|
def test_clean_quant_after_package_move(self):
|
||
|
|
"""
|
||
|
|
A product is at WH/Stock in a package PK. We deliver PK. The user should
|
||
|
|
not find any quant at WH/Stock with PK anymore.
|
||
|
|
"""
|
||
|
|
package = self.env['stock.quant.package'].create({})
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0, package_id=package)
|
||
|
|
|
||
|
|
move = self.env['stock.move'].create({
|
||
|
|
'name': 'OUT 1 product',
|
||
|
|
'product_id': self.product.id,
|
||
|
|
'product_uom_qty': 1,
|
||
|
|
'product_uom': self.product.uom_id.id,
|
||
|
|
'location_id': self.stock_location.id,
|
||
|
|
'location_dest_id': self.ref('stock.stock_location_customers'),
|
||
|
|
})
|
||
|
|
move._action_confirm()
|
||
|
|
move._action_assign()
|
||
|
|
move.move_line_ids.write({
|
||
|
|
'result_package_id': package.id,
|
||
|
|
'qty_done': 1,
|
||
|
|
})
|
||
|
|
move._action_done()
|
||
|
|
|
||
|
|
self.assertFalse(self.env['stock.quant'].search_count([
|
||
|
|
('product_id', '=', self.product.id),
|
||
|
|
('package_id', '=', package.id),
|
||
|
|
('location_id', '=', self.stock_location.id),
|
||
|
|
]))
|
||
|
|
|
||
|
|
def test_serial_constraint_with_package_and_return(self):
|
||
|
|
"""
|
||
|
|
Receive product with serial S
|
||
|
|
Return it in a package
|
||
|
|
Confirm a new receipt with S
|
||
|
|
"""
|
||
|
|
stock_location = self.env['stock.warehouse'].search([], limit=1).lot_stock_id
|
||
|
|
supplier_location = self.env.ref('stock.stock_location_suppliers')
|
||
|
|
picking_type_in = self.env.ref('stock.picking_type_in')
|
||
|
|
|
||
|
|
receipt01 = self.env['stock.picking'].create({
|
||
|
|
'picking_type_id': picking_type_in.id,
|
||
|
|
'location_id': supplier_location.id,
|
||
|
|
'location_dest_id': stock_location.id,
|
||
|
|
'move_ids': [(0, 0, {
|
||
|
|
'name': self.product_serial.name,
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'location_id': supplier_location.id,
|
||
|
|
'location_dest_id': stock_location.id,
|
||
|
|
'product_uom_qty': 1,
|
||
|
|
'product_uom': self.product_serial.uom_id.id,
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
receipt01.action_confirm()
|
||
|
|
receipt01.move_line_ids.write({
|
||
|
|
'lot_name': 'Michel',
|
||
|
|
'qty_done': 1.0
|
||
|
|
})
|
||
|
|
receipt01.button_validate()
|
||
|
|
|
||
|
|
quant = self.env['stock.quant'].search([('product_id', '=', self.product_serial.id), ('location_id', '=', stock_location.id)])
|
||
|
|
|
||
|
|
wizard_form = Form(self.env['stock.return.picking'].with_context(active_ids=receipt01.ids, active_id=receipt01.ids[0], active_model='stock.picking'))
|
||
|
|
wizard = wizard_form.save()
|
||
|
|
wizard.product_return_moves.quantity = 1.0
|
||
|
|
stock_return_picking_action = wizard.create_returns()
|
||
|
|
|
||
|
|
return_pick = self.env['stock.picking'].browse(stock_return_picking_action['res_id'])
|
||
|
|
return_pick.move_ids.move_line_ids.qty_done = 1.0
|
||
|
|
return_pick.action_put_in_pack()
|
||
|
|
return_pick._action_done()
|
||
|
|
|
||
|
|
self.assertEqual(return_pick.move_line_ids.lot_id, quant.lot_id)
|
||
|
|
self.assertTrue(return_pick.move_line_ids.result_package_id, quant.lot_id)
|
||
|
|
|
||
|
|
receipt02 = self.env['stock.picking'].create({
|
||
|
|
'picking_type_id': picking_type_in.id,
|
||
|
|
'location_id': supplier_location.id,
|
||
|
|
'location_dest_id': stock_location.id,
|
||
|
|
'move_ids': [(0, 0, {
|
||
|
|
'name': self.product_serial.name,
|
||
|
|
'product_id': self.product_serial.id,
|
||
|
|
'location_id': supplier_location.id,
|
||
|
|
'location_dest_id': stock_location.id,
|
||
|
|
'product_uom_qty': 1,
|
||
|
|
'product_uom': self.product_serial.uom_id.id,
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
receipt02.action_confirm()
|
||
|
|
receipt02.move_line_ids.write({
|
||
|
|
'lot_name': 'Michel',
|
||
|
|
'qty_done': 1.0
|
||
|
|
})
|
||
|
|
receipt02.button_validate()
|
||
|
|
|
||
|
|
quant = self.env['stock.quant'].search([('product_id', '=', self.product_serial.id), ('location_id', '=', stock_location.id)])
|
||
|
|
self.assertEqual(len(quant), 1)
|
||
|
|
self.assertEqual(quant.lot_id.name, 'Michel')
|
||
|
|
|
||
|
|
def test_update_quant_with_forbidden_field(self):
|
||
|
|
"""
|
||
|
|
Test that updating a quant with a forbidden field raise an error.
|
||
|
|
"""
|
||
|
|
product = self.env['product.product'].create({
|
||
|
|
'name': 'Product',
|
||
|
|
'type': 'product',
|
||
|
|
'tracking': 'serial',
|
||
|
|
})
|
||
|
|
sn1 = self.env['stock.lot'].create({
|
||
|
|
'name': 'SN1',
|
||
|
|
'product_id': product.id,
|
||
|
|
})
|
||
|
|
self.env['stock.quant']._update_available_quantity(product, self.stock_subloc2, 1.0, lot_id=sn1)
|
||
|
|
self.assertEqual(len(product.stock_quant_ids), 1)
|
||
|
|
self.env['stock.quant']._update_available_quantity(product, self.stock_subloc3, 1.0, lot_id=sn1)
|
||
|
|
self.assertEqual(len(product.stock_quant_ids), 2)
|
||
|
|
quant_2 = product.stock_quant_ids[1]
|
||
|
|
self.assertEqual(quant_2.with_context(inventory_mode=True).sn_duplicated, True)
|
||
|
|
with self.assertRaises(UserError):
|
||
|
|
quant_2.with_context(inventory_mode=True).write({'location_id': self.stock_subloc2})
|
||
|
|
|
||
|
|
def test_update_quant_with_forbidden_field_02(self):
|
||
|
|
"""
|
||
|
|
Test that updating the package from the quant raise an error
|
||
|
|
but if the package is unpacked, the quant can be updated.
|
||
|
|
"""
|
||
|
|
package = self.env['stock.quant.package'].create({
|
||
|
|
'name': 'Package',
|
||
|
|
})
|
||
|
|
self.env['stock.quant']._update_available_quantity(self.product, self.stock_location, 1.0, package_id=package)
|
||
|
|
quant = self.product.stock_quant_ids
|
||
|
|
self.assertEqual(len(self.product.stock_quant_ids), 1)
|
||
|
|
with self.assertRaises(UserError):
|
||
|
|
quant.with_context(inventory_mode=True).write({'package_id': False})
|
||
|
|
package.with_context(inventory_mode=True).unpack()
|
||
|
|
self.assertFalse(quant.package_id)
|
||
|
|
self.assertTrue(True)
|
||
|
|
|
||
|
|
def test_unpack_and_quants_history(self):
|
||
|
|
"""
|
||
|
|
Test that after unpacking the quant history is preserved
|
||
|
|
"""
|
||
|
|
product = self.env['product.product'].create({
|
||
|
|
'name': 'Product',
|
||
|
|
'type': 'product',
|
||
|
|
'tracking': 'lot',
|
||
|
|
})
|
||
|
|
lot_a = self.env['stock.lot'].create({
|
||
|
|
'name': 'A',
|
||
|
|
'product_id': product.id,
|
||
|
|
'product_qty': 5,
|
||
|
|
})
|
||
|
|
package = self.env['stock.quant.package'].create({
|
||
|
|
'name': 'Super Package',
|
||
|
|
})
|
||
|
|
stock_location = self.stock_location
|
||
|
|
dst_location = self.stock_subloc2
|
||
|
|
picking_type = self.env.ref('stock.picking_type_internal')
|
||
|
|
|
||
|
|
self.env['stock.quant']._update_available_quantity(product, stock_location, 5.0, lot_id=lot_a, package_id=package)
|
||
|
|
|
||
|
|
picking = self.env['stock.picking'].create({
|
||
|
|
'picking_type_id': picking_type.id,
|
||
|
|
'location_id': dst_location.id,
|
||
|
|
'location_dest_id': stock_location.id,
|
||
|
|
'move_ids': [(0, 0, {
|
||
|
|
'name': 'In 5 x %s' % product.name,
|
||
|
|
'product_id': product.id,
|
||
|
|
'location_id': stock_location.id,
|
||
|
|
'location_dest_id': dst_location.id,
|
||
|
|
'product_uom_qty': 5,
|
||
|
|
'product_uom': product.uom_id.id,
|
||
|
|
})],
|
||
|
|
})
|
||
|
|
picking.action_confirm()
|
||
|
|
|
||
|
|
picking.move_ids.move_line_ids.write({
|
||
|
|
'qty_done': 5,
|
||
|
|
'lot_id': lot_a.id,
|
||
|
|
'package_id': package.id,
|
||
|
|
'result_package_id': package.id,
|
||
|
|
})
|
||
|
|
picking.button_validate()
|
||
|
|
package.unpack()
|
||
|
|
|
||
|
|
quant = self.env['stock.quant'].search([
|
||
|
|
('product_id', '=', product.id),
|
||
|
|
('location_id', '=', dst_location.id),
|
||
|
|
])
|
||
|
|
action = quant.action_view_stock_moves()
|
||
|
|
history = self.env['stock.move.line'].search(action['domain'])
|
||
|
|
self.assertTrue(history)
|