Misleading error message when using invalid character in scenario name

[dAnjou]

Brief summary

When using an invalid character in a scenario name, K6's error message is referring to the executor name instead:

ERRO[0000] There were problems with the specified script configuration:
        - scenario danjou.dev has configuration errors: the executor name should contain only numbers, latin letters, underscores, and dashes 

k6 version

k6 v0.49.0 (go1.21.6, darwin/arm64)

OS

macOS 14.3 (23D56)

Docker version and image (if applicable)

No response

Steps to reproduce the problem

Using the following setup: https://gist.github.com/dAnjou/930e0263d7be45b97233959a4c02f792

k6 run --env CONFIG=default.json script.js
k6 run --env CONFIG=scenario.json script.js
k6 run --env CONFIG=buggy.json script.js

Expected behaviour

The first 2 commands run a test.

The last one fails telling me there is an invalid character in the scenario name.

Actual behaviour

The first 2 commands run a test.

The last one fails telling me there is an invalid character in the executor name.

[joanlopez]

Yeah, after a quick look, I'm pretty sure the validation function could be easily improved in terms of user experience and error message correctness.

I'll open a PR soon with some refinements, including the one your reported.

Good catch @dAnjou, thanks! 🙌🏻