Render in arguments section optional computed properties with default values

[dikhan]

What problem does this Pull Request solve?

terraformdocsgenerator does not render optional computed properties with default values in the arguments section

• Added helper method IsOptionalComputedWithDefault to check within the renderer
  whether the property with default value is to be treated as optional computed too.
• Renderer now includes both optioanl computed properties and optioanl computed properties
  with default values in the arguments section along with the default value if populated.
• Other tests were failing due to ordering not matching anymore. This is due to
  the hashstructure.Hash now computing the new property added which changed the hash order.

Please link to the issue number here (issue will be closed when PR is merged): Closes #289

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:

• New feature (change that adds new functionality)
• Bug-fix (change that fixes current functionality)
• Tech debt (enhances the current functionality)
• New release (pumps the version)

Checklist

Please put an x (w/o heading/trailing white spaces) in the boxes that apply:

• I have read and followed the CONTRIBUTING guidelines
• I have added tests that prove my fix is effective or that my feature works
• I have made sure code compiles correctly and all tests are passing by running make test-all
• I have added/updated necessary documentation (if appropriate)
• I have added the following info to the title of the PR (pick the appropriate option for the type of change). This is important because the release notes will include this information.
  • Feature Request: PRs related to feature requests should have in the title [FeatureRequest: Issue #X] <PR Title>
  • Bug Fixes: PRs related to bug fixes should have in the title [BugFix: Issue #X] <PR Title>
  • Tech Debt: PRs related to technical debt should have in the title [TechDebt: Issue #X] <PR Title>
  • New Release: PRs related to a new release should have in the title [NewRelease] vX.Y.Z

Checklist for Admins

• Label is populated
• PR is assigned to the corresponding project
• PR has at least 1 reviewer and 1 assignee

[codecov]

Codecov Report

Merging #290 (5d2c8e9) into master (dac381f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #290   +/-   ##
=======================================
  Coverage   92.13%   92.14%           
=======================================
  Files          60       60           
  Lines        3066     3069    +3     
=======================================
+ Hits         2825     2828    +3     
  Misses        163      163           
  Partials       78       78           

Impacted Files	Coverage Δ
...penapi_spec_resource_schema_definition_property.go	100.00% <100.00%> (ø)
...erator/openapi_terraform_provider_doc_generator.go	100.00% <100.00%> (ø)
...api_terraform_provider_documentation_properties.go	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dac381f...5d2c8e9. Read the comment docs.