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.

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

test_devops_group_and_member_tests fails #1174

Closed
3 tasks done
Tracked by #1165
roshan-sy opened this issue Aug 25, 2021 · 3 comments · Fixed by #1183, #1201 or #1204
Closed
3 tasks done
Tracked by #1165

test_devops_group_and_member_tests fails #1174

roshan-sy opened this issue Aug 25, 2021 · 3 comments · Fixed by #1183, #1201 or #1204

Comments

@roshan-sy
Copy link
Contributor

roshan-sy commented Aug 25, 2021

https://dev.azure.com/ms/azure-devops-cli-extension/_build/results?buildId=214635&view=ms.vss-test-web.build-test-results-tab&runId=3223678&resultId=100007&paneView=debug

  • RCA
  • Propose Fix
  • Code change
@roshan-sy
Copy link
Contributor Author

roshan-sy commented Aug 25, 2021

self.sleep_in_live_run(1)

This command in multiple line induces a pause for the live test. (In seconds)
Sometime the pause is not sufficient for the execution of assert command.

  • A simple fix could be increase the pause time
    It passes when pause is increased to 60 seconds, but also increases the total time for test execution

  • We can explore for the callback mechanism

Build link: https://dev.azure.com/ms/azure-devops-cli-extension/_build/results?buildId=214725&view=ms.vss-test-web.build-test-results-tab
Here though other tests are failing, this one is passing

@roshan-sy
Copy link
Contributor Author

self.sleep_in_live_run(1)

This command in multiple line induces a pause for the live test. (In seconds)
Sometime the pause is not sufficient for the execution of assert command.

  • A simple fix could be increase the pause time
    It passes when pause is increased to 60 seconds, but also increases the total time for test execution
  • We can explore for the callback mechanism

Build link: https://dev.azure.com/ms/azure-devops-cli-extension/_build/results?buildId=214725&view=ms.vss-test-web.build-test-results-tab
Here though other tests are failing, this one is passing

@gauravsaralMs

@gauravsaralMs
Copy link
Contributor

@roshan-sy sounds good, can you raise a PR with this change ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants