Commit Graph

4330 Commits

Author SHA1 Message Date
Brendan Forster 0ebda0de81 reactions are in! 2016-07-11 17:44:56 -07:00
Brendan Forster 3268d38254 refactor fields into properties 2016-07-11 17:01:22 -07:00
Brendan Forster 8e75d8652f a bit more grouping 2016-07-11 17:01:22 -07:00
Brendan Forster e89f1c6277 we now exclude this The Right Way 2016-07-11 17:01:22 -07:00
Brendan Forster 00c23934d3 words are hard 2016-07-11 17:01:22 -07:00
Brendan Forster 2db1ae4701 and have a peek at the obsoleted endpoints 2016-07-11 17:01:22 -07:00
Brendan Forster 1ee57b9c98 sprinkle a bit of SRP on it 2016-07-11 17:01:22 -07:00
Brendan Forster 1f883a5f66 a better way to detect legacy APIs 2016-07-11 17:01:22 -07:00
Brendan Forster 4e67f9dd8c a bit of cleanup 2016-07-11 17:01:22 -07:00
Brendan Forster a32817d67a moving scraping code out for readability 2016-07-11 17:01:22 -07:00
Brendan Forster 36ac863ef3 wrote a simple test to audit the current implementation of the octokit API 2016-07-11 17:01:22 -07:00
Brendan Forster 35ec8acc4f ported HtmlAgilityPack-based test which retrieves all the documented APIs 2016-07-11 17:01:22 -07:00
Martin Scholz 72e30a7f90 Add reactions to issue, commit comment and review payload (#1405)
* 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.
2016-07-08 20:29:08 +10:00
Brendan Forster 3e18580de2 Merge pull request #1379 from dampir/add-repo-id-issue-comments-client
Add repositoryId overloads to methods on I(Observable)IssueCommentsClient
2016-07-07 13:47:22 +10:00
Brendan Forster 7453899b51 Merge pull request #1380 from dampir/add-repo-id-issues-client
Add repositoryId overloads to methods on I(Observable)IssuesClient
2016-07-07 10:26:33 +10:00
Brendan Forster a25e537f3b Merge pull request #1369 from dampir/add-repo-id-tree-client
Add repositoryId overloads to methods on I(Observable)TreesClient
2016-07-07 10:19:47 +10:00
Brendan Forster 6f22b7197a Merge pull request #1371 from dampir/add-repo-id-statistics-client
Add repositoryId overloads to methods on I(Observable)StatisticsClient
2016-07-07 10:09:53 +10:00
aedampir@gmail.com 3a68f87fdf removed accidentally added namespace 2016-07-07 03:36:34 +07:00
aedampir@gmail.com 567220c28b fixed error in tests 2016-07-07 03:35:03 +07:00
aedampir@gmail.com d1c89b1ffa removed <returns> tags 2016-07-07 03:33:00 +07:00
aedampir@gmail.com 36c20079a7 removed <retuns> tags 2016-07-07 03:10:31 +07:00
aedampir@gmail.com 20df7ed711 removed <retunrs> tags 2016-07-07 03:08:54 +07:00
aedampir@gmail.com 91073ec127 removed <retuns> tags 2016-07-07 03:06:29 +07:00
Brendan Forster 51d2167491 Merge pull request #1382 from dampir/add-repo-id-commit-comment-reactions-client
Add repositoryId overloads to methods on I(Observable)CommitCommentReactionsClient
2016-07-06 11:39:12 +10:00
Brendan Forster 0521477e4d Merge pull request #1414 from TattsGroup/ctor-convention-test
Add convention test to ensure all response models have a public parameterless ctor
2016-07-06 11:05:56 +10:00
Brendan Forster 85e339d2fa Merge pull request #1387 from dampir/add-repo-id-repo-collaborators-client
Add repositoryId overloads to methods on I(Observable)RepoCollaboratorsClient
2016-07-06 10:40:42 +10:00
Brendan Forster 8f2cea5609 Merge pull request #1394 from dampir/add-repo-id-pull-requests-client
Add repositoryId overloads to methods on I(Observable)PullRequestsClient
2016-07-06 10:12:35 +10:00
Brendan Forster 3032c74b93 Merge pull request #1385 from dampir/add-repo-id-pull-request-review-comment-reactions-client
Add repositoryId overloads to methods on I(Observable)PullRequestReviewCommentReactionsClient
2016-07-06 10:12:12 +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
Brendan Forster 671db74aa1 Merge pull request #1368 from dampir/add-repo-id-deployment-status
Add repositoryId overloads to methods on I(Observable)DeploymentStatusClient
2016-07-06 10:11:53 +10:00
Brendan Forster 082c615aab Merge pull request #1372 from dampir/add-repo-id-watched-client
Add repositoryId overloads to methods on I(Observable)WatchedClient
2016-07-06 10:11:08 +10:00
Brendan Forster 722f0b447d Merge pull request #1393 from dampir/add-repo-id-issues-labels-client
Add repositoryId overloads to methods on I(Observable)IssuesLabelsClient
2016-07-06 10:07:28 +10:00
Brendan Forster c4f8c888b9 Merge pull request #1395 from dampir/add-repo-id-pull-request-review-comments-client
Add repositoryId overloads to methods on I(Observable)PullRequestReviewCommentsClient
2016-07-06 10:07:07 +10:00
Brendan Forster 5ff80c9e27 Merge pull request #1383 from dampir/add-repo-id-issue-comment-reactions-client
Add repositoryId overloads to methods on I(Observable)IssueCommentReactionsClient
2016-07-06 10:06:11 +10:00
aedampir@gmail.com a52b13deff burned <returns> tags 2016-07-06 03:04:52 +07:00
aedampir@gmail.com bfb1b3f991 burned<returns> tags 2016-07-06 02:58:01 +07:00
aedampir@gmail.com 0ff2325702 Merge branch 'add-repo-id-repo-collaborators-client' of https://github.com/dampir/octokit.net.git 2016-07-06 02:56:59 +07:00
aedampir@gmail.com e5d8297b13 added integration tests 2016-07-06 02:56:54 +07:00
aedampir@gmail.com 5ab9523044 fixed some docs
adden null checks
2016-07-06 02:48:09 +07:00
aedampir@gmail.com 91bcb128a6 burned <returns> tags 2016-07-06 02:41:13 +07:00
aedampir@gmail.com 26ba6ddf25 burned <retunrs> tags 2016-07-06 02:38:05 +07:00
aedampir@gmail.com 0c1a4e0d00 burned <returns> tags 2016-07-06 02:35:20 +07:00
aedampir@gmail.com eb569736f8 burned <returns> tags 2016-07-06 02:32:42 +07:00
aedampir@gmail.com b9e82086b5 burned <returns> tags 2016-07-06 02:27:38 +07:00
aedampir@gmail.com 0029e01457 burned <returns> tags 2016-07-06 02:21:49 +07:00
Ryan Gribble c8072391d4 Make consistent with other classes 2016-07-05 23:45:20 +10:00
Ryan Gribble e3fa865381 add convention test to ensure all response models have a public parameterless ctor
2 model classes were missing one - technically not needed for these 2 classes due to their api calls being basic data types that then get populated into these objects, but it's easiest to just add them to these 2 classes so the test can pass on all response models
2016-07-05 23:32:40 +10:00
Alexander Efremov e701ae763c Add repositoryId overloads to methods on I(Observable)StarredClient (#1389)
* modified XML docs

* updated xml docs

* added new overloads

* added new unit tests

* added new integration tests

* revert xml back a little

* fixed failed tests

* cleared <returns> tags

* removed <returns> tags

* remove <returns> tags
2016-07-03 22:01:31 +10:00
Brendan Forster 9cf39defc1 Merge pull request #1373 from dampir/add-repo-id-issues-events-client
Add repositoryId overloads to methods on I(Observable)IssuesEventsClient
2016-07-01 09:53:22 +10:00
Brendan Forster 51eb848174 Merge pull request #1408 from octokit/serialization-unit-tests
Add some unit tests for enum serialization
2016-07-01 09:04:04 +10:00