Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: split testing utilities out of main bundle #707

Open
wants to merge 16 commits into
base: dev
Choose a base branch
from

Conversation

SamuelAlev
Copy link
Member

@SamuelAlev SamuelAlev commented Jan 22, 2024

Otherwise sinon gets bundled into the blocks/themes/web-app.

You will need to update the import paths in your tests:

- import { AssetDummy, withAppBridgeBlockStubs } from '@frontify/app-bridge';
+ import { AssetDummy, withAppBridgeBlockStubs } from '@frontify/app-bridge/testing';

I also had to add getAppBridgeTestingPackage to lazy load the @frontify/app-bridge (v3.0.x) or @frontify/app-bridge/testing (future)

Copy link

changeset-bot bot commented Jan 22, 2024

🦋 Changeset detected

Latest commit: 1175f80

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@frontify/app-bridge Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@SamuelAlev SamuelAlev requested review from anxobotana and removed request for oliverschwendener January 22, 2024 16:06
@ragi96
Copy link
Contributor

ragi96 commented Jan 23, 2024

Do we need some conditional import of the testing stuff within guideline-blocks-settings 😅

@SamuelAlev SamuelAlev force-pushed the refactor/split-testing-utilities-from-main-bundle branch from 6b3642a to b700348 Compare January 24, 2024 16:50
@SamuelAlev SamuelAlev changed the base branch from main to chore/add-typing-package-json January 24, 2024 17:46
Base automatically changed from chore/add-typing-package-json to main January 24, 2024 19:22
@SamuelAlev SamuelAlev force-pushed the refactor/split-testing-utilities-from-main-bundle branch 2 times, most recently from d3ec91e to 8779a32 Compare January 24, 2024 19:27
@SamuelAlev SamuelAlev force-pushed the refactor/split-testing-utilities-from-main-bundle branch from e7650ba to 69eebd2 Compare January 25, 2024 14:28
@SamuelAlev SamuelAlev changed the base branch from main to dev January 25, 2024 14:28
Copy link

sonarcloud bot commented Jan 26, 2024

Quality Gate Passed Quality Gate passed for 'App Bridge'

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants