details:   https://code.tryton.org/tryton/commit/790b437452a4
branch:    default
user:      Cédric Krier <[email protected]>
date:      Thu Nov 27 16:59:05 2025 +0100
description:
        Standardise the skipping of warnings in the test scenario
diffstat:

 modules/account/tests/scenario_move_line_group.rst                             
    |  6 +++---
 modules/account_invoice/tests/scenario_cancelling_invoice_move.rst             
    |  4 ++--
 modules/account_invoice/tests/scenario_invoice_alternate_currency.rst          
    |  3 +--
 modules/account_invoice/tests/scenario_invoice_tax_deductible.rst              
    |  3 +--
 
modules/account_stock_shipment_cost/tests/scenario_account_stock_shipment_cost.rst
 |  3 +--
 modules/stock_supply_production/tests/scenario_stock_supply_production.rst     
    |  7 +++----
 6 files changed, 11 insertions(+), 15 deletions(-)

diffs (117 lines):

diff -r 8443fa348a0d -r 790b437452a4 
modules/account/tests/scenario_move_line_group.rst
--- a/modules/account/tests/scenario_move_line_group.rst        Thu Nov 27 
16:28:21 2025 +0100
+++ b/modules/account/tests/scenario_move_line_group.rst        Thu Nov 27 
16:59:05 2025 +0100
@@ -19,6 +19,8 @@
 
     >>> config = activate_modules('account', create_company, create_chart)
 
+    >>> Warning = Model.get('res.user.warning')
+
 Get currency::
 
     >>> usd = get_currency('USD')
@@ -144,14 +146,12 @@
    >>> try:
    ...     cancel.execute('cancel')
    ... except CancelDelegatedWarning as warning:
-   ...     _, (key, *_) = warning.args
+   ...     Warning(user=config.user, name=warning.name).save()
    ...     raise
    Traceback (most recent call last):
       ...
    CancelDelegatedWarning: ...
 
-   >>> Warning = Model.get('res.user.warning')
-   >>> Warning(user=config.user, name=key).save()
    >>> cancel.execute('cancel')
    >>> Reconciliation.find([('id', '=', reconciliations[0].id)])
    []
diff -r 8443fa348a0d -r 790b437452a4 
modules/account_invoice/tests/scenario_cancelling_invoice_move.rst
--- a/modules/account_invoice/tests/scenario_cancelling_invoice_move.rst        
Thu Nov 27 16:28:21 2025 +0100
+++ b/modules/account_invoice/tests/scenario_cancelling_invoice_move.rst        
Thu Nov 27 16:59:05 2025 +0100
@@ -86,8 +86,8 @@
 
     >>> try:
     ...     cancel_move.execute('cancel')
-    ... except CancelInvoiceMoveWarning as e:
-    ...     Warning(user=config.user, name=e.name).save()
+    ... except CancelInvoiceMoveWarning as warning:
+    ...     Warning(user=config.user, name=warning.name).save()
     >>> cancel_move.execute('cancel')
     >>> cancel_move.state
     'end'
diff -r 8443fa348a0d -r 790b437452a4 
modules/account_invoice/tests/scenario_invoice_alternate_currency.rst
--- a/modules/account_invoice/tests/scenario_invoice_alternate_currency.rst     
Thu Nov 27 16:28:21 2025 +0100
+++ b/modules/account_invoice/tests/scenario_invoice_alternate_currency.rst     
Thu Nov 27 16:59:05 2025 +0100
@@ -177,12 +177,11 @@
     >>> try:
     ...     invoice.click('post')
     ... except InvoiceTaxesWarning as warning:
-    ...     _, (key, *_) = warning.args
+    ...     Warning(user=config.user, name=warning.name).save()
     ...     raise
     Traceback (most recent call last):
         ...
     InvoiceTaxesWarning: ...
-    >>> Warning(user=config.user, name=key).save()
     >>> invoice.click('post')
     >>> invoice.state
     'posted'
diff -r 8443fa348a0d -r 790b437452a4 
modules/account_invoice/tests/scenario_invoice_tax_deductible.rst
--- a/modules/account_invoice/tests/scenario_invoice_tax_deductible.rst Thu Nov 
27 16:28:21 2025 +0100
+++ b/modules/account_invoice/tests/scenario_invoice_tax_deductible.rst Thu Nov 
27 16:59:05 2025 +0100
@@ -89,12 +89,11 @@
     >>> try:
     ...     invoice.click('post')
     ... except InvoiceTaxesWarning as warning:
-    ...     _, (key, *_) = warning.args
+    ...     Warning(user=config.user, name=warning.name).save()
     ...     raise
     Traceback (most recent call last):
         ...
     InvoiceTaxesWarning: ...
-    >>> Warning(user=config.user, name=key).save()
     >>> invoice.click('post')
     >>> invoice.untaxed_amount, invoice.tax_amount, invoice.total_amount
     (Decimal('550.00'), Decimal('0.00'), Decimal('550.00'))
diff -r 8443fa348a0d -r 790b437452a4 
modules/account_stock_shipment_cost/tests/scenario_account_stock_shipment_cost.rst
--- 
a/modules/account_stock_shipment_cost/tests/scenario_account_stock_shipment_cost.rst
        Thu Nov 27 16:28:21 2025 +0100
+++ 
b/modules/account_stock_shipment_cost/tests/scenario_account_stock_shipment_cost.rst
        Thu Nov 27 16:59:05 2025 +0100
@@ -200,12 +200,11 @@
     >>> try:
     ...     post_shipment_cost.execute('post')
     ... except SamePartiesWarning as warning:
-    ...     _, (key, *_) = warning.args
+    ...     Warning(user=config.user, name=warning.name).save()
     ...     raise
     Traceback (most recent call last):
         ...
     SamePartiesWarning: ...
-    >>> Warning(user=config.user, name=key).save()
     >>> post_shipment_cost.execute('post')
     >>> shipment_cost2.state
     'posted'
diff -r 8443fa348a0d -r 790b437452a4 
modules/stock_supply_production/tests/scenario_stock_supply_production.rst
--- 
a/modules/stock_supply_production/tests/scenario_stock_supply_production.rst    
    Thu Nov 27 16:28:21 2025 +0100
+++ 
b/modules/stock_supply_production/tests/scenario_stock_supply_production.rst    
    Thu Nov 27 16:59:05 2025 +0100
@@ -18,6 +18,8 @@
 
     >>> config = activate_modules('stock_supply_production', create_company)
 
+    >>> Warning = Model.get('res.user.warning')
+
 Create product::
 
     >>> ProductUom = Model.get('product.uom')
@@ -64,10 +66,7 @@
     >>> try:
     ...     move.click('do')
     ... except MoveFutureWarning as warning:
-    ...     _, (key, *_) = warning.args
-
-    >>> Warning = Model.get('res.user.warning')
-    >>> Warning(user=config.user, name=key).save()
+    ...     Warning(user=config.user, name=warning.name).save()
     >>> move.click('do')
 
 There is no production request::

Reply via email to