IntenseWebs/mattermost
3
0
Fork 0
mirror of https://github.com/mattermost/mattermost.git synced 2025-02-25 18:55:24 -06:00
Code Issues Packages Projects Releases Wiki Activity

test: Add test cases for preserving IDs in custom profile attributes

Browse Source
This commit is contained in:
Julien Tant (aider)
2025-02-12 14:57:24 -07:00
parent bbb7f3e261
commit b6fb5f274a
1 changed files with 42 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

45
server/public/model/custom_profile_attributes_test.go
View File

@@ -25,6 +25,15 @@ func TestNewCustomProfileAttributeSelectOption(t *testing.T) {
assert.Equal(t, "Test Option", option.Name)
assert.Equal(t, "#FF0000", option.Color)
})
t.Run("preserves provided ID", func(t *testing.T) {
providedID := NewId()
option := NewCustomProfileAttributesSelectOption(providedID, "Test Option", "#FF0000")
assert.Equal(t, providedID, option.ID)
assert.Equal(t, "Test Option", option.Name)
assert.Equal(t, "#FF0000", option.Color)
})
}
func TestCustomProfileAttributeSelectOptionIsValid(t *testing.T) {
@@ -142,6 +151,32 @@ func TestNewCustomProfileAttributesSelectOptionFromMap(t *testing.T) {
Color: "#FF0000",
},
},
{
name: "with provided ID",
input: map[string]any{
"ID": "existingid123456789012345678",
"Name": "Test Option",
"Color": "#FF0000",
},
expected: CustomProfileAttributesSelectOption{
ID: "existingid123456789012345678",
Name: "Test Option",
Color: "#FF0000",
},
},
{
name: "with provided lowercase id",
input: map[string]any{
"id": "existingid123456789012345678",
"name": "Test Option",
"color": "#FF0000",
},
expected: CustomProfileAttributesSelectOption{
ID: "existingid123456789012345678",
Name: "Test Option",
Color: "#FF0000",
},
},
{
name: "with non-string values",
input: map[string]any{
@@ -166,11 +201,15 @@ func TestNewCustomProfileAttributesSelectOptionFromMap(t *testing.T) {
for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
result := NewCustomProfileAttributesSelectOptionFromMap(tt.input)
// Don't compare IDs since they are randomly generated
if tt.expected.ID != "" {
// When an ID is expected, verify it matches exactly
assert.Equal(t, tt.expected.ID, result.ID)
} else {
// When no ID is provided, verify generated ID is valid
assert.True(t, IsValidId(result.ID))
}
assert.Equal(t, tt.expected.Name, result.Name)
assert.Equal(t, tt.expected.Color, result.Color)
// Verify ID is valid
assert.True(t, IsValidId(result.ID))
})
}
}