-
Notifications
You must be signed in to change notification settings - Fork 49
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
Feature/post opeartion with no inputs #175
Conversation
…eation) is not considered terraform compatible
…ing post body operation - test is currently failing, need to get to green state - updated convey statements to match actual method being called
…on (bodyParameterExists) - use the first body parameter returned - no longer check for multiple body params - remove test case checking for multiple body params
- refactor bodyParameterExists to no longer return error - refactor tests with new bodyParameterExists signature - add tests for validateResourceSchemaDefWithOptions
- only testing happy path - moved other test cases to TestValidateResourceSchemaDefWithOptions
- added unit tests
…t expect any input) logic into validateRootPath
- added delete operation as the int test fw requires the delete operation to be implemented... - fixed type format issue int vs string
Codecov Report
@@ Coverage Diff @@
## master #175 +/- ##
==========================================
+ Coverage 88.84% 88.91% +0.06%
==========================================
Files 51 51
Lines 3047 3066 +19
==========================================
+ Hits 2707 2726 +19
Misses 219 219
Partials 121 121
Continue to review full report at Codecov.
|
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.
👍
Proposed changes
Please add as many details as possible about the change here. Does this Pull Request resolve any open issue? If so, please make sure to link to that issue:
Fixes: #171
Type of change
What type of change does your code introduce to the provider? Please put an
x
(w/o heading/trailing white spaces)in the boxes that apply:
Checklist
Please put an
x
(w/o heading/trailing white spaces) in the boxes that apply: