Skip to content

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)

@SamuelAlev SamuelAlev requested a review from ragi96 January 22, 2024 16:06
@SamuelAlev SamuelAlev self-assigned this Jan 22, 2024
@SamuelAlev SamuelAlev requested review from a team as code owners January 22, 2024 16:06
@changeset-bot
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
Collaborator

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
@sonarqubecloud
Copy link

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.

4 participants