Skip to content

Commit 8f2c269

Browse files
authored
fix: auto assign dependabot[bot] (#1306)
Signed-off-by: Egbaiyelo <moteniolaegbaiyelo@trentu.ca>
1 parent 41638f9 commit 8f2c269

File tree

2 files changed

+30
-1
lines changed

2 files changed

+30
-1
lines changed

.github/scripts/helpers/validation.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ function isNonNegativeInteger(value) {
3232
* @returns {boolean}
3333
*/
3434
function isSafeSearchToken(value) {
35-
return typeof value === 'string' && /^[a-zA-Z0-9._/-]+$/.test(value);
35+
return typeof value === 'string' && /^[a-zA-Z0-9._/-]+(\[bot\])?$/.test(value);
3636
}
3737

3838
/**

.github/scripts/tests/test-api.js

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ const {
1515
hasLabel,
1616
} = require('../helpers/api');
1717
const { LABELS } = require('../helpers/constants');
18+
const { isSafeSearchToken } = require('../helpers/validation');
1819

1920
// =============================================================================
2021
// MOCK FACTORY
@@ -457,6 +458,34 @@ const unitTests = [
457458
return calls.labelsRemoved.length === 0 && calls.labelsAdded.length === 0;
458459
},
459460
},
461+
462+
// ---------------------------------------------------------------------------
463+
// SafeSearchToken
464+
// ---------------------------------------------------------------------------
465+
{
466+
name: 'isSafeSearchToken: dependabot[bot] → true',
467+
test: () => isSafeSearchToken('dependabot[bot]') === true,
468+
},
469+
{
470+
name: 'isSafeSearchToken: string with spaces → false',
471+
test: () => isSafeSearchToken('bad username') === false,
472+
},
473+
{
474+
name: 'isSafeSearchToken: string with bad characters → false',
475+
test: () => isSafeSearchToken('bad<username>') === false,
476+
},
477+
{
478+
name: 'isSafeSearchToken: string with bad characters → false',
479+
test: () => isSafeSearchToken('bad;username') === false,
480+
},
481+
{
482+
name: 'isSafeSearchToken: string with brackets but not bot inside → false',
483+
test: () => isSafeSearchToken('bad[admin]') === false,
484+
},
485+
{
486+
name: 'isSafeSearchToken: string with multiple brackets → false',
487+
test: () => isSafeSearchToken('bad[[admin]') === false,
488+
},
460489
];
461490

462491
// =============================================================================

0 commit comments

Comments
 (0)