Fix validate on save

[jpogran]

The /didSave handler was missing WithModuleManager and WithExecutorFactory, which the validate command handler needs to find terraform to execute validate.

Closes #792

[dbanck]

Closes #792

[radeksimko]

LGTM, but I'd prefer to rebase & merge this after #782

[radeksimko]

@jpogran Do you mind rebasing this one? I think it's just the ModuleManager removal that needs reflecting here.

[jpogran]

Rebased and pushed

[jpogran]

Confirmed works as expected:

2022/02/24 12:42:39 rpc_logger.go:29: Incoming notification for "textDocument/didSave": {"textDocument":{"uri":"file:///c%3A/Users/james/src/hashicorp/learn/example/main.tf"}}
2022/02/24 12:42:40 opts.go:254: Posting server notification "textDocument/publishDiagnostics" {"uri":"file:///C:/Users/james/src/hashicorp/learn/example","diagnostics":[]}
2022/02/24 12:42:40 opts.go:254: Posting server notification "textDocument/publishDiagnostics" {"uri":"file:///C:/Users/james/src/hashicorp/learn/example/main.tf","diagnostics":[{"range":{"start":{"line":15,"character":43},"end":{"line":15,"character":44}},"severity":1,"source":"terraform validate","message":"Missing required argument: The argument \"resource_group_name\" is required, but no definition was found."},{"range":{"start":{"line":15,"character":43},"end":{"line":15,"character":44}},"severity":1,"source":"terraform validate","message":"Missing required argument: The argument \"data_residency_location\" is required, but no definition was found."},{"range":{"start":{"line":15,"character":43},"end":{"line":15,"character":44}},"severity":1,"source":"terraform validate","message":"Missing required argument: The argument \"sku_name\" is required, but no definition was found."},{"range":{"start":{"line":15,"character":43},"end":{"line":15,"character":44}},"severity":1,"source":"terraform validate","message":"Missing required argument: The argument \"domain_name\" is required, but no definition was found."}]}

[github-actions]

This functionality has been released in v0.26.0 of the language server.
If you use the official Terraform VS Code extension, it will prompt you to upgrade to this version automatically upon next launch or within the next 24 hours.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

[github-actions]

I'm going to lock this pull request 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 related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.