audrius/stock-logistics-warehouse
1
0
Fork 1
mirror of https://github.com/OCA/stock-logistics-warehouse.git synced 2025-01-21 14:27:28 +02:00
Code Issues Packages Projects Releases Wiki Activity

[IMP] stock_account_change_qty_reason: black, isort

Browse Source
This commit is contained in:
Adrià Gil Sorribes
2019-12-23 09:49:48 +01:00
parent 33405d4688
commit 0217015810
4 changed files with 130 additions and 112 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
stock_account_change_qty_reason/__manifest__.py
View File

@@ -1,21 +1,15 @@
# Copyright 2019 Eficent Business and IT Consulting Services, S.L.
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html).
{
'name': "Stock Account Change Quantity Reason",
'summary': """
"name": "Stock Account Change Quantity Reason",
"summary": """
Stock Account Change Quantity Reason """,
'author': 'Eficent, Odoo Community Association (OCA)',
'website': "https://github.com/OCA/stock-logistics-warehouse",
'category': 'Warehouse Management',
'version': '12.0.1.0.0',
'license': 'AGPL-3',
'depends': [
'stock_account',
'stock_change_qty_reason'
],
'data': [
'views/stock_inventory_line_reason_view.xml',
'views/stock_move_view.xml',
],
'installable': True,
"author": "Eficent, Odoo Community Association (OCA)",
"website": "https://github.com/OCA/stock-logistics-warehouse",
"category": "Warehouse Management",
"version": "12.0.1.0.0",
"license": "AGPL-3",
"depends": ["stock_account", "stock_change_qty_reason"],
"data": ["views/stock_inventory_line_reason_view.xml", "views/stock_move_view.xml"],
"installable": True,
}

14
stock_account_change_qty_reason/models/stock_inventory_line_reason.py
View File

@@ -4,13 +4,17 @@ from odoo import fields, models
class StockAccountInventoryChangeReason(models.Model):
_inherit = 'stock.inventory.line.reason'
_inherit = "stock.inventory.line.reason"
account_reason_input_id = fields.Many2one(
'account.account', string="Input Account for Stock Valuation",
"account.account",
string="Input Account for Stock Valuation",
help="When set, it will be used as offsetting account when "
"products are received into the company.")
"products are received into the company.",
)
account_reason_output_id = fields.Many2one(
'account.account', string="Output Account for Stock Valuation",
"account.account",
string="Output Account for Stock Valuation",
help="When set, it will be used as offsetting account when "
"products are issued from the company.")
"products are issued from the company.",
)

5
stock_account_change_qty_reason/models/stock_move.py
View File

@@ -9,8 +9,9 @@ class StockMove(models.Model):
@api.multi
def _get_accounting_data_for_valuation(self):
self.ensure_one()
journal_id, acc_src, acc_dest, acc_valuation = \
super(StockMove, self)._get_accounting_data_for_valuation()
journal_id, acc_src, acc_dest, acc_valuation = super(
StockMove, self
)._get_accounting_data_for_valuation()
if self.preset_reason_id:
if self.preset_reason_id.account_reason_input_id:
acc_src = self.preset_reason_id.account_reason_input_id.id

197
stock_account_change_qty_reason/tests/test_stock_account_change_qty_reason.py
View File

@@ -5,135 +5,154 @@ from odoo.tests.common import SavepointCase
class TestStockAccountChangeQtyReason(SavepointCase):
@classmethod
def setUpClass(cls):
super(TestStockAccountChangeQtyReason, cls).setUpClass()
# MODELS
cls.product_product_model = cls.env['product.product']
cls.product_category_model = cls.env['product.category']
cls.wizard_model = cls.env['stock.change.product.qty']
cls.preset_reason = cls.env['stock.inventory.line.reason']
cls.product_product_model = cls.env["product.product"]
cls.product_category_model = cls.env["product.category"]
cls.wizard_model = cls.env["stock.change.product.qty"]
cls.preset_reason = cls.env["stock.inventory.line.reason"]
# INSTANCES
cls.stock_valuation_account = cls.env['account.account'].create({
'name': 'Stock Valuation',
'code': 'Stock Valuation',
'user_type_id': cls.env.ref(
'account.data_account_type_current_assets').id
})
cls.category = cls.product_category_model.create({
'name': 'Physical (test)',
'property_cost_method': 'standard',
'property_valuation': 'real_time',
'property_stock_valuation_account_id':
cls.stock_valuation_account
})
cls.stock_valuation_account = cls.env["account.account"].create(
{
"name": "Stock Valuation",
"code": "Stock Valuation",
"user_type_id": cls.env.ref(
"account.data_account_type_current_assets"
).id,
}
)
cls.category = cls.product_category_model.create(
{
"name": "Physical (test)",
"property_cost_method": "standard",
"property_valuation": "real_time",
"property_stock_valuation_account_id": cls.stock_valuation_account,
}
)
company = cls.env.ref('base.main_company')
company = cls.env.ref("base.main_company")
# Instance: account type (receivable)
cls.type_recv = cls.env.ref('account.data_account_type_receivable')
cls.type_recv = cls.env.ref("account.data_account_type_receivable")
# Instance: account type (payable)
cls.type_payable = cls.env.ref('account.data_account_type_payable')
cls.type_payable = cls.env.ref("account.data_account_type_payable")
# account (receivable)
cls.account_input = cls.env['account.account'].create({
'name': 'test_account_reason_input',
'code': '1234',
'user_type_id': cls.type_recv.id,
'company_id': company.id,
'reconcile': True
})
cls.account_input = cls.env["account.account"].create(
{
"name": "test_account_reason_input",
"code": "1234",
"user_type_id": cls.type_recv.id,
"company_id": company.id,
"reconcile": True,
}
)
# account (payable)
cls.account_output = cls.env['account.account'].create({
'name': 'test_account_reason_output',
'code': '4321',
'user_type_id': cls.type_payable.id,
'company_id': company.id,
'reconcile': True
})
cls.account_output = cls.env["account.account"].create(
{
"name": "test_account_reason_output",
"code": "4321",
"user_type_id": cls.type_payable.id,
"company_id": company.id,
"reconcile": True,
}
)
cls.reason = cls.preset_reason.create({
'name': 'Test Reason',
'description': 'Test Reason Description',
'account_reason_input_id': cls.account_input.id,
'account_reason_output_id': cls.account_output.id,
})
cls.reason = cls.preset_reason.create(
{
"name": "Test Reason",
"description": "Test Reason Description",
"account_reason_input_id": cls.account_input.id,
"account_reason_output_id": cls.account_output.id,
}
)
# Start Inventory with 10 units
cls.product = cls._create_product(cls, 'product_product')
cls.product = cls._create_product(cls, "product_product")
cls._product_change_qty(cls, cls.product, 10)
def _create_product(self, name):
return self.product_product_model.create({
'name': name,
'categ_id': self.category.id,
'type': 'product',
'standard_price': 100,
})
return self.product_product_model.create(
{
"name": name,
"categ_id": self.category.id,
"type": "product",
"standard_price": 100,
}
)
def _product_change_qty(self, product, new_qty, reason=None,
preset_reason_id=None):
values = {
'product_id': product.id,
'new_quantity': new_qty,
}
def _product_change_qty(self, product, new_qty, reason=None, preset_reason_id=None):
values = {"product_id": product.id, "new_quantity": new_qty}
if reason:
values = {**values, 'reason': reason}
values = {**values, "reason": reason}
if preset_reason_id:
values = {**values, 'preset_reason_id': preset_reason_id.id}
values = {**values, "preset_reason_id": preset_reason_id.id}
wizard = self.wizard_model.create(values)
wizard.change_product_qty()
def _create_reason(self, name, description=None):
return self.preset_reason.create({
'name': name,
'description': description})
return self.preset_reason.create({"name": name, "description": description})
def test_product_change_qty_account_input(self):
# update qty on hand and add reason
self._product_change_qty(self.product, 100, self.reason.name,
self.reason)
self._product_change_qty(self.product, 100, self.reason.name, self.reason)
# check stock moves and account moves created
stock_move = self.env['stock.move'].search([
('product_id', '=', self.product.id),
('product_qty', '=', 90)])
account_move = self.env['account.move'].search(
[('stock_move_id', '=', stock_move.id)])
stock_move = self.env["stock.move"].search(
[("product_id", "=", self.product.id), ("product_qty", "=", 90)]
)
account_move = self.env["account.move"].search(
[("stock_move_id", "=", stock_move.id)]
)
# asserts
account_move_line1 = self.env['account.move.line'].search(
[('move_id', '=', account_move.id),
('account_id', '=', self.account_input.id)])
account_move_line2 = self.env['account.move.line'].search(
[('move_id', '=', account_move.id),
('account_id', '=', self.stock_valuation_account.id)])
self.assertEqual(abs(account_move_line1.balance),
abs(account_move_line2.balance))
account_move_line1 = self.env["account.move.line"].search(
[
("move_id", "=", account_move.id),
("account_id", "=", self.account_input.id),
]
)
account_move_line2 = self.env["account.move.line"].search(
[
("move_id", "=", account_move.id),
("account_id", "=", self.stock_valuation_account.id),
]
)
self.assertEqual(
abs(account_move_line1.balance), abs(account_move_line2.balance)
)
def test_product_change_qty_account_output(self):
# update qty on hand and add reason
self._product_change_qty(self.product, 5, self.reason.name,
self.reason)
self._product_change_qty(self.product, 5, self.reason.name, self.reason)
# check stock moves and account moves created
stock_move = self.env['stock.move'].search([
('product_id', '=', self.product.id),
('product_qty', '=', 5)])
account_move = self.env['account.move'].search(
[('stock_move_id', '=', stock_move.id)])
stock_move = self.env["stock.move"].search(
[("product_id", "=", self.product.id), ("product_qty", "=", 5)]
)
account_move = self.env["account.move"].search(
[("stock_move_id", "=", stock_move.id)]
)
# asserts
account_move_line3 = self.env['account.move.line'].search(
[('move_id', '=', account_move.id),
('account_id', '=', self.account_output.id)])
account_move_line4 = self.env['account.move.line'].search(
[('move_id', '=', account_move.id),
('account_id', '=', self.stock_valuation_account.id)])
self.assertEqual(abs(account_move_line3.balance),
abs(account_move_line4.balance))
account_move_line3 = self.env["account.move.line"].search(
[
("move_id", "=", account_move.id),
("account_id", "=", self.account_output.id),
]
)
account_move_line4 = self.env["account.move.line"].search(
[
("move_id", "=", account_move.id),
("account_id", "=", self.stock_valuation_account.id),
]
)
self.assertEqual(
abs(account_move_line3.balance), abs(account_move_line4.balance)
)