NoppesTheFolf / E621Client

.NET Standard wrapper for the e621 API.
MIT License
9 stars 1 forks source link

Refactor/improve testability #12

Closed GizmoCaritate closed 3 years ago

GizmoCaritate commented 3 years ago

PR to solve issue #11

This is a refactor of the code, putting an interface between the E621Client. Having this interface allows for better unit testing and mocking of the client

I also moved the static values that were present in the different partials of E621Client to a static class E621Constants. I have moved the static values from the E621ClientDefaults class to the static E621Constants class. The documentation (Readme.md). Has been updated to reflect this change. Keep in mind this is a breaking change for any users of the package.

Let me know if there are any issues or questions.