* 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 overloads to the GetAll methods in the IEventsClient Interface
* Implement GetAll* overload methods in the EventClient class to allow ApiOptions as parameter
* Fix the failling tests after refactoring the GetAll* methds
* Add the overload GetAlls methods to the IObservableEventsClient interface
* Implement the GetAll* overloads in the ObservableEventsClient class
* Fix the failling tests after refactoring the GetAll* methods
* Make sure the ApiOtions arn't null in the newly added overload methods + implement RequestsCorrectUrlWithApiOptions test
* Implement unit tests for the EventClient's GetAll* overload methods
* Add missing test cases in the ReleaseClientTest's EnsuresNonNullArguments test
* Fix the last commit
* Add the ObservableEventsClientTests class
* Add an ArgumentException check to the EnsuresArgumentsNotNull methodS + some clean up
* Events GetAll Method Integration Tests
* Implement the GetAll* IntegrationTests
* TheGetAllForAnOrganizationMethod Integrations test [WIP]
* Fix a failing integration test
* Proper way to call Organization and UserName in the integration test
* Remove .swp file
* Fixe after merging
* Add more checks to EnsuresNonNullArguments
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.
Chris Simpson