fix: empty array flag values should be ignored

[sduquej]

Fixes foo-software/lighthouse-check-orb#8
Similar to #29

Our setup requires that we're able to create urls dynamically. To do this we generate a config file before running the audit where we define urls. Because the CircleCI Orb always passes in urls as part of the CLI call we were running into a problem when the flags were being parsed as urls had no value and was consequently being evaluated as a boolean.

[sduquej]

👋 @adamhenson, would really appreciate a review or guidance in getting this shipped. Happy to make any changes you think make sense. This is currently blocking us on using the lighthouse check effectively.

[adamhenson]

Nice catch @sduquej, and thanks for adding the unit test. I'll pull in this change and publish a new version in the various projects. Will update you here shortly. Thanks again for contributing!

[sduquej]

awesome, thank you!

[adamhenson]

Hi @sduquej - I published the Docker image. In a standard case, you won't need to do anything (because the default Orb executor uses the latest Docker image) and just try re-running your job. If you can, let me know here if your issue is fixed.

I also added a test in the lighthouse-check-orb project:

https://github.com/foo-software/lighthouse-check-orb/blob/4eef641da65087c63c99771976c7a224742c9b62/.circleci/config.yml#L38-L43

[sduquej]

Thanks @adamhenson! I can confirm our workflow is now passing.

[adamhenson]

Awesome! Thanks again for you help.