Fix backtick escaping and add more tests

[zackradisic]

What does this PR do?

This fixes shell not escaping a string with backticks and no other special characters

Also added more tests for shell escaping

[Jarred-Sumner]

Are there any other characters that need to be escaped? How do we avoid this from happening again?

[github-actions]

❌ @zackradisic, your commit has failing tests :(

💪 5 failing tests Darwin AARCH64

• test/bundler/bundler_npm.test.ts 1 failing
• test/cli/install/bun-install.test.ts 1 failing
• test/js/bun/http/serve.test.ts 1 failing
• test/js/bun/shell/commands/rm.test.ts 1 failing
• test/js/node/http2/node-http2.test.js 1 failing

💻 3 failing tests Darwin x64 baseline

• test/cli/install/bun-create.test.ts 2 failing
• test/cli/install/registry/bun-install-registry.test.ts 2 failing
• test/js/bun/test/test-test.test.ts 1 failing

💻 2 failing tests Darwin x64

• test/bundler/large_asset_regression.test.ts 1 failing
• test/cli/install/bun-create.test.ts 2 failing

🐧🖥 1 failing tests Linux x64

• test/js/node/child_process/child_process-node.test.js 1 failing

🪟💻 10 failing tests Windows x64 baseline

• test/cli/install/bunx.test.ts 1 failing
• test/cli/test/bun-test.test.ts 1 failing
• test/integration/esbuild/esbuild.test.ts 1 failing
• test/integration/next-pages/test/dev-server-ssr-100.test.ts 1 failing
• test/integration/next-pages/test/dev-server.test.ts 1 failing
• test/integration/next-pages/test/next-build.test.ts 1 failing
• test/js/bun/shell/bunshell.test.ts 1 failing
• test/js/node/child_process/child_process.test.ts 1 failing
• test/js/node/dns/node-dns.test.js 2 failing
• test/js/node/process/process.test.js 2 failing

🪟💻 13 failing tests Windows x64

• test/cli/install/bunx.test.ts 1 failing
• test/cli/install/registry/bun-install-registry.test.ts SIGTERM
• test/cli/test/bun-test.test.ts 1 failing
• test/integration/next-pages/test/dev-server-ssr-100.test.ts 1 failing
• test/integration/next-pages/test/dev-server.test.ts 1 failing
• test/integration/next-pages/test/next-build.test.ts 1 failing
• test/js/bun/shell/bunshell.test.ts 1 failing
• test/js/node/child_process/child_process.test.ts 1 failing
• test/js/node/dns/node-dns.test.js 2 failing
• test/js/node/process/process.test.js 2 failing
• test/js/third_party/body-parser/express-body-parser-test.test.ts 1 failing
• test/js/third_party/body-parser/express-bun-build-compile.test.ts 1 failing
• test/js/web/websocket/websocket.test.js STATUS_SEVERITY_ERROR

View logs

[gvilums]

Looks good, but env.positionals.test.ts is failing

[Jarred-Sumner]

@zackradisic merge conflict sorry

[Jarred-Sumner]

re-running because hard to tell if flaky or new test failures

[Jarred-Sumner]

Thank you

[BalduinB]

Is there a date for when this merged PR is getting published?