Keeping the local counts in sync with the server counts is now working, I believe, although more testing is needed.

What is interesting is that it seems that if you ask to move a subscription from a folder to a folder called "" this will break your ability to use the API of TheOldReader. If I subsequently attempted to stream article IDs after doing that it gave me a 404 during the stream. I had to "low level" unsubscribe from the feed before I could make use of the stream API again.

I think this means I should tweak the move method in OldAS so that it handles an empty string as a "remove from folder" operation (currently it only does that if you pass None as that parameter, or omit it); but I might also want to drop a line to the ToR folk and point out what looks to be a wee bug in their API.

Also see davep/oldas#27 which relates to this. Right now the PR won't let you move a subscription outwith of any folder (because of the API problem I ran into), but I think this should be safe and possible after I tweak OldAS to handle an empty folder name.

I could also test/fix this by passing None when it's an empty folder name, but I feel it's safer to fix it in the core library.