helper/schema: Fix timeout parsing during Provider.Diff

[radeksimko]

I'm not sure when/where/why we decided to produce a simple map instead of slice (read as I didn't go through all the refactoring commits), but it makes more sense to me, so I'm assuming it was intentional.

Also I don't know why are we were not originally erroring out on invalid timeout types as I couldn't find a corresponding PR to this commit aa3677c
cc @catsby ^

[jbardin]

Thanks @radeksimko,

This makes sense, since blocks were always decoded in hcl as []map[string]interface{}, and in special cases they were hoisted up into a single map[string]interface{}. Since timeouts were never in the actual schema for some reason, that differentiation couldn't be done when the config was loading.

Now that they are present in the config as a NestingSingle block, they will be converted correctly by the shims before making it to the ConfigDecode function.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.