You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I found parts of the server for which it would be nice to add additional tests. We generally want to increase our test coverage. The following parts need additional tests (all inside server/src/server folder):
server.ts: Rate limiting (lines 194 - 234) should be tested
session.ts: Memory session is not tested
controllers/verification/session-state/session-state.handlers.ts: In this file, we should test adding remote files (line 35), and the addInputContractEndpoint is completely uncovered.
services/compiler/lambda and services/compiler/lambda-with-fallback: These are not covered at all.
services/storageServices/AllianceDatabaseService.ts is not really tested. Generally, we want to have separate tests for the different storage services. It would be nice to add these as part of this issue.
The text was updated successfully, but these errors were encountered:
follow-up of #1429
I found parts of the server for which it would be nice to add additional tests. We generally want to increase our test coverage. The following parts need additional tests (all inside
server/src/server
folder):server.ts
: Rate limiting (lines 194 - 234) should be testedsession.ts
: Memory session is not testedcontrollers/verification/session-state/session-state.handlers.ts
: In this file, we should test adding remote files (line 35), and theaddInputContractEndpoint
is completely uncovered.services/compiler/lambda
andservices/compiler/lambda-with-fallback
: These are not covered at all.services/storageServices/AllianceDatabaseService.ts
is not really tested. Generally, we want to have separate tests for the different storage services. It would be nice to add these as part of this issue.The text was updated successfully, but these errors were encountered: