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

Unit tests for HystrixCommandExecutionHooks (Manual merge of #327) #378

Merged
merged 8 commits into from
Dec 27, 2014
Merged

Unit tests for HystrixCommandExecutionHooks (Manual merge of #327) #378

merged 8 commits into from
Dec 27, 2014

Commits on Dec 27, 2014

  1. Hook execution sequence issue

    Sergey Krutsko authored and Matt Jacobs committed Dec 27, 2014
    Configuration menu
    Copy the full SHA
    89730c2 View commit details
    Browse the repository at this point in the history

  2. Unit tests to reproduce Hook execution sequence issue.

    Sergey Krutsko authored and Matt Jacobs committed Dec 27, 2014
    Configuration menu
    Copy the full SHA
    a21a253 View commit details
    Browse the repository at this point in the history

  3. Unit tests to reproduce Hook execution sequence issue. 

    testExecutionHookFailedOnHystrixBadRequestWithSemaphoreIsolation
    Sergey Krutsko authored and Matt Jacobs committed Dec 27, 2014
    Configuration menu
    Copy the full SHA
    f95c694 View commit details
    Browse the repository at this point in the history

  4. Changing expectations of hook order to match 1.3

    Matt Jacobs committed Dec 27, 2014
    Configuration menu
    Copy the full SHA
    912bb9e View commit details
    Browse the repository at this point in the history

  5. Switching order on onRunStart, onThreadStart hook execution to match 1.3

    Matt Jacobs committed Dec 27, 2014
    Configuration menu
    Copy the full SHA
    cd0e6c2 View commit details
    Browse the repository at this point in the history

  6. Removed assertions that executionHook.onRunSuccess gets called in tim… 

    …eout case
    Matt Jacobs committed Dec 27, 2014
    Configuration menu
    Copy the full SHA
    f18919b View commit details
    Browse the repository at this point in the history

  7. Switch assertion order of hooks, as onThreadComplete is firing earlie… 

    …r in 1.4
    Matt Jacobs committed Dec 27, 2014
    Configuration menu
    Copy the full SHA
    65ed5fb View commit details
    Browse the repository at this point in the history

  8. Added ExecutionHook.onError call to HystrixBadRequestException handling

    Matt Jacobs committed Dec 27, 2014
    Configuration menu
    Copy the full SHA
    7d740d8 View commit details
    Browse the repository at this point in the history