microsoft / PowerApps-TestEngine

MIT License
93 stars 28 forks source link

Adding UserInput or UserApp Exceptions #293

Closed NavneetThekkumpat closed 10 months ago

NavneetThekkumpat commented 10 months ago

User input errors are not handled gracefully. This is keeping in mind the behavior with the consumption of test engine in PAC CLI. Few cases below handled. Also updated a case to handle as User app exception. This will help us isolate this type and localize in PAC CLI side. Categorizing mainly these specific errors into

  1. UserInputExceptionInvalidFilePath.
  2. UserInputExceptionLoginCredential.
  3. UserInputExceptionTestConfig.
  4. UserInputExceptionYAMLFormat
  5. UserAppException

Validation

Console Output Scenarios

With this PAC test run will handle such scenarios gracefully (second run has this fix).

image

Checklist

landonmsft commented 10 months ago

Looks good, besides flip-flopped names for two 'cs' files, UserAppException/UserInputException

github-actions[bot] commented 10 months ago

Code Coverage

Package Line Rate Branch Rate Complexity Health
Microsoft.PowerApps.TestEngine 91% 87% 974
Summary 91% (2536 / 2781) 87% (593 / 678) 974

Minimum allowed line rate is 85%