audrius/stock-logistics-reporting
1
0
Fork 1
mirror of https://github.com/OCA/stock-logistics-reporting.git synced 2025-02-16 17:13:21 +02:00
Code Issues Packages Projects Releases Wiki Activity

[FIX] stock_average_daily_sale: Don't check the view existence in tests

Browse Source 
As tests create a Savepoint, there is a concurrent query when checking the
view existence.
This commit is contained in:
Denis Roussel
2023-10-11 13:41:14 +02:00
committed by twalter-c2c
parent ce97bf0ef3
commit 0c2f631a4c
2 changed files with 2 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
stock_average_daily_sale/models/stock_average_daily_sale.py
View File

@@ -7,6 +7,7 @@ from psycopg2.errors import ObjectNotInPrerequisiteState
from psycopg2.extensions import AsIs from psycopg2.extensions import AsIs
from odoo import _, api, fields, models, registry from odoo import _, api, fields, models, registry
from odoo.tools import config
from odoo.addons.stock_storage_type_putaway_abc.models.stock_location import ( from odoo.addons.stock_storage_type_putaway_abc.models.stock_location import (
ABC_SELECTION, ABC_SELECTION,
@@ -118,7 +119,7 @@ class StockAverageDailySale(models.Model):
# pylint: disable=redefined-outer-name # pylint: disable=redefined-outer-name
@api.model @api.model
def search(self, domain, offset=0, limit=None, order=None, count=False): def search(self, domain, offset=0, limit=None, order=None, count=False):
if not self._check_view(): if not config["test_enable"] and not self._check_view():
return self.browse() return self.browse()
return super().search( return super().search(
domain=domain, offset=offset, limit=limit, order=order, count=count domain=domain, offset=offset, limit=limit, order=order, count=count

11
stock_average_daily_sale/tests/test_average_daily_sale.py
View File

@@ -186,17 +186,6 @@ class TestAverageSale(CommonAverageSaleTest, TransactionCase):
) )
self.assertFalse(avg_product_2) self.assertFalse(avg_product_2)
def test_view(self):
# Check no exception is raised if materialized view is not loaded
with self.assertLogs(
"odoo.addons.stock_average_daily_sale.models.stock_average_daily_sale"
) as log:
result = self.env["stock.average.daily.sale"].search_read(
[("product_id", "=", self.product_1.id)]
)
self.assertFalse(result)
self.assertIn("The materialized view has not been populated", log.output[0])
def test_view_refreshed(self): def test_view_refreshed(self):
self._refresh() self._refresh()
with self.assertNoLogs( with self.assertNoLogs(