[edit controller] fix handling of removal of subentities from an edit form
In CubicWeb 3.18 the semantics of dropping a composite relation were changed.
It had before the effect of deleting the subentities.
Now the edit controller must handle this by itself (it was not adapted then by
mistake).
So if it detects the removal of a composite relation (on the browser side, the
user clicked on the "remove" action), the subentity is now explicitly deleted.
It was chosen not to change the form generation but the EditController:
- to ensure pre-3.18 forms backward compat
- to avoid the introduction of a specific entity-deletion API (RQL is probably
our best API for complex DB operations)
Related to #8529868.
# -*- coding: iso-8859-1 -*-# copyright 2003-2010 LOGILAB S.A. (Paris, FRANCE), all rights reserved.# contact http://www.logilab.fr/ -- mailto:contact@logilab.fr## This file is part of CubicWeb.## CubicWeb is free software: you can redistribute it and/or modify it under the# terms of the GNU Lesser General Public License as published by the Free# Software Foundation, either version 2.1 of the License, or (at your option)# any later version.## CubicWeb 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 Lesser General Public License for more# details.## You should have received a copy of the GNU Lesser General Public License along# with CubicWeb. If not, see <http://www.gnu.org/licenses/>."""unit tests for database value generator"""importos.pathasospimportreimportdatetimefromlogilab.common.testlibimportTestCase,unittest_mainfromcubicweb.devtools.fillimportValueGenerator,make_telfromcubicweb.devtoolsimportApptestConfigurationDATADIR=osp.join(osp.abspath(osp.dirname(__file__)),'data')ISODATE_SRE=re.compile('(?P<year>\d{4})-(?P<month>\d{2})-(?P<day>\d{2})$')classMyValueGenerator(ValueGenerator):defgenerate_Bug_severity(self,entity,index):returnu'dangerous'defgenerate_Any_description(self,entity,index,format=None):returnu'yo'classValueGeneratorTC(TestCase):"""test case for ValueGenerator"""def_choice_func(self,etype,attrname):try:returngetattr(self,'_available_%s_%s'%(etype,attrname))(etype,attrname)exceptAttributeError:returnNonedef_available_Person_firstname(self,etype,attrname):return[f.strip()forfinfile(osp.join(DATADIR,'firstnames.txt'))]defsetUp(self):config=ApptestConfiguration('data',apphome=DATADIR)config.bootstrap_cubes()schema=config.load_schema()e_schema=schema.eschema('Person')self.person_valgen=ValueGenerator(e_schema,self._choice_func)e_schema=schema.eschema('Bug')self.bug_valgen=MyValueGenerator(e_schema)self.config=configdeftest_string(self):"""test string generation"""surname=self.person_valgen.generate_attribute_value({},'surname',12)self.assertEqual(surname,u'�&surname12')deftest_domain_value(self):"""test value generation from a given domain value"""firstname=self.person_valgen.generate_attribute_value({},'firstname',12)possible_choices=self._choice_func('Person','firstname')self.assertTrue(firstnameinpossible_choices,'%s not in %s'%(firstname,possible_choices))deftest_choice(self):"""test choice generation"""# Test for random indexforindexinrange(5):sx_value=self.person_valgen.generate_attribute_value({},'civility',index)self.assertTrue(sx_valuein('Mr','Mrs','Ms'))deftest_integer(self):"""test integer generation"""# Test for random indexforindexinrange(5):cost_value=self.bug_valgen.generate_attribute_value({},'cost',index)self.assertIn(cost_value,range(index+1))deftest_date(self):"""test date generation"""# Test for random indexforindexinrange(10):date_value=self.person_valgen.generate_attribute_value({},'birthday',index)self.assertTrue(isinstance(date_value,datetime.date))deftest_phone(self):"""tests make_tel utility"""self.assertEqual(make_tel(22030405),'22 03 04 05')deftest_customized_generation(self):self.assertEqual(self.bug_valgen.generate_attribute_value({},'severity',12),u'dangerous')self.assertEqual(self.bug_valgen.generate_attribute_value({},'description',12),u'yo')self.assertEqual(self.person_valgen.generate_attribute_value({},'description',12),u'yo')classConstraintInsertionTC(TestCase):deftest_writeme(self):self.skipTest('Test automatic insertion / Schema Constraints')if__name__=='__main__':unittest_main()