498c2aa0a5
Add ability to pass ApiOptions to PullRequestsClient.Files ( #2553 )
...
* Add ability to pass ApiOptions to PullRequestsClient.Files
* Change Reactive PullRequestClient + Fix Test Mocking
2022-08-28 15:12:27 +01:00
2f7bd00dd6
feat: Removing accept header previews ( #2515 )
2022-08-01 09:37:23 -05:00
287861e4a7
rewrite the setup of responses to use a standard helper function ( #2177 )
2020-04-14 11:30:15 -03:00
4b0e7255b1
remove all usages of TaskFactory in test suite ( #2164 )
2020-03-31 16:57:16 -03:00
fafbf33b78
Implement Draft Pull Requests ( #2009 )
...
* Add draft PR preview header
* Add Draft property to models
* Update pull requests client and tests to use draft PR accept header
* Update observable pull requests client and tests to use draft PR accept header
* Add integration tests to create and retrieve draft pull requests
2019-09-22 13:38:56 -03:00
23b25f5922
add GraphQL Node ID's to response models ( #1806 )
...
* add node_id to Deployments payloads (Deployment and DeploymentStatus and Account/User/Organization)
* add node_id to gist responses
* add node_id to Git Blob
* add node_id to Git Commit response
* add node_id to GitReference response
* add node_id to everything that inherits GitReference
* add node_id to Issue
* add node_id to IssueComment/IssueEvent
* add node_id to Label
* add node_id to Milestone
* add node_id to Project/ProjectCard/ProjectColumn
* add node_id to Reaction
* add node_id to Release/ReleaseAsset
* add node_id to Team
* add node_id to Repository.RepositoryContributor/RepositoryInvitation/RepositoryTag
* add node_id to Commit related responses
* Add node_id to PullRequest related responses
* Add node_id to any response models it was found to be missing, based on auditing integration test responses
* remove unused test variable that was using a response ctor
* fix tests that need to handle node_id now
* Committer is a request object as well as response, so make nodeId optional
* fix test
2018-06-29 15:41:50 +10:00
e430a9ef14
Add PreviousFileName to the PullRequestFile model ( #1770 )
...
* Add PreviousFileName to the PullRequestFile model
* fix white space issue
2018-02-24 06:47:56 +10:00
0a9478b2e9
added new unit tests
2016-06-16 19:45:26 +07:00
9d6eada6ab
fixed integration tests
2016-05-25 15:05:29 +07:00
04a91d60f9
added unit tests
2016-05-25 14:00:04 +07:00
36c08248fa
New convention test was added. Now each "TheCtor" class must have "EnsuresNonNullArguments" method were throwing of ArgumentNullException from client constructor class should be tested.
...
All unused "using" directives were removed.
2016-04-18 12:51:35 +07:00
56a0970d15
Redundant string.Format() call was removed.
2016-03-22 18:57:12 +07:00
870a4e817a
Rename Message to CommitMessage
...
According to the docs
(https://developer.github.com/v3/pulls/#merge-a-pull-request-merge-button ),
this should be sent as "commit_message" thus we need to name it
`CommitMessage`
Fixes #913
2015-09-27 14:45:11 -07:00
e1ea7c30ad
fix expected exceptions
2015-05-19 11:51:27 +03:00
8079f3012c
O-PullRequestReviewComments, O-PullRequest
2015-05-19 11:35:08 +03:00
d8cef691b7
Add SHA and make message optional
2015-05-19 07:53:01 +10:00
189ae42c3a
Update tests
2015-03-24 00:12:54 +10:00
2c424448a7
Add a couple of unit tests
2015-03-19 09:57:24 +10:00
b21fddaea3
Fix existing unit tests
2015-03-19 09:57:22 +10:00
2508205cfd
Fixed failing tests
2015-03-18 14:33:06 +01:00
e947fac62d
Merge branch 'master' into haacked/api-response-refactor
...
Conflicts:
Octokit.Tests/Clients/OauthClientTests.cs
2015-01-26 09:31:55 -08:00
c3a67ae777
Creates constructors for all Models.Response.
...
Resolves https://github.com/octokit/octokit.net/issues/677 .
Removes obscolete properties (gravatar).
Makes Models.Response properties all be protected (most were already).
2015-01-24 16:07:03 -08:00
7fdbbf0ae3
Merge branch 'master' into haacked/api-response-refactor
...
Conflicts:
Octokit.Tests/Reactive/ObservableMilestonesClientTests.cs
Octokit.Tests/Reactive/ObservablePullRequestReviewCommentsClientTests.cs
Octokit.Tests/Reactive/ObservablePullRequestsClientTests.cs
Octokit.Tests/Reactive/ObservableRepositoriesClientTests.cs
Octokit/Octokit-Mono.csproj
Octokit/Octokit-MonoAndroid.csproj
Octokit/Octokit-Monotouch.csproj
Octokit/Octokit-Portable.csproj
Octokit/Octokit-netcore45.csproj
2015-01-18 12:35:17 -08:00
21ca33817d
Fixed failing tests
2015-01-06 00:27:16 +01:00
ff3506f3e8
Make some of the IResponse properties readonly
2015-01-04 19:13:32 -08:00
4c8bab20eb
IApiResponse no longer inherits IResponse
2015-01-04 18:58:53 -08:00
09d5c236a4
Rename IResponse<T> to IApiResponse
2015-01-04 18:33:48 -08:00
3b39d0d95a
Fixed some async void tests
...
These tests weren't getting run.
2014-11-07 15:29:07 -08:00
d731f1bd97
More jacked up tests
2014-04-30 21:47:34 -07:00
7e3089ba04
From Assert.Throw to AssertEx.Throw
...
These should have been in the previous cleanup. Premature pull request
without a WIP tag, yay!
2014-02-26 13:44:31 -07:00
d58bfc6a9a
Fixing some AssertEx.Throws that were not
...
awaited by awaiting them. Also removed `async`/`await` from test lambdas
whenever they were unnecessary.
2014-02-26 11:39:21 -07:00
43be7994b0
corrected impacted tests
2014-02-18 21:13:12 +11:00
a3190aa0bb
ok, now the tests should pass
2014-02-03 21:18:43 +11:00
3f306fb9b0
address some documentation feedback
2014-02-03 19:16:27 +11:00
ad216b5c1a
NewPullRequest required parameters altered
...
NewPullRequest constructor only takes required params. Each required
parameter is now a readonly property.
2014-02-03 18:58:46 +11:00
654622371a
PullRequest model now contains all of necessary fields
...
- Initial integration test for pull requests (can't seem to do much
without the tree api)
- Added Observable unit tests for the rest of the pull request client
2014-02-03 18:58:44 +11:00
6323b62f71
Fixed some goofed up test names
2014-02-03 18:31:06 +11:00
c81d8d08f7
Tested out all of the existing Pull Request code
...
Still missing integration tests for Pull Requests, though. I need to do
a bit more research before I can start to tackle that one.
2014-02-03 18:31:05 +11:00
Gamer025