Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

Add an acceptance test that ensures that the installation of extensio… #1545

Merged
merged 4 commits into from
Sep 20, 2021

Conversation

pepicrft
Copy link
Contributor

WHY are these changes introduced?

This PR extends this PR to add an acceptance test that ensures that the installation of Shopify Extensions doesn't break.

WHAT is this pull request doing?

Add an acceptance test that ensures the installation of extensions works.

Update checklist

  • I've added a CHANGELOG entry for this PR (if the change is public-facing)
  • I've considered possible cross-platform impacts (Mac, Linux, Windows).
  • I've left the version number as is (we'll handle incrementing this when releasing).

@pepicrft pepicrft requested a review from t6d September 17, 2021 15:01
@pepicrft pepicrft self-assigned this Sep 17, 2021
Then("I have the right binary for my system's architecture") do
system_architecture = `uname -m`
binary_architecture = `file #{File.expand_path("shopify-extensions", @working_dir)}`
assert binary_architecture.include?(system_architecture)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not 100% this works in environment others than macOS but I'll let CI tests surface any incompatibilities :)

@t6d t6d force-pushed the feature/download-shopify-cli-extensions-during-installation branch 2 times, most recently from 7f224a5 to 0f98a82 Compare September 17, 2021 19:43
Base automatically changed from feature/download-shopify-cli-extensions-during-installation to main September 17, 2021 20:25
@pepicrft pepicrft force-pushed the pull-cli-extension-acceptance-test branch from 1ecc971 to 399ad12 Compare September 20, 2021 10:42
@pepicrft pepicrft merged commit fe1baec into main Sep 20, 2021
@pepicrft pepicrft deleted the pull-cli-extension-acceptance-test branch September 20, 2021 11:02
cecyc pushed a commit that referenced this pull request Sep 20, 2021
Add an acceptance test that ensures that the installation of extensio…
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants