-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: entity sub-grid count when 0 #139
fix: entity sub-grid count when 0 #139
Conversation
…sCount as it fails for zero items (line 116)
…ding Gherkin test file to the LookupSteps.cs file (line 127 onwards)
@tdashworth, here is the pull request relating to the fixes we have made on the specflow bindings project. |
Looks good to me |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution 😄 a couple of comments if you could look at, please.
Also, wasn't the same fix required for #137?
bindings/src/Capgemini.PowerApps.SpecFlowBindings/Steps/LookupSteps.cs
Outdated
Show resolved
Hide resolved
bindings/src/Capgemini.PowerApps.SpecFlowBindings/Steps/EntitySubGridSteps.cs
Outdated
Show resolved
Hide resolved
After reviewing #137 again, it doesn't look to be the same solution. |
Commented out some code from EasyRepro and included the relevant link on GitHub relating to the issue. Co-authored-by: Tom Ashworth <tom.ashworth@capgemini.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Thanks for the contribution @AhmedD43 @osagiestar and team. When merging you can specify the commit message. Please follow to convention commit pattern e.g. ‘fix: asserting sub-grid count when there are 0 actual rows’. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Purpose
Fixes #138.
Approach
Bypasses the EasyRepro calls and instead retrieved the count of rows from the
x or y
text in the sub-grid footer.TODOs