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

address SmokeTestWatcherWithSecurityIT#testSearchInputWithInsufficientPrivileges #42764

Merged
merged 1 commit into from
May 31, 2019
Merged

address SmokeTestWatcherWithSecurityIT#testSearchInputWithInsufficientPrivileges #42764

merged 1 commit into from
May 31, 2019

Commits on May 31, 2019

  1. address SmokeTestWatcherWithSecurityIT#testSearchInputWithInsufficien… 

    …tPrivileges

This commit adds busy wait and increases the interval for
SmokeTestWatcherWithSecurityIT#testSearchInputWithInsufficientPrivileges.

Watcher will not allow the same watch to be executed concurrently. If it
finds that case, it will update the watch history with a "not_executed_already_queued"
status. Given a slow machine, and 1 second interval this is possible.

To address this, this commit increases the interval so the watch can fire at most 2
times with a greater interval between the executions and adds a busy wait for the
expected state.

While this does not gaurntee a fix, it should greatly reduce the chances of this
test erroring.
    @jakelandis
    jakelandis committed May 31, 2019
    Configuration menu
    Copy the full SHA
    76eea67 View commit details
    Browse the repository at this point in the history