-
Notifications
You must be signed in to change notification settings - Fork 398
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
resource group entitlement check fix and detecting changes fix #549
Conversation
Codecov Report
@@ Coverage Diff @@
## master #549 +/- ##
==========================================
+ Coverage 81.81% 82.15% +0.34%
==========================================
Files 78 78
Lines 6768 6771 +3
==========================================
+ Hits 5537 5563 +26
+ Misses 818 800 -18
+ Partials 413 408 -5
|
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.
lgtm from code perspective, can we add a test to cover that changes?
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.
Code looks good, but we have to get unit tests to cover these changes.
hi @stikkireddy, do you have any updates on the test coverage? |
@nfx i should have some time later today and tomorrow to work on this. |
…-databricks into resource-group-changes-patch
@nfx everything should be up to date now with coverage of resource_group |
…ricks#549) * resource group entitlement check fix and detecting changes fix * updated test stub * fixed up the update tests for appropriate coverage * cover all lines in group resource but the terraform data set statements * added statement into changelog
Entitlements for resource groups had typos and logic for detecting changes was not working correctly. This fix resolves those issues.