12 Commits

Author SHA1 Message Date
Brendan Forster
eb65e090ca next batch of test cleanup (#2004) 2019-09-20 10:41:04 -03:00
Ryan Gribble
db74c3b4ad Fix async tests (#1631)
* Fix up Assert.ThrowsAsync tests to actually await the call

* ... and it even picked up a missing null check... pay day!
2017-07-23 09:03:10 +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
Martin Scholz
5b9e23c2fb Fix timeout getting multiple repositories (#1411)
* add test

* [WIP]

* put logic for redirects outside of delegating handler

* change send method

* format code

* reorganized http client adapter

* change HttpClientAdapter

* rework http redirect tests - still an issue with accessing the response.RequestMessage.Content property as it is disposed

* remove some unused lines in httpclientadapter

* Reworked redirect implementation to fully clone http request and re-use it later
Now the skipped test from #874 works!
Also had to fix the new ReturnsRenamedRepository test as the ionide repo was renamed again
2016-08-15 06:57:50 +10:00
Alexander Efremov
db839f58d4 Redundant braces of argument list were removed (#1277) 2016-04-21 16:23:42 +10:00
aedampir@gmail.com
79c38b4822 Redundant member initializer was removed.
Unused "using" directives were removed.
2016-03-23 12:23:30 +07:00
aedampir@gmail.com
6133210b97 Inconsistent modifiers declaration order was fixed. 2016-03-22 19:13:24 +07:00
Brendan Forster
80719c0033 aaaand format the code 2015-11-04 13:38:51 -08:00
Phil Haack
5be1cae361 🎨 Really anal fix-ups of extra newlines
Not worth commenting on in a code review, so I fixed it myself.
2015-07-17 10:17:21 -07:00
Brendan Forster
eafd63d768 oops, corrected the test 2015-06-05 16:57:27 +09:30
Brendan Forster
3acdd10be4 deprecate the optional AllowAutoRedirect parameter 2015-05-31 22:44:41 +09:30
Darrel
ea9f3a4761 Fixed issues with RedirectHandler and added tests 2015-05-31 18:08:37 +09:30