* add test
* [WIP]
* put logic for redirects outside of delegating handler
* change send method
* format code
* reorganized http client adapter
* change HttpClientAdapter
* rework http redirect tests - still an issue with accessing the response.RequestMessage.Content property as it is disposed
* remove some unused lines in httpclientadapter
* Reworked redirect implementation to fully clone http request and re-use it later
Now the skipped test from #874 works!
Also had to fix the new ReturnsRenamedRepository test as the ionide repo was renamed again
* Tidy up location of existing EditBranch tests
* Create RepositoryBranchesClient and move the GetBranch GetAllBranches and EditBranch methods to it, obsoleting the old ones
* Add tests for the new RepositoryBranchesClient (keeping old tests for RepositoriesClient around for now)
* Disable obsolete warning on reactive client temporarily
* Create observable repository branches client and move GetBranch, GetAllBranches, EditBranch methods to it, obsoleting the old ones
* Add tests for observable repository branches client, leave old tests in place for now
* Fix projects...
* Fix whitespace
I was trying to create a repository and I wasn't sure which parameters
were required. Following our philosophy of exposing required parameters
in the constructor, I change the `NewRepository` object to take in a
repository name and to make that property readonly.
- This also changes SearchUsersRequest to use AccountSearchType as this is a slightly
different representation of the user/org role.
- Added some tests to check existing API usage.
Added self tests to find async void methods. It found some cases where
we passed an async lambda to a method that did not accept Action<Task>
or Func<Task>. I fixed those tests.
