Error message regarding plugin version string should be better

[fdrozdowski]

When I was trying out different version strings for the python plugins, and tried out this config file:
$ cat plugin_config.yml
id: 8da10137-ae59-45ea-9139-6f39b698040a
name: ga-plugin-1
version: 1.0
language: PYTHON27
hostTypes:

• UNIX
  pluginType: DIRECT
  entryPoint: plugin_runner:plugin
  srcDir: src
  schemaFile: schema.json

When I ran dvp build, I noticed the following error message not indicative of the issue:
$dvp build
Error: 1.0 is not of type 'string' on ['version']
1.0
Validation failed on /Users/vidyaraichur/APPDATA-virtual-env/ga-plugin-1/plugin_config.yml.
0 Warning(s). 1 Error(s)

BUILD FAILED.

The error message should indicate that we require it be of the format x.y.z and there is also an extra line 1.0 in the error output which is unnecessary.

┆Issue is synchronized with this Jira Bug by Unito

[fdrozdowski]

➤ Vidya Raichur commented:

Looks like the 1.0 in the YAML file was interpreted as a decimal value instead of a string.
When I tried specifying the version string in quotes. likes this in the YAML file:
version: "1.0"

I saw this error message during dvp build:
$ dvp build
Error: '1.0' does not match '^([0-9]+)\.([0-9]+)\.([a-zA-Z0-9_]+)$' on ['version']
"1.0"
Validation failed on /Users/vidyaraichur/APPDATA-virtual-env/ga-plugin-1/plugin_config.yml.
0 Warning(s). 1 Error(s)

BUILD FAILED.

[fdrozdowski]

➤ Ankur Sarin commented:

We should check if we can give type hints to the YAML parser and/or manually set it to string after parsing to a dict before validation.