Skip to content

Commit 2417b39

Browse files
committed
💚 validate line numbers and valid two-letter abbrevation
1 parent c5c547c commit 2417b39

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

‎scripts/check-delegates-test.mjs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@ const uppercaseLatin = `Chris de Almeida (CDA)\nRob Palmer (ç«„)\nUjjwal Sharma
77
const twoLetter = `Chris de Almeida (CDA)\nRob Palmer (RP)\nUjjwal Sharma (USA)`;
88
const threeLetter = `Chris de Almeida (CDA)\nRob Palmer (ROBPALMER)\nUjjwal Sharma (USA)`;
99
const duplicate = `Chris de Almeida (CDA)\nRob Palmer (RPR)\nUjjwal Sharma (USA)\nUjjwal Sharma (USA)`;
10-
const valid = `Chris de Almeida (CDA)\nRob Palmer (RPR)\nUjjwal Sharma (USA)`;
10+
const valid = `Chris de Almeida (CDA)\nMichael Ficarra (MF)\nRob Palmer (RPR)\nUjjwal Sharma (USA)`;
1111

12-
assert.throws(() => checkDelegates(lex), { message: /Not in lexicographic order/});
12+
assert.throws(() => checkDelegates(lex), { message: 'Line 3: Not in lexicographic order.'}); // also validates expected line number
1313
assert.throws(() => checkDelegates(missing), { message: /Missing abbreviation for/});
1414
assert.throws(() => checkDelegates(uppercaseLatin), { message: /Abbreviations must be all uppercase Latin letters/});
1515
assert.throws(() => checkDelegates(twoLetter), { message: /not in allowlist. New delegate abbreviations must be three letters/});

0 commit comments

Comments
 (0)