Simmy

Simmy copied to clipboard

Implement BeforeInject callback

1

Closes #65

iCodeSometime

Feature Proposal - Add callback to be run when chaos is injected

1

It would be nice to be able to run code when chaos is injected (by any of the simmy injection policies). Example use-case: ``` var chaosExceptionPolicy = MonkeyPolicy .InjectExceptionAsync(with =>...

iCodeSometime

Hey @reisenberger! Just cleaning up the old syntax which isn't gonna be available since v1. Putting this PR just to start to work over the remaining issues and also for...

vany0114

Adopt Nullable Reference Types

1

Throwing this here to take into consideration, according to Microsoft, we should consider adopting NRT by next year: https://devblogs.microsoft.com/dotnet/embracing-nullable-reference-types/ There are some caveats tho regarding C# 8 and multitargeting that...

vany0114

Build frequently fails on second build for PR (clean up build-numbering strategy)

1

A PR typically builds twice - for the branch and then the PR. The second build often fails with: ``` ======================================== __UpdateAppVeyorBuildNumber ======================================== Executing task: __UpdateAppVeyorBuildNumber Build with specified version...

reisenberger

Currently, all the monkeys are cancellable, which is fine, but we want to allow the user to make the latency monkey no cancellable for those scenarios where even if a...

vany0114

Consider optimisations of async configuration hotpaths for synchronously-provided results

2

It seems _reasonable_ that the delegates used to configure async `MonkeyPolicy` instances allow users to provide an async lambda. This _permits_ users make async calls to consult some config source,...

reisenberger

Consider shorter syntax for result stubbing, for use in unit-tests in other systems on how those systems handle faults

4

All current overloads are of the form: MonkeyPolicy.InjectX(/* what to inject */, Double injectionRate, bool enabled) // or similar Func variants for injectionRate and enabled Unit tests are likely to...

reisenberger

We need to update the project file to include all the metadata that we have into the `.nuspec` file, in order to remove the duplication with the releases notes, that...

vany0114

We need to increase the coverage for `InjectBehaviour` policies, there are overloads that don't even have unit tests. 

vany0114