Merge pull request #4973 from edx/tim/single-group-is-ok
Update test_group_configuration_validation on single group allowed
This commit is contained in:
@@ -642,9 +642,9 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
When I set a name
|
||||
And I delete the name of one of the groups and try to save
|
||||
Then I see error message "All groups must have a name"
|
||||
When I delete the group without name and try to save
|
||||
Then I see error message "Please add at least two groups."
|
||||
When I add new group and try to save
|
||||
When I delete all the groups and try to save
|
||||
Then I see error message "There must be at least one group."
|
||||
When I add a group and try to save
|
||||
Then I see the group configuration is saved successfully
|
||||
"""
|
||||
def try_to_save_and_verify_error_message(message):
|
||||
@@ -668,7 +668,9 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
config.name = "Name of the Group Configuration"
|
||||
config.groups[1].name = ''
|
||||
try_to_save_and_verify_error_message("All groups must have a name.")
|
||||
config.groups[1].remove()
|
||||
config.groups[0].remove()
|
||||
config.groups[0].remove()
|
||||
try_to_save_and_verify_error_message("There must be at least one group.")
|
||||
config.add_group()
|
||||
|
||||
# Save the configuration
|
||||
@@ -678,7 +680,7 @@ class GroupConfigurationsTest(ContainerBase, SplitTestMixin):
|
||||
config,
|
||||
name="Name of the Group Configuration",
|
||||
description="Description of the group configuration.",
|
||||
groups=["Group A", "Group B"]
|
||||
groups=["Group A"]
|
||||
)
|
||||
|
||||
def test_group_configuration_empty_usage(self):
|
||||
|
||||
Reference in New Issue
Block a user