Chore: Improvement of tests script to allow param passing (Issue/3472)

[cahirodoherty-learningpool]

Addresses part 1 and part 2 of #3472

• Split up the test prepare command to allow only install/only grunt diff based on input and/or state
• Allowed for passing of a limited set of arguments to the test command

[cahirodoherty-learningpool]

Note on this - I had considered creating a 'utils' folder and splitting out some functions to a testHelpers file but felt it was potentially a step too far for this PR and also something worth getting peoples thoughts on. I'll happily add this or extend from this branch and do that if it is desired. @oliverfoster Any thoughts there?

[oliverfoster]

Note on this - I had considered creating a 'utils' folder and splitting out some functions to a testHelpers file but felt it was potentially a step too far for this PR and also something worth getting peoples thoughts on. I'll happily add this or extend from this branch and do that if it is desired. @oliverfoster Any thoughts there?

I think it's a waste of time and space unless the functions are going to be used elsewhere.

[cahirodoherty-learningpool]

Hi @oliverfoster thanks for your feedback on this. Do you feel the null default is suitable for the testfiles argument and trust in any user looking to specify a subset that they will provide the correct string(s)?
Or do you think we need to split the arguments into two; one for unittestfiles and one for e2etestfiles?

[oliverfoster]

Hi @oliverfoster thanks for your feedback on this. Do you feel the null default is suitable for the testfiles argument and trust in any user looking to specify a subset that they will provide the correct string(s)? Or do you think we need to split the arguments into two; one for unittestfiles and one for e2etestfiles?

I imagine it's only going to be used when writing tests, to specify which tests to run. Is there any other use-case so far?

[cahirodoherty-learningpool]

I imagine it's only going to be used when writing tests, to specify which tests to run. Is there any other use-case so far?

No I think thats all. I feel if we meet any use cases in the future we can amend to meet 🤷‍♂️

Thanks for the approval 👍

[github-actions]

🎉 This PR is included in version 5.33.12 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀