Commit Graph

11 Commits

Author SHA1 Message Date
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
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
Alexander Efremov 85a87dade3 Add ApiOptions overloads to methods on I(Observable)StarredClient (#1336) 2016-06-02 12:15:35 +02: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
aedampir@gmail.com d0c8e82453 Red Tests were fixed
Unused 'using' directive were removed.
2016-04-18 12:46:13 +07:00
Mordechai Zuber c1aa707378 more of above 2015-05-19 11:53:04 +03:00
Mordechai Zuber e1ea7c30ad fix expected exceptions 2015-05-19 11:51:27 +03:00
Mordechai Zuber 306cffac7d O-Starred, O-Statistics, O-Trees 2015-05-19 11:37:40 +03:00
Paul Betts d731f1bd97 More jacked up tests 2014-04-30 21:47:34 -07:00
Kristian Hellang ae23a2ae7a Fixed minor namespace and whitespace issues 2013-12-01 23:42:04 +01:00
James R Sconfitto 71fc7e76f8 Add tests for ObservableStarredClient 2013-11-16 11:52:34 -05:00