Commit Graph

11 Commits

Author SHA1 Message Date
Chris Simpson 251c3a26fc feat: Adds 4 new Reaction delete routes and obsoleting existing routes (#2494) 2022-07-22 11:28:30 -05:00
Brendan Forster ce5ea64428 migrate to dotnet-format tool (#2016)
* migrate to new dotnet-format tool in build script (run on local build)
* reformat code
2019-09-24 11:34:50 -03:00
Henrik Andersson 43381c4a53 Pagination support to Reactions Clients (#1948)
* Add pagination to *CommitCommentReactionsClient

* Add unit tests for *CommitCommentReactionsClient

* Add integration tests for *CommitCommentReactionsClient

* Add pagination to *IssueCommentReactionsClient

* Add unit tests for *IssueCommentReactionsClient

* Add integration tests for *IssueCommentReactionsClient

* Add pagination to *IssueReactionsClient

* Add unit tests for *IssueReactionsClient

* Add integration tests for *IssueReactionsClient

* Add pagination to *PullRequestReviewCommentReactionsClient

* Add unit tests for *PullRequestReviewCommentReactionsClient

* Add integration tests for *PullRequestReviewCommentReactionsClient

* Remove rogue using statement and whitespace

* Add null check tests for GetAll with repositoryid overload
2019-02-27 21:29:33 +10:00
Ryan Gribble 3c818934b8 Release v0.24 - A Sight For Sore Eyes (#1539)
* 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
2017-01-17 18:56:55 +10:00
Brendan Forster 79a4749d5d Merge pull request #1384 from dampir/add-repo-id-issue-reactions-client
Add repositoryId overloads to methods on I(Observable)IssueReactionsClient
2016-07-06 10:12:03 +10:00
Martin Scholz f7a464cc3a foreach loop for reaction types in integration tests 2016-06-23 08:35:04 +02:00
Martin Scholz bdc222132b test if all reactions can be created 2016-06-22 08:47:34 +02:00
aedampir@gmail.com 9ca231bb27 added new integration tests 2016-06-15 18:18:41 +07:00
maddin2016 ccd9654727 fix some grammar and tests 2016-06-09 15:06:02 +02:00
maddin2016 af2894364f Fix unit tests 2016-06-09 08:51:48 +02:00
maddin2016 eff9ef4a56 add integration tests for reaction clients 2016-06-07 09:16:25 +02:00