hectorcorrea / marcli

MARC command line parser
MIT License
117 stars 8 forks source link

Adding pkg/marc unit testing #13

Closed blackerby closed 1 year ago

blackerby commented 1 year ago

This pull request adds unit testing to pkg/marc. It adds five files and one directory to the pkg/marc root directory. Those files and directory are:

The PR also includes an addition to the .gitignore file, adds https://github.com/google/go-cmp as a dependency for testing, fixes some compiler warnings and typos in the comments, and lightly refactors some of the library files.

Some behavior remains untested, primarily the error handling code.

Closes #12

hectorcorrea commented 1 year ago

I probably won't look closely into this until after Code4Lib, but it looks good. Thanks for submitting it.

hectorcorrea commented 1 year ago

@blackerby sorry for the long delay. Thank you so much for the work on this PR.

blackerby commented 1 year ago

Happy to contribute!