Fix ruby bundler config when app is of rails type and uses bundler local config

[alvaro-shopify]

WHY are these changes introduced?

Fixes https://github.com/Shopify/develop-app-management/issues/1517

WHAT is this pull request doing?

• Instead of modifying the local bundle configuration that could be used by the app, equivalent env variables are used
• Fix ruby unit tests as some of them were not being run because the method inTemporaryDirectory was mocked

How to test your changes?

• Create a spin instance for a 1P app that uses local bundle config

spin up shopify_chat:theme-extension -c shopify_chat.branch=theme-extension-spin

• Access the spin terminal

spin ssh

• Modify the package.json to use the cli version 0.0.0-experimental-20231027101204
• Restart the cli process
• Restart the server process
• No error should be shown

Tested on Windows and Ubuntu

• npm init @shopify/app@0.0.0-experimental-20231027101204 -- --template ruby
• cd in the app folder and npm run shopify app generate extension -- --template theme_app_extension
• npm run dev
• Theme app extension server should be run without any error

• Measuring impact

  How do we know this change was effective? Please choose one:

  • n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
  • Existing analytics will cater for this addition
  • PR includes analytics changes to measure impact

  Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

[github-actions]

Thanks for your contribution!

Depending on what you are working on, you may want to request a review from a Shopify team:

• Themes: @shopify/theme-code-tools
• UI extensions: @shopify/ui-extensions-cli
  • Checkout UI extensions: @shopify/checkout-ui-extensions-api-stewardship
• Hydrogen: @shopify/hydrogen
• Other: @shopify/cli-foundations

[github-actions]

Coverage report

St.❔	Category	Percentage	Covered / Total
🟡	Statements	73.48% (+0.4% 🔼)	6216/8460
🟡	Branches	70.42% (+0.49% 🔼)	3047/4327
🟡	Functions	71.95% (+0.5% 🔼)	1580/2196
🟡	Lines	74.67% (+0.37% 🔼)	5898/7899

Show files with reduced coverage 🔻

St.❔	File	Statements	Branches	Functions	Lines
🟢	... / ConcurrentOutput.tsx	97.62% (-2.38% 🔻)	75% (-8.33% 🔻)	100%	97.44% (-2.56% 🔻)

Test suite run success

1452 tests passing in 675 suites.

Report generated by 🧪jest coverage report action from 28e6290

[github-actions]

Differences in type declarations

We detected differences in the type declarations generated by Typescript for this branch compared to the baseline ('main' branch). Please, review them to ensure they are backward-compatible. Here are some important things to keep in mind:

• Some seemingly private modules might be re-exported through public modules.
• If the branch is behind main you might see odd diffs, rebase main into this branch.

New type declarations

We found no new type declarations in this PR

Existing type declarations

packages/cli-kit/dist/public/common/version.d.ts

@@ -1 +1,2 @@
-export declare const CLI_KIT_VERSION = "3.50.0";
\ No newline at end of file
+export declare const CLI_KIT_VERSION = "3.50.0";
+export declare const getCliKitVersion: () => string;
\ No newline at end of file

[Arkham]

Really good fix, LGTM 🙏