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 lib-sourcify for which it would be nice to add additional tests. We generally want to increase our test coverage. The following parts need additional tests:
The hashFunctions folder has only about 50 % coverage. We should write tests that hash some file with ipfsHash and swarmHash. It should be easy to get a high coverage above 90 % here.
SourcifyChain: There is some code regarding contracts created by factories that is uncovered. I specifically mean lines 388 - 417 and 194 - 234. These should be tested.
validation.ts and verification.ts: These are quite critical components and they have some functions and rather important branches uncovered. I won't go into details here. I think it would be sufficient to get them at least to 90 % coverage.
The text was updated successfully, but these errors were encountered:
follow-up of #1429
I found parts of
lib-sourcify
for which it would be nice to add additional tests. We generally want to increase our test coverage. The following parts need additional tests:hashFunctions
folder has only about 50 % coverage. We should write tests that hash some file withipfsHash
andswarmHash
. It should be easy to get a high coverage above 90 % here.SourcifyChain
: There is some code regarding contracts created by factories that is uncovered. I specifically mean lines 388 - 417 and 194 - 234. These should be tested.validation.ts
andverification.ts
: These are quite critical components and they have some functions and rather important branches uncovered. I won't go into details here. I think it would be sufficient to get them at least to 90 % coverage.The text was updated successfully, but these errors were encountered: