26 Commits

Author SHA1 Message Date
Chris Simpson
2f7bd00dd6 feat: Removing accept header previews (#2515) 2022-08-01 09:37:23 -05:00
Brendan Forster
287861e4a7 rewrite the setup of responses to use a standard helper function (#2177) 2020-04-14 11:30:15 -03:00
Brendan Forster
4b0e7255b1 remove all usages of TaskFactory in test suite (#2164) 2020-03-31 16:57:16 -03: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
aedampir@gmail.com
a2df9a781b added new unit tests 2016-06-16 17:02:04 +07:00
aedampir@gmail.com
4c36a9757e fixed all remarks 2016-05-24 21:29:17 +07:00
aedampir@gmail.com
9b68b788d6 fixed errors in tests 2016-05-24 14:46:10 +07:00
aedampir@gmail.com
fe2fd53991 fixed inconsistency in tests 2016-05-23 15:39:07 +07:00
aedampir@gmail.com
b0a2cfe987 added unit tests 2016-05-22 18:58:27 +07:00
aedampir@gmail.com
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
Brendan Forster
80719c0033 aaaand format the code 2015-11-04 13:38:51 -08:00
Mordechai Zuber
e1ea7c30ad fix expected exceptions 2015-05-19 11:51:27 +03:00
Mordechai Zuber
b36e133049 O-Issues, O-Milestones, O-OrgMembers 2015-05-19 11:32:10 +03:00
Henrik Andersson
189ae42c3a Update tests 2015-03-24 00:12:54 +10: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
Paul Betts
d731f1bd97 More jacked up tests 2014-04-30 21:47:34 -07:00
Amy Palamountain
36427ea173 Merge branch 'master' into ammeep/statistics-api 2014-02-18 16:05:13 +13:00
Amy Palamountain
6a66bf9a1e Observable Client tests 2014-02-16 17:04:15 +13:00
Peter MacNaughton
b8c18e2da6 Fixing some unit tests that were not doing
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.
2014-02-09 01:12:31 -07:00
Kristian Hellang
ae23a2ae7a Fixed minor namespace and whitespace issues 2013-12-01 23:42:04 +01:00
Haacked
3c3febaf77 Remove use of .Wait() from tests 2013-11-05 16:02:33 -08:00
Keith Dahlby
e549376bb8 Add ObservableMilestoneClient unit tests 2013-10-31 22:51:10 -05:00