feat(TeamRepository): Prevent removal of MLS from supported protocols if it was previously supported #6888
Annotations
4 errors and 10 warnings
|
TeamRepository › getTeamSupportedProtocols › returns team supported protocols from mls feature config:
src/script/team/TeamRepository.test.ts#L152
expect(received).toEqual(expected) // deep equality
- Expected - 0
+ Received + 1
Array [
"proteus",
"mls",
+ "mls",
]
at Object.toEqual (src/script/team/TeamRepository.test.ts:152:25)
|
|
TeamRepository › getTeamSupportedProtocols › returns proteus if supported protocols field does not exist on mls feature:
src/script/team/TeamRepository.ts#L543
TypeError: teamSupportedProtocols is not iterable
at TeamRepository.teamSupportedProtocols [as getTeamSupportedProtocols] (src/script/team/TeamRepository.ts:543:18)
at Object.getTeamSupportedProtocols (src/script/team/TeamRepository.test.ts:194:34)
|
|
TeamRepository › getTeamSupportedProtocols › returns proteus if supported protocols on mls feature config is an empty list:
src/script/team/TeamRepository.test.ts#L210
expect(received).toEqual(expected) // deep equality
- Expected - 1
+ Received + 1
Array [
- "proteus",
+ "mls",
]
at Object.toEqual (src/script/team/TeamRepository.test.ts:210:25)
|
|
test
Process completed with exit code 1.
|
|
test:
server/config/client.config.ts#L82
No magic number: 4
|
|
test:
server/config/client.config.ts#L85
No magic number: 500
|
|
test:
server/config/client.config.ts#L86
No magic number: 4
|
|
test:
server/config/client.config.ts#L87
No magic number: 8
|
|
test:
server/config/index.ts#L32
Caution: `dotenv` also has a named export `load`. Check if you meant to write `import {load} from 'dotenv-extended'` instead
|
|
test:
server/config/server.config.ts#L116
No magic number: 21080
|
|
test:
server/routes/RedirectRoutes.ts#L26
Caution: `express` also has a named export `Router`. Check if you meant to write `import {Router} from 'express'` instead
|
|
test:
server/routes/Root.ts#L29
Caution: `maxmind` also has a named export `open`. Check if you meant to write `import {open} from 'maxmind'` instead
|
|
test:
server/routes/Root.ts#L29
Caution: `geolite2` also has a named export `paths`. Check if you meant to write `import {paths} from 'geolite2'` instead
|
|
test:
server/routes/error/ErrorRoutes.ts#L26
Caution: `express` also has a named export `Router`. Check if you meant to write `import {Router} from 'express'` instead
|