Fix `dependsOn` for symbolic names

[andyleejordan]

Looking at the ARM docs https://learn.microsoft.com/en-us/azure/azure-resource-manager/templates/resource-declaration#use-symbolic-name, I don't think you should just clobber the name with the symbolic name. Instead, looking at https://learn.microsoft.com/en-us/azure/azure-resource-manager/templates/resource-dependency, it appears that symbolic names can be used in dependsOn in addition to resourceId() in the classical sense. So I think what we may need to eventually (this can be a follow-up PR) do is maintain a list of symbolic names and for resolving dependencies allow for either symbolic name or resourceid().

For this PR, I would just change the TODO comment to reflect what I said above.

Originally posted by @SteveL-MSFT in #1210 (comment)

[SteveL-MSFT]

Yes, my understanding with 2.0 is that dependsOn() can take a symbolic name or resourceId().