Wait for tenancy call if security enabled and bump version

[kavilla]

Description

Bumping to version 2.0.0, fix linter problem.

Also, preventing the test runner from navigating to a new
page when security plugin is enabled. The reason why is
because depending on how long the test takes to run, there
is an API call to refresh the permissions that take a little
bit longer than what tests are expecting. So if the test runner
makes the call to switch tenants but navigates to a new page
before that call is finished another call might run into a
permissions error. This ensures that if that call is made
then the test runner waits before navigating to the destination.

Signed-off-by: Kawika Avilla [email protected]

Issues Resolved

#231

Check List

• Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[kavilla]

Failures would be due to chicken and egg problem. Needs a 2.0 build but this needs to be in the 2.0 build.

[tianleh]

Failures would be due to chicken and egg problem. Needs a 2.0 build but this needs to be in the 2.0 build.

By checking the error in this check, https://github.com/opensearch-project/opensearch-dashboards-functional-test/runs/6514651442?check_suite_focus=true , the error is

{"type":"response","@timestamp":"2022-05-19T20:43:07Z","tags":["api"],"pid":2089,"method":"get","statusCode":401,"req":{"url":"/api/status","method":"get","headers":{"host":"localhost:5601","user-agent":"curl/7.68.0","accept":"*/*"},"remoteAddress":"127.0.0.1","userAgent":"curl/7.68.0"},"res":{"statusCode":401,"responseTime":1,"contentLength":9},"message":"GET /api/status 401 1ms - 9.0B"}

It is related to the breaking change where /api/status needs authentication.( opensearch-project/opensearch-build#2112 (comment))

@kavilla Since you already have the version bump in this PR, can you add such authentication along the way? We already have an existing example for 9200 check .

cc @seraphjiang @CCongWang @peterzhuamazon

[kavilla]

@tianleh seeing some issues. but might be related to the current existing failures from AD and Observability. The other PR might help but I believe both plugins are working on it.

[peterzhuamazon]

@kavilla @tianleh I vote for merging this PR now:

1. OBS Add data-test-subj to app analytics tests #216 is waiting for this PR to merge.
2. ADDash is also waiting for this to verify their test again.
   Thanks.

[tianleh]

@kavilla @tianleh I vote for merging this PR now:

1. OBS

+1