Refactor client into single class

HeloiseS / atlasapiclient

1 stars 0 forks source link

Refactor client into single class #2

Open jackleland opened 2 months ago

jackleland commented 2 months ago

The client code is made up of lots of classes that do a single thing with underlying core functionality. I reckon this is probably better organised as a single Client class with different methods, one for each of the different data types that should be returned.

jackleland commented 16 hours ago

This is now the priority, this can be approached with the following two steps:

[ ] Write tests for existing calls to this from psat-server, using mocks, so that we can know if functionality changes
[ ] Snapshots would be good too
[ ] Draw up a plan for the proposed changes to the apiclient
[ ] Draw up a plan for folding St3ph3n into the apiclient