Conversation
…e module installation.
estate/demo/estate_property.xml
Outdated
There was a problem hiding this comment.
This should preferably be relative to the date of install (see codebase for examples)
estate/demo/estate_property.xml
Outdated
There was a problem hiding this comment.
As it is the default value, you're not forced to provide it
| <field name="state">new</field> |
estate/demo/estate_property.xml
Outdated
There was a problem hiding this comment.
Why a selling_price if the state is cancelled?
…d/cancelled property. -Added check to raise an error if a property is sold without any offer accepted. -Created tests cases to verify total area calculation, verify action_property_sold, and verify offer creation for sold property. -Added missing new lines in some files.
estate/tests/test_estate_property.py
Outdated
There was a problem hiding this comment.
id?! don't touch this 😄
| 'id': 'test_partner', |
estate/tests/test_estate_property.py
Outdated
There was a problem hiding this comment.
Do you need all this? I bet a name is sufficient for your case
| 'company_name': 'test_company', | |
| 'street': 'test_street', | |
| 'city': 'test_city', | |
| 'zip': '12345', | |
| 'country_id': cls.env.ref('base.us').id, | |
| 'state_id': cls.env.ref('base.state_us_39').id, | |
| 'phone': '+1 555-555-5555', | |
| 'email': 'test@testing.example.com', | |
| 'tz': 'Europe/Brussels', |
estate/tests/test_estate_property.py
Outdated
There was a problem hiding this comment.
Plural is misleading
| cls.properties = cls.env['estate.property'].create([{'name': 'test_house'}]) | |
| cls.property = cls.env['estate.property'].create([{'name': 'test_house'}]) |
estate/tests/test_estate_property.py
Outdated
There was a problem hiding this comment.
assertEqual if only one value to check, no?
estate/tests/test_estate_property.py
Outdated
There was a problem hiding this comment.
Not related to tests, but I have the feeling that action_offer_accept should mark the property as sold, so why do you need the call to action_property_sold?
There was a problem hiding this comment.
In the estate tutorial accepting the offer and selling the property are two different states.
After accepting an offer you can still cancel the property, not sell it.
estate/tests/test_estate_property.py
Outdated
There was a problem hiding this comment.
Weird way to create an offer. Either use self.env['property.offer'].create(...), either create it while editing the field offer_ids with Command
2 participants
No description provided.