* Add release notes and bump version to 0.24
* run "build FormatCode" to fix up whitespace/formatting issues
* Fix failing Ssh key tests due to "validation exception". This key must be in use on github (under another user, most likely from these tests failing). Changed to a new SSH key and tweaked tests to reduce chance of a key being created and not destroyed
* Assignee and Assignees cant both be specified on NewIssue. We missed this one in the PR. Marked Assignee as [Obsolete] and fixed tests to use Assignees
* Fix a couple of Reactions tests that were calling the wrong client methods
* Fix timeline tests - looks like the response class has changed shape a bit, it now has an Issue object in the payload and Id field isnt present (leaving Id field there in case other timeline events do use it)
* Fix some following tests that require the test user to follow more than 1 other user
* Unskip these Event tests now because apparently they work!
* add breaking changes notes
* Update ApiErrorMessageSafe to return null for empty and whitespace strings (#1540)
* return null if ApiError.Message is empty or whitespace
* Uncomment test, which now passes
* update release notes to include PR1540
* Add "Bot" AccountType, was causing a deserialization exception when running the integration test "SearchForExcludedLanguage" (#1541)
* Update to include PR1541
* add bullets to make release notes easier to read
* markup additional code mentions in notes
* Fix grammar
fields => field
* 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.
any assertions.
Adding Received() where it was missing.
The majority of these instances were unawaited AssertEx.Throws
They are now all either awaited or replaced with Assert.Throw
Assert.Throw was used when possible and when not an option
AssertEx.Throw was awaited.
Brendan Forster