Skip to content

Commit

Permalink
[WIP][T1902]product_configurator: add test case for issue OCA#59 and …
Browse files Browse the repository at this point in the history 
…fix an bug while going to previous stage at time of configuring product
  • Loading branch information
@bizzappdev @pledrateam
bizzappdev authored and pledrateam committed Oct 11, 2022
1 parent d22d395 commit 5596b4e
Show file tree
Hide file tree
Showing 2 changed files with 150 additions and 4 deletions.
128 changes: 126 additions & 2 deletions product_configurator/tests/test_create.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,60 @@

class ConfigurationCreate(TransactionCase):

def setUp(self):
super(ConfigurationCreate, self).setUp()

self.ProductAttribute = self.env['product.attribute']
self.ProductAttributeValue = self.env['product.attribute.value']
self.ProductAttributeLine = self.env['product.attribute.line']
self.ProductConfigDomain = self.env['product.config.domain']
self.ProductConfigSteps = self.env['product.config.step']
self.ProductConfWizard = self.env['product.configurator']
self.attr_color = self.ProductAttribute.create({
'name': 'Color',
})
self.attr_size = self.ProductAttribute.create({
'name': 'Size',
})
self.attr_width = self.ProductAttribute.create({
'name': 'Width',
})
self.value_blue = self.ProductAttributeValue.create({
'name': 'Blue',
'attribute_id': self.attr_color.id
})
self.value_green = self.ProductAttributeValue.create({
'name': 'Green',
'attribute_id': self.attr_color.id
})
self.value_S = self.ProductAttributeValue.create({
'name': 'S',
'attribute_id': self.attr_size.id
})
self.value_M = self.ProductAttributeValue.create({
'name': 'M',
'attribute_id': self.attr_size.id
})
self.value_L = self.ProductAttributeValue.create({
'name': 'L',
'attribute_id': self.attr_size.id
})
self.value_A = self.ProductAttributeValue.create({
'name': 'A',
'attribute_id': self.attr_width.id
})
self.value_B = self.ProductAttributeValue.create({
'name': 'B',
'attribute_id': self.attr_width.id
})
self.product_category = self.env.ref('product.product_category_5')
self.step_1 = self.ProductConfigSteps.create({
'name': 'Step: 1'
})
self.step_2 = self.ProductConfigSteps.create({
'name': 'Step: 2'
})

def test_01_create(self):
"""Test configuration item does not make variations"""

Expand All @@ -18,7 +72,7 @@ def test_01_create(self):
'name': 'Test Configuration',
'config_ok': True,
'type': 'consu',
'categ_id': self.env.ref('product.product_category_5').id,
'categ_id': self.product_category.id,
'attribute_line_ids': [(0, 0, {
'attribute_id': attr_test.id,
'value_ids': [
Expand All @@ -34,4 +88,74 @@ def test_01_create(self):

def test_02_product_configuration_restriction(self):
"""Values on future steps breaking current wizard view """
pass

restriction = self.ProductConfigDomain.create({
'name': 'restriction-color',
'domain_line_ids': [(0, 0, {
'attribute_id': self.attr_color.id,
'condition': 'in',
'value_ids': [(6, 0, [self.value_blue.id])]
})]
})
product = self.env['product.template'].create({
'name': 'bug #59',
'config_ok': True,
'type': 'consu',
'categ_id': self.product_category.id,
'list_price': '20000',
})
attr_line1 = self.ProductAttributeLine.create({
'attribute_id': self.attr_color.id,
'value_ids': [(6, 0, [self.value_blue.id, self.value_green.id])],
'required': True,
'product_tmpl_id': product.id,
})
attr_line2 = self.ProductAttributeLine.create({
'attribute_id': self.attr_size.id,
'value_ids': [(6, 0, [self.value_S.id, self.value_M.id, self.value_L.id])],
'required': True,
'product_tmpl_id': product.id,
})
attr_line3 = self.ProductAttributeLine.create({
'attribute_id': self.attr_width.id,
'value_ids': [(6, 0, [self.value_A.id, self.value_B.id])],
'required': True,
'product_tmpl_id': product.id,
})
step1 = {
'config_step_id': self.step_1.id,
'attribute_line_ids': [(6, 0, [attr_line1.id])]
}
step2 = {
'config_step_id': self.step_2.id,
'attribute_line_ids': [(6, 0, [attr_line2.id, attr_line3.id])]
}
product.write({
'config_step_line_ids': [(0, 0, step1), (0, 0, step2)],
'config_line_ids': [(0, 0, {
'attribute_line_id': attr_line2.id,
'value_ids': [(6, 0, [self.value_M.id])],
'domain_id': restriction.id
})]
})
product_config_wizard = self.ProductConfWizard.create({
'product_tmpl_id': product.id,
})
product_config_wizard.action_next_step()
setattr(product_config_wizard, '__attribute-' + str(self.attr_color.id), str(self.value_blue.id))
product_config_wizard.action_next_step()
setattr(product_config_wizard, '__attribute-' + str(self.attr_size.id), str(self.value_M.id))
product_config_wizard.action_previous_step()
setattr(product_config_wizard, '__attribute-' + str(self.attr_color.id), str(self.value_green.id))
product_config_wizard.action_next_step()
setattr(product_config_wizard, '__attribute-' + str(self.attr_size.id), str(self.value_S.id))
setattr(product_config_wizard, '__attribute-' + str(self.attr_width.id), str(self.value_A.id))
product_config_wizard.action_next_step()
new_variant = product.product_variant_ids.filtered(
lambda variant: variant.attribute_value_ids == (self.value_green + self.value_S + self.value_A)
)
self.assertNotEqual(
new_variant.id,
False,
'Error while configure product from ptoduct template form'
)
26 changes: 24 additions & 2 deletions product_configurator/wizard/product_configurator.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -777,6 +777,29 @@ def unlink(self):
self.mapped('config_session_id').unlink()
return super(ProductConfigurator, self).unlink()

def clear_values_in_future_steps(self, active_cfg_line_id):
"""Set values of fields in step 'active_cfg_line_id' to null/false"""
field_prefix = self._prefixes.get('field_prefix')
custom_field_prefix = self._prefixes.get('custom_field_prefix')

fields = self.fields_get()
active_cfg_line_fields_dict = {}

for attr_line in active_cfg_line_id.attribute_line_ids:
attr_id = attr_line.attribute_id.id
field_name = field_prefix + str(attr_id)
custom_field_name = custom_field_prefix + str(attr_id)

if field_name in fields:
if fields[field_name]['type'] == 'many2one':
active_cfg_line_fields_dict.update({field_name: False})
elif fields[field_name]['type'] == 'many2many':
active_cfg_line_fields_dict.update({field_name: []})
if custom_field_name in fields:
active_cfg_line_fields_dict.update({custom_field_name: False})

self.write(active_cfg_line_fields_dict)

@api.multi
def action_next_step(self):
"""Proceeds to the next step of the configuration process. This usually
Expand Down Expand Up @@ -820,8 +843,7 @@ def action_next_step(self):
next_step = adjacent_steps.get('next_step')

session_config_step = self.config_session_id.config_step

if session_config_step and self.state != session_config_step:
if session_config_step and str(self.state) != session_config_step:
next_step = self.config_session_id.config_step
else:
next_step = str(next_step.id) if next_step else None
Expand Down

0 comments on commit 5596b4e

Please sign in to comment.