Implement the mock translate function (from Puppetlabs-translate)

[PierreR]

I was about to push this PR but it does not accept the the following syntax with the double ((.

Any idea ?

 fail translate(('log_level must be one of debug, info, warn, error or fatal'))

https://github.com/puppetlabs/puppetlabs-docker/blob/master/manifests/init.pp#L498

ERROR: (swarm.manager.dev) cannot parse ./modules/application/manifests/role/swarm/test.pp:3:17:
  |
3 |   fail translate(('You must provide the $bridge parameter.'))
  |                 ^
unexpected '('
expecting ',', '}', operator, or the rest of Statement
 at ./modules/application/manifests/role/swarm/manager.pp:2:3

[bartavelle]

Is it accepted with single parens?

[PierreR]

I guess it is. Looking at the help from puppetlabs-translate there no usage of double parens. I am actually surprised this is valid puppet syntax.

[PierreR]

I guess it is. Looking at the help from puppetlabs-translate there no usage of double parens. I am actually surprised this is valid puppet syntax.

[bartavelle]

I think puppetresources doesn't like that fail is called without parens, that is why I wonder if it worked with single parens.

[PierreR]

No it does not work with single parens:

ERROR: (swarm.manager.dev) cannot parse ./modules/application/manifests/role/swarm/test.pp:3:17:
  |
3 |   fail translate('You must provide the $bridge parameter.')
  |                 ^
unexpected '('
expecting ',', '}', operator, or the rest of Statement

[PierreR]

Do you want to support that idiom with the single parens ? Not at all or only in permissive mode ?

fail translate('Custom package source is currently not implemented on windows.')

[bartavelle]

The problem is not with the parens, it is that you can apparently call functions like fail with no parens ...

[bartavelle]

Your translate function is fine, it is a parser problem with fail "foo".

[PierreR]

Ok thanks I will fill in a different issue if needed.