microsoft / PowerApps-TestEngine

MIT License
93 stars 28 forks source link

Fixing how test steps are extracted in case we need to go step by step #264

Closed akshar2401 closed 1 year ago

akshar2401 commented 1 year ago

Pull Request Template

Description

Splitting by semicolon is not the right way to extract out the test steps from the whole expression as it could be present anywhere in comments, string literals, or interpolated strings. This pr utilizes PowerFx lexer to attempt to correctly compute the test steps in the formula. The logic is locale specific.

Checklist

github-actions[bot] commented 1 year ago

Code Coverage

Package Line Rate Branch Rate Complexity Health
Microsoft.PowerApps.TestEngine 91% 89% 838
Summary 91% (2250 / 2470) 89% (514 / 578) 838

Minimum allowed line rate is 85%