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.

Sign up for GitHub

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

Jump to bottom

Allow annotations to be explicitly enabled via config #275

Merged
merged 5 commits into from
May 15, 2018
Merged

Allow annotations to be explicitly enabled via config #275

merged 5 commits into from
May 15, 2018

Commits on May 12, 2018

  1. eclipse#266 improve the spec to take in explicit trues 

    Signed-off-by: Emily Jiang <[email protected]>
    @Emily-Jiang @Azquelt
    Emily-Jiang authored and Azquelt committed May 12, 2018
    Configuration menu
    Copy the full SHA
    aee7c84 View commit details
    Browse the repository at this point in the history

  2. Address spec review comments 

    Signed-off-by: Andrew Rouse <[email protected]>
    @Azquelt
    Azquelt committed May 12, 2018
    Configuration menu
    Copy the full SHA
    61fcaa1 View commit details
    Browse the repository at this point in the history

  3. Move TestException into general util package 

    This is to allow TestException to be used in other tests.

Also refactor how the metrics tests include util packages to simplify
the Deployment methods.

Signed-off-by: Andrew Rouse <[email protected]>
    @Azquelt
    Azquelt committed May 12, 2018
    Configuration menu
    Copy the full SHA
    f341098 View commit details
    Browse the repository at this point in the history

  4. Extract logic for tracking concurrent execution counts into new bean 

    Signed-off-by: Andrew Rouse <[email protected]>
    @Azquelt
    Azquelt committed May 12, 2018
    Configuration menu
    Copy the full SHA
    5649077 View commit details
    Browse the repository at this point in the history

  5. Tests for enabling and disabling annotations using config 

    * Simplify existing tests.
* Add new tests to check that method config overrides class config and
class config overrides global config
* Add tests for Bulkhead which were previously missing

Also changes tests from throwing RuntimeException to throwing
TestException. Some of the old tests would incorrectly catch
FaultToleranceExceptions when the intent was to only catch the
RuntimeException thrown by the test method.

Signed-off-by: Andrew Rouse <[email protected]>
    @Azquelt
    Azquelt committed May 12, 2018
    Configuration menu
    Copy the full SHA
    d7e99b0 View commit details
    Browse the repository at this point in the history