TBD54566975 / dwn-sdk-js

Decentralized Web Node (DWN) Reference implementation
https://identity.foundation/decentralized-web-node/spec/
Apache License 2.0
326 stars 105 forks source link

Increase code coverage (even if just by a little bit) #136

Closed thehenrytsai closed 1 month ago

thehenrytsai commented 2 years ago

Background:

The code coverage goal is 100% for this repo, so any PR adding any test to increase the overall code coverage would be super useful!

Task Details:

To know what code hasn't been covered:

  1. Run npm run test:node
  2. Ignore the code coverage summary at the end of the run, there is a better way:
  3. Use MacOS Finder or Windows Explorer to open the coverage/index.html page, it will show you precisely what line of code is not covered

Files that can be ignored as they will likely be undergoing drastic changes:

Picking Up This Issue:

Questions:

Resources:

Remember, communication is key! If you have any questions or face any challenges, we're here to help so please don't hesitate to reach out.

Good Luck! 🍁

alexlemons commented 1 year ago

PR to test utils/object/removeUndefinedProperties: 154

tux7P commented 1 year ago

picking this up

EbonyLouis commented 1 year ago

@tux7P Yay! I'll assign it to you now.

I'd also like to note that Hacktoberfest officially kicks off on October 1st. Contributions made before the event may not be eligible for the challenge. However, feel free to start working on it, but I'd recommend holding off on submitting your PR until the event begins.

to be eligible for Hacktoberfest's official prizes make sure you're registered, registration is now open. Reach out if you need any help 🥳

Good Luck 🍁

EbonyLouis commented 1 year ago

More than one person can work on this issue, feel free to comment "picking this up" if it interest you! 🍁

shobitb commented 1 year ago

@EbonyLouis, picking this up!

EbonyLouis commented 1 year ago

Heyy @tux7P how are things going with this issue? Is this something you still want to work on?

Tushar98644 commented 1 year ago

@EbonyLouis picking this up !

ANIRUDH-333 commented 1 month ago

picking this up

ANIRUDH-333 commented 1 month ago

.take

github-actions[bot] commented 1 month ago

The issue you are trying to assign to yourself is already assigned.

ANIRUDH-333 commented 1 month ago

Can I still pick this up? @EbonyLouis

EbonyLouis commented 1 month ago

@ANIRUDH-333, yes! I just assigned it to you thank you for contributing ✨