Get folder mount name

[soapy1]

fixes #12277

additional tests in the vagrant-spec project https://github.com/hashicorp/vagrant-spec/blob/main/acceptance/synced_folder/smb_spec.rb

[chrisroberts]

We should also update the test for this by updating the allow mock to only match on the correct number of passed arguments.

[DanSibbernsen]

Hi! I'm relatively new to the Vagrant world. Would this fix also need to be incorporated into the Vagrant-Parallels provider's use of mount? Example here

If so, I can open a change request over there (I'm stuck on this issue too)

[soapy1]

@DanSibbernsen yep, I think you are right. That mount_name bit should look something like this smb capability. You can double check that this change is required by running vagrant 2.2.15 with the parallels plugin.

[legal90]

Thank you, @soapy1.
It's a bit unfortunate that the positional arguments passed to mount_name capability were changed in Vagrant 2.2.15 (by #12281). That was a breaking change for 3rd party plugins like vagrant-parallels, implementing this capability on their side. :(

We have to use some smelly hacks in order to work it around and keep the backward compatibility: https://github.com/Parallels/vagrant-parallels/pull/386/files

Hopefully, we can avoid such things in the future 👍

[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.