Henrik Andersson
2c424448a7
Add a couple of unit tests
2015-03-19 09:57:24 +10:00
Henrik Andersson
b21fddaea3
Fix existing unit tests
2015-03-19 09:57:22 +10:00
Kristian Hellang
2508205cfd
Fixed failing tests
2015-03-18 14:33:06 +01:00
Haacked
e947fac62d
Merge branch 'master' into haacked/api-response-refactor
...
Conflicts:
Octokit.Tests/Clients/OauthClientTests.cs
2015-01-26 09:31:55 -08:00
Micah
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
Haacked
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
Kristian Hellang
21ca33817d
Fixed failing tests
2015-01-06 00:27:16 +01:00
Haacked
ff3506f3e8
Make some of the IResponse properties readonly
2015-01-04 19:13:32 -08:00
Haacked
4c8bab20eb
IApiResponse no longer inherits IResponse
2015-01-04 18:58:53 -08:00
Haacked
09d5c236a4
Rename IResponse<T> to IApiResponse
2015-01-04 18:33:48 -08:00
Haacked
3b39d0d95a
Fixed some async void tests
...
These tests weren't getting run.
2014-11-07 15:29:07 -08:00
Paul Betts
d731f1bd97
More jacked up tests
2014-04-30 21:47:34 -07:00
pmacnaughton
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
pmacnaughton
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
Brendan Forster
43be7994b0
corrected impacted tests
2014-02-18 21:13:12 +11:00
Brendan Forster
a3190aa0bb
ok, now the tests should pass
2014-02-03 21:18:43 +11:00
Brendan Forster
3f306fb9b0
address some documentation feedback
2014-02-03 19:16:27 +11:00
Josh Sullivan
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
Josh Sullivan
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
Josh Sullivan
6323b62f71
Fixed some goofed up test names
2014-02-03 18:31:06 +11:00
Josh Sullivan
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
