microsoft / PowerApps-TestEngine

MIT License
93 stars 28 forks source link

Implementing ready function #277

Closed NavneetThekkumpat closed 12 months ago

NavneetThekkumpat commented 12 months ago
  1. When the canvas published app is initialized >> there could be a delay in registration of published app JSSDK.
  2. If the TestEngine functions including getAppStatus is executed it returns COMMAND NOT FOUND error >> which in turn is interpreted as published app Test Engine JSSDK not found >> though it is still in process.

PROPOSAL

We just need to make sure TE continue supporting old apps (back compatibility >> older apps won't have this new ready function).

SUPPORTING DIFFERENT SCENARIOS

  1. Webplayer app and published app versions with no testEngineReady() >> Continue test execution.
  2. Webplayer app version with testEngineReady() and published app with no testEngineReady() >> Continue test execution.
  3. Webplayer app version with no testEngineReady() >> Continue test execution.
  4. Webplayer and published app with testEngineReady() >> Continue if true returned, else appropriate error codes are thrown in this case and handled appropriately.

Checklist

github-actions[bot] commented 12 months ago

Code Coverage

Package Line Rate Branch Rate Complexity Health
Microsoft.PowerApps.TestEngine 91% 88% 924
Summary 91% (2453 / 2682) 88% (559 / 636) 924

Minimum allowed line rate is 85%