search

UTDNebula / api-tools

CLI-based tool which facilitates the scraping, parsing, and uploading of data for Nebula Labs' API.
MIT License
5 stars 13 forks source link

Add unit testing to api-tools #22

Open jpahm opened 9 months ago

jpahm commented 9 months ago

The toolkit would greatly benefit from both unit and integration testing, and ideally sooner rather than later.

We'll have to talk about how exactly we want to go about effectively performing integration testing, but unit testing should be fairly straightforward via Go's built-in testing library. I would personally be in favor of the test files being within the same package as the files being tested so that non-exported methods can be tested more easily.

Any comments on this are welcome, as usual.

TortCode commented 7 months ago

I can work on this, starting with the unit testing at least

jpahm commented 7 months ago

Yeah, no worries on integration testing yet. You've got it!

jpahm commented 2 months ago

@TortCode Are you still working on this?

jpahm commented 3 weeks ago

Reopening this since I haven't heard any updates for a while.

jpahm commented 3 weeks ago

*Also going to bump this up to an L3 since it's a rather large undertaking at this point