fix: correctly split quoted args

[gregmagolan]

Fixes #891 (requested by Bazel Open Source Support customer).

---

Changes are visible to end-users: yes

• Searched for relevant documentation and updated as needed: yes
• Breaking change (forces users to change their own code or config): no
• Suggested release notes appear below: yes

New function split_args is added to the public API of strings.bzl.

This fix touches a number of different rules:

• jq rule expansions now include make vars and rule fixed to single quote args that contain spaces
• expand_template optimized to not to bother with pointless .split(" ") followed by " ".join()
• bats_test optimized to not to bother with pointless .split(" ") followed by " ".join()
• params_file rule fixed to correctly split args on quotes
• run_binary rule fixed to correctly split args on quotes

Test plan

• Covered by existing test cases
• New test cases added

Unit tests added for the new split_args function that handles parsing args and splitting them taking into account single quotes, double quotes and escapes.

[aspect-workflows]

Test

1 test target passed

Targets

//docs:update_29_test [k8-fastbuild] 69ms

Total test execution time was 69ms. 220 tests (99.5%) were fully cached saving 30s.

---

Test

e2e/copy_action

All tests were cache hits

1 test (100.0%) was fully cached saving 59ms.

---

Test

e2e/copy_to_directory

All tests were cache hits

6 tests (100.0%) were fully cached saving 750ms.

---

Test

e2e/coreutils

All tests were cache hits

4 tests (100.0%) were fully cached saving 742ms.

---

Test

e2e/external_copy_to_directory

All tests were cache hits

1 test (100.0%) was fully cached saving 59ms.

---

Test

e2e/smoke

All tests were cache hits

6 tests (100.0%) were fully cached saving 2s.

---

Buildifier      Format      Gazelle