(MAINT) Add locale to vmpooler Jap/French defs

[jcoconnor]

This is an outcome of moving the Windows-2012r2 localised vmpooler platforms to Windows 2019. When trying to test the new BHG strings in the ci-job-configs repo using the Puppet Repo Jenkins test harness it was obvious that there were issues with the parsing of the embedded "locale" string in the BHG name.

So since BHG already supports additional Beaker variables in the definition (see the use of the "user" variable for the French platform definitions to allow Windows use a localised Administrator name) it
makes sense to properly embed the "locale" definition here for the French and Japanese platforms.

This will allow the removal of the windows2019_ja-64%7Blocale=ja%7D type string from the Puppet Agent CI job definitions.

[jcoconnor]

Hmm - this is not liking the addition of locale, whereas user is ok.
Does the test suite also need updated ?

13:17:02          "CONFIG" => {"consoleport"=>443, "nfs_server"=>"none", "pooling_api"=>"http://vmpooler.delivery.puppetlabs.net/"},
13:17:02         -"HOSTS" => {"windows2016_fr-6432-1"=>{"hypervisor"=>"vmpooler", "packaging_platform"=>"windows-2012-x64", "pe_dir"=>nil, "pe_upgrade_dir"=>nil, "pe_upgrade_ver"=>nil, "pe_ver"=>nil, "platform"=>"windows-2016-64", "roles"=>["agent", "frictionless"], "ruby_arch"=>"x86", "template"=>"win-2016-fr-x86_64", "user"=>"Administrateur"}},
13:17:02         +"HOSTS" => {"windows2016_fr-6432-1"=>{"hypervisor"=>"vmpooler", "locale"=>"fr", "packaging_platform"=>"windows-2012-x64", "pe_dir"=>nil, "pe_upgrade_dir"=>nil, "pe_upgrade_ver"=>nil, "pe_ver"=>nil, "platform"=>"windows-2016-64", "roles"=>["agent", "frictionless"], "ruby_arch"=>"x86", "template"=>"win-2016-fr-x86_64", "user"=>"Administrateur"}},```

[jcoconnor]

Test fixtures sorted.

[kevpl]

lgtm 👍