Merge lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill2-lga into lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-hierarchical

[Laetitia Gangloff (Acsone)]

I make some change in the module stock_inventory_hierarchical_location.

The goal is to take account of sub-inventories in case of an exhaustive inventory in two case :

-fill inventory

I also correct a little mistake in demo data of this module.

[Lionel Sausin - Initiatives/Numérigraphe (ls-initiatives)]

Thanks for your help.
I've just merged our own fixes of the day, will you please merge them to avoid conflicts ?

I think I understand what you're trying to do in fill_inventory and it's not exactly how we had it in v6 - which is broken right now in v7, and would require a patch on the code addons.
Your patch completely excludes the locations of sub-inventories, whereas we used to only exclude the generated inventory lines. That would allow us to "catch" the lines that could have been missing from sub-inventories, but since they're exhaustive anyway there should not be any.
So I guess your approach is correct.
I'll let Loïc decide, he's working on the problem too.

review: Needs Resubmitting (has conflicts)

[Lionel Sausin - Initiatives/Numérigraphe (ls-initiatives)]

review: Needs Fixing (has conflicts)

[Laetitia Gangloff (Acsone) (laetitia-gangloff)]

I redo a branch based on your last version (38).

So this one contains change for fill inventory. The fill inventory, is used when confirm inventory or when click on button.

In case of confirm inventory, the sub-inventory are already confirmed -> no missing lines.

In case of click on button, it is true that we can have some missing lines, but the confirmation, is not possible (sub-inventories are not confirmed). I think it is 'dangerous' if lines are added, because the confirmation on sub-inventory will also add these lines !

Maybe can we prevent this action when sub-inventories are not confirmed ? (like confirm)

[lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill2-lga]

40. By Laetitia Gangloff (Acsone) on 2014-06-13

stock_inventory_location : use SUPERUSER for check in stock.location and stock.move

[Laetitia Gangloff (Acsone) (laetitia-gangloff)]

I complete this mp, with the user of SUPERUSER for check on stock.location and stock.move.
It is in the module stock_inventory_location.

A user with no access to stock.inventory, have acl error with these check. And record rules can give wrong result.

[lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill2-lga]

41. By Laetitia Gangloff (Acsone) on 2014-06-13

stock_inventory_location : avoid recursive write on stock.location

[Laetitia Gangloff (Acsone) (laetitia-gangloff)]

I resolve another problem on the write of stock_location.
There is a "RuntimeError: maximum recursion depth exceeded" when change location_id. ids was just assigned in ids_to_check so, when ids_to_check change, ids change too and the write is not done with right parameter.

[lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill2-lga]

42. By Laetitia Gangloff (Acsone) on 2014-06-13

stock_inventory_hierarchical_location : in fill inventory, if exclude location is empty do not add it in domain

[Laetitia Gangloff (Acsone) (laetitia-gangloff)]

Following Loïc's test, in fill inventory I remove the exclude clause when location list to exclude is empty.

[lp:~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill2-lga]

43. By Laetitia Gangloff (Acsone) on 2014-06-13

stock_inventory_hierarchical_location : remove unused method _fill_location_lines

44. By Laetitia Gangloff (Acsone) on 2014-06-13

stock_inventory_hierarchical_location: fill_inventory ignore no product exception in case of an exhaustive inventory

[Loïc Bellier - Numérigraphe (lb-b)]

After lot's of test, your module seems to work !

review: Approve (code review, testing)

=== modified file 'stock_inventory_hierarchical_location/__init__.py'
--- stock_inventory_hierarchical_location/__init__.py	2014-06-12 12:48:28 +0000
+++ stock_inventory_hierarchical_location/__init__.py	2014-06-13 13:19:05 +0000
@@ -19,3 +19,4 @@
 ##############################################################################
 
 from . import inventory_hierarchical_location
+from . import wizard

=== modified file 'stock_inventory_hierarchical_location/__openerp__.py'
--- stock_inventory_hierarchical_location/__openerp__.py	2014-06-10 14:04:38 +0000
+++ stock_inventory_hierarchical_location/__openerp__.py	2014-06-13 13:19:05 +0000
@@ -43,6 +43,6 @@
     "data": [
         "inventory_hierarchical_location_view.xml",
     ],
-    "test": ["test/inventory_hierarchical_location_test.yml"],
+    "test": ["tests/inventory_hierarchical_location_test.yml"],
     "demo": ["inventory_hierarchical_location_demo.xml"],
 }

=== modified file 'stock_inventory_hierarchical_location/inventory_hierarchical_location.py'
--- stock_inventory_hierarchical_location/inventory_hierarchical_location.py	2014-06-12 12:48:28 +0000
+++ stock_inventory_hierarchical_location/inventory_hierarchical_location.py	2014-06-13 13:19:05 +0000
@@ -86,13 +86,3 @@
                                          "It cannot be added.")}
                     }
         return {}
-
-    def _fill_location_lines(self, cr, uid, inventory_id, location_ids,
-                             set_stock_zero, context=None):
-        """Add ids of children inventory into list """
-        children_inventory_ids = self.search(
-            cr, uid, [('parent_id', 'child_of', inventory_id)])
-        context['children_inventory_ids'] = children_inventory_ids
-        return super(HierarchicalExhInventory, self)._fill_location_lines(
-            cr, uid, inventory_id, location_ids, set_stock_zero,
-            context=context)

=== modified file 'stock_inventory_hierarchical_location/inventory_hierarchical_location_demo.xml'
--- stock_inventory_hierarchical_location/inventory_hierarchical_location_demo.xml	2014-06-12 16:40:46 +0000
+++ stock_inventory_hierarchical_location/inventory_hierarchical_location_demo.xml	2014-06-13 13:19:05 +0000
@@ -22,7 +22,7 @@
         <record id="child_2_id" model="stock.inventory">
             <field name="name">Team B</field>
             <field name="parent_id" ref="parent_inventory"/>
-            <field name="location_id" model="stock.location" search="stock.stock_location_components"/>
+            <field name="location_id" model="stock.location" ref="stock.stock_location_components"/>
         </record>
     </data>
 </openerp>

=== renamed directory 'stock_inventory_hierarchical_location/test' => 'stock_inventory_hierarchical_location/tests'
=== added file 'stock_inventory_hierarchical_location/tests/__init__.py'
--- stock_inventory_hierarchical_location/tests/__init__.py	1970-01-01 00:00:00 +0000
+++ stock_inventory_hierarchical_location/tests/__init__.py	2014-06-13 13:19:05 +0000
@@ -0,0 +1,39 @@
+# -*- coding: utf-8 -*-
+#
+#
+#    Authors: Laetitia Gangloff
+#    Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu)
+#    All Rights Reserved
+#
+#    WARNING: This program as such is intended to be used by professional
+#    programmers who take the whole responsibility of assessing all potential
+#    consequences resulting from its eventual inadequacies and bugs.
+#    End users who are looking for a ready-to-use solution with commercial
+#    guarantees and support are strongly advised to contact a Free Software
+#    Service Company.
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+#
+
+import fill_inventory_test
+
+fast_suite = [
+]
+
+checks = [
+    fill_inventory_test,
+]
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'stock_inventory_hierarchical_location/tests/fill_inventory_test.py'
--- stock_inventory_hierarchical_location/tests/fill_inventory_test.py	1970-01-01 00:00:00 +0000
+++ stock_inventory_hierarchical_location/tests/fill_inventory_test.py	2014-06-13 13:19:05 +0000
@@ -0,0 +1,118 @@
+# -*- coding: utf-8 -*-
+#
+#
+#    Authors: Laetitia Gangloff
+#    Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu)
+#    All Rights Reserved
+#
+#    WARNING: This program as such is intended to be used by professional
+#    programmers who take the whole responsibility of assessing all potential
+#    consequences resulting from its eventual inadequacies and bugs.
+#    End users who are looking for a ready-to-use solution with commercial
+#    guarantees and support are strongly advised to contact a Free Software
+#    Service Company.
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+#
+
+import openerp.tests.common as common
+
+DB = common.DB
+ADMIN_USER_ID = common.ADMIN_USER_ID
+
+
+class fill_inventory_test(common.TransactionCase):
+
+    def setUp(self):
+        super(fill_inventory_test, self).setUp()
+
+    def test_missing_location(self):
+        """
+        Test that when confirm a parent inventory, the child location are not in the confirmation result
+        """
+        parent_inventory_id = self.ref('stock_inventory_hierarchical_location.parent_inventory')
+        self.registry('stock.inventory').action_open(self.cr, self.uid, [parent_inventory_id])
+        # confirm shelf 1 inventory
+        inventory_id = self.ref('stock_inventory_hierarchical_location.child_2_id')
+        self.registry('stock.inventory').action_open(self.cr, self.uid, [inventory_id])
+        missing_location = self.registry('stock.inventory').get_missing_locations(self.cr, self.uid, [inventory_id])
+        self.assertEqual(len(missing_location), 1, "1 missing location should be find, because the inventory is empty")
+        wizard_id = self.registry('stock.inventory.uninventoried.locations').create(self.cr, self.uid, {}, context={'active_ids': [inventory_id]})
+        self.registry('stock.inventory.uninventoried.locations').confirm_uninventoried_locations(self.cr, self.uid, wizard_id, context={'active_ids': [inventory_id]})
+        missing_location = self.registry('stock.inventory').get_missing_locations(self.cr, self.uid, [inventory_id])
+        self.assertEqual(len(missing_location), 0, "No missing location should be find, because the inventory is confirmed")
+        # confirm shelf 2 inventory
+        inventory_id = self.ref('stock_inventory_hierarchical_location.child_1_id')
+        self.registry('stock.inventory').action_open(self.cr, self.uid, [inventory_id])
+        missing_location = self.registry('stock.inventory').get_missing_locations(self.cr, self.uid, [inventory_id])
+        self.assertEqual(len(missing_location), 1, "1 missing location should be fine, because the inventory is empty")
+        self.registry('stock.inventory.line').create(self.cr, self.uid, {'product_id': self.ref('product.product_product_7'),
+                                                                         'product_uom': self.ref('product.product_uom_unit'),
+                                                                         'company_id': self.ref('base.main_company'),
+                                                                         'inventory_id': inventory_id,
+                                                                         'product_qty': 18.0,
+                                                                         'location_id': self.ref('stock.stock_location_14')})
+        missing_location = self.registry('stock.inventory').get_missing_locations(self.cr, self.uid, [inventory_id])
+        self.assertEqual(len(missing_location), 0, "No missing location should be find, because the inventory is filled")
+        wizard_id = self.registry('stock.inventory.uninventoried.locations').create(self.cr, self.uid, {}, context={'active_ids': [inventory_id]})
+        wizard = self.registry('stock.inventory.uninventoried.locations').browse(self.cr, self.uid, wizard_id, context={'active_ids': [inventory_id]})
+        self.assertEqual(len(wizard.location_ids), 0, "The wizard should not contain any lines but contains %s." % wizard.location_ids)
+        self.registry('stock.inventory.uninventoried.locations').confirm_uninventoried_locations(self.cr, self.uid, wizard_id, context={'active_ids': [inventory_id]})
+        # confirm parent inventory
+        missing_location = self.registry('stock.inventory').get_missing_locations(self.cr, self.uid, [parent_inventory_id])
+        self.assertEqual(len(missing_location), 1, "Only 1 missing location should be find, because there is some location in child inventory")
+
+    def test_fill_inventory(self):
+        """
+        Test that when fill a parent inventory, the child location are not in the result
+        """
+        parent_inventory_id = self.ref('stock_inventory_hierarchical_location.parent_inventory')
+        self.registry('stock.inventory').action_open(self.cr, self.uid, [parent_inventory_id])
+        # confirm shelf 1 inventory
+        inventory_id = self.ref('stock_inventory_hierarchical_location.child_2_id')
+        self.registry('stock.inventory').action_open(self.cr, self.uid, [inventory_id])
+        wizard_id = self.registry('stock.fill.inventory').create(self.cr, self.uid, {'location_id': self.ref('stock.stock_location_components'),
+                                                                                     'recursive': True,
+                                                                                     'exhaustive': True,
+                                                                                     'set_stock_zero': True}, context={'active_ids': [inventory_id]})
+        self.registry('stock.fill.inventory').fill_inventory(self.cr, self.uid, [wizard_id], context={'active_ids': [inventory_id]})
+        inventory_line_ids = self.registry('stock.inventory.line').search(self.cr, self.uid, [('inventory_id', '=', inventory_id)])
+        self.assertEqual(len(inventory_line_ids), 12, "12 inventory line is fount after filling inventory")
+        # confirm shelf 2 inventory
+        inventory_id = self.ref('stock_inventory_hierarchical_location.child_1_id')
+        self.registry('stock.inventory').action_open(self.cr, self.uid, [inventory_id])
+        wizard_id = self.registry('stock.fill.inventory').create(self.cr, self.uid, {'location_id': self.ref('stock.stock_location_14'),
+                                                                                     'recursive': True,
+                                                                                     'exhaustive': True,
+                                                                                     'set_stock_zero': True}, context={'active_ids': [inventory_id]})
+        self.registry('stock.fill.inventory').fill_inventory(self.cr, self.uid, [wizard_id], context={'active_ids': [inventory_id]})
+        inventory_line_ids = self.registry('stock.inventory.line').search(self.cr, self.uid, [('inventory_id', '=', inventory_id)])
+        self.assertEqual(len(inventory_line_ids), 4, "1 inventory line is fount after filling inventory")
+        # confirm parent inventory
+        wizard_id = self.registry('stock.fill.inventory').create(self.cr, self.uid, {'location_id': self.ref('stock.stock_location_stock'),
+                                                                                     'recursive': True,
+                                                                                     'exhaustive': True,
+                                                                                     'set_stock_zero': True}, context={'active_ids': [parent_inventory_id]})
+        try:
+            self.registry('stock.fill.inventory').fill_inventory(self.cr, self.uid, [wizard_id], context={'active_ids': [parent_inventory_id]})
+        except Exception, e:
+            self.assertEqual(e.value, 'No product in this location. Please select a location in the product form.', "The message should be ''No product in this location. Please select a location in the product form.''")
+            exception_happened = True
+            pass
+        self.assertTrue(exception_happened)
+        inventory_line_ids = self.registry('stock.inventory.line').search(self.cr, self.uid, [('inventory_id', '=', parent_inventory_id)])
+        self.assertEqual(len(inventory_line_ids), 0, "No inventory line is fount after filling inventory")
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added directory 'stock_inventory_hierarchical_location/wizard'
=== added file 'stock_inventory_hierarchical_location/wizard/__init__.py'
--- stock_inventory_hierarchical_location/wizard/__init__.py	1970-01-01 00:00:00 +0000
+++ stock_inventory_hierarchical_location/wizard/__init__.py	2014-06-13 13:19:05 +0000
@@ -0,0 +1,21 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    This module is copyright (C) 2013 Numérigraphe SARL. All Rights Reserved.
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU General Public License as published by
+#    the Free Software Foundation, either version 3 of the License, or
+#    (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU General Public License for more details.
+#
+#    You should have received a copy of the GNU General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+from . import stock_fill_location_inventory

=== added file 'stock_inventory_hierarchical_location/wizard/stock_fill_location_inventory.py'
--- stock_inventory_hierarchical_location/wizard/stock_fill_location_inventory.py	1970-01-01 00:00:00 +0000
+++ stock_inventory_hierarchical_location/wizard/stock_fill_location_inventory.py	2014-06-13 13:19:05 +0000
@@ -0,0 +1,89 @@
+# -*- coding: utf-8 -*-
+#
+#
+#    Authors: Laetitia Gangloff
+#    Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu)
+#    All Rights Reserved
+#
+#    WARNING: This program as such is intended to be used by professional
+#    programmers who take the whole responsibility of assessing all potential
+#    consequences resulting from its eventual inadequacies and bugs.
+#    End users who are looking for a ready-to-use solution with commercial
+#    guarantees and support are strongly advised to contact a Free Software
+#    Service Company.
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+#
+
+from openerp.osv import orm
+from openerp.tools.translate import _
+
+
+class FillInventoryWizard(orm.TransientModel):
+    """If inventory as sub inventories, do not fill with sub inventories location"""
+    _inherit = 'stock.fill.inventory'
+
+    def fill_inventory(self, cr, uid, ids, context=None):
+        """ To Import stock inventory according to products available in the location and not already in a sub inventory
+
+        We split fill_inventory on many fill_inventory (one for each location)
+        @param self: The object pointer.
+        @param cr: A database cursor
+        @param uid: ID of the user currently logged in
+        @param ids: the ID or list of IDs if we want more than one
+        @param context: A standard dictionary
+        @return:
+        """
+        if context is None:
+            context = {}
+
+        if ids and len(ids):
+            ids = ids[0]
+        else:
+            return {'type': 'ir.actions.act_window_close'}
+        fill_inventory = self.browse(cr, uid, ids, context=context)
+        if fill_inventory.recursive and fill_inventory.exhaustive:
+            exclude_location_ids = []
+            for i in self.pool['stock.inventory'].browse(cr, uid, context['active_ids']):
+                for sub_inventory in i.inventory_ids:
+                    # exclude these location
+                    exclude_location_ids.append(sub_inventory.location_id.id)
+            domain = [('location_id', 'child_of', [fill_inventory.location_id.id])]
+            if exclude_location_ids:
+                domain.append('!')
+                domain.append(('location_id', 'child_of', exclude_location_ids))
+            location_ids = self.pool['stock.location'].search(cr, uid, domain,
+                                                              order="id",
+                                                              context=context)
+            all_in_exception = 0
+            for location_id in location_ids:
+                try:
+                    super(FillInventoryWizard, self).fill_inventory(cr, uid,
+                                                                    [self.copy(cr, uid, ids, {'location_id': location_id,
+                                                                                              'recursive': False, }, context=context)],
+                                                                    context=context)
+                except Exception, e:
+                    if e.value == _('No product in this location. Please select a location in the product form.'):
+                        all_in_exception = all_in_exception + 1
+                        pass
+                    else:
+                        raise e
+            if all_in_exception == len(location_ids):
+                raise orm.except_orm(_('Warning!'), _('No product in this location. Please select a location in the product form.'))
+            return {'type': 'ir.actions.act_window_close'}
+        else:
+            return super(FillInventoryWizard, self).fill_inventory(cr, uid, [ids], context=context)
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== modified file 'stock_inventory_location/__openerp__.py'
--- stock_inventory_location/__openerp__.py	2014-06-10 16:02:09 +0000
+++ stock_inventory_location/__openerp__.py	2014-06-13 13:19:05 +0000
@@ -62,9 +62,9 @@
         "wizard/stock_fill_location_inventory_view.xml",
     ],
     "test": [
-        "test/inventory_standard_test.yml",
-        "test/inventory_exhaustive_test.yml",
-        "test/inventory_future_test.yml",
+        "tests/inventory_standard_test.yml",
+        "tests/inventory_exhaustive_test.yml",
+        "tests/inventory_future_test.yml",
     ],
     "images": [
         "images/inventory_form.png",

=== modified file 'stock_inventory_location/stock_inventory_location.py'
--- stock_inventory_location/stock_inventory_location.py	2014-06-12 16:49:28 +0000
+++ stock_inventory_location/stock_inventory_location.py	2014-06-13 13:19:05 +0000
@@ -27,6 +27,7 @@
 # TODOv8! remove, feature is included upstream
 from openerp.osv import osv
 from openerp.tools import DEFAULT_SERVER_DATETIME_FORMAT
+from openerp import SUPERUSER_ID
 
 from .exceptions import ExhaustiveInventoryException
 
@@ -270,7 +271,7 @@
         """Error if an exhaustive Inventory is being conducted here"""
         inv_obj = self.pool['stock.inventory']
         location_inventory_open_ids = inv_obj._get_locations_open_inventories(
-            cr, uid, context=context)
+            cr, SUPERUSER_ID, context=context)
         if not isinstance(ids, Iterable):
             ids = [ids]
         for inv_id in ids:
@@ -286,7 +287,7 @@
         self._check_inventory(cr, uid, ids, context=context)
         if not isinstance(ids, Iterable):
             ids = [ids]
-        ids_to_check = ids
+        ids_to_check = list(ids)
         # If changing the parent, no inventory must conducted there either
         if vals.get('location_id'):
             ids_to_check.append(vals['location_id'])
@@ -326,7 +327,7 @@
         message = ""
         inv_obj = self.pool['stock.inventory']
         locked_location_ids = inv_obj._get_locations_open_inventories(
-            cr, uid, context=context)
+            cr, SUPERUSER_ID, context=context)
         if not locked_location_ids:
             # Nothing to verify
             return True

=== renamed directory 'stock_inventory_location/test' => 'stock_inventory_location/tests'
=== added file 'stock_inventory_location/tests/__init__.py'
--- stock_inventory_location/tests/__init__.py	1970-01-01 00:00:00 +0000
+++ stock_inventory_location/tests/__init__.py	2014-06-13 13:19:05 +0000
@@ -0,0 +1,39 @@
+# -*- coding: utf-8 -*-
+#
+#
+#    Authors: Laetitia Gangloff
+#    Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu)
+#    All Rights Reserved
+#
+#    WARNING: This program as such is intended to be used by professional
+#    programmers who take the whole responsibility of assessing all potential
+#    consequences resulting from its eventual inadequacies and bugs.
+#    End users who are looking for a ready-to-use solution with commercial
+#    guarantees and support are strongly advised to contact a Free Software
+#    Service Company.
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+#
+
+import stock_inventory_location_test
+
+fast_suite = [
+]
+
+checks = [
+    stock_inventory_location_test,
+]
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

=== added file 'stock_inventory_location/tests/stock_inventory_location_test.py'
--- stock_inventory_location/tests/stock_inventory_location_test.py	1970-01-01 00:00:00 +0000
+++ stock_inventory_location/tests/stock_inventory_location_test.py	2014-06-13 13:19:05 +0000
@@ -0,0 +1,47 @@
+# -*- coding: utf-8 -*-
+#
+#
+#    Authors: Laetitia Gangloff
+#    Copyright (c) 2014 Acsone SA/NV (http://www.acsone.eu)
+#    All Rights Reserved
+#
+#    WARNING: This program as such is intended to be used by professional
+#    programmers who take the whole responsibility of assessing all potential
+#    consequences resulting from its eventual inadequacies and bugs.
+#    End users who are looking for a ready-to-use solution with commercial
+#    guarantees and support are strongly advised to contact a Free Software
+#    Service Company.
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+#
+
+import openerp.tests.common as common
+
+DB = common.DB
+ADMIN_USER_ID = common.ADMIN_USER_ID
+
+
+class stock_inventory_location_test(common.TransactionCase):
+
+    def setUp(self):
+        super(stock_inventory_location_test, self).setUp()
+
+    def test_update_parent_location(self):
+        """
+        Test the update of the parent of a location (no inventory in progress
+        """
+        self.registry('stock.location').write(self.cr, self.uid, self.ref('stock.stock_location_5'), {'location_id': self.ref('stock.stock_location_4')})
+
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: