* add reactions to issue response payload
* add reactions to commit comment payload
* add reactions to review comment payload
* create tests for issue client
* tests for commitcomment client
* tests for pull request review comment
* change observable tests
* simplify strings
* remove unnecessary clients
* change integration tests to retrieve all reaction types
* create integration test for issue comment client
* fix merge conflicts
* fix merge conflicts
* gets tests passing again
* fix some reaction integration tests
* Fixup unit tests wth preview accepts header
Also applied preview header to a couple of repositoryId based calls that were added by another PR
* Fixup unit tests wth preview accepts header
Also applied preview header to a couple of repositoryId based calls that were added by another PR
* Rework reaction payload tests for IssueComments to handle Get, GetAllForIssue and GetAllForRepository calls
* [WIP] reaction payload tests
* Rework reaction payload tests for IssueComments to handle Get, GetAllForIssue and GetAllForRepository calls
* Rework reaction payload tests for Issues client
* Rework reaction payload tests for PullRequestReviews client
* Rework reaction payload tests for CommitComments client
* Revert "[WIP] reaction payload tests"
This reverts commit a6179b0f21a3ddfe36bfd3ae5eafae0e69b52252.
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.
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.
Martin Scholz