test: nepali numbers module

[golesuman]

Description

Add tests with full test coverage

Fixes Issue

• 64

  Type of Change

Please mark the appropriate option below to describe the type of change your pull request introduces:

• [ ] Bug fix
• [ ] New feature
• [x] Enhancement
• [ ] Documentation update
• [ ] Refactor
• [ ] Other (please specify)

Checklist

• [x] I have used semantic commit messages. Examples: "fix(google): Fixed foobar bug", "feat(accounts): Added foobar feature".
• [x] I have used pre-commit hooks.
• [ ] I have added/updated the necessary documentation on README.md.
• [ ] I have updated CHANGELOG.md for the significant changes.
• [x] I have added appropriate test cases (if applicable) to ensure the changes are functioning correctly.
• [ ] My pull request has a clear title and description.

Additional Notes

[Add any additional notes or context that you think the reviewers should know about.]

By submitting this pull request, I confirm that I have read and complied with the contribution guidelines of this project.

[sugat009]

Hello @golesuman , thank you for your contribution. The tests that you have added might have been misplaced. TestNumber class is for the tests of the number package as a whole. We try to make sure that the test classes only test one particular entity, so TestNepaliNumberMethods class in the same test file might be a better choice to add your tests. Also, we follow the convention of writing commit messages of the form <type>: <verb> <what was done>. For example in your case it might be test: Add object representation and equality tests for nepalinumber.

[sugat009]

@golesuman Can you also squash your commit into one commit?

[aj3sh]

Closing this stale PR. cc: @sugat009