Skip to content

Commit fd841fe

Browse files
test: cover null email value in other_emails derivation
Exercises the entry?.value ?? '' fallback for malformed/missing email entries so they are skipped, closing the last uncovered patch branch. Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 6405898 commit fd841fe

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

tests/unit/components/lobbyside-widget-test.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,10 +77,16 @@ module('Unit | Component | lobbyside-widget', function () {
7777
assert.strictEqual(lobbysideCustomFields(user).other_emails, '');
7878
});
7979

80-
test('other_emails dedupes and drops blank values', function (assert) {
80+
test('other_emails dedupes and drops blank or missing values', function (assert) {
8181
const user = createUser({
8282
primaryEmailAddress: 'primary@example.com',
83-
emailAddresses: [{ value: 'work@example.com' }, { value: ' work@example.com ' }, { value: '' }, { value: 'alt@example.com' }],
83+
emailAddresses: [
84+
{ value: 'work@example.com' },
85+
{ value: ' work@example.com ' },
86+
{ value: '' },
87+
{ value: null },
88+
{ value: 'alt@example.com' },
89+
],
8490
} as unknown as Partial<UserModel>);
8591

8692
assert.strictEqual(lobbysideCustomFields(user).other_emails, 'work@example.com, alt@example.com');

0 commit comments

Comments
 (0)