search

microsoft / PowerAppsTestAutomation

MIT License
63 stars 677 forks source link

Jofinn/fix session id crash #24

Closed jfinnecy closed 2 years ago

jfinnecy commented 2 years ago
  1. Move the fetching of sessionId to disrupt a race condition where we get intermittent failures (exceptions) trying to access Core scripts.
  2. Wrap the fetch in try/catch so that even if it fails it will still execute tests, it just won't report sessionId. If we see a lot of reports of test results with no session Id's we can revisit to look for a better solution/placement of the code.
jfinnecy commented 2 years ago

DOC modernization may eventually make this method of fetching sessionId invalid. We should consider adding SessionId to the test results in the future so we don't need to query for it separately.