Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge development branch for v5.6.0 #373

Merged
merged 24 commits into from
Nov 26, 2017
Merged

Merge development branch for v5.6.0 #373

merged 24 commits into from
Nov 26, 2017

Conversation

reisenberger
Copy link
Member

Delivers:

reisenberger and others added 24 commits November 12, 2017 13:12
Implemtation for #312 from @MartinSStewart , Polly Knowabunga Hackathon
at Tretton37
#362)

* Changed tests to be ran in parallel

Using [xUnit's CollectionAttribute](https://xunit.github.io/docs/running-tests-in-parallel.html) to mark some tests that cannot run in parallel.
Implement HandleInner<>() and OrInner<>(), for inner exceptions.  Specs
on all aspects, within FallbackPolicy.
Add IPolicyWrap.GetPolicy/ies<TPolicy> extension methods
* Prepare syntax for setting wait based on returned error
* Let WaitAndResponse calculate wait based on exception
WaitAndRetryForever: take error as input to sleepDurationProvider

* Allow WaitAndRetryForever to take error as input to sleepDurationProvider.
* Tidy up other WaitAndRetry overloads.
* More tests.
* Doco
De-parallelize tests which exercise policies which depend on
SystemClock, even if they don't manipulate the SystemClock themselves.
These tests may be vulnerable to failure if parallelized against a test
which does manipulate the SystemClock.
New Feature: Tooling for InnerExceptions of AggregateException

Another fruit of the weekend Hackathon
@reisenberger reisenberger added this to the v5.6.0 milestone Nov 26, 2017
@reisenberger reisenberger merged commit 9c133b6 into master Nov 26, 2017
@reisenberger reisenberger deleted the v560dev branch December 20, 2017 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants