microsoft / HydraLab

Intelligent cloud testing made easy.
MIT License
1.09k stars 99 forks source link

Fix sleep between retries of startRecordingScreen #603

Closed DavidAsulin1 closed 11 months ago

DavidAsulin1 commented 12 months ago

Description

A minor fix for the sleep time between startRecordingScreen. The retryAndSleepWhenFalse takes sleepSeconds param and it's set to 1000 second (probably a mistake). I changed the value to 10 second (align with the other usages of this method).

Pull Request Checklist

Does this introduce a breaking change?

How you tested it

I encounter an issue running test on a mac machine. If appium failed to start the recording the retryAndSleepWhenFalse enter a very long sleep of 1000 seconds which block the test. Changing the value to 10 second (align with the other usages of this method) did fix the issue.

Please check the type of change your PR introduces:

hydraxman commented 11 months ago

/azp run

azure-pipelines[bot] commented 11 months ago
No pipelines are associated with this pull request.
hydraxman commented 11 months ago

@DavidAsulin1 thank you for the contribution!

you may need to agree to microsoft-github-policy by commenting:

@microsoft-github-policy-service agree

cc @zhou9584

hydraxman commented 11 months ago

/azp run

azure-pipelines[bot] commented 11 months ago
No pipelines are associated with this pull request.
DavidAsulin1 commented 11 months ago

@microsoft-github-policy-service agree

hydraxman commented 11 months ago

/azp run

azure-pipelines[bot] commented 11 months ago
No pipelines are associated with this pull request.
hydraxman commented 11 months ago

Do we know why this keeps hanging in this state? @zhou9584 @olivershen-wow @TedaLIEz

image