Resource & property loop decompilation

[anthony-c-martin]

Contributing a Pull Request

If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check. Once you are done and ready to submit your PR, run through the relevant checklist below.

Contributing a feature

• I have opened a new issue for the proposal, or commented on an existing one, and ensured that the bicep maintainers are good with the design of the feature being implemented
• I have included "Fixes #{issue_number}" in the PR description, so GitHub can link to the issue and close it when the PR is merged
• I have appropriate test coverage of my new feature

Fixes #1682
Fixes #1603

[codecov-io]

Codecov Report

Merging #1640 (37f84fa) into main (6cc8139) will decrease coverage by 0.09%.
The diff coverage is 71.34%.

@@            Coverage Diff             @@
##             main    #1640      +/-   ##
==========================================
- Coverage   95.13%   95.03%   -0.10%     
==========================================
  Files         362      364       +2     
  Lines       19882    20282     +400     
  Branches       13       13              
==========================================
+ Hits        18914    19276     +362     
- Misses        968     1006      +38     

Flag	Coverage Δ
dotnet	95.49% <71.34%> (-0.11%)	⬇️
typescript	27.40% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
....Decompiler.IntegrationTests/DecompilationTests.cs	100.00% <ø> (ø)
src/Bicep.Decompiler/ArmHelpers/TemplateHelpers.cs	94.44% <60.00%> (-5.56%)	⬇️
src/Bicep.Decompiler/TemplateConverter.cs	83.51% <64.65%> (-4.51%)	⬇️
src/Bicep.Decompiler/ScopedNamingResolver.cs	70.00% <70.00%> (ø)
...c/Bicep.Decompiler/ArmHelpers/ExpressionHelpers.cs	96.42% <92.30%> (-0.55%)	⬇️
src/Bicep.Core/Syntax/SyntaxFactory.cs	70.68% <100.00%> (+6.10%)	⬆️
src/Bicep.Core/Emit/ResourceDependencyVisitor.cs	93.33% <0.00%> (-1.75%)	⬇️
src/Bicep.Core/Semantics/SymbolValidator.cs	84.21% <0.00%> (-0.52%)	⬇️
src/Bicep.Core/TypeSystem/TypeAssignmentVisitor.cs	97.64% <0.00%> (-0.48%)	⬇️
....LangServer/Completions/BicepCompletionProvider.cs	96.54% <0.00%> (-0.14%)	⬇️
... and 17 more

[shenglol]

Looks good! Will output copy loop decompilation come with a separate PR, though?

[anthony-c-martin]

Looks good! Will output copy loop decompilation come with a separate PR, though?

Thanks for pointing that out - output loops were merged after I started this PR. I'll do it in a separate PR.