From 973dee4c97f437d7ff4c166f346e07aa8d284a24 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Thu, 4 Aug 2011 16:57:40 +0200
Subject: [PATCH] Add test for depends

---
 opportunity.py                 | 49 +++++++++++++++++++---------------
 tests/test_sale_opportunity.py |  8 +++++-
 2 files changed, 34 insertions(+), 23 deletions(-)

diff --git a/opportunity.py b/opportunity.py
index 17173dceea..fb02e29968 100644
--- a/opportunity.py
+++ b/opportunity.py
@@ -20,9 +20,11 @@ STATES = [
 _STATES_START = {
     'readonly': Not(Equal(Eval('state'), 'lead')),
 }
+_DEPENDS_START = ['state']
 _STATES_STOP = {
     'readonly': In(Eval('state'), ['converted', 'lost', 'cancelled']),
 }
+_DEPENDS_STOP = ['state']
 
 
 class SaleOpportunity(ModelWorkflow, ModelSQL, ModelView):
@@ -33,16 +35,16 @@ class SaleOpportunity(ModelWorkflow, ModelSQL, ModelView):
     _rec_name = 'description'
 
     party = fields.Many2One('party.party', 'Party', required=True, select=1,
-            states=_STATES_STOP, depends=['state'])
+            states=_STATES_STOP, depends=_DEPENDS_STOP)
     address = fields.Many2One('party.address', 'Address',
-            domain=[('party', '=', Eval('party'))],
-            select=2, depends=['party', 'state'],
-            states=_STATES_STOP)
+        domain=[('party', '=', Eval('party'))],
+        select=2, depends=['party', 'state'],
+        states=_STATES_STOP)
     company = fields.Many2One('company.company', 'Company', required=True,
-            select=1, states=_STATES_STOP, domain=[
-                ('id', If(In('company', Eval('context', {})), '=', '!='),
-                    Get(Eval('context', {}), 'company', 0)),
-            ], on_change=['company'], depends=['state'])
+        select=1, states=_STATES_STOP, domain=[
+            ('id', If(In('company', Eval('context', {})), '=', '!='),
+                Get(Eval('context', {}), 'company', 0)),
+            ], on_change=['company'], depends=_DEPENDS_STOP)
     currency = fields.Function(fields.Many2One('currency.currency',
         'Currency'), 'get_currency')
     currency_digits = fields.Function(fields.Integer('Currency Digits'),
@@ -50,31 +52,34 @@ class SaleOpportunity(ModelWorkflow, ModelSQL, ModelView):
     amount = fields.Numeric('Amount', digits=(16, Eval('currency_digits', 2)),
             depends=['currency_digits'], help='Estimated revenue amount')
     warehouse = fields.Many2One('stock.location', 'Warehouse',
-            domain=[('type', '=', 'warehouse')], states={
-                'required': Equal(Eval('state'), 'converted'),
-                'readonly': In(Eval('state'),
-                    ['converted', 'lost', 'cancelled']),
-            })
+        domain=[('type', '=', 'warehouse')], states={
+            'required': Equal(Eval('state'), 'converted'),
+            'readonly': In(Eval('state'),
+                ['converted', 'lost', 'cancelled']),
+            },
+        depends=['state'])
     payment_term = fields.Many2One('account.invoice.payment_term',
-            'Payment Term', states={
-                'required': Equal(Eval('state'), 'converted'),
-                'readonly': In(Eval('state'),
-                    ['converted', 'lost', 'cancelled']),
-            })
+        'Payment Term', states={
+            'required': Equal(Eval('state'), 'converted'),
+            'readonly': In(Eval('state'),
+                ['converted', 'lost', 'cancelled']),
+            },
+        depends=['state'])
     employee = fields.Many2One('company.employee', 'Employee', required=True,
             states=_STATES_STOP, depends=['state', 'company'],
             domain=[('company', '=', Eval('company'))])
     start_date = fields.Date('Start Date', required=True, select=1,
-            states=_STATES_START, depends=['state'])
+        states=_STATES_START, depends=_DEPENDS_START)
     end_date = fields.Date('End Date', select=2, readonly=True, states={
         'invisible': Not(In(Eval('state'),
             ['converted', 'cancelled', 'lost'])),
     }, depends=['state'])
     description = fields.Char('Description', required=True,
-            states=_STATES_STOP, depends=['state'])
-    comment = fields.Text('Comment', states=_STATES_STOP)
+        states=_STATES_STOP, depends=_DEPENDS_STOP)
+    comment = fields.Text('Comment', states=_STATES_STOP,
+        depends=_DEPENDS_STOP)
     lines = fields.One2Many('sale.opportunity.line', 'opportunity', 'Lines',
-            states=_STATES_STOP, depends=['state'])
+        states=_STATES_STOP, depends=_DEPENDS_STOP)
     state = fields.Selection(STATES, 'State', required=True, select=1,
             sort=False, readonly=True)
     probability = fields.Integer('Conversion Probability',
diff --git a/tests/test_sale_opportunity.py b/tests/test_sale_opportunity.py
index b92dc7629a..b260192696 100644
--- a/tests/test_sale_opportunity.py
+++ b/tests/test_sale_opportunity.py
@@ -10,7 +10,7 @@ if os.path.isdir(DIR):
 
 import unittest
 import trytond.tests.test_tryton
-from trytond.tests.test_tryton import test_view
+from trytond.tests.test_tryton import test_view, test_depends
 
 
 class SaleOpportunityTestCase(unittest.TestCase):
@@ -27,6 +27,12 @@ class SaleOpportunityTestCase(unittest.TestCase):
         '''
         test_view('sale_opportunity')
 
+    def test0006depends(self):
+        '''
+        Test depends.
+        '''
+        test_depends()
+
 def suite():
     suite = trytond.tests.test_tryton.suite()
     suite.addTests(unittest.TestLoader().loadTestsFromTestCase(